Re: [ovirt-devel] Subject: Looking for advice regarding user portal development.

[Greg Padgett]

On 12/11/2015 05:07 PM, Alexander Wels wrote:

[...] Note as far as I know
there is currently no way to upload templates with either the User Portal or
the REST api. So you will have to find some other mechanism to get them into
the system. And when I say templates I mean VM templates in the oVirt sense.


Hi Thomas,

More on Alexander's point above, we are actively working on a project [1] 
to allow uploading images into the oVirt environment via WebAdmin and/or 
the REST API.  Some parts may be useful to you despite it not being 
targeted for inclusion in the User Portal at this time.


Alternatively, using a Glance Provider [2] may also help with your goals, 
though I'm not too familiar with the details.


Greg

[1] http://www.ovirt.org/Features/Image_Upload
[2] http://ovedou.blogspot.com/2014/03/importing-glance-images-as-ovirt.html

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

[ovirt-devel] Making GWT UI build logs a bit less confusing

[Vojtech Szocs]

Hi guys,

I wrote https://gerrit.ovirt.org/#/c/50545/ and would like to hear your 
feedback.

(Many times, people are confused by those [ERROR]'s which are not really 
errors.)

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

Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on master

[Einav Cohen]

> (a) such patch cannot be merged because a unit-test, *even if ran during
> compilation for English only*, should fail (see [1]).

update: after discussing with Alexander: the unit-test doesn't 
cover the UIMessages file - that's why it was not detectable 
when compiling in English only. 

@Alexander - can you please extend the unit-test to include UIMessages?

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

- Original Message -
> From: "Einav Cohen" 
> To: "Tal Nisan" , "Alexander Wels" , 
> "Tomas Jelinek" 
> Cc: "devel" 
> Sent: Tuesday, December 15, 2015 7:30:40 AM
> Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on 
> master
> 
> > - Original Message -
> > From: "Tal Nisan" 
> > Sent: Tuesday, December 15, 2015 5:29:17 AM
> > 
> > Einav, it seems like an old issue we had when the default message contains
> > the necessary placeholders but since no translations patch was released yet
> > we don't have the placeholders in the translated messages, did we decide on
> > a workaround back then?
> 
> (a) such patch cannot be merged because a unit-test, *even if ran during
> compilation for English only*, should fail (see [1]).
> 
> @Alexander - can you please take a look at it with Tomas and see what
> happened (original patch merged without running unit-tests / problem
> with the unit-test itself [1] / ...)?
> 
> (b) non-English .properties files should typically not be modified as part
> of regular patches (i.e. patches that are not dedicated "translation
> updates" which include updates to the files pulled directly from Zanata).
> 
> *However*, there is one exception for that, and the exception is compilation
> error that happens due to the reason that you mentioned above - changes in
> English message variable-placeholders.
> In this case - the patch author must change the matching messages in the non-
> English .properties files to include the exact same variable-placeholders as
> the (updated) English message (i.e. remove placeholders / add placeholders
> as necessary).
> 
> [1] http://lists.ovirt.org/pipermail/engine-commits/2014-October/018907.html
> 
> > 
> > On Tue, Dec 15, 2015 at 12:04 PM, Sandro Bonazzola 
> > wrote:
> > 
> > > *05:22:42* [INFO]   [ERROR] Required argument 3 not
> > > present: {0} ({1} Sockets, {2} Kerne pro Socket)*05:22:42* [INFO]
> > > [ERROR] Errors in
> > > 'org/ovirt/engine/ui/uicompat/ConstantsManager.java'*05:22:42* [INFO]
> > > [ERROR] Line 38: Failed to resolve
> > > 'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding
> > >
> > >
> > > See
> > > http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console
> > >
> > >
> > > Please fix ASAP
> > >
> > >
> > >
> > > --
> > > Sandro Bonazzola
> > > Better technology. Faster innovation. Powered by community collaboration.
> > > See how it works at redhat.com
> > >
> > > ___
> > > 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-users] [QE] oVirt 3.6.1 Hosted Engine test day

[Simone Tiraboschi]

On Mon, Dec 14, 2015 at 4:39 PM, Gianluca Cecchi 
wrote:

