March 2012 - Devel - oVirt List Archives

[Engine-devel] Gluster Support Feature
by Shireesh Anjal 16 Mar '12

16 Mar '12

Hi all, Here is the feature page for Gluster Support in oVirt: http://www.ovirt.org/wiki/Features/Gluster_Support Comments / Suggestions welcome. -- Thanks, Shireesh

1 0

[Engine-devel] [Jenkins] ovirt-engine-cli and ovirt-engine-sdk rpms availiable
by Eyal Edri 15 Mar '12

15 Mar '12

fyi, you can now d/l ovirt engine cli & rpms built per commit directly from jenkins: http://jenkins.ovirt.org/view/ovirt_engine_sdk/job/ovirt_engine_sdk_create_… http://jenkins.ovirt.org/view/ovirt_engine_cli/job/ovirt_engine_cli_create_… Enjoy, Eyal Edri oVirt Infra Team

1 0

[Engine-devel] Can we add a engine_3.0 tag to the git repository?
by Juan Hernandez 15 Mar '12

15 Mar '12

Currently we have only a engine_3.0 branch, that can be modified. Is it possible to add a tag?

2 2

[Engine-devel] Hints for building up a spicec connection in Ovirt
by Andrew Wells 15 Mar '12

15 Mar '12

I am trying to build a connection manually with spicec (or what ever I need to build the connection with) in Ovirt/RHEV and I am wondering what do I need to form this connection? And if this is the wrong place to ask, can you point me in the correct direction? Andrew Wells

5 14

[Engine-devel] revievers needed
by Laszlo Hornyak 15 Mar '12

15 Mar '12

Hi, I have a big pile of patches waiting for reviewers, some of them are very old. If you have some spare-time, please take any: http://gerrit.ovirt.org/#dashboard,1000027 Thank you, Laszlo

3 2

[Engine-devel] Shared Raw Disk - Design change
by Maor Lipchuk 14 Mar '12

14 Mar '12

Hello all, The shared raw disk feature description has been changed. The changes can be found in the following wiki: links: http://www.ovirt.org/wiki/Features/DetailedSharedRawDisk http://www.ovirt.org/wiki/Features/SharedRawDisk Please feel free, to share your comments. Regards, Maor

2 4

[Engine-devel] agenda for todays meeting
by Livnat Peer 14 Mar '12

14 Mar '12

Hi All, We don't have an agenda for today - open for suggestions. If no other requests will be suggested for today's agenda we'll discuss storage live migration and who ever is interested can join us. Federico/ayal can you make the slot today at 1600? Thanks, Livnat

1 0

[Engine-devel] [backend] logger declarations
by Laszlo Hornyak 11 Mar '12

11 Mar '12

