From vitor.lima at eldorado.org.br Thu Aug 29 14:18:31 2013 Content-Type: multipart/mixed; boundary="===============0574226310938208291==" MIME-Version: 1.0 From: Vitor de Lima To: devel at ovirt.org Subject: [Engine-devel] About the "New Pool" dialog box Date: Thu, 29 Aug 2013 18:18:28 +0000 Message-ID: --===============0574226310938208291== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable --_000_B2CAFC4D5E2D574A883EF61ACD5ADE3301811EEASERV070corpeldo_ Content-Type: text/plain; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable Hi everyone, I noticed some issues with the "New Pool" dialog in the webadmin interface.= =3D When creating a new pool, I cannot change the selected cluster and the sel= =3D ected template, it automatically resets into the initial selection, that ap= =3D peared when the dialog was first shown, when I try to choose something else= =3D . I also noticed that, in the revision 0d9da7d0d922bc9f0eecaf6f4eecad77e8b2= =3D e3e2, the interface automatically changed the selected cluster after the us= =3D er had selected a template. How should this dialog behave? Was the previous= =3D behavior the correct one? There are also some issues on how to introduce the multi-architecture suppo= =3D rt into the webadmin interface. When the user is creating a Pool or a VM, h= =3D e shouldn't be able to use templates that are based on a x86 VM to create a= =3D VM/Pool in a PPC64 cluster (and vice-versa). This kind of problem could be= =3D avoided if the interface hid incompatible templates given a cluster, or hi= =3D d incompatible clusters given a template, or maybe it could let the user ch= =3D oose any template or cluster, and only show an error after the OK button is= =3D pressed. What would be a proper way to handle this? Thanks, Vitor de Lima --_000_B2CAFC4D5E2D574A883EF61ACD5ADE3301811EEASERV070corpeldo_ Content-Type: text/html; charset=3D"us-ascii" Content-Transfer-Encoding: quoted-printable

Hi everyone,<=3D /p>

I noticed some issues with = the =3D “New Pool” dialog in the webadmin interface. When creating a ne= =3D w pool, I cannot change the selected cluster and the selected template, it = =3D automatically resets into the initial selection, that appeared when the dialog was first shown, when I try to choose someth= =3D ing else. I also noticed that, in the revision 0d9da7d0d922bc9f0eecaf6f4eec= =3D ad77e8b2e3e2, the interface automatically changed the selected cluster afte= =3D r the user had selected a template. How should this dialog behave? Was the previous behavior the correct one?<= =3D o:p>

There are also some issues = on h=3D ow to introduce the multi-architecture support into the webadmin interface.= =3D When the user is creating a Pool or a VM, he shouldn’t be able to us= =3D e templates that are based on a x86 VM to create a VM/Pool in a PPC64 cluster (and vice-versa). This kind of problem= =3D could be avoided if the interface hid incompatible templates given a clust= =3D er, or hid incompatible clusters given a template, or maybe it could let th= =3D e user choose any template or cluster, and only show an error after the OK button is pressed. What would be a pro= =3D per way to handle this?

Thanks,

Vitor de Lima

--_000_B2CAFC4D5E2D574A883EF61ACD5ADE3301811EEASERV070corpeldo_-- --===============0574226310938208291== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" LS1fMDAwX0IyQ0FGQzRENUUyRDU3NEE4ODNFRjYxQUNENUFERTMzMDE4MTFFRUFTRVJWMDcwY29y cGVsZG9fCkNvbnRlbnQtVHlwZTogdGV4dC9wbGFpbjsgY2hhcnNldD0idXMtYXNjaWkiCkNvbnRl bnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKCkhpIGV2ZXJ5b25lLAoKSSBu b3RpY2VkIHNvbWUgaXNzdWVzIHdpdGggdGhlICJOZXcgUG9vbCIgZGlhbG9nIGluIHRoZSB3ZWJh ZG1pbiBpbnRlcmZhY2UuPQogV2hlbiBjcmVhdGluZyBhIG5ldyBwb29sLCBJIGNhbm5vdCBjaGFu Z2UgdGhlIHNlbGVjdGVkIGNsdXN0ZXIgYW5kIHRoZSBzZWw9CmVjdGVkIHRlbXBsYXRlLCBpdCBh dXRvbWF0aWNhbGx5IHJlc2V0cyBpbnRvIHRoZSBpbml0aWFsIHNlbGVjdGlvbiwgdGhhdCBhcD0K cGVhcmVkIHdoZW4gdGhlIGRpYWxvZyB3YXMgZmlyc3Qgc2hvd24sIHdoZW4gSSB0cnkgdG8gY2hv b3NlIHNvbWV0aGluZyBlbHNlPQouIEkgYWxzbyBub3RpY2VkIHRoYXQsIGluIHRoZSByZXZpc2lv biAwZDlkYTdkMGQ5MjJiYzlmMGVlY2FmNmY0ZWVjYWQ3N2U4YjI9CmUzZTIsIHRoZSBpbnRlcmZh Y2UgYXV0b21hdGljYWxseSBjaGFuZ2VkIHRoZSBzZWxlY3RlZCBjbHVzdGVyIGFmdGVyIHRoZSB1 cz0KZXIgaGFkIHNlbGVjdGVkIGEgdGVtcGxhdGUuIEhvdyBzaG91bGQgdGhpcyBkaWFsb2cgYmVo YXZlPyBXYXMgdGhlIHByZXZpb3VzPQogYmVoYXZpb3IgdGhlIGNvcnJlY3Qgb25lPwoKVGhlcmUg YXJlIGFsc28gc29tZSBpc3N1ZXMgb24gaG93IHRvIGludHJvZHVjZSB0aGUgbXVsdGktYXJjaGl0 ZWN0dXJlIHN1cHBvPQpydCBpbnRvIHRoZSB3ZWJhZG1pbiBpbnRlcmZhY2UuIFdoZW4gdGhlIHVz ZXIgaXMgY3JlYXRpbmcgYSBQb29sIG9yIGEgVk0sIGg9CmUgc2hvdWxkbid0IGJlIGFibGUgdG8g dXNlIHRlbXBsYXRlcyB0aGF0IGFyZSBiYXNlZCBvbiBhIHg4NiBWTSB0byBjcmVhdGUgYT0KIFZN L1Bvb2wgaW4gYSBQUEM2NCBjbHVzdGVyIChhbmQgdmljZS12ZXJzYSkuIFRoaXMga2luZCBvZiBw cm9ibGVtIGNvdWxkIGJlPQogYXZvaWRlZCBpZiB0aGUgaW50ZXJmYWNlIGhpZCBpbmNvbXBhdGli bGUgdGVtcGxhdGVzIGdpdmVuIGEgY2x1c3Rlciwgb3IgaGk9CmQgaW5jb21wYXRpYmxlIGNsdXN0 ZXJzIGdpdmVuIGEgdGVtcGxhdGUsIG9yIG1heWJlIGl0IGNvdWxkIGxldCB0aGUgdXNlciBjaD0K b29zZSBhbnkgdGVtcGxhdGUgb3IgY2x1c3RlciwgYW5kIG9ubHkgc2hvdyBhbiBlcnJvciBhZnRl ciB0aGUgT0sgYnV0dG9uIGlzPQogcHJlc3NlZC4gV2hhdCB3b3VsZCBiZSBhIHByb3BlciB3YXkg dG8gaGFuZGxlIHRoaXM/CgpUaGFua3MsClZpdG9yIGRlIExpbWEKCi0tXzAwMF9CMkNBRkM0RDVF MkQ1NzRBODgzRUY2MUFDRDVBREUzMzAxODExRUVBU0VSVjA3MGNvcnBlbGRvXwpDb250ZW50LVR5 cGU6IHRleHQvaHRtbDsgY2hhcnNldD0idXMtYXNjaWkiCkNvbnRlbnQtVHJhbnNmZXItRW5jb2Rp bmc6IHF1b3RlZC1wcmludGFibGUKCjxodG1sIHhtbG5zOnY9M0QidXJuOnNjaGVtYXMtbWljcm9z b2Z0LWNvbTp2bWwiIHhtbG5zOm89M0QidXJuOnNjaGVtYXMtbWljcj0Kb3NvZnQtY29tOm9mZmlj ZTpvZmZpY2UiIHhtbG5zOnc9M0QidXJuOnNjaGVtYXMtbWljcm9zb2Z0LWNvbTpvZmZpY2U6d29y ZCIgPQp4bWxuczptPTNEImh0dHA6Ly9zY2hlbWFzLm1pY3Jvc29mdC5jb20vb2ZmaWNlLzIwMDQv MTIvb21tbCIgeG1sbnM9M0QiaHR0cDo9Ci8vd3d3LnczLm9yZy9UUi9SRUMtaHRtbDQwIj4KPGhl YWQ+CjxtZXRhIGh0dHAtZXF1aXY9M0QiQ29udGVudC1UeXBlIiBjb250ZW50PTNEInRleHQvaHRt bDsgY2hhcnNldD0zRHVzLWFzY2lpIj0KPgo8bWV0YSBuYW1lPTNEIkdlbmVyYXRvciIgY29udGVu dD0zRCJNaWNyb3NvZnQgV29yZCAxNCAoZmlsdGVyZWQgbWVkaXVtKSI+CjxzdHlsZT48IS0tCi8q IEZvbnQgRGVmaW5pdGlvbnMgKi8KQGZvbnQtZmFjZQoJe2ZvbnQtZmFtaWx5OkNhbGlicmk7Cglw YW5vc2UtMToyIDE1IDUgMiAyIDIgNCAzIDIgNDt9Ci8qIFN0eWxlIERlZmluaXRpb25zICovCnAu TXNvTm9ybWFsLCBsaS5Nc29Ob3JtYWwsIGRpdi5Nc29Ob3JtYWwKCXttYXJnaW46MGNtOwoJbWFy Z2luLWJvdHRvbTouMDAwMXB0OwoJZm9udC1zaXplOjExLjBwdDsKCWZvbnQtZmFtaWx5OiJDYWxp YnJpIiwic2Fucy1zZXJpZiI7Cgltc28tZmFyZWFzdC1sYW5ndWFnZTpFTi1VUzt9CmE6bGluaywg c3Bhbi5Nc29IeXBlcmxpbmsKCXttc28tc3R5bGUtcHJpb3JpdHk6OTk7Cgljb2xvcjpibHVlOwoJ dGV4dC1kZWNvcmF0aW9uOnVuZGVybGluZTt9CmE6dmlzaXRlZCwgc3Bhbi5Nc29IeXBlcmxpbmtG b2xsb3dlZAoJe21zby1zdHlsZS1wcmlvcml0eTo5OTsKCWNvbG9yOnB1cnBsZTsKCXRleHQtZGVj b3JhdGlvbjp1bmRlcmxpbmU7fQpzcGFuLkVtYWlsU3R5bGUxNwoJe21zby1zdHlsZS10eXBlOnBl cnNvbmFsLWNvbXBvc2U7Cglmb250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiOwoJY29s b3I6d2luZG93dGV4dDt9Ci5Nc29DaHBEZWZhdWx0Cgl7bXNvLXN0eWxlLXR5cGU6ZXhwb3J0LW9u bHk7Cglmb250LWZhbWlseToiQ2FsaWJyaSIsInNhbnMtc2VyaWYiOwoJbXNvLWZhcmVhc3QtbGFu Z3VhZ2U6RU4tVVM7fQpAcGFnZSBXb3JkU2VjdGlvbjEKCXtzaXplOjYxMi4wcHQgNzkyLjBwdDsK CW1hcmdpbjo3MC44NXB0IDMuMGNtIDcwLjg1cHQgMy4wY207fQpkaXYuV29yZFNlY3Rpb24xCgl7 cGFnZTpXb3JkU2VjdGlvbjE7fQotLT48L3N0eWxlPjwhLS1baWYgZ3RlIG1zbyA5XT48eG1sPgo8 bzpzaGFwZWRlZmF1bHRzIHY6ZXh0PTNEImVkaXQiIHNwaWRtYXg9M0QiMTAyNiIgLz4KPC94bWw+ PCFbZW5kaWZdLS0+PCEtLVtpZiBndGUgbXNvIDldPjx4bWw+CjxvOnNoYXBlbGF5b3V0IHY6ZXh0 PTNEImVkaXQiPgo8bzppZG1hcCB2OmV4dD0zRCJlZGl0IiBkYXRhPTNEIjEiIC8+CjwvbzpzaGFw ZWxheW91dD48L3htbD48IVtlbmRpZl0tLT4KPC9oZWFkPgo8Ym9keSBsYW5nPTNEIlBULUJSIiBs aW5rPTNEImJsdWUiIHZsaW5rPTNEInB1cnBsZSI+CjxkaXYgY2xhc3M9M0QiV29yZFNlY3Rpb24x Ij4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBsYW5nPTNEIkVOLVVTIj5IaSBldmVyeW9u ZSw8bzpwPjwvbzpwPjwvc3Bhbj48PQovcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBs YW5nPTNEIkVOLVVTIj48bzpwPiZuYnNwOzwvbzpwPjwvc3Bhbj48L3A+CjxwIGNsYXNzPTNEIk1z b05vcm1hbCI+PHNwYW4gbGFuZz0zRCJFTi1VUyI+SSBub3RpY2VkIHNvbWUgaXNzdWVzIHdpdGgg dGhlID0KJiM4MjIwO05ldyBQb29sJiM4MjIxOyBkaWFsb2cgaW4gdGhlIHdlYmFkbWluIGludGVy ZmFjZS4gV2hlbiBjcmVhdGluZyBhIG5lPQp3IHBvb2wsIEkgY2Fubm90IGNoYW5nZSB0aGUgc2Vs ZWN0ZWQgY2x1c3RlciBhbmQgdGhlIHNlbGVjdGVkIHRlbXBsYXRlLCBpdCA9CmF1dG9tYXRpY2Fs bHkgcmVzZXRzIGludG8gdGhlIGluaXRpYWwgc2VsZWN0aW9uLAogdGhhdCBhcHBlYXJlZCB3aGVu IHRoZSBkaWFsb2cgd2FzIGZpcnN0IHNob3duLCB3aGVuIEkgdHJ5IHRvIGNob29zZSBzb21ldGg9 CmluZyBlbHNlLiBJIGFsc28gbm90aWNlZCB0aGF0LCBpbiB0aGUgcmV2aXNpb24gMGQ5ZGE3ZDBk OTIyYmM5ZjBlZWNhZjZmNGVlYz0KYWQ3N2U4YjJlM2UyLCB0aGUgaW50ZXJmYWNlIGF1dG9tYXRp Y2FsbHkgY2hhbmdlZCB0aGUgc2VsZWN0ZWQgY2x1c3RlciBhZnRlPQpyIHRoZSB1c2VyIGhhZCBz ZWxlY3RlZCBhIHRlbXBsYXRlLgogSG93IHNob3VsZCB0aGlzIGRpYWxvZyBiZWhhdmU/IFdhcyB0 aGUgcHJldmlvdXMgYmVoYXZpb3IgdGhlIGNvcnJlY3Qgb25lPzw9Cm86cD48L286cD48L3NwYW4+ PC9wPgo8cCBjbGFzcz0zRCJNc29Ob3JtYWwiPjxzcGFuIGxhbmc9M0QiRU4tVVMiPjxvOnA+Jm5i c3A7PC9vOnA+PC9zcGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBsYW5nPTNE IkVOLVVTIj5UaGVyZSBhcmUgYWxzbyBzb21lIGlzc3VlcyBvbiBoPQpvdyB0byBpbnRyb2R1Y2Ug dGhlIG11bHRpLWFyY2hpdGVjdHVyZSBzdXBwb3J0IGludG8gdGhlIHdlYmFkbWluIGludGVyZmFj ZS49CiBXaGVuIHRoZSB1c2VyIGlzIGNyZWF0aW5nIGEgUG9vbCBvciBhIFZNLCBoZSBzaG91bGRu JiM4MjE3O3QgYmUgYWJsZSB0byB1cz0KZSB0ZW1wbGF0ZXMgdGhhdCBhcmUgYmFzZWQgb24gYSB4 ODYgVk0gdG8KIGNyZWF0ZSBhIFZNL1Bvb2wgaW4gYSBQUEM2NCBjbHVzdGVyIChhbmQgdmljZS12 ZXJzYSkuIFRoaXMga2luZCBvZiBwcm9ibGVtPQogY291bGQgYmUgYXZvaWRlZCBpZiB0aGUgaW50 ZXJmYWNlIGhpZCBpbmNvbXBhdGlibGUgdGVtcGxhdGVzIGdpdmVuIGEgY2x1c3Q9CmVyLCBvciBo aWQgaW5jb21wYXRpYmxlIGNsdXN0ZXJzIGdpdmVuIGEgdGVtcGxhdGUsIG9yIG1heWJlIGl0IGNv dWxkIGxldCB0aD0KZSB1c2VyIGNob29zZSBhbnkgdGVtcGxhdGUgb3IgY2x1c3RlciwKIGFuZCBv bmx5IHNob3cgYW4gZXJyb3IgYWZ0ZXIgdGhlIE9LIGJ1dHRvbiBpcyBwcmVzc2VkLiBXaGF0IHdv dWxkIGJlIGEgcHJvPQpwZXIgd2F5IHRvIGhhbmRsZSB0aGlzPzxvOnA+PC9vOnA+PC9zcGFuPjwv cD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFsIj48c3BhbiBsYW5nPTNEIkVOLVVTIj48bzpwPiZuYnNw OzwvbzpwPjwvc3Bhbj48L3A+CjxwIGNsYXNzPTNEIk1zb05vcm1hbCI+PHNwYW4gbGFuZz0zRCJF Ti1VUyI+VGhhbmtzLDxvOnA+PC9vOnA+PC9zcGFuPjwvcD4KPHAgY2xhc3M9M0QiTXNvTm9ybWFs Ij5WaXRvciBkZSBMaW1hPG86cD48L286cD48L3A+CjwvZGl2Pgo8L2JvZHk+CjwvaHRtbD4KCi0t XzAwMF9CMkNBRkM0RDVFMkQ1NzRBODgzRUY2MUFDRDVBREUzMzAxODExRUVBU0VSVjA3MGNvcnBl bGRvXy0tCg== --===============0574226310938208291==-- From tjelinek at redhat.com Fri Aug 30 04:32:28 2013 Content-Type: multipart/mixed; boundary="===============0201351913206768138==" MIME-Version: 1.0 From: Tomas Jelinek To: devel at ovirt.org Subject: Re: [Engine-devel] About the "New Pool" dialog box Date: Fri, 30 Aug 2013 04:32:24 -0400 Message-ID: <1844357177.6428122.1377851544750.JavaMail.root@redhat.com> In-Reply-To: B2CAFC4D5E2D574A883EF61ACD5ADE3301811EEA@SERV070.corp.eldorado.org.br --===============0201351913206768138== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Hi Vitor, comments inline have a nice day, Tomas ----- Original Message ----- > From: "Vitor de Lima" > To: engine-devel(a)ovirt.org > Sent: Thursday, August 29, 2013 8:18:28 PM > Subject: [Engine-devel] About the "New Pool" dialog box > = > = > = > Hi everyone, > = > = > = > I noticed some issues with the =E2=80=9CNew Pool=E2=80=9D dialog in the w= ebadmin interface. > When creating a new pool, I cannot change the selected cluster and the > selected template, it automatically resets into the initial selection, th= at > appeared when the dialog was first shown, when I try to choose something > else. I also noticed that, in the revision > 0d9da7d0d922bc9f0eecaf6f4eecad77e8b2e3e2, the interface automatically > changed the selected cluster after the user had selected a template. How > should this dialog behave? Was the previous behavior the correct one? Thank you for reporting this! I have created a BZ for it (https://bugzilla.= redhat.com/show_bug.cgi?id=3D1002902) and submitted a patch fixing it (http://gerrit.ovirt.org/#/c/18713/). The correct behavior is that the selection of the template should not touch= the DC/Cluster. > = > = > = > There are also some issues on how to introduce the multi-architecture sup= port > into the webadmin interface. When the user is creating a Pool or a VM, he > shouldn=E2=80=99t be able to use templates that are based on a x86 VM to = create a > VM/Pool in a PPC64 cluster (and vice-versa). This kind of problem could be > avoided if the interface hid incompatible templates given a cluster, or h= id > incompatible clusters given a template, or maybe it could let the user > choose any template or cluster, and only show an error after the OK button > is pressed. What would be a proper way to handle this? Normally we hide the incompatible selections according to the selected DC/C= luster. = So, if it was me I would filter the list of the templates according to the = selected DC/Cluster. > = > = > = > Thanks, > = > Vitor de Lima > = > _______________________________________________ > Engine-devel mailing list > Engine-devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/engine-devel > --===============0201351913206768138==--