> On Mon, Dec 14, 2015 at 4:10 PM, Sandro Bonazzola  wrote:
>
>>
>>>
>>> You should just activate it.
>>> Adding Roy, looks like this doesn't happen to me only.
>>>
>>>
>> See *Bug 1290518*  -
>>  failed Activating hosted engine domain during auto-import on NFS
>>
>>
> I confirm that I hit the bug and I was able to simply activate the storage
> domain and able to see the engine vm
>
> Dec 14, 2015 4:22:23 PM Hosted Engine VM was imported successfully
> Dec 14, 2015 4:22:23 PM Starting to import Vm HostedEngine to Data Center
> Default, Cluster Default
> Dec 14, 2015 4:22:19 PM Storage Domain hosted_storage (Data Center
> Default) was activated by admin@internal
>
>
> NOTES:
> 1) I configured the engine vm as a vnc one during install (in 3.6.0, as
> the appliance was the 3.6.0 one as we noted)
>
> [ INFO  ] Stage: Setup validation
>
>   --== CONFIGURATION PREVIEW ==--
>
>   Bridge interface   : eth0
>   Engine FQDN: shengine.localdomain.local
>   Bridge name: ovirtmgmt
>   SSH daemon port: 22
>   Firewall manager   : iptables
>   Gateway address: 192.168.122.1
>   Host name for web application  : hosted_engine_1
>   Host ID: 1
>   Image size GB  : 10
>   GlusterFS Share Name   : hosted_engine_glusterfs
>   GlusterFS Brick Provisioning   : False
>   Storage connection :
> ovc71.localdomain.local:/SHE_DOMAIN
>   Console type   : vnc
>   Memory size MB : 8192
>   MAC address: 00:16:3e:72:e7:26
>   Boot type  : disk
>   Number of CPUs : 1
>   OVF archive (for disk boot):
> /usr/share/ovirt-engine-appliance/ovirt-engine-appliance-3.6-20151211.1.el7.centos.ova
>   Restart engine VM after engine-setup: True
>   CPU Type   : model_Nehalem
>
>   Please confirm installation settings (Yes, No)[Yes]:
>
> but after import in 3.6.1 I see it configured as with spice console in web
> admin gui.
> I indeed am able to access it via spice, but just to notice that if one
> eventually would like to have it vnc, he/she has to edit the engine vm
> after update to 3.6.1.
> BTW: is it possible/supported to edit engine vm from the gui and change
> for example the console mode?
>
> 2) During setup, see above, it was configured with iptables as firewall
> manager
> During the update to 3.6.1 this was the workflow shown at video:
>
> [root@shengine ~]# engine-setup
> [ INFO  ] Stage: Initializing
> [ INFO  ] Stage: Environment setup
>   Configuration files:
> ['/etc/ovirt-engine-setup.conf.d/10-packaging-jboss.conf',
> '/etc/ovirt-engine-setup.conf.d/10-packaging.conf',
> '/etc/ovirt-engine-setup.conf.d/20-setup-ovirt-post.conf']
>   Log file:
> /var/log/ovirt-engine/setup/ovirt-engine-setup-20151214134331-ga1lxy.log
>   Version: otopi-1.4.0 (otopi-1.4.0-1.el7.centos)
> [ INFO  ] Stage: Environment packages setup
> [ INFO  ] Stage: Programs detection
> [ INFO  ] Stage: Environment setup
> [ INFO  ] Stage: Environment customization
>
>   --== PRODUCT OPTIONS ==--
>
>
>   --== PACKAGES ==--
>
> [ INFO  ] Checking for product updates...
>   Setup has found updates for some packages:
>   PACKAGE: [updated] ovirt-engine-3.6.0.3-1.el7.centos.noarch
> ...
>   PACKAGE: [update] ovirt-engine-wildfly-overlay-8.0.4-1.el7.noarch
>   do you wish to update them now? (Yes, No) [Yes]:
>  [ INFO  ] Checking for an update for Setup...
>
>   --== ALL IN ONE CONFIGURATION ==--
>
>   --== NETWORK CONFIGURATION ==--
>
>   Setup can automatically configure the firewall on this system.
>   Note: automatic configuration of the firewall may overwrite
> current settings.
>   Do you want Setup to configure the firewall? (Yes, No) [Yes]:
> [ INFO  ] firewalld will be configured as firewall manager.
>
> So the former setup was with iptables durinh sh engine setup in 3.6.0
> while then with firewalld in 3.6.1.
>
> In fact on the engine I see
>
>
> [root@shengine ~]# systemctl status iptables
> iptables.service
>Loaded: not-found (Reason: No such file or directory)
>Active: inactive (dead)
>
> [root@shengine ~]# systemctl status firewalld
> firewalld.service - firewalld - dynamic firewall daemon
>Loaded: loaded (/usr/lib/systemd/system/firewalld.service; enabled)
>Active: active (running) since Mon 2015-12-14 14:19:56 UTC; 1h 15min ago
>  Main PID: 431 (firewalld)
>CGroup: 

Re: [ovirt-devel] [ovirt-users] [QE] oVirt 3.6.1 Hosted Engine test day

[Roman Mohr]

On Mon, Dec 14, 2015 at 8:07 PM, Roy Golan  wrote:

>
>
> On Mon, Dec 14, 2015 at 7:07 PM, Roman Mohr  wrote:
>
>>
>>
>> On Fri, Dec 11, 2015 at 12:45 PM, Roman Mohr  wrote:
>>
>>>
>>>
>>> On Fri, Dec 11, 2015 at 12:41 PM, Sandro Bonazzola 
>>> wrote:
>>>
 Hi,
 oVirt 3.6.1 RC4 included latest bits allowing to auto import Hosted
 Engine storage domain within the Engine.
 In order to have a wider coverage of the Hosted Engine workflow we've
 scheduled a Hosted Engine test day for next week
 on Monday, December 14th.
 Please join us testing Hosted Engine with oVirt 3.6.1 RC4, both on a
 clean install and on an upgrade from 3.5 flow.

>>>
>>> +1
>>>
>>> Will at least do a clean install with nfs on monday.
>>>
>>
>>
>> The hosted-engine-setup part worked for me. But activating the host which
>> was added by engine-setup failed. After the host was added to the cluster I
>> saw that it was in 'Non Operational' state.
>> Searching the logs I found
>>
>> > Message: Host hosted_engine_1 does not comply with the cluster Default
>> emulated machines. The current cluster compatibility level supports
>> [pc-i440fx-rhel7.2.0, pc-i440fx-2.1, pseries-rhel7.2.0] and the host
>> emulated machines are
>> pc-i440fx-rhel7.1.0,rhel6.3.0,pc-q35-rhel7.0.0,rhel6.1.0,rhel6.6.0,
>> rhel6.2.0,pc,pc-q35-rhel7.1.0,q35,rhel6.4.0,rhel6.0.0,rhel6.5.0,pc-i440fx-rhel7.0.0.
>>
>>
> your host emulated machine is "pc-i440fx-rhel7.1.0..." meaning either its
> RHEL 7.1 or qemu version isn't latest.
>
>
Ok thx, after updating qemu on the host I could activate it. Should not the
first host determine which emulated machines are supported in the cluster?

>
> My host is RHEL 7.1. After I tried to klick 'Reset emulated machines' on
>> the default cluster, the engine tried to activate the host but failed with
>> the following SQL exception on the AddExistingFileStorageDomainCommand:
>>
>> > 2015-12-14 16:25:40,336 ERROR
>> [org.ovirt.engine.core.bll.storage.AddExistingFileStorageDomainCommand]
>> (org.ovirt.thread.pool-8-thread-7) [789ca4b] Command
>> 'org.ovirt.engine.core.bll.sto
>> rage.AddExistingFileStorageDomainCommand' failed:
>> CallableStatementCallback; SQL [{call insertstorage_domain_static(?, ?, ?,
>> ?, ?, ?, ?, ?, ?, ?, ?, ?)}]; ERROR: null value in column "s torage"
>> violates not-null constraint
>> [...]
>>
>>
I guess we should not execute an invalid SQL statement when the host does
not comply with the cluster, right?


> Log is attached.
>>
>> I am using the following repo:
>> http://resources.ovirt.org/pub/yum-repo/mirrorlist-ovirt-3.6-pre-el7
>>
>> ovirt-engine version is 3.6.1.3-1.el7.
>>
>>
>>
>>> Thanks,
 --
 Sandro Bonazzola
 Better technology. Faster innovation. Powered by community
 collaboration.
 See how it works at redhat.com

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

>>>
>>> Roman
>>>
>>
>> Best regards,
>> Roman
>>
>> ___
>> Users mailing list
>> us...@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/users
>>
>>
>
Thanks,
Roman
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

[ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on master

[Sandro Bonazzola]

*05:22:42* [INFO]   [ERROR] Required argument 3 not
present: {0} ({1} Sockets, {2} Kerne pro Socket)*05:22:42* [INFO]
[ERROR] Errors in
'org/ovirt/engine/ui/uicompat/ConstantsManager.java'*05:22:42* [INFO]
 [ERROR] Line 38: Failed to resolve
'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding


See 
http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console


Please fix ASAP



-- 
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Proposal: Hystrix for realtime command monitoring

[Roman Mohr]

On Fri, Dec 11, 2015 at 10:28 AM, Roman Mohr  wrote:

> Hi,
>
> a status update, a request and a question:
>
> [...]
>
>>
 A first implementation can be found on gerrit[3].

 The implementation should be almost ready. The new rpms for hystrix are
> also moving forward. I would need some people which can give me karma on
> bodhi. See be
>
>  [...]
>
>> # How to monitor the engine?

 It is as easy as starting a hystrix-dashboard [2] with

   $ git clone https://github.com/Netflix/Hystrix.git
   $ cd Hystrix/hystrix-dashboard
   $ ../gradlew jettyRun


> As part of the hystrix rpms there is now also a 'hystrix-dashboard' rpm.
> Using it is pretty simple. Just install it with 'dnf install
> hystrix-dashboard' and start jetty with 'systemctrl start jetty'. Jetty
> will then listen on 8080 by default (and if you told selinux that jetty is
> allowed to access the network).
>
> [...]
>
>
 # Security?

 In the provided patches the hystrix-metrics-servlet is accessible at
 /ovirt-engine/api/hystrix.stream. It is protected by basic auth but
 accessible
 for everyone who can authenticate. We should probably restrict it to
 admins.

 that would be great if it doesn't require too much work. If it does
 then we can start with enabling/disabling via JMX using Roy's recent patch
 [8]


> Since I had to implement JMX support anyway to enable and disable hystrix
> (disabled by default) I am wondering if I can remove the authentication
> part. There is no sensible data in the hystrix stream and all other
> services like the db health check are not protected either. It would make
> it again a little bit easier to use.
>
>  [...]
>
>> 3) Three unpackaged dependencies: archaius, hystrix-core, hystrix-contrib



> All required packages will be available in rawhide the next few hours. All
> builds on koji succeeded.
> Also all packages for f23 were successfully build.
>
> I would appreciate if some of you find the time to give these f23 pacakges
> some karma:
>
> archaius-0.7.3-3.fc23
>  [9]
> (includes archaius-core and archaius-zookeeper)
> hystrix-1.4.21-4.fc23
>   [10]
> (includes hystrix-core, hystrix-metrics-event-stream and hystrix-dashboard)
>
> On el7 I had to package a little bit more and the final hystrix package
> itself is still missing, but some karma on the first round of packages
> would be very helpful:
>
> archaius-0.4.1-1.el7
> 
> [11] (includes archaius-core)
> mockito-1.9.0-19.el7
>  [12]
> assertj-core-2.2.0-2.el7
>  [13]
> jctools-1.1-0.3.alpha.el7
>  [14]
> rxjava-1.0.13-2.el7
> 
> [15]
>
>

All additional packages for el7 are now also available on testing:

jackson-core-2.6.3-1.el7 [16]

hystrix-1.4.21-5.el7
 [17]

As always, I am grateful for every karma!


>



>>> Since you yesterday volunteered to package them I think this should not
 stop us!:-)

 thanks a lot for the effort, I miss a proper analysis for s long.
 Thanks for stepping up!

 michal


 # References

 [1] https://github.com/Netflix/Hystrix
 [2] https://github.com/Netflix/Hystrix/tree/master/hystrix-dashboard
 [3] https://gerrit.ovirt.org/#/q/topic:hystrix
 [4]
 http://www.nurkiewicz.com/2015/02/storing-months-of-historical-metrics.html
 [5]
 https://github.com/Netflix/Hystrix/wiki/FAQ#what-is-the-processing-overhead-of-using-hystrix
 [5] https://bugzilla.redhat.com/show_bug.cgi?id=1268216
 [6] https://bugzilla.redhat.com/show_bug.cgi?id=1268224
 [7] http://graphite.wikidot.com

 [8] https://gerrit.ovirt.org/#/c/29693/


> [9]   https://bodhi.fedoraproject.org/updates/FEDORA-2015-3ae4cc39c5
> [10] https://bodhi.fedoraproject.org/updates/FEDORA-2015-35994552ed
> [11] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-dd72806724
> [12] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-7bf9b82936
> [13] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-f02466a5da
> [14] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-27b59f8bf2
> [15] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-37400bf69d
>
[16] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-b711a01041
[17] https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-d770404d8b

>
>
> 

Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on master

[Tomas Jelinek]

- Original Message -
> From: "Tal Nisan" 
> To: "Sandro Bonazzola" , "Einav Cohen" 
> 
> Cc: "devel" 
> Sent: Tuesday, December 15, 2015 11:29:17 AM
> Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on
> master
> 
> Einav, it seems like an old issue we had when the default message contains
> the necessary placeholders but since no translations patch was released yet
> we don't have the placeholders in the translated messages, did we decide on
> a workaround back then?

This particular issue is only happening when you change a message which has 
already been in localized property files
in a way that you change the num of parameters of it.

This fixes it: https://gerrit.ovirt.org/#/c/50506/
Tested with 2 locales and seems to be working - Tal, adding you as a reviewer 
since you are already in the thread ;)

> 
> On Tue, Dec 15, 2015 at 12:04 PM, Sandro Bonazzola < sbona...@redhat.com >
> wrote:
> 
> 
> 
> 05:22:42 [INFO]   [ERROR] Required argument 3 not present:
> {0} ({1} Sockets, {2} Kerne pro Socket) 05:22:42 [INFO][ERROR] Errors in
> 'org/ovirt/engine/ui/uicompat/ConstantsManager.java' 05:22:42 [INFO]
> [ERROR] Line 38: Failed to resolve
> 'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding
> 
> See
> http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console
> 
> Please fix ASAP
> 
> 
> --
> Sandro Bonazzola
> Better technology. Faster innovation. Powered by community collaboration.
> See how it works at redhat.com
> 
> ___
> 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] [ACTION REQUIRED] ovirt-engine build failing on master

[Tal Nisan]

Acked, let me know if you need me to merge

On Tue, Dec 15, 2015 at 12:36 PM, Tomas Jelinek  wrote:

>
>
> - Original Message -
> > From: "Tal Nisan" 
> > To: "Sandro Bonazzola" , "Einav Cohen" <
> eco...@redhat.com>
> > Cc: "devel" 
> > Sent: Tuesday, December 15, 2015 11:29:17 AM
> > Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing
> onmaster
> >
> > Einav, it seems like an old issue we had when the default message
> contains
> > the necessary placeholders but since no translations patch was released
> yet
> > we don't have the placeholders in the translated messages, did we decide
> on
> > a workaround back then?
>
> This particular issue is only happening when you change a message which
> has already been in localized property files
> in a way that you change the num of parameters of it.
>
> This fixes it: https://gerrit.ovirt.org/#/c/50506/
> Tested with 2 locales and seems to be working - Tal, adding you as a
> reviewer since you are already in the thread ;)
>
> >
> > On Tue, Dec 15, 2015 at 12:04 PM, Sandro Bonazzola < sbona...@redhat.com
> >
> > wrote:
> >
> >
> >
> > 05:22:42 [INFO]   [ERROR] Required argument 3 not
> present:
> > {0} ({1} Sockets, {2} Kerne pro Socket) 05:22:42 [INFO][ERROR]
> Errors in
> > 'org/ovirt/engine/ui/uicompat/ConstantsManager.java' 05:22:42 [INFO]
> > [ERROR] Line 38: Failed to resolve
> > 'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding
> >
> > See
> >
> http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console
> >
> > Please fix ASAP
> >
> >
> > --
> > Sandro Bonazzola
> > Better technology. Faster innovation. Powered by community collaboration.
> > See how it works at redhat.com
> >
> > ___
> > 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] [ACTION REQUIRED] ovirt-engine build failing on master

[Martin Perina]

- Original Message -
> From: "Tomas Jelinek" 
> To: "Martin Perina" 
> Cc: "Sandro Bonazzola" , "devel" 
> Sent: Tuesday, December 15, 2015 11:27:23 AM
> Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on 
> master
> 
> 
> 
> - Original Message -
> > From: "Martin Perina" 
> > To: "Sandro Bonazzola" 
> > Cc: "devel" 
> > Sent: Tuesday, December 15, 2015 11:24:50 AM
> > Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on
> > master
> > 
> > Hi,
> > 
> > it seems to like some CI issue, I just verified that local build works
> > fine.
> > I also looked at the job and last successful build is 8 days old ...
> 
> it should reproduce only when you build with other than EN locale enabled
> since it is a mismatch between the UIMessages java and properties files

Ah, sorry I missed that.

> 
> > 
> > Martin
> > 
> > - Original Message -
> > > From: "Sandro Bonazzola" 
> > > To: "devel" 
> > > Sent: Tuesday, December 15, 2015 11:04:30 AM
> > > Subject: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on
> > > master
> > > 
> > > 05:22:42 [INFO]   [ERROR] Required argument 3 not
> > > present:
> > > {0} ({1} Sockets, {2} Kerne pro Socket) 05:22:42 [INFO][ERROR] Errors
> > > in
> > > 'org/ovirt/engine/ui/uicompat/ConstantsManager.java' 05:22:42 [INFO]
> > > [ERROR] Line 38: Failed to resolve
> > > 'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding
> > > 
> > > See
> > > http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console
> > > 
> > > Please fix ASAP
> > > 
> > > 
> > > --
> > > Sandro Bonazzola
> > > Better technology. Faster innovation. Powered by community collaboration.
> > > See how it works at redhat.com
> > > 
> > > ___
> > > 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] [ACTION REQUIRED] ovirt-engine build failing on master

[Tomas Jelinek]

- Original Message -
> From: "Tal Nisan" 
> To: "Tomas Jelinek" 
> Cc: "Sandro Bonazzola" , "Einav Cohen" 
> , "devel" 
> Sent: Tuesday, December 15, 2015 1:00:23 PM
> Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on 
> master
> 
> Acked, let me know if you need me to merge

Thanx, I will merge it as soon as jenkins passes.

> 
> On Tue, Dec 15, 2015 at 12:36 PM, Tomas Jelinek  wrote:
> 
> >
> >
> > - Original Message -
> > > From: "Tal Nisan" 
> > > To: "Sandro Bonazzola" , "Einav Cohen" <
> > eco...@redhat.com>
> > > Cc: "devel" 
> > > Sent: Tuesday, December 15, 2015 11:29:17 AM
> > > Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing
> > onmaster
> > >
> > > Einav, it seems like an old issue we had when the default message
> > contains
> > > the necessary placeholders but since no translations patch was released
> > yet
> > > we don't have the placeholders in the translated messages, did we decide
> > on
> > > a workaround back then?
> >
> > This particular issue is only happening when you change a message which
> > has already been in localized property files
> > in a way that you change the num of parameters of it.
> >
> > This fixes it: https://gerrit.ovirt.org/#/c/50506/
> > Tested with 2 locales and seems to be working - Tal, adding you as a
> > reviewer since you are already in the thread ;)
> >
> > >
> > > On Tue, Dec 15, 2015 at 12:04 PM, Sandro Bonazzola < sbona...@redhat.com
> > >
> > > wrote:
> > >
> > >
> > >
> > > 05:22:42 [INFO]   [ERROR] Required argument 3 not
> > present:
> > > {0} ({1} Sockets, {2} Kerne pro Socket) 05:22:42 [INFO][ERROR]
> > Errors in
> > > 'org/ovirt/engine/ui/uicompat/ConstantsManager.java' 05:22:42 [INFO]
> > > [ERROR] Line 38: Failed to resolve
> > > 'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding
> > >
> > > See
> > >
> > http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console
> > >
> > > Please fix ASAP
> > >
> > >
> > > --
> > > Sandro Bonazzola
> > > Better technology. Faster innovation. Powered by community collaboration.
> > > See how it works at redhat.com
> > >
> > > ___
> > > 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] [ACTION REQUIRED] ovirt-engine build failing on master

[Tomas Jelinek]

- Original Message -
> From: "Martin Perina" 
> To: "Sandro Bonazzola" 
> Cc: "devel" 
> Sent: Tuesday, December 15, 2015 11:24:50 AM
> Subject: Re: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on 
> master
> 
> Hi,
> 
> it seems to like some CI issue, I just verified that local build works fine.
> I also looked at the job and last successful build is 8 days old ...

it should reproduce only when you build with other than EN locale enabled
since it is a mismatch between the UIMessages java and properties files

> 
> Martin
> 
> - Original Message -
> > From: "Sandro Bonazzola" 
> > To: "devel" 
> > Sent: Tuesday, December 15, 2015 11:04:30 AM
> > Subject: [ovirt-devel] [ACTION REQUIRED] ovirt-engine build failing on
> > master
> > 
> > 05:22:42 [INFO]   [ERROR] Required argument 3 not present:
> > {0} ({1} Sockets, {2} Kerne pro Socket) 05:22:42 [INFO][ERROR] Errors
> > in
> > 'org/ovirt/engine/ui/uicompat/ConstantsManager.java' 05:22:42 [INFO]
> > [ERROR] Line 38: Failed to resolve
> > 'org.ovirt.engine.ui.uicompat.UIMessages' via deferred binding
> > 
> > See
> > http://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/682/console
> > 
> > Please fix ASAP
> > 
> > 
> > --
> > Sandro Bonazzola
> > Better technology. Faster innovation. Powered by community collaboration.
> > See how it works at redhat.com
> > 
> > ___
> > 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] Lago testing environment contribution

[Barak Korren]

>
> I would love to see a short paragraph on github and on readthedocs on what
> this project is about and what the aims are.
> It is really hard to even find out what it is for when you are reading the
> documentation.
>
Thank you for that:
https://bugzilla.redhat.com/show_bug.cgi?id=1291753


-- 
Barak Korren
bkor...@redhat.com
RHEV-CI Team
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Lago testing environment contribution

[David Caro Estevez]

On 12/15 15:29, Roman Mohr wrote:
> Hi,
> 
> On Tue, Dec 15, 2015 at 3:19 PM, Yaniv Bronheim  wrote:
> 
> > Hi list,
> >
> > If you're not aware yet about Lago you should. This our engine to test
> > functional flows over variant distributions for ovirt-engine and vdsm
> > This project is currently maintained by dcaro but it will love to have
> > more contribution in reviews, tests and more.
> >
> > If you want to follow it I encourage to join the mailing list
> > lago-de...@ovirt.org and help with reviews if possible ([2])
> >
> > You can also checkout following links for more info
> > In [3] you can find full readme about how to use it to run ovirt-engine
> > tests. should be intuitive and quick. If not, please raise
> >
> >
> I would love to see a short paragraph on github and on readthedocs on what
> this project is about and what the aims are.
> It is really hard to even find out what it is for when you are reading the
> documentation.

Agree, though it's not yet so clear what it is for or what the aims are...

But for now one could say something like:

"""
Provide a simple and easy way to create virtual environments aimed for
functional/system tests, being able to run them on a standard development
laptop.
"""

If that makes sense I'll add it to the docs


> 
> 
> > [1] https://gerrit.ovirt.org/#/admin/projects/lago
> > [2] https://gerrit.ovirt.org/#/q/project:+lago+status:+open
> > [3] https://github.com/ovirt/lago
> >
> > Hope you'll find it interesting and Lago will get more attention by our
> > community
> >
> > Greetings,
> >
> > --
> > *Yaniv Bronhaim.*
> >
> > ___
> > Devel mailing list
> > Devel@ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/devel
> >

-- 
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R

Tel.: +420 532 294 605
Email: dc...@redhat.com
IRC: dcaro|dcaroest@{freenode|oftc|redhat}
Web: www.redhat.com
RHT Global #: 82-62605


signature.asc
Description: PGP signature
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Lago testing environment contribution

[David Caro Estevez]

On 12/15 15:46, David Caro Estevez wrote:
> On 12/15 15:29, Roman Mohr wrote:
> > Hi,
> > 
> > On Tue, Dec 15, 2015 at 3:19 PM, Yaniv Bronheim  wrote:
> > 
> > > Hi list,
> > >
> > > If you're not aware yet about Lago you should. This our engine to test
> > > functional flows over variant distributions for ovirt-engine and vdsm
> > > This project is currently maintained by dcaro but it will love to have
> > > more contribution in reviews, tests and more.
> > >
> > > If you want to follow it I encourage to join the mailing list
> > > lago-de...@ovirt.org and help with reviews if possible ([2])
> > >
> > > You can also checkout following links for more info
> > > In [3] you can find full readme about how to use it to run ovirt-engine
> > > tests. should be intuitive and quick. If not, please raise
> > >
> > >
> > I would love to see a short paragraph on github and on readthedocs on what
> > this project is about and what the aims are.
> > It is really hard to even find out what it is for when you are reading the
> > documentation.
> 
> Agree, though it's not yet so clear what it is for or what the aims are...
> 
> But for now one could say something like:
> 
> """
> Provide a simple and easy way to create virtual environments aimed for
> functional/system tests, being able to run them on a standard development
> laptop.
> """
> 
> If that makes sense I'll add it to the docs


I see barak ever opened a bug on it :)

https://bugzilla.redhat.com/show_bug.cgi?id=1291753


> 
> 
> > 
> > 
> > > [1] https://gerrit.ovirt.org/#/admin/projects/lago
> > > [2] https://gerrit.ovirt.org/#/q/project:+lago+status:+open
> > > [3] https://github.com/ovirt/lago
> > >
> > > Hope you'll find it interesting and Lago will get more attention by our
> > > community
> > >
> > > Greetings,
> > >
> > > --
> > > *Yaniv Bronhaim.*
> > >
> > > ___
> > > Devel mailing list
> > > Devel@ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/devel
> > >
> 
> -- 
> David Caro
> 
> Red Hat S.L.
> Continuous Integration Engineer - EMEA ENG Virtualization R
> 
> Tel.: +420 532 294 605
> Email: dc...@redhat.com
> IRC: dcaro|dcaroest@{freenode|oftc|redhat}
> Web: www.redhat.com
> RHT Global #: 82-62605



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


-- 
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R

Tel.: +420 532 294 605
Email: dc...@redhat.com
IRC: dcaro|dcaroest@{freenode|oftc|redhat}
Web: www.redhat.com
RHT Global #: 82-62605


signature.asc
Description: PGP signature
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] AppErrors cleanup

[Moti Asayag]

Any removal of entries from AuditLogType enum should be followed by an
upgrade script to remove the matching entries from the audit_log table.
Without it, there will be NPEs due to AuditLogType.valueOf()

On Sun, Dec 13, 2015 at 2:46 PM, Allon Mureinik  wrote:

> Same has been done for AuditLogMessages.properties (vs. the AuditLogType
> enum).
>
> On Wed, Jul 29, 2015 at 3:51 PM, Allon Mureinik 
> wrote:
>
>> Hi all,
>>
>> A recent bug [1] reported as part of the translation effort alerted me to
>> the fact that we have a lot (and I mean a LOT - over 100 per file) of
>> deprecated, unused keys in the various AppErrors files that serve no
>> purpose and just take up space and waste translators time when they examine
>> them.
>>
>> To make a long story short - I've just merged a patch to remove all these
>> useless messages, and enforce via unit tests that EVERY key there should
>> have a corresponding constant in the EngineMessage or EngineError enums.
>>
>> Many thanks to my reviewers!
>> I know this was an tedious patch that couldn't have been too fun to
>> review.
>>
>>
>> -Allon
>>
>> [1] https://bugzilla.redhat.com/show_bug.cgi?id=1244766
>> ___
>> 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
>



-- 
Regards,
Moti
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Merged UI infra patch

[Vojtech Szocs]

Nice work, Alexander!

(GWTP's @ProxyEvent is a convenient mechanism, but as you wrote, it can lead
to UI code being lazy-loaded too aggresively which hurts UI responsiveness.)

Vojtech


- Original Message -
> From: "Alexander Wels" 
> To: devel@ovirt.org
> Sent: Monday, December 14, 2015 9:32:41 PM
> Subject: [ovirt-devel] Merged UI infra patch
> 
> Hi guys,
> 
> I just merged [1] which removes all occurrences of the @ProxyEvent mechanism
> from the code base. As part of this patch I have to fix some Java generics
> issues with some sub tabs. I believe I fixed them all properly but I might
> have
> missed some.
> 
> As part of this fix, now it will lazy load the code associated with sub tabs
> instead of loading all the code at once when you select an item in the main
> tab grid. This should result in a more responsive UI and less memory usage.
> 
> If you see any kind of strange behavior in sub tabs let me know and I will
> see
> if I can fix it.
> 
> Alexander
> 
> 
> [1] https://gerrit.ovirt.org/#/c/47912/
> ___
> 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-users] [QE] oVirt 3.6.1 Hosted Engine test day

[Sandro Bonazzola]

Adding Allon and Tal

On Tue, Dec 15, 2015 at 4:09 PM, wodel youchi 
wrote:

> Hello again,
>
> Here is another problem :-)
>
> today I tried to update my installation with the latest packages
> available, but after starting engine-setup I get this message :
>
>   --== CONFIGURATION PREVIEW ==--
>
>   Default SAN wipe after delete   : False
>   Firewall manager: iptables
>   Update Firewall : True
>   Host FQDN   : engine.domain.tld
>   Upgrade packages: True
>   Engine database secured connection  : False
>   Engine database host: localhost
>   Engine database user name   : engine
>   Engine database name: engine
>   Engine database port: 5432
>   Engine database host name validation: False
>   Engine installation : True
>   PKI organization: wodel.wd
>   Configure VMConsole Proxy   : True
>   Engine Host FQDN: engine.domain.tld
>   Configure WebSocket Proxy   : True
>
>   Please confirm installation settings (OK, Cancel) [OK]:
> [ INFO  ] Cleaning async tasks and compensations
>   The following system tasks have been found running in the system:
>   The following commands have been found running in the system:
>   The following compensations have been found running in the
> system:
>
>
>
>
> *
> org.ovirt.engine.core.bll.storage.AddExistingFileStorageDomainCommand
> org.ovirt.engine.core.common.businessentities.StorageDomainStatic
> org.ovirt.engine.core.bll.storage.AddExistingFileStorageDomainCommand
> org.ovirt.engine.core.common.businessentities.StorageDomainDynamic
> org.ovirt.engine.core.bll.storage.AddExistingFileStorageDomainCommand
> org.ovirt.engine.core.common.businessentities.StorageDomainDynamic
> Would you like to try to wait for that?  (Answering "no" will stop
> the upgrade (Yes, No)*
>
> If I hit no, the setup is aborted,
>
> if I hit yes, the setup will wait forever, and Ctrl+C aborts the setup too.
>
>
>
>
>
>
>
>
> *Waiting for the completion of 3 running tasks during the next 20
> seconds.  Press Ctrl+C to interrupt.Waiting for the completion of 3
> running tasks during the next 20 seconds.  Press Ctrl+C to
> interrupt.Waiting for the completion of 3 running tasks during the next 20
> seconds.  Press Ctrl+C to interrupt.*
> On hypervisor, vdsClient does not show any task running
>
> *[root@server ~]# vdsClient -s 0 getAllTasksStatuses{'status': {'message':
> 'OK', 'code': 0}, 'allTasksStatus': {}}*
>
>
> I am stuck ...!!!
>
> Regards
>
> 
>  Cet
> e-mail a été envoyé depuis un ordinateur protégé par Avast.
> www.avast.com
> 
> <#151a6305d772671f_DDB4FAA8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
>
> 2015-12-15 11:21 GMT+01:00 Sandro Bonazzola :
>
>>
>>
>> On Fri, Dec 11, 2015 at 12:41 PM, Sandro Bonazzola 
>> wrote:
>>
>>> Hi,
>>> oVirt 3.6.1 RC4 included latest bits allowing to auto import Hosted
>>> Engine storage domain within the Engine.
>>> In order to have a wider coverage of the Hosted Engine workflow we've
>>> scheduled a Hosted Engine test day for next week
>>> on Monday, December 14th.
>>> Please join us testing Hosted Engine with oVirt 3.6.1 RC4, both on a
>>> clean install and on an upgrade from 3.5 flow.
>>>
>>> Thanks,
>>> --
>>> Sandro Bonazzola
>>> Better technology. Faster innovation. Powered by community collaboration.
>>> See how it works at redhat.com
>>>
>>
>>
>> Thanks everyone who joined the test day, your feedback is really
>> appreciated!
>>
>>
>> --
>> Sandro Bonazzola
>> Better technology. Faster innovation. Powered by community collaboration.
>> See how it works at redhat.com
>>
>> ___
>> Users mailing list
>> us...@ovirt.org
>> http://lists.ovirt.org/mailman/listinfo/users
>>
>>
>


-- 
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] new UI i18n mechanism

[Vojtech Szocs]

- Original Message -
> From: "Einav Cohen" 
> To: "Vojtech Szocs" 
> Cc: "devel" 
> Sent: Tuesday, December 15, 2015 4:33:20 PM
> Subject: new UI i18n mechanism
> 
> [changed thread title since it's a new subject]
> 
> > Hmmm, I think we should do something similar in UI's i18n mechanism.
> 
> can you please explain/elaborate?

Sure :)

As we're planning to improve existing oVirt UI i18n mechanism,
we could also include unused message key detection into this
effort. As Allon wrote, we don't want translators to waste time
translating something that isn't used at all at runtime.

We have some high-level goals we'd like to accomplish within
this "UI i18n overhaul" effort, in short:

- improve GWT compile performance
  (should make all-langs + all-user-agents builds faster)

- externalize i18n messages out of GWT permutations
  (UI properties deployed on Engine for better handling)

- improve UI translation process
  (make life easier for translators, we have many ideas)

Scott and me should be driving this effort so if you have any
other ideas, we'd love to hear them.

Vojtech


> 
> - Original Message -
> > From: "Vojtech Szocs" 
> > To: "Allon Mureinik" 
> > Cc: "devel" 
> > Sent: Tuesday, December 15, 2015 10:28:27 AM
> > Subject: Re: [ovirt-devel] AppErrors cleanup
> > 
> > Hmmm, I think we should do something similar in UI's i18n mechanism.
> > 
> > Vojtech
> > 
> > 
> > - Original Message -
> > > From: "Allon Mureinik" 
> > > To: "devel" 
> > > Sent: Sunday, December 13, 2015 1:46:01 PM
> > > Subject: Re: [ovirt-devel] AppErrors cleanup
> > > 
> > > Same has been done for AuditLogMessages.properties (vs. the AuditLogType
> > > enum).
> > > 
> > > On Wed, Jul 29, 2015 at 3:51 PM, Allon Mureinik < amure...@redhat.com >
> > > wrote:
> > > 
> > > 
> > > Hi all,
> > > 
> > > A recent bug [1] reported as part of the translation effort alerted me to
> > > the
> > > fact that we have a lot (and I mean a LOT - over 100 per file) of
> > > deprecated, unused keys in the various AppErrors files that serve no
> > > purpose
> > > and just take up space and waste translators time when they examine them.
> > > 
> > > To make a long story short - I've just merged a patch to remove all these
> > > useless messages, and enforce via unit tests that EVERY key there should
> > > have a corresponding constant in the EngineMessage or EngineError enums.
> > > 
> > > Many thanks to my reviewers!
> > > I know this was an tedious patch that couldn't have been too fun to
> > > review.
> > > 
> > > 
> > > -Allon
> > > 
> > > [1] https://bugzilla.redhat.com/show_bug.cgi?id=1244766
> > > ___
> > > 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
> > 
> 
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel