Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Martin Perina 
Hi,

unfortunately we have a new blocker [1] related to upgrade from 4.1.z, but
we will try to do our best to post a fix before 10:00 today.

Martin

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1519301




On Thu, Nov 30, 2017 at 8:49 AM, Sandro Bonazzola 
wrote:

> Hi,
> we still have 4 open acknowledged blockers according to
> https://bugzilla.redhat.com/buglist.cgi?quicksearch=
> flag%3Ablocker%2B%20target_milestone%3Aovirt-4.2.0%
> 20status%3Anew%2Cassigned%2Cpost
>
> Bug ID Product Assignee Status Summary
> 1518887 
> ovirt-hosted-engine-ha b...@ovirt.org NEW ovirt-ha-agent fails parsing
> the OVF_STORE due to a change in OVF namespace URI
> 1516113 
> cockpit-ovirt phbai...@redhat.com POST Deploy the HostedEngine failed
> with the default CPU type
> 1518693  ovirt-engine
> akrej...@redhat.com POST Quota is needed to copy template disk
> 1507277  ovirt-engine
> era...@redhat.com POST [RFE][DR] - Vnic Profiles mapping in VMs register
> from data storage domain should be supported also for templates
>
> There are also 3 proposed blockers that need either to be acknowledged or
> rejected: https://bugzilla.redhat.com/buglist.cgi?
> quicksearch=flag%3Ablocker%3F%20target_milestone%3Aovirt-4.
> 2.0%20status%3Anew%2Cassigned%2Cpost
>
> Bug ID Product Assignee Summary
> 1450061  Red Hat
> Enterprise Virtualization Manager rh-spice-b...@redhat.com Copy-paste:
> filename encoding in Win guest
> 1517810  ovirt-engine
> stira...@redhat.com Adding additional ha-host fails.
> 1502920  Red Hat
> Enterprise Virtualization Manager rba...@redhat.com File missing after
> upgrade of RHVH node from version RHVH-4.1-20170925.0 to latest.
>
>
> Given current status I would reschedule oVirt 4.2.0 RC to next week,
> tentatively on Monday.
>
> --
>
> SANDRO BONAZZOLA
>
> ASSOCIATE MANAGER, SOFTWARE ENGINEERING, EMEA ENG VIRTUALIZATION R
>
> Red Hat EMEA 
> 
> TRIED. TESTED. TRUSTED. 
>
>
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>



-- 
Martin Perina
Associate Manager, Software Engineering
Red Hat Czech s.r.o.
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

[ovirt-devel] 4.2.1 bugs

2017-11-30 Thread Greg Sheremeta 
Hi,

I have some 4.2.1 bugs I want to start working on, but I don't want them
getting into 4.2.0 GA. What should I do?

Best wishes,
Greg

-- 

GREG SHEREMETA

SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX

Red Hat NA



gsher...@redhat.comIRC: gshereme

___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2 and UI plugin infrastructure

2017-11-30 Thread Vojtech Szocs 
Hello Martin,

UI plugin API in 4.2 should be generally backwards compatible with 4.1.

Comparing PluginManager.java between master and 4.1.8 - the
exposePluginApi() function is exactly the same for now.

However, we might do some API changes in the future, mainly due to 4.2 UI
redesign. For example:

   - taking out the word "Tab" out of function names, since the UI concept
   of "main tab" is no more
   - as mentioned by Alex, support adding buttons inside kebab drop-down,
   as opposed to adding them directly to the toolbar (UX wise, important
   things should be outside kebab, less important things should go inside it)
   - support adding secondary level menu items when adding new "main tab"
   (this should become "add menu item" function)
   - support customizing the icon for primary menu items

As for the visuals, as Greg wrote, UI plugins should generally follow the
PatternFly styling to ensure consistent looks.

WebAdmin has its own branding infra and UI plugins can use that. Just
inspect the HTML source and see how WebAdmin requests PatternFly & branding
related assets.

Vojtech


On Wed, Nov 29, 2017 at 1:22 AM, Greg Sheremeta  wrote:

> Hey,
>
> We haven't changed the API, so everything should still work. For example,
> I just installed 4.1 versions of dashboard and support-plugin in master,
> and all's well [*]:
>
> API:
>
> api.addSubTab('Template', 'Red Hat Documentation',
> 'my-host-subtab-template', '', {alignRight: true});
> api.setTabAccessible('my-host-subtab-template', true);
>
> [image: Inline image 1]
>
> [*] however, note that "alignRight" is now ignored
>
> And dashboard 4.1.8 installed in master engine:
>
> [image: Inline image 2]
>
>
> That said, we probably should have sent some announcement about making
> sure the plugins actually look good with the new theme. If some plugins are
> greenish, that will look bad.
>
> To help with styling external things, we've started exposing more CSS via
> a new SASS brand module:
> https://github.com/oVirt/ovirt-engine/tree/master/frontend/brands/
> ​​
> ovirt-brand/src/main/sass
> 
> Also, we're heavily relying on base PatternFly styles, and plugins should
> do the same. For example, the new dialog styles are completely PatternFly [
> http://www.patternfly.org/pattern-library/forms-and-
> controls/modal-overlay/] and we don't do anything extra on top of that.
>
> Let us know if something doesn't work, or you need help styling something.
>
> Greg
>
> On Tue, Nov 28, 2017 at 6:25 PM, Martin Sivak  wrote:
>
>> Hi,
>>
>> I just got a question from an ovirt-optimizer user about the support
>> in 4.2. And I realized I haven't heard anything about how UI plugins
>> should be updated to work with the new UI.
>>
>> Is there anything special that needs to be done to make the plugin
>> functional? Or will everything still work somehow?
>>
>> Best regards
>>
>> Martin Sivak
>> ___
>> Devel mailing list
>> Devel@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
>>
>
>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2 and UI plugin infrastructure

2017-11-30 Thread Alexander Wels 
On Thursday, November 30, 2017 10:48:26 AM EST Martin Sivak wrote:
> It stays on a single line with just slightly wider window than I normally
> have. But all plugins will be adding buttons here so the line can
> theoretically get really long.
> 
> Btw, is 1920px the required resolution now? Because I have no screen or
> machine with that :D
> 
> Martin
> 

Right and we have no api option (yet) to allow plugins to put their buttons in 
the kebab menu. The buttons (and kebab as you noticed) will simply wrap to the 
next line if there is not enough space to display them all. You just happened 
to have a width where just the kebab menu wrapped, which looks sort of weird. 
I guess we can try to force the kebab on the top right regardless of if the 
buttons wrap.

Btw there is no resolution requirement. In fact it should be somewhat usable 
on low resolution and mobile type screens now.