Hi, In general loggers are recommended to be private static final, also have the correct class. (e.g. commons logging standards from PMD http://pmd.sourceforge.net/rules/logging-jakarta-commons.html) In oVirt backend, very often the logger is just package protected, which allows classes in the same package to use it. Actually the classes that really use it are usualy subclasses, but what I believe is misleading in this case is that when reading a log, you will see that e.g. org.foo.Bar logged 'X', and you look into the sourcecode of org.foo.Bar and you do not see any code that would ever log that message, you will have to look for that message in all of it's subclasses and this will make debugging harder especially if the number of subclasses is high. I am trying to maintain a coding standard here, please feel free to edit: http://www.ovirt.org/wiki/Backend_Coding_Standards Thank you, Laszlo

2 1

[Engine-devel] Wiki page for audit logs and event notifications
by Shireesh Anjal 09 Mar '12

09 Mar '12

Hi all, Based on my recent experience with audit logs and event notifications, I have created a Wiki page explaining how to introduce the same for a new entity in oVirt. It is available at following URL: http://www.ovirt.org/wiki/Development/Audit_Logs_and_Event_Notifications Please feel free to contribute to the same in order to correct any discrepancies or enhance the information provided. -- Thanks, Shireesh

1 0

[Engine-devel] setup newtorks UI sync meeting MOM
by Roy Golan 06 Mar '12

06 Mar '12

MTU ∘ provision MTU when network is down ∘ valid values? -1, minimum = 68, maximum = configurable ∘ MTU -1 -> dont send the value to VDSM let the default take place ∘ mtu checkbox, label : "override MTU" , unchecked, value box always blank when uncheck. ∘ mtu box checked, show a box for a value ∘ support matrix of co-exisiting networks MTU values on single interface - open issue  STP • remove stp from the dialog VM network • checked by default for management network as well Setup network ∘ On what host status we allow to run setup networks on ? - open issue ∘ boxes are a bit naked. Add icons for allowed actions on each interface box.

2 1

[Engine-devel] [backend] some graphs from sonar
by Laszlo Hornyak 06 Mar '12

06 Mar '12

Hi, I am running sonar (http://www.sonarsource.org/) on oVirt source code since last October and I would like to share some results with you. http://twitpic.com/8son6o/full - showing the backend complexity, rules compliance and coverage. While complexity increased about 10%, the code coverage doubled. http://twitpic.com/8soojz/full - size vs coverage in backend (100% would be green) Unfortunately this server is on the internal network but it would be great to have a public sonar instance. Laszlo

1 0

[Engine-devel] network - UI Sync meeting
by Roy Golan 05 Mar '12

05 Mar '12

--=_ecaa160f-be85-47b6-8d28-5b3159110abb Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable The following meeting has been modified: Subject: network - UI Sync meeting Organizer: "Roy Golan" <rgolan(a)redhat.com> Location: [MODIFIED] Time: Monday, March 5, 2012, 4:00:00 PM - 5:00:00 PM GMT +02:00 Jerusalem [= MODIFIED] Invitees: mkenneth(a)redhat.com; sgrinber(a)redhat.com; lpeer(a)redhat.com; dfedi= uck(a)redhat.com; drankevi(a)redhat.com; ecohen(a)redhat.com; iheim(a)redhat.com; o= vedo(a)redhat.com; acathrow(a)redhat.com; engine-devel(a)ovirt.org; kroberts@redh= at.com ... *~*~*~*~*~*~*~*~*~* Follow-up meeting on setup networks UI. Last meeting AIs: =E2=88=98 "Edit..." button should be placed on the network to cover cases o= f single interface to multiple network. =E2=88=98 "i" icon on each logical network to have details: vlan, mtu, VmNe= tworks, mandatoy... =E2=88=98 right-hand breakdown VM-networks|VLAN/Non-VLAN|optional issues to follow: 1. can VDSM attach many non-vlan and many vlan networks to a single nic? (D= an - please reply if its doable) Bridge ID: 1814335863 https://www.intercallonline.com/portlets/scheduling/viewNumbers/listNumbers= ByCode.do?confCode=3D1814335863 --=_ecaa160f-be85-47b6-8d28-5b3159110abb Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable <html><body><h3>The following meeting has been modified:</h3> <p> <table border=3D'0'> <tr><th align=3Dleft>Subject:</th><td>network - UI Sync meeting </td></tr> <tr><th align=3Dleft>Organizer:</th><td>"Roy Golan" <rgolan(a)redhat.com&g= t; </td></tr> </table> <p> <table border=3D'0'> <tr><th align=3Dleft>Location:</th><td> [MODIFIED]</td></tr> <tr><th align=3Dleft>Time:</th><td>Monday, March 5, 2012, 4:00:00 PM - 5:00= :00 PM GMT +02:00 Jerusalem [MODIFIED] </td></tr></table> <p> <table border=3D'0'> <tr><th align=3Dleft>Invitees:</th><td>mkenneth(a)redhat.com; sgrinber@redhat= .com; lpeer(a)redhat.com; dfediuck(a)redhat.com; drankevi(a)redhat.com; ecohen@re= dhat.com; iheim(a)redhat.com; ovedo(a)redhat.com; acathrow(a)redhat.com; engine-d= evel(a)ovirt.org; kroberts(a)redhat.com ... </td></tr> </table> <div>*~*~*~*~*~*~*~*~*~*</div><br><br>Follow-up meeting on setup networks U= I. <br><br>Last meeting AIs:<br>=E2=88=98 "Edit..." button should be placed= on the network to cover cases of single interface to multiple network.<br>= =E2=88=98 "i" icon on each logical network to have details: vlan, mtu, VmNe= tworks, mandatoy...<br>=E2=88=98 right-hand breakdown VM-networks|VLAN/Non= -VLAN|optional<br><br>issues to follow: <br>1. can VDSM attach many non-vla= n and many vlan networks to a single nic? (Dan - please reply if its doable= ) <br><br><br>Bridge ID: 1814335863 <br><br>https://www.intercallonline.com= /portlets/scheduling/viewNumbers/listNumbersByCode.do?confCode=3D1814335863= </body></html> --=_ecaa160f-be85-47b6-8d28-5b3159110abb Content-Type: text/calendar; charset=utf-8; method=REQUEST; name=meeting.ics Content-Transfer-Encoding: quoted-printable BEGIN:VCALENDAR PRODID:Zimbra-Calendar-Provider VERSION:2.0 METHOD:REQUEST BEGIN:VTIMEZONE TZID:Asia/Jerusalem BEGIN:STANDARD DTSTART:19710101T020000 TZOFFSETTO:+0200 TZOFFSETFROM:+0300 RRULE:FREQ=3DYEARLY;WKST=3DMO;INTERVAL=3D1;BYMONTH=3D9;BYDAY=3D2SU TZNAME:IST END:STANDARD BEGIN:DAYLIGHT DTSTART:19710101T020000 TZOFFSETTO:+0300 TZOFFSETFROM:+0200 RRULE:FREQ=3DYEARLY;WKST=3DMO;INTERVAL=3D1;BYMONTH=3D3;BYDAY=3D-1FR TZNAME:IDT END:DAYLIGHT END:VTIMEZONE BEGIN:VEVENT UID:e6d6ce3a-e7f1-492b-9c52-65365ce2e235 SUMMARY:network - UI Sync meeting ATTENDEE;CN=3DMiki Kenneth;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;R= SVP=3DTRU E:mailto:mkenneth@redhat.com ATTENDEE;CN=3DSimon Grinberg;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION= ;RSVP=3DT RUE:mailto:sgrinber@redhat.com ATTENDEE;CN=3DLivnat Peer;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;RS= VP=3DTRUE :mailto:lpeer@redhat.com ATTENDEE;CN=3DDoron Fediuck;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;= RSVP=3DTR UE:mailto:dfediuck@redhat.com ATTENDEE;CN=3DDanny Rankevich;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTIO= N;RSVP=3D TRUE:mailto:drankevi@redhat.com ATTENDEE;CN=3DEinav Cohen;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;RS= VP=3DTRUE :mailto:ecohen@redhat.com ATTENDEE;CN=3DItamar Heim;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;RS= VP=3DTRUE :mailto:iheim@redhat.com ATTENDEE;CN=3DOved Ourfalli;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;= RSVP=3DTR UE:mailto:ovedo@redhat.com ATTENDEE;CN=3DAndrew Cathrow;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION= ;RSVP=3DT RUE:mailto:acathrow@redhat.com ATTENDEE;CN=3Dengine-devel;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;R= SVP=3DTRU E:mailto:engine-devel@ovirt.org ATTENDEE;CN=3DKeith Robertson;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTIO= N;RSVP=3D TRUE:mailto:kroberts@redhat.com ATTENDEE;CN=3DWilliam Henry;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;= RSVP=3DTR UE:mailto:whenry@redhat.com ATTENDEE;CN=3DYaniv Kaul;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;RSV= P=3DTRUE: mailto:ykaul@redhat.com ATTENDEE;CN=3DAyal Baron;ROLE=3DREQ-PARTICIPANT;PARTSTAT=3DNEEDS-ACTION;RSV= P=3DTRUE: mailto:abaron@redhat.com X-MOZ-LASTACK:20120228T133242Z X-MOZ-GENERATION:1 ORGANIZER;CN=3DRoy Golan:mailto:rgolan@redhat.com DTSTART;TZID=3D"Asia/Jerusalem":20120305T160000 DTEND;TZID=3D"Asia/Jerusalem":20120305T170000 STATUS:CONFIRMED CLASS:PUBLIC X-MICROSOFT-CDO-INTENDEDSTATUS:BUSY TRANSP:OPAQUE LAST-MODIFIED:20120305T131309Z DTSTAMP:20120305T131309Z SEQUENCE:4 DESCRIPTION:The following meeting has been modified:\n\nSubject: network - = U I Sync meeting \nOrganizer: "Roy Golan" <rgolan(a)redhat.com> \n\nLocation: = [ MODIFIED]\nTime: Monday\, March 5\, 2012\, 4:00:00 PM - 5:00:00 PM GMT +02= :0 0 Jerusalem [MODIFIED]\n \nInvitees: mkenneth(a)redhat.com\; sgrinber@redhat= .c om\; lpeer(a)redhat.com\; dfediuck(a)redhat.com\; drankevi(a)redhat.com\; ecohen= @r edhat.com\; iheim(a)redhat.com\; ovedo(a)redhat.com\; acathrow(a)redhat.com\; en= gi ne-devel(a)ovirt.org\; kroberts(a)redhat.com ... \n\n\n*~*~*~*~*~*~*~*~*~*\n\n= \n Follow-up meeting on setup networks UI. \n\nLast meeting AIs:\n=E2=88=98 "= Edit..." b utton should be placed on the network to cover cases of single interface t= o=20 multiple network.\n=E2=88=98 "i" icon on each logical network to have deta= ils: vlan\ , mtu\, VmNetworks\, mandatoy...\n=E2=88=98 right-hand breakdown VM-netwo= rks|VLAN/N on-VLAN|optional\n\nissues to follow: \n1. can VDSM attach many non-vlan a= nd many vlan networks to a single nic? (Dan - please reply if its doable) \n= \n \nBridge ID: 1814335863 \n\nhttps://www.intercallonline.com/portlets/sched= ul ing/viewNumbers/listNumbersByCode.do?confCode=3D1814335863=20 X-ALT-DESC;FMTTYPE=3Dtext/html:<html><body><h3>The following meeting has be= en=20 modified:</h3>\n\n<p>\n<table border=3D'0'>\n<tr><th align=3Dleft>Subject:= </th>< td>network - UI Sync meeting </td></tr>\n<tr><th align=3Dleft>Organizer:</= th>< td>"Roy Golan" &lt\;rgolan(a)redhat.com&gt\; </td></tr>\n</table>\n<p>\n<tab= le border=3D'0'>\n<tr><th align=3Dleft>Location:</th><td> [MODIFIED]</td></t= r>\n<t r><th align=3Dleft>Time:</th><td>Monday\, March 5\, 2012\, 4:00:00 PM - 5:= 00:0 0 PM GMT +02:00 Jerusalem [MODIFIED]\n </td></tr></table>\n<p>\n<table bor= de r=3D'0'>\n<tr><th align=3Dleft>Invitees:</th><td>mkenneth(a)redhat.com\; sgr= inber@ redhat.com\; lpeer(a)redhat.com\; dfediuck(a)redhat.com\; drankevi(a)redhat.com\= ;=20 ecohen(a)redhat.com\; iheim(a)redhat.com\; ovedo(a)redhat.com\; acathrow(a)redhat.= co m\; engine-devel(a)ovirt.org\; kroberts(a)redhat.com ... </td></tr>\n</table>\= n< div>*~*~*~*~*~*~*~*~*~*</div><br><br>Follow-up meeting on setup networks U= I. <br><br>Last meeting AIs:<br>=E2=88=98 "Edit..." button should be placed = on the net work to cover cases of single interface to multiple network.<br>=E2=88=98 = "i" icon o n each logical network to have details: vlan\, mtu\, VmNetworks\, mandatoy= .. .<br>=E2=88=98 right-hand breakdown VM-networks|VLAN/Non-VLAN|optional<br= ><br>issue s to follow: <br>1. can VDSM attach many non-vlan and many vlan networks t= o=20 a single nic? (Dan - please reply if its doable) <br><br><br>Bridge ID: 18= 14 335863 <br><br>https://www.intercallonline.com/portlets/scheduling/viewNum= be rs/listNumbersByCode.do?confCode=3D1814335863 </body></html> BEGIN:VALARM ACTION:DISPLAY TRIGGER;RELATED=3DSTART:-PT5M DESCRIPTION:Reminder END:VALARM END:VEVENT END:VCALENDAR --=_ecaa160f-be85-47b6-8d28-5b3159110abb--

1 0

[Engine-devel] [ovirt Jenkins] new HIGH priority bugs introduced
by Eyal Edri 05 Mar '12

05 Mar '12

FYI, one of these commits inserted 3 new HIGH Priority bugs: core : Fixing import of vm with snapshots (detail / gitweb) restapi: rename case-sensitive matrix param (detail / gitweb) restapi: Add JSON support (detail / gitweb) pleae check here for more details: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/278/ Eyal Edri oVirt Infrastructure

3 3

[Engine-devel] UI Related: Removal of SharedGwt module [imprtant notes]
by Asaf Shakarchi 05 Mar '12

05 Mar '12

All, Gerrit ticket: http://gerrit.ovirt.org/#change,2164 was just merged, This commit removes the SharedGwt module from oVirt maven structure, This module aggregated backend source code of three modules: Common, Compat and SearchBackend for UI (gwt) modules during compile phase, This is replaced now by a direct access to the backend modules using sources artifacts, GWT module descriptor of SharedGwt maven module (named: SharedGwt.gwt.xml) contained a list of java classes that should be visible for the GWT compiler, Since this module does not exist anymore, this file was split into three files per backend module: Common.gwt.xml - Contains backend Common code that is visible to GWT compiler. SearchBackend.gwt.xml - Contains SearchBackend code that is visible to GWT compiler. Compat.gwt.xml - Contains backend Compat code that is visible to GWT compiler. To avoid pushing UI related files directly into the backend modules, these files are located in gwt-common module, which contains shared code for UI modules. In short: any backend source code that should be available during UI compilation which exist in Common|Compat|SearchBackend must be included in the appropriate GWT module descriptor according to its parent module. Please contact me if this is causing any issue, Regards, +972-543094949 Asaf.

1 0

[Engine-devel] Introducing virt / gluster flags at cluster level
by Shireesh Anjal 04 Mar '12

04 Mar '12

Hi, In order to identify whether a cluster exposes Gluster / Virtualization capabilities, we plan to introduce two boolean columns - virt_service and gluster_service in the vds_groups table. As per immediate plans, it is intended to support only one service per cluster, meaning only one of these two values can be true. We plan to make following changes as part of the patch: 1) Introduce an upgrade script for adding following columns to the table "vds_groups" - virt_service - boolean - NOT_NULL - default value true - gluster_service - boolean - NOT_NULL - default value false => What is the naming convention for naming the upgrade script? Is it <major-version>_<minor-version>_<running-sequence-with-step-10>_script_description.sql ? By that logic, the name of our script will be something like 03_01_0560_add_service_columns_to_vds_groups.sql 2) Modify vds_groups_sp.sql to introduce these arguments / variables in save and update methods 3) Modify the DAO interface and implementation to introduce these arguments / variables in the save/update methods and VdsGroupRawMapper. 4) Modify the DAO JUnit test class to take care of these new fields, if required. 5) Modify class VDSGroup to introduce the two new boolean variables virtService and glusterService and modify the methods equals and hashCode to use these. => One question here. Most of the attributes of class VDSGroup have annotations like @XmlElement and @Column. I think these are related to jaxb and JPA. Are these annotations really required? If yes, how are they useful? 6) REST API : Default populate these variables (virtService = true, glusterService = false) before invoking the BLL command (AddVdsGroupCommand / UpdateVdsGroupCommand). This makes sure that the existing API won't break. No change in api.xsd for now. 7) webadmin create cluster code: Default populate the new variables, same as above. No change in UI screen for now. => Alternatively, the variable declaration on VDSGroup itself can initialize these variables with default values, so that changes to UI / REST API code (points 6 and 7) may not be required immediately. What do you suggest? Inputs / comments / suggestions welcome. -- Thanks, Shireesh

8 11

Re: [Engine-devel] [ovirt Jenkins] new HIGH priority bugs introduced
by Yaniv Kaul 04 Mar '12

04 Mar '12

If Jenkins is bored, it could be cool if it could perform 'git bisect' on such failures. Y. Eyal Edri <eedri(a)redhat.com> wrote: FYI, one of these commits inserted 3 new HIGH Priority bugs: core : Fixing import of vm with snapshots (detail / gitweb) restapi: rename case-sensitive matrix param (detail / gitweb) restapi: Add JSON support (detail / gitweb) pleae check here for more details: http://jenkins.ovirt.org/job/ovirt_engine_find_bugs/278/ Eyal Edri oVirt Infrastructure _______________________________________________ Engine-devel mailing list Engine-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

2 1

[Engine-devel] synthetic-access
by Laszlo Hornyak 01 Mar '12

01 Mar '12

Hi, I am reviewing Allon's patches (e.g. http://gerrit.ovirt.org/#patch,sidebyside,2188,11,backend/manager/modules/d… ) and this is the first time I have met @SuppressWarnings("synthetic-access") annotations in the ovirt code. It is right, eclipse warns about the performance problem synthetic access (if turned on, by default it is turned off). This mostly happens in DAO's because rowmappers are private inner classes. What if, instead of adding an annotation to ignore this - we could make the rowmapper classes package protected? - or since most of these classes are stateless and thread safe, we can add a public final static rowmapper instance and instead of instantiating the rowmapper over and over again, use that single instance. Please share your thoughts. Thank you, Laszlo

4 3