> On Thu, Nov 30, 2017 at 4:44 PM, Greg Sheremeta  wrote:
> > How does it look at 1920px wide?
> > 
> > On Thu, Nov 30, 2017 at 10:40 AM, Martin Sivak  wrote:
> >> Hi,
> >> 
> >> I see it working with a small UI glitch:
> >> 
> >> Martin
> >> 
> >> On Wed, Nov 29, 2017 at 6:25 PM, Martin Sivak  wrote:
> >>> I will try finding time tomorrow. Thanks.
> >>> 
> >>> Martin
> >>> 
> >>> On Wed, Nov 29, 2017 at 6:02 PM, Greg Sheremeta 
> >>> 
> >>> wrote:
>  Martin, we've merged the fix. (Thanks to Alexander for the super fast
>  fix.)
>  
>  Care to test with master?
>  
>  On Wed, Nov 29, 2017 at 10:50 AM, Alexander Wels 
>  
>  wrote:
> > On Wednesday, November 29, 2017 10:16:27 AM EST Martin Sivak wrote:
> > > Thanks to you too for checking.
> > > 
> > > Martin
> > 
> > Fix is pending review [1]. We moved the main views to a different
> > class
> > hierarchy and it wasn't inheriting the methods to add the buttons.
> > 
> > Alexander
> > 
> > [1] https://gerrit.ovirt.org/c/84901/
> > 
> > > On Wed, Nov 29, 2017 at 3:07 PM, Greg Sheremeta
> > > 
> > 
> > wrote:
> > > > Discussed with Alexander, and yep this is an issue.
> > > > I opened https://bugzilla.redhat.com/show_bug.cgi?id=1518724
> > > > 
> > > > Thanks for checking and reporting!
> > > > 
> > > > On Wed, Nov 29, 2017 at 8:35 AM, Greg Sheremeta <
> > 
> > gsher...@redhat.com>
> > 
> > > > wrote:
> > > >> +Alexander, can you take a look?
> > > >> 
> > > >> On Wed, Nov 29, 2017 at 8:33 AM, Martin Sivak 
> > 
> > wrote:
> > > >>> Hi,
> > > >>> 
> > > >>> I think I found one issue.. I do not see the extra menu items
> > 
> > that were
> > 
> > > >>> supposed to be added to VMs (used to be available in context
> > 
> > menu as
> > 
> > > >>> well).
> > > >>> 
> > > >>> You can check it with
> > > >>> 
> > > >>> http://jenkins.ovirt.org/job/ovirt-optimizer_master_check-pa
> > > >>> tch-el7-x86_64/65/
> > > >>> 
> > > >>> Martin
> > > >>> 
> > > >>> On Wed, Nov 29, 2017 at 1:22 AM, Greg Sheremeta <
> > 
> > gsher...@redhat.com>
> > 
> > > >>> wrote:
> > >  Hey,
> > >  
> > >  We haven't changed the API, so everything should still work.
> > >  For
> > >  example, I just installed 4.1 versions of dashboard and
> > 
> > support-plugin
> > 
> > >  in
> > >  master, and all's well [*]:
> > >  
> > >  API:
> > >  
> > >  api.addSubTab('Template', 'Red Hat Documentation',
> > >  'my-host-subtab-template', '', {alignRight: true});
> > >  api.setTabAccessible('my-host-subtab-template', true);
> > >  
> > >  [image: Inline image 1]
> > >  
> > >  [*] however, note that "alignRight" is now ignored
> > >  
> > >  And dashboard 4.1.8 installed in master engine:
> > >  
> > >  [image: Inline image 2]
> > >  
> > >  
> > >  That said, we probably should have sent some announcement about
> > 
> > making
> > 
> > >  sure the plugins actually look good with the new theme. If some
> > 
> > plugins
> > 
> > >  are
> > >  greenish, that will look bad.
> > >  
> > >  To help with styling external things, we've started exposing
> > 
> > more CSS
> > 
> > >  via a new SASS brand module:
> > >  https://github.com/oVirt/ovirt-engine/tree/master/frontend/b
> > >  rands/ovirt-brand/src/main/sass
> > >  Also, we're heavily relying on base PatternFly styles, and
> > 
> > plugins
> > 
> > >  should do the same. For example, the new dialog styles are
> > 
> > completely
> > 
> > >

Re: [ovirt-devel] oVirt 4.2 and UI plugin infrastructure

2017-11-30 Thread Martin Sivak 
It stays on a single line with just slightly wider window than I normally
have. But all plugins will be adding buttons here so the line can
theoretically get really long.

Btw, is 1920px the required resolution now? Because I have no screen or
machine with that :D

Martin

On Thu, Nov 30, 2017 at 4:44 PM, Greg Sheremeta  wrote:

> How does it look at 1920px wide?
>
> On Thu, Nov 30, 2017 at 10:40 AM, Martin Sivak  wrote:
>
>> Hi,
>>
>> I see it working with a small UI glitch:
>>
>> Martin
>>
>> On Wed, Nov 29, 2017 at 6:25 PM, Martin Sivak  wrote:
>>
>>> I will try finding time tomorrow. Thanks.
>>>
>>> Martin
>>>
>>> On Wed, Nov 29, 2017 at 6:02 PM, Greg Sheremeta 
>>> wrote:
>>>
 Martin, we've merged the fix. (Thanks to Alexander for the super fast
 fix.)

 Care to test with master?

 On Wed, Nov 29, 2017 at 10:50 AM, Alexander Wels 
 wrote:

> On Wednesday, November 29, 2017 10:16:27 AM EST Martin Sivak wrote:
> > Thanks to you too for checking.
> >
> > Martin
> >
>
> Fix is pending review [1]. We moved the main views to a different class
> hierarchy and it wasn't inheriting the methods to add the buttons.
>
> Alexander
>
> [1] https://gerrit.ovirt.org/c/84901/
>
> > On Wed, Nov 29, 2017 at 3:07 PM, Greg Sheremeta 
> wrote:
> > > Discussed with Alexander, and yep this is an issue.
> > > I opened https://bugzilla.redhat.com/show_bug.cgi?id=1518724
> > >
> > > Thanks for checking and reporting!
> > >
> > > On Wed, Nov 29, 2017 at 8:35 AM, Greg Sheremeta <
> gsher...@redhat.com>
> > >
> > > wrote:
> > >> +Alexander, can you take a look?
> > >>
> > >> On Wed, Nov 29, 2017 at 8:33 AM, Martin Sivak 
> wrote:
> > >>> Hi,
> > >>>
> > >>> I think I found one issue.. I do not see the extra menu items
> that were
> > >>> supposed to be added to VMs (used to be available in context
> menu as
> > >>> well).
> > >>>
> > >>> You can check it with
> > >>>
> > >>> http://jenkins.ovirt.org/job/ovirt-optimizer_master_check-pa
> > >>> tch-el7-x86_64/65/
> > >>>
> > >>> Martin
> > >>>
> > >>> On Wed, Nov 29, 2017 at 1:22 AM, Greg Sheremeta <
> gsher...@redhat.com>
> > >>>
> > >>> wrote:
> >  Hey,
> > 
> >  We haven't changed the API, so everything should still work. For
> >  example, I just installed 4.1 versions of dashboard and
> support-plugin
> >  in
> >  master, and all's well [*]:
> > 
> >  API:
> > 
> >  api.addSubTab('Template', 'Red Hat Documentation',
> >  'my-host-subtab-template', '', {alignRight: true});
> >  api.setTabAccessible('my-host-subtab-template', true);
> > 
> >  [image: Inline image 1]
> > 
> >  [*] however, note that "alignRight" is now ignored
> > 
> >  And dashboard 4.1.8 installed in master engine:
> > 
> >  [image: Inline image 2]
> > 
> > 
> >  That said, we probably should have sent some announcement about
> making
> >  sure the plugins actually look good with the new theme. If some
> plugins
> >  are
> >  greenish, that will look bad.
> > 
> >  To help with styling external things, we've started exposing
> more CSS
> >  via a new SASS brand module:
> >  https://github.com/oVirt/ovirt-engine/tree/master/frontend/b
> >  rands/ovirt-brand/src/main/sass
> >  Also, we're heavily relying on base PatternFly styles, and
> plugins
> >  should do the same. For example, the new dialog styles are
> completely
> >  PatternFly [http://www.patternfly.org/pat
> tern-library/forms-and-control
> >  s/modal-overlay/] and we don't do anything extra on top of that.
> > 
> >  Let us know if something doesn't work, or you need help styling
> >  something.
> > 
> >  Greg
> > 
> >  On Tue, Nov 28, 2017 at 6:25 PM, Martin Sivak <
> msi...@redhat.com>
> > 
> >  wrote:
> > > Hi,
> > >
> > > I just got a question from an ovirt-optimizer user about the
> support
> > > in 4.2. And I realized I haven't heard anything about how UI
> plugins
> > > should be updated to work with the new UI.
> > >
> > > Is there anything special that needs to be done to make the
> plugin
> > > functional? Or will everything still work somehow?
> > >
> > > Best regards
> > >
> > > Martin Sivak
> > > ___
> > > Devel mailing list
> > >

Re: [ovirt-devel] oVirt 4.2 and UI plugin infrastructure

2017-11-30 Thread Greg Sheremeta 
How does it look at 1920px wide?

On Thu, Nov 30, 2017 at 10:40 AM, Martin Sivak  wrote:

> Hi,
>
> I see it working with a small UI glitch:
>
> Martin
>
> On Wed, Nov 29, 2017 at 6:25 PM, Martin Sivak  wrote:
>
>> I will try finding time tomorrow. Thanks.
>>
>> Martin
>>
>> On Wed, Nov 29, 2017 at 6:02 PM, Greg Sheremeta 
>> wrote:
>>
>>> Martin, we've merged the fix. (Thanks to Alexander for the super fast
>>> fix.)
>>>
>>> Care to test with master?
>>>
>>> On Wed, Nov 29, 2017 at 10:50 AM, Alexander Wels 
>>> wrote:
>>>
 On Wednesday, November 29, 2017 10:16:27 AM EST Martin Sivak wrote:
 > Thanks to you too for checking.
 >
 > Martin
 >

 Fix is pending review [1]. We moved the main views to a different class
 hierarchy and it wasn't inheriting the methods to add the buttons.

 Alexander

 [1] https://gerrit.ovirt.org/c/84901/

 > On Wed, Nov 29, 2017 at 3:07 PM, Greg Sheremeta 
 wrote:
 > > Discussed with Alexander, and yep this is an issue.
 > > I opened https://bugzilla.redhat.com/show_bug.cgi?id=1518724
 > >
 > > Thanks for checking and reporting!
 > >
 > > On Wed, Nov 29, 2017 at 8:35 AM, Greg Sheremeta <
 gsher...@redhat.com>
 > >
 > > wrote:
 > >> +Alexander, can you take a look?
 > >>
 > >> On Wed, Nov 29, 2017 at 8:33 AM, Martin Sivak 
 wrote:
 > >>> Hi,
 > >>>
 > >>> I think I found one issue.. I do not see the extra menu items
 that were
 > >>> supposed to be added to VMs (used to be available in context menu
 as
 > >>> well).
 > >>>
 > >>> You can check it with
 > >>>
 > >>> http://jenkins.ovirt.org/job/ovirt-optimizer_master_check-pa
 > >>> tch-el7-x86_64/65/
 > >>>
 > >>> Martin
 > >>>
 > >>> On Wed, Nov 29, 2017 at 1:22 AM, Greg Sheremeta <
 gsher...@redhat.com>
 > >>>
 > >>> wrote:
 >  Hey,
 > 
 >  We haven't changed the API, so everything should still work. For
 >  example, I just installed 4.1 versions of dashboard and
 support-plugin
 >  in
 >  master, and all's well [*]:
 > 
 >  API:
 > 
 >  api.addSubTab('Template', 'Red Hat Documentation',
 >  'my-host-subtab-template', '', {alignRight: true});
 >  api.setTabAccessible('my-host-subtab-template', true);
 > 
 >  [image: Inline image 1]
 > 
 >  [*] however, note that "alignRight" is now ignored
 > 
 >  And dashboard 4.1.8 installed in master engine:
 > 
 >  [image: Inline image 2]
 > 
 > 
 >  That said, we probably should have sent some announcement about
 making
 >  sure the plugins actually look good with the new theme. If some
 plugins
 >  are
 >  greenish, that will look bad.
 > 
 >  To help with styling external things, we've started exposing
 more CSS
 >  via a new SASS brand module:
 >  https://github.com/oVirt/ovirt-engine/tree/master/frontend/b
 >  rands/ovirt-brand/src/main/sass
 >  Also, we're heavily relying on base PatternFly styles, and
 plugins
 >  should do the same. For example, the new dialog styles are
 completely
 >  PatternFly [http://www.patternfly.org/pat
 tern-library/forms-and-control
 >  s/modal-overlay/] and we don't do anything extra on top of that.
 > 
 >  Let us know if something doesn't work, or you need help styling
 >  something.
 > 
 >  Greg
 > 
 >  On Tue, Nov 28, 2017 at 6:25 PM, Martin Sivak 
 > 
 >  wrote:
 > > Hi,
 > >
 > > I just got a question from an ovirt-optimizer user about the
 support
 > > in 4.2. And I realized I haven't heard anything about how UI
 plugins
 > > should be updated to work with the new UI.
 > >
 > > Is there anything special that needs to be done to make the
 plugin
 > > functional? Or will everything still work somehow?
 > >
 > > Best regards
 > >
 > > Martin Sivak
 > > ___
 > > Devel mailing list
 > > Devel@ovirt.org
 > > http://lists.ovirt.org/mailman/listinfo/devel
 > >>
 > >> --
 > >>
 > >> GREG SHEREMETA
 > >>
 > >> SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX
 > >>
 > >> Red Hat NA
 > >>
 > >> 
 > >>
 > >> gsher...@redhat.comIRC: gshereme
 > >> 
 > >
 > > --
 > >
 > > GREG SHEREMETA
 > >
 > > SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX
 > >
 > > Red Hat

Re: [ovirt-devel] oVirt 4.2 and UI plugin infrastructure

2017-11-30 Thread Martin Sivak 
Hi,

I see it working with a small UI glitch:

Martin

On Wed, Nov 29, 2017 at 6:25 PM, Martin Sivak  wrote:

> I will try finding time tomorrow. Thanks.
>
> Martin
>
> On Wed, Nov 29, 2017 at 6:02 PM, Greg Sheremeta 
> wrote:
>
>> Martin, we've merged the fix. (Thanks to Alexander for the super fast
>> fix.)
>>
>> Care to test with master?
>>
>> On Wed, Nov 29, 2017 at 10:50 AM, Alexander Wels 
>> wrote:
>>
>>> On Wednesday, November 29, 2017 10:16:27 AM EST Martin Sivak wrote:
>>> > Thanks to you too for checking.
>>> >
>>> > Martin
>>> >
>>>
>>> Fix is pending review [1]. We moved the main views to a different class
>>> hierarchy and it wasn't inheriting the methods to add the buttons.
>>>
>>> Alexander
>>>
>>> [1] https://gerrit.ovirt.org/c/84901/
>>>
>>> > On Wed, Nov 29, 2017 at 3:07 PM, Greg Sheremeta 
>>> wrote:
>>> > > Discussed with Alexander, and yep this is an issue.
>>> > > I opened https://bugzilla.redhat.com/show_bug.cgi?id=1518724
>>> > >
>>> > > Thanks for checking and reporting!
>>> > >
>>> > > On Wed, Nov 29, 2017 at 8:35 AM, Greg Sheremeta >> >
>>> > >
>>> > > wrote:
>>> > >> +Alexander, can you take a look?
>>> > >>
>>> > >> On Wed, Nov 29, 2017 at 8:33 AM, Martin Sivak 
>>> wrote:
>>> > >>> Hi,
>>> > >>>
>>> > >>> I think I found one issue.. I do not see the extra menu items that
>>> were
>>> > >>> supposed to be added to VMs (used to be available in context menu
>>> as
>>> > >>> well).
>>> > >>>
>>> > >>> You can check it with
>>> > >>>
>>> > >>> http://jenkins.ovirt.org/job/ovirt-optimizer_master_check-pa
>>> > >>> tch-el7-x86_64/65/
>>> > >>>
>>> > >>> Martin
>>> > >>>
>>> > >>> On Wed, Nov 29, 2017 at 1:22 AM, Greg Sheremeta <
>>> gsher...@redhat.com>
>>> > >>>
>>> > >>> wrote:
>>> >  Hey,
>>> > 
>>> >  We haven't changed the API, so everything should still work. For
>>> >  example, I just installed 4.1 versions of dashboard and
>>> support-plugin
>>> >  in
>>> >  master, and all's well [*]:
>>> > 
>>> >  API:
>>> > 
>>> >  api.addSubTab('Template', 'Red Hat Documentation',
>>> >  'my-host-subtab-template', '', {alignRight: true});
>>> >  api.setTabAccessible('my-host-subtab-template', true);
>>> > 
>>> >  [image: Inline image 1]
>>> > 
>>> >  [*] however, note that "alignRight" is now ignored
>>> > 
>>> >  And dashboard 4.1.8 installed in master engine:
>>> > 
>>> >  [image: Inline image 2]
>>> > 
>>> > 
>>> >  That said, we probably should have sent some announcement about
>>> making
>>> >  sure the plugins actually look good with the new theme. If some
>>> plugins
>>> >  are
>>> >  greenish, that will look bad.
>>> > 
>>> >  To help with styling external things, we've started exposing more
>>> CSS
>>> >  via a new SASS brand module:
>>> >  https://github.com/oVirt/ovirt-engine/tree/master/frontend/b
>>> >  rands/ovirt-brand/src/main/sass
>>> >  Also, we're heavily relying on base PatternFly styles, and plugins
>>> >  should do the same. For example, the new dialog styles are
>>> completely
>>> >  PatternFly [http://www.patternfly.org/pat
>>> tern-library/forms-and-control
>>> >  s/modal-overlay/] and we don't do anything extra on top of that.
>>> > 
>>> >  Let us know if something doesn't work, or you need help styling
>>> >  something.
>>> > 
>>> >  Greg
>>> > 
>>> >  On Tue, Nov 28, 2017 at 6:25 PM, Martin Sivak 
>>> > 
>>> >  wrote:
>>> > > Hi,
>>> > >
>>> > > I just got a question from an ovirt-optimizer user about the
>>> support
>>> > > in 4.2. And I realized I haven't heard anything about how UI
>>> plugins
>>> > > should be updated to work with the new UI.
>>> > >
>>> > > Is there anything special that needs to be done to make the
>>> plugin
>>> > > functional? Or will everything still work somehow?
>>> > >
>>> > > Best regards
>>> > >
>>> > > Martin Sivak
>>> > > ___
>>> > > Devel mailing list
>>> > > Devel@ovirt.org
>>> > > http://lists.ovirt.org/mailman/listinfo/devel
>>> > >>
>>> > >> --
>>> > >>
>>> > >> GREG SHEREMETA
>>> > >>
>>> > >> SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX
>>> > >>
>>> > >> Red Hat NA
>>> > >>
>>> > >> 
>>> > >>
>>> > >> gsher...@redhat.comIRC: gshereme
>>> > >> 
>>> > >
>>> > > --
>>> > >
>>> > > GREG SHEREMETA
>>> > >
>>> > > SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX
>>> > >
>>> > > Red Hat NA
>>> > >
>>> > > 
>>> > >
>>> > > gsher...@redhat.comIRC: gshereme
>>> > > 
>>>
>>>
>>>
>>
>>
>> --
>>
>> GREG SHEREMETA
>>
>> SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX
>>
>> Red Hat NA
>>
>> 
>>
>>

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ 20-11-2017 ] [ 001_initialize_engine.test_initialize_engine ]

2017-11-30 Thread Dafna Ron 
On 11/30/2017 03:03 PM, Yedidyah Bar David wrote:
> On Wed, Nov 29, 2017 at 4:57 PM, Yedidyah Bar David  wrote:
>> On Wed, Nov 29, 2017 at 3:56 PM, Dafna Ron  wrote:
>>> we had a failure on 002_bootstrap.verify_add_hosts but the error is on
>>> imageio
>>>
>>> I looked at the host log that Nir added and I can only see that the
>>> address is in use which seems to be the same issue we have in initialize
>>> engine.
>>>
>>>
>>> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4205/artifact/exported-artifacts/basic-suit-master-el7/test_logs/basic-suite-master/post-002_bootstrap.py/lago-basic-suite-master-host-0/_var_log/ovirt-imageio-daemon/
>>>
>>> I cannot see anything in host-deploy.
>>> Didi, would we be able to see anything here?
>>
>> Sorry, seems like my plugin is not enough. Will have a look.
> Now merged an updated plugin, should hopefully pass changequeue soon.
> Let's see what happens next time a service fails.
> Search engine-setup/host-deploy logs for 'tcp connections'.
>
> Best regards,
thank you :)
will update once we have another failure.
>>>
>>> Thanks,
>>> Dafna
>>>
>>>
>>>
>>> On 11/29/2017 11:03 AM, Yedidyah Bar David wrote:
>>>
>>> On Wed, Nov 29, 2017 at 1:00 PM, Dafna Ron  wrote:
 this is the plugin info from steup log but I don't see anything more than
 we have seen except a timeout.

 https://pastebin.com/QVtNRNWV


 http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/artifact/exported-artifacts/upgrade-from-release-suit-master-el7/test_logs/upgrade-from-release-suite-master/post-001_initialize_engine.py/lago-upgrade-from-release-suite-master-engine/_var_log/ovirt-engine/setup/ovirt-engine-setup-20171128123116-mmjen3.log

 Didi, is there anywhere else I should look?
>>>
>>> Sadly, as already replied, not yet. Hopefully next time...
>>>


 On 11/29/2017 10:18 AM, Nir Soffer wrote:

 Do we have more info from Didi's debug plugin now?

 On Wed, Nov 29, 2017 at 12:07 PM Dafna Ron  wrote:
> Hi,
>
> We have failed cq with ovirt-imageio failing to start on upgrade suite.
> I can still only see errors in the messages log.
>
> I'm writing the reported patch but I don't think it has anything to do
> with this issue.
>
> Link and headline of suspected patches:
>
> restapi: Enable update to no default network provider of cluster -
> https://gerrit.ovirt.org/#/c/84814/
>
> Link to Job:
>
> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/
>
> Link to all logs:
>
>
> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/artifact/
>
>
> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/testReport/junit/(root)/001_initialize_engine/test_initialize_engine/
>
>
> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/artifact/exported-artifacts/upgrade-from-release-suit-master-el7/test_logs/upgrade-from-release-suite-master/post-001_initialize_engine.py/lago-upgrade-from-release-suite-master-engine/_var_log/messages/*view*/
>
> (Relevant) error snippet from the log:
>
> 
>
> From messages log
>
>
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Started oVirt Engine.
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Reloading.
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Configuration file /usr/lib/systemd/system/ebtables.service is marked
> executable. Please remove executable permission bits. Proceeding anyway.
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Starting oVirt Engine Data Warehouse...
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Started oVirt Engine Data Warehouse.
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Reloading.
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Configuration file /usr/lib/systemd/system/ebtables.service is marked
> executable. Please remove executable permission bits. Proceeding anyway.
> Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
> Starting oVirt ImageIO Proxy...
> Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
> ovirt-imageio-proxy: Traceback (most recent call last):
> Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
> ovirt-imageio-proxy: File "/usr/bin/ovirt-imageio-proxy", line 85, in
> 
> Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
> ovirt-imageio-proxy: status = image_proxy.main(args, config)
> Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
> ovirt-imageio-proxy: File
>

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ 20-11-2017 ] [ 001_initialize_engine.test_initialize_engine ]

2017-11-30 Thread Yedidyah Bar David 
On Wed, Nov 29, 2017 at 4:57 PM, Yedidyah Bar David  wrote:
> On Wed, Nov 29, 2017 at 3:56 PM, Dafna Ron  wrote:
>>
>> we had a failure on 002_bootstrap.verify_add_hosts but the error is on
>> imageio
>>
>> I looked at the host log that Nir added and I can only see that the
>> address is in use which seems to be the same issue we have in initialize
>> engine.
>>
>>
>> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4205/artifact/exported-artifacts/basic-suit-master-el7/test_logs/basic-suite-master/post-002_bootstrap.py/lago-basic-suite-master-host-0/_var_log/ovirt-imageio-daemon/
>>
>> I cannot see anything in host-deploy.
>> Didi, would we be able to see anything here?
>
>
> Sorry, seems like my plugin is not enough. Will have a look.

Now merged an updated plugin, should hopefully pass changequeue soon.
Let's see what happens next time a service fails.
Search engine-setup/host-deploy logs for 'tcp connections'.

Best regards,

>
>>
>>
>> Thanks,
>> Dafna
>>
>>
>>
>> On 11/29/2017 11:03 AM, Yedidyah Bar David wrote:
>>
>> On Wed, Nov 29, 2017 at 1:00 PM, Dafna Ron  wrote:
>>>
>>> this is the plugin info from steup log but I don't see anything more than
>>> we have seen except a timeout.
>>>
>>> https://pastebin.com/QVtNRNWV
>>>
>>>
>>> http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/artifact/exported-artifacts/upgrade-from-release-suit-master-el7/test_logs/upgrade-from-release-suite-master/post-001_initialize_engine.py/lago-upgrade-from-release-suite-master-engine/_var_log/ovirt-engine/setup/ovirt-engine-setup-20171128123116-mmjen3.log
>>>
>>> Didi, is there anywhere else I should look?
>>
>>
>> Sadly, as already replied, not yet. Hopefully next time...
>>
>>>
>>>
>>>
>>> On 11/29/2017 10:18 AM, Nir Soffer wrote:
>>>
>>> Do we have more info from Didi's debug plugin now?
>>>
>>> On Wed, Nov 29, 2017 at 12:07 PM Dafna Ron  wrote:

 Hi,

 We have failed cq with ovirt-imageio failing to start on upgrade suite.
 I can still only see errors in the messages log.

 I'm writing the reported patch but I don't think it has anything to do
 with this issue.

 Link and headline of suspected patches:

 restapi: Enable update to no default network provider of cluster -
 https://gerrit.ovirt.org/#/c/84814/

 Link to Job:

 http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/

 Link to all logs:


 http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/artifact/


 http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/testReport/junit/(root)/001_initialize_engine/test_initialize_engine/


 http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/4194/artifact/exported-artifacts/upgrade-from-release-suit-master-el7/test_logs/upgrade-from-release-suite-master/post-001_initialize_engine.py/lago-upgrade-from-release-suite-master-engine/_var_log/messages/*view*/

 (Relevant) error snippet from the log:

 

 From messages log


 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Started oVirt Engine.
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Reloading.
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Configuration file /usr/lib/systemd/system/ebtables.service is marked
 executable. Please remove executable permission bits. Proceeding anyway.
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Starting oVirt Engine Data Warehouse...
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Started oVirt Engine Data Warehouse.
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Reloading.
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Configuration file /usr/lib/systemd/system/ebtables.service is marked
 executable. Please remove executable permission bits. Proceeding anyway.
 Nov 28 12:32:13 lago-upgrade-from-release-suite-master-engine systemd:
 Starting oVirt ImageIO Proxy...
 Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
 ovirt-imageio-proxy: Traceback (most recent call last):
 Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
 ovirt-imageio-proxy: File "/usr/bin/ovirt-imageio-proxy", line 85, in
 
 Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
 ovirt-imageio-proxy: status = image_proxy.main(args, config)
 Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
 ovirt-imageio-proxy: File
 "/usr/lib/python2.7/site-packages/ovirt_imageio_proxy/image_proxy.py", line
 21, in main
 Nov 28 12:32:14 lago-upgrade-from-release-suite-master-engine
 ovirt-imageio-proxy: image_server.start(config)
 Nov 28 12:32:14

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Martin Sivak 
>
>> 1517810 
>>> ovirt-engine stira...@redhat.com Adding additional ha-host fails.
>>>
>>
> This one has a verified engine fix now, we just need to merge it and
> update Node 0 setup (also verified).
>

Merged.
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Martin Sivak 
On Thu, Nov 30, 2017 at 11:28 AM, Martin Sivak  wrote:

> I think a reasonable alternative is to revert the change, adjust HE to use
>> both formats, and then introduce the new format. The first step (revert)
>> should be done for 4.2.0 (today?) and the rest for 4.2.1.
>> Y.
>>
>>
> The format is still the same, we just needed to autodetect the used NS and
> then query for attributes using the detected namespace. Patch posted, tests
> are passing, but since we want to be 100% sure we will test with OST and
> Node 0 before pushing it. Should not take long.
>


Verified and Merged.


> Martin
>
>
>
>>
>>>


>
> 1516113 
>> cockpit-ovirt phbai...@redhat.com POST Deploy the HostedEngine
>> failed with the default CPU type
>>
>
> Would be happy if the remaining patch could get reviewed quickly.
>




>
> 1518693 
>> ovirt-engine akrej...@redhat.com POST Quota is needed to copy
>> template disk
>>
>
> This is only via REST and the default quota can be used as a
> workaround - why is this a blocker?
>

 Automation added it because Raz marked it as Regression. But the change
 was intentional.



> 1517810 
>> ovirt-engine stira...@redhat.com Adding additional ha-host fails.
>>
>
 This one has a verified engine fix now, we just need to merge it and
 update Node 0 setup (also verified).


 Martin


 ___
 Devel mailing list
 Devel@ovirt.org
 http://lists.ovirt.org/mailman/listinfo/devel

>>>
>>>
>>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Martin Sivak 
>
> I think a reasonable alternative is to revert the change, adjust HE to use
> both formats, and then introduce the new format. The first step (revert)
> should be done for 4.2.0 (today?) and the rest for 4.2.1.
> Y.
>
>
The format is still the same, we just needed to autodetect the used NS and
then query for attributes using the detected namespace. Patch posted, tests
are passing, but since we want to be 100% sure we will test with OST and
Node 0 before pushing it. Should not take long.

Martin



>
>>
>>>
>>>

 1516113 
> cockpit-ovirt phbai...@redhat.com POST Deploy the HostedEngine failed
> with the default CPU type
>

 Would be happy if the remaining patch could get reviewed quickly.

>>>
>>>
>>>
>>>

 1518693 
> ovirt-engine akrej...@redhat.com POST Quota is needed to copy
> template disk
>

 This is only via REST and the default quota can be used as a workaround
 - why is this a blocker?

>>>
>>> Automation added it because Raz marked it as Regression. But the change
>>> was intentional.
>>>
>>>
>>>
 1517810 
> ovirt-engine stira...@redhat.com Adding additional ha-host fails.
>

>>> This one has a verified engine fix now, we just need to merge it and
>>> update Node 0 setup (also verified).
>>>
>>>
>>> Martin
>>>
>>>
>>> ___
>>> Devel mailing list
>>> Devel@ovirt.org
>>> http://lists.ovirt.org/mailman/listinfo/devel
>>>
>>
>>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Michal Skrivanek 

> On 30 Nov 2017, at 05:13, Yaniv Kaul  wrote:
> 
> 
> 
> On Thu, Nov 30, 2017 at 11:38 AM, Arik Hadas  > wrote:
> 
> 
> On Thu, Nov 30, 2017 at 11:10 AM, Martin Sivak  > wrote:
> 
> 
> Bug IDProduct AssigneeStatus  Summary
> 1518887  
> ovirt-hosted-engine-ha  b...@ovirt.org   NEW 
> ovirt-ha-agent fails parsing the OVF_STORE due to a change in OVF namespace 
> URI
> 
> I'm in favor of reverting the virt change personally.
> 
> Unless something else depends on it, the commit message said vdsm needs this.
> 
> vdsm needs it because we would like vdsm to be able to parse OVA files that 
> are generated by us and OVA files that  are generated by others (and are 
> VMware-compatbile) with the existing code in vdsm. The existing code is 
> tailored to VMware-compatible OVA files that are generated by others, in 
> which the uri doesn't include that slash at the end.
> 
> It would be best to adjust ovirt-ha-agent to parse the right uri.
> However, it that's too complicated, an alternative solution is to keep 
> writing the previous uri to ovirt's OVFs, i.e., those in OVF_STORE and in 
> snapshot's configuration. That would be a pitty since we want to minimize the 
> differences between the OVFs we generate, but it would be better than 
> reverting the change..
> 
> I think a reasonable alternative is to revert the change, adjust HE to use 
> both formats, and then introduce the new format. The first step (revert) 
> should be done for 4.2.0 (today?) and the rest for 4.2.1.

It shouldn’t take too long to polish [1] and rebuild ovirt-hosted-engine-ha
Thanks Martin for posting it!

Thanks,
michal

[1] https://gerrit.ovirt.org/#/c/84933/

> Y.
> 
>  
>  
> 
> 1516113  cockpit-ovirt   
> phbai...@redhat.com POSTDeploy the 
> HostedEngine failed with the default CPU type
> 
> Would be happy if the remaining patch could get reviewed quickly.
> 
> 
>  
> 
> 1518693  ovirt-engine
> akrej...@redhat.com POSTQuota is 
> needed to copy template disk
> 
> This is only via REST and the default quota can be used as a workaround - why 
> is this a blocker?
> 
> Automation added it because Raz marked it as Regression. But the change was 
> intentional.
>  
>  
> 1517810  ovirt-engine
> stira...@redhat.com Adding additional 
> ha-host fails.
> 
> This one has a verified engine fix now, we just need to merge it and update 
> Node 0 setup (also verified).
> 
> 
> Martin
> 
> 
> ___
> Devel mailing list
> Devel@ovirt.org 
> http://lists.ovirt.org/mailman/listinfo/devel 
> 
> 
> 
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel

___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Yaniv Kaul 
On Thu, Nov 30, 2017 at 11:38 AM, Arik Hadas  wrote:

>
>
> On Thu, Nov 30, 2017 at 11:10 AM, Martin Sivak  wrote:
>
>>

 Bug ID Product Assignee Status Summary
 1518887 
 ovirt-hosted-engine-ha b...@ovirt.org NEW ovirt-ha-agent fails parsing
 the OVF_STORE due to a change in OVF namespace URI

>>>
>>> I'm in favor of reverting the virt change personally.
>>>
>>
>> Unless something else depends on it, the commit message said vdsm needs
>> this.
>>
>
> vdsm needs it because we would like vdsm to be able to parse OVA files
> that are generated by us and OVA files that  are generated by others (and
> are VMware-compatbile) with the existing code in vdsm. The existing code is
> tailored to VMware-compatible OVA files that are generated by others, in
> which the uri doesn't include that slash at the end.
>
> It would be best to adjust ovirt-ha-agent to parse the right uri.
> However, it that's too complicated, an alternative solution is to keep
> writing the previous uri to ovirt's OVFs, i.e., those in OVF_STORE and in
> snapshot's configuration. That would be a pitty since we want to minimize
> the differences between the OVFs we generate, but it would be better than
> reverting the change..
>

I think a reasonable alternative is to revert the change, adjust HE to use
both formats, and then introduce the new format. The first step (revert)
should be done for 4.2.0 (today?) and the rest for 4.2.1.
Y.


>
>>
>>
>>>
>>> 1516113 
 cockpit-ovirt phbai...@redhat.com POST Deploy the HostedEngine failed
 with the default CPU type

>>>
>>> Would be happy if the remaining patch could get reviewed quickly.
>>>
>>
>>
>>
>>
>>>
>>> 1518693 
 ovirt-engine akrej...@redhat.com POST Quota is needed to copy template
 disk

>>>
>>> This is only via REST and the default quota can be used as a workaround
>>> - why is this a blocker?
>>>
>>
>> Automation added it because Raz marked it as Regression. But the change
>> was intentional.
>>
>>
>>
>>> 1517810 
 ovirt-engine stira...@redhat.com Adding additional ha-host fails.

>>>
>> This one has a verified engine fix now, we just need to merge it and
>> update Node 0 setup (also verified).
>>
>>
>> Martin
>>
>>
>> ___
>> Devel mailing list
>> Devel@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/devel
>>
>
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Arik Hadas 
On Thu, Nov 30, 2017 at 11:10 AM, Martin Sivak  wrote:

>
>>>
>>> Bug ID Product Assignee Status Summary
>>> 1518887 
>>> ovirt-hosted-engine-ha b...@ovirt.org NEW ovirt-ha-agent fails parsing
>>> the OVF_STORE due to a change in OVF namespace URI
>>>
>>
>> I'm in favor of reverting the virt change personally.
>>
>
> Unless something else depends on it, the commit message said vdsm needs
> this.
>

vdsm needs it because we would like vdsm to be able to parse OVA files that
are generated by us and OVA files that  are generated by others (and are
VMware-compatbile) with the existing code in vdsm. The existing code is
tailored to VMware-compatible OVA files that are generated by others, in
which the uri doesn't include that slash at the end.

It would be best to adjust ovirt-ha-agent to parse the right uri.
However, it that's too complicated, an alternative solution is to keep
writing the previous uri to ovirt's OVFs, i.e., those in OVF_STORE and in
snapshot's configuration. That would be a pitty since we want to minimize
the differences between the OVFs we generate, but it would be better than
reverting the change..


>
>
>>
>> 1516113 
>>> cockpit-ovirt phbai...@redhat.com POST Deploy the HostedEngine failed
>>> with the default CPU type
>>>
>>
>> Would be happy if the remaining patch could get reviewed quickly.
>>
>
>
>
>
>>
>> 1518693 
>>> ovirt-engine akrej...@redhat.com POST Quota is needed to copy template
>>> disk
>>>
>>
>> This is only via REST and the default quota can be used as a workaround -
>> why is this a blocker?
>>
>
> Automation added it because Raz marked it as Regression. But the change
> was intentional.
>
>
>
>> 1517810 
>>> ovirt-engine stira...@redhat.com Adding additional ha-host fails.
>>>
>>
> This one has a verified engine fix now, we just need to merge it and
> update Node 0 setup (also verified).
>
>
> Martin
>
>
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Martin Sivak 
>
>
>>
>> Bug ID Product Assignee Status Summary
>> 1518887 
>> ovirt-hosted-engine-ha b...@ovirt.org NEW ovirt-ha-agent fails parsing
>> the OVF_STORE due to a change in OVF namespace URI
>>
>
> I'm in favor of reverting the virt change personally.
>

Unless something else depends on it, the commit message said vdsm needs
this.


>
> 1516113 
>> cockpit-ovirt phbai...@redhat.com POST Deploy the HostedEngine failed
>> with the default CPU type
>>
>
> Would be happy if the remaining patch could get reviewed quickly.
>




>
> 1518693  ovirt-engine
>> akrej...@redhat.com POST Quota is needed to copy template disk
>>
>
> This is only via REST and the default quota can be used as a workaround -
> why is this a blocker?
>

Automation added it because Raz marked it as Regression. But the change was
intentional.



> 1517810  ovirt-engine
>> stira...@redhat.com Adding additional ha-host fails.
>>
>
This one has a verified engine fix now, we just need to merge it and update
Node 0 setup (also verified).


Martin
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] [OST Failure] [oVirt Master] [HC] Hosted engine fails to install

2017-11-30 Thread Yaniv Kaul 
On Thu, Nov 30, 2017 at 8:58 AM, Sahina Bose  wrote:

> Hi,
>
> The error with HE install is :  Starting vdsmd", "[ ERROR ] Failed to
> execute stage 'Misc configuration': Couldn't  connect to VDSM within 15
> seconds".
>
> Is there a configuration parameter that needs to be set to change the
> timeout, or is this a bug?
>

>From the log it doesn't seem it even waited 15 secs:
2017-11-29 21:45:03,031-0500 DEBUG otopi.plugins.otopi.services.systemd
plugin.executeRaw:813 execute: ('/usr/bin/systemctl', 'start',
'vdsmd.service'), executable='None', cwd='None', env=None
2017-11-29 21:45:05,881-0500 DEBUG otopi.plugins.otopi.services.systemd
plugin.executeRaw:863 execute-result: ('/usr/bin/systemctl', 'start',
'vdsmd.service'), rc=0
2017-11-29 21:45:05,882-0500 DEBUG otopi.plugins.otopi.services.systemd
plugin.execute:921 execute-output: ('/usr/bin/systemctl', 'start',
'vdsmd.service') stdout:


2017-11-29 21:45:05,882-0500 DEBUG otopi.plugins.otopi.services.systemd
plugin.execute:926 execute-output: ('/usr/bin/systemctl', 'start',
'vdsmd.service') stderr:


2017-11-29 21:45:07,001-0500 DEBUG otopi.plugins.gr_he_setup.system.vdsmenv
util.__log_debug:374 VDSM jsonrpc connection is not ready
2017-11-29 21:45:07,002-0500 DEBUG otopi.plugins.gr_he_setup.system.vdsmenv
util.__log_debug:374 Creating a new json-rpc connection to VDSM
2017-11-29 21:45:07,202-0500 DEBUG otopi.plugins.gr_he_setup.system.vdsmenv
util.__log_debug:374 VDSM jsonrpc connection is not ready
2017-11-29 21:45:07,203-0500 DEBUG otopi.context context._executeMethod:143
method exception
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/otopi/context.py", line 133, in
_executeMethod
method['method']()
  File
"/usr/share/ovirt-hosted-engine-setup/scripts/../plugins/gr-he-setup/system/vdsmenv.py",
line 158, in _misc
timeout=ohostedcons.Const.VDSCLI_SSL_TIMEOUT,
  File
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/lib/util.py", line
442, in connect_vdsm_json_rpc
__vdsm_json_rpc_connect(logger, timeout)
  File
"/usr/lib/python2.7/site-packages/ovirt_hosted_engine_ha/lib/util.py", line
398, in __vdsm_json_rpc_connect
timeout=VDSM_MAX_RETRY * VDSM_DELAY
RuntimeError: Couldn't  connect to VDSM within 15 seconds
2017-11-29 21:45:07,204-0500 ERROR otopi.context context._executeMethod:152
Failed to execute stage 'Misc configuration': Couldn't  connect to VDSM
within 15 seconds
2017-11-29 21:45:07,205-0500 DEBUG otopi.transaction transaction.abort:119
aborting 'Yum Transaction'
2017-11-29 21:45:07,205-0500 INFO otopi.plugins.otopi.packagers.yumpackager
yumpackager.info:80 Yum Performing yum transaction rollback



> Logs at : http://jenkins.ovirt.org/job/ovirt-system-tests_hc-basic-
> suite-master/107/artifact/exported-artifacts/test_logs/
> hc-basic-suite-master/post-002_bootstrap.py/lago-hc-
> basic-suite-master-host0/_var_log/ovirt-hosted-engine-setup/
> ovirt-hosted-engine-setup-20171129214218-7vns3t.log
>
> thanks
> sahina
>
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] oVirt 4.2.0 blockers review - Day 3

2017-11-30 Thread Yaniv Kaul 
On Thu, Nov 30, 2017 at 9:49 AM, Sandro Bonazzola 
wrote:

> Hi,
> we still have 4 open acknowledged blockers according to
> https://bugzilla.redhat.com/buglist.cgi?quicksearch=
> flag%3Ablocker%2B%20target_milestone%3Aovirt-4.2.0%
> 20status%3Anew%2Cassigned%2Cpost
>
> Bug ID Product Assignee Status Summary
> 1518887 
> ovirt-hosted-engine-ha b...@ovirt.org NEW ovirt-ha-agent fails parsing
> the OVF_STORE due to a change in OVF namespace URI
>

I'm in favor of reverting the virt change personally.

1516113  cockpit-ovirt
> phbai...@redhat.com POST Deploy the HostedEngine failed with the default
> CPU type
>

Would be happy if the remaining patch could get reviewed quickly.

1518693  ovirt-engine
> akrej...@redhat.com POST Quota is needed to copy template disk
>

This is only via REST and the default quota can be used as a workaround -
why is this a blocker?

1507277  ovirt-engine
> era...@redhat.com POST [RFE][DR] - Vnic Profiles mapping in VMs register
> from data storage domain should be supported also for templates
>

I'm not sure why it's a blocker for 4.2.0.


> There are also 3 proposed blockers that need either to be acknowledged or
> rejected: https://bugzilla.redhat.com/buglist.cgi?
> quicksearch=flag%3Ablocker%3F%20target_milestone%3Aovirt-4.
> 2.0%20status%3Anew%2Cassigned%2Cpost
>
> Bug ID Product Assignee Summary
> 1450061  Red Hat
> Enterprise Virtualization Manager rh-spice-b...@redhat.com Copy-paste:
> filename encoding in Win guest
>

Moved to 4.2.1. The Spice team did not touch it for quite some time since
it was opened.

1517810  ovirt-engine
> stira...@redhat.com Adding additional ha-host fails.
> 1502920  Red Hat
> Enterprise Virtualization Manager rba...@redhat.com File missing after
> upgrade of RHVH node from version RHVH-4.1-20170925.0 to latest.
>

Stuck on POST since2017-10-20 00:47:07 IDT ?


>
> Given current status I would reschedule oVirt 4.2.0 RC to next week,
> tentatively on Monday.
>

I'd review the status tomorrow as well - perhaps we can build the RC
tomorrow. I don't see too many blockers that cannot be completed today
above.
Y.

>
> --
>
> SANDRO BONAZZOLA
>
> ASSOCIATE MANAGER, SOFTWARE ENGINEERING, EMEA ENG VIRTUALIZATION R
>
> Red Hat EMEA 
> 
> TRIED. TESTED. TRUSTED. 
>
>
> ___
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel