January 2013 - Engine-commits - oVirt List Archives

Change in otopi[master]: core: filetransaction: support unsafe visible file transacti...
by Alon Bar-Lev 29 Jan '13

29 Jan '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: filetransaction: support unsafe visible file transaction ...................................................................... core: filetransaction: support unsafe visible file transaction In some cases there are utilities that need to use the file that is writing during transaction, this implies that we must modify the original file. In the unsafe visible mode we do exactly that, and revert by moving the backup file to original file. Change-Id: I66af1694922730a377257a907a9a80f4d071ec74 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/otopi/filetransaction.py 1 file changed, 68 insertions(+), 22 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11515 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66af1694922730a377257a907a9a80f4d071ec74 Gerrit-PatchSet: 3 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: core: plugin: add executable and cwd to debug of execute
by Alon Bar-Lev 29 Jan '13

29 Jan '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: plugin: add executable and cwd to debug of execute ...................................................................... core: plugin: add executable and cwd to debug of execute Change-Id: I6b6631ddf25f9863143bf9470eb85e449c99d5d1 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/otopi/plugin.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11514 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b6631ddf25f9863143bf9470eb85e449c99d5d1 Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: Make release number based on date
by juan.hernandez＠redhat.com 29 Jan '13

29 Jan '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: Make release number based on date ...................................................................... packaging: Make release number based on date This patch changes the default release number of the packages generated with "make rpm" so that they have the following format: 0.1.$(shell date +%Y%m%d%H%M%S) This has the advantage that builds in development environment can be easily updated and also means that any release we do will always be greater. Change-Id: Ia9820caa71723e929cae5fb873013e52acce221c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9704 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia9820caa71723e929cae5fb873013e52acce221c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: use APP_NAME in /etc/exports comments
by juan.hernandez＠redhat.com 29 Jan '13

29 Jan '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: use APP_NAME in /etc/exports comments ...................................................................... packaging: use APP_NAME in /etc/exports comments when configuring /etc/exports use APP_NAME instead of the hardcoded value "rhev installer". Change-Id: Ie7c343408d586f756251247da51a3412a5a9afdd Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=904149#c3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/fedora/setup/engine-setup.py 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11506 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7c343408d586f756251247da51a3412a5a9afdd Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: tools: Adding Setting ldap servers for domain
by yzaslavs＠redhat.com 29 Jan '13

29 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Adding Setting ldap servers for domain ...................................................................... tools: Adding Setting ldap servers for domain This patch adds support for setting ldap servers for a given domain when adding/editting a domain. This patch also removes some main methods which are no longer in use (all domain management must be done using the main method of ManageDomains) Bug-url: https://bugzilla.redhat.com/894681 change-id: Ic0904f9f6d9432ecac1cebee2790dae902a1329c Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/conf/kerberos/engine-manage-domains M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ipa/SimpleAuthenticationCheck.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ConfigurationProvider.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/DomainsConfigurationEntry.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KerberosConfigCheck.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KrbConfCreator.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ManageDomains.java 8 files changed, 190 insertions(+), 153 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11065 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0904f9f6d9432ecac1cebee2790dae902a1329c Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Supporting hard coded ldap servers per domain
by yzaslavs＠redhat.com 29 Jan '13

29 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Supporting hard coded ldap servers per domain ...................................................................... core: Supporting hard coded ldap servers per domain Adding support for hard coded ldap server list per domain. If such a list is configured for a given domain, no DNS SRV lookup will be made to obtain the servers list. This patch also removes ordering the ldap servers by score, and uses the ordering algorithms instead. Change-Id: Ic0914f2f6d9432ecac0cebee3790eae902a1329a Bug-Url: https://bugzilla.redhat.com/894681 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/DirectorySearcher.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/Domain.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/GetRootDSETask.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapSearchExceptionHandler.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapSearchExceptionHandlingResponse.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/ScorableLDAPServer.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/Score.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/UsersDomainsCacheManagerService.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/dns/DnsSRVLocator.java 9 files changed, 141 insertions(+), 158 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11064 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0914f2f6d9432ecac0cebee3790eae902a1329a Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Adding ldap server ordering algorithms factory
by yzaslavs＠redhat.com 29 Jan '13

29 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Adding ldap server ordering algorithms factory ...................................................................... core: Adding ldap server ordering algorithms factory The following adds ldap server ordering logic as preparation for using it instead of Ldap server scoring. This is required in order to support hardcoded ldap server list per domain, and let the ordering of server effect the scording Bug-Url: https://bugzilla.redhat.com/894681 Change-Id: Ic0904f9f6d9431ecac1cebee2790eae902a1329c Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/serverordering/LdapServersNoOpOrderingAlgorithm.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/serverordering/LdapServersOrderingAlgorithm.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/serverordering/LdapServersOrderingAlgorithmFactory.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/serverordering/LdapServersPutAtLastPlaceAlgorithm.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/serverordering/OrderingAlgorithmType.java 5 files changed, 67 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11063 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0904f9f6d9431ecac1cebee2790eae902a1329c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Removal of DNS root from host name
by yzaslavs＠redhat.com 29 Jan '13

29 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Removal of DNS root from host name ...................................................................... core: Removal of DNS root from host name This fix removes the DNS root name from the host name. The DNS root name appears as "." at the end of the host name. This patch also address the kerberos error when adding a second domain. Change-Id: I14ffafec4853f23a3bfa50793febafbbf97321f2 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/ldap/LdapSRVLocator.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/dns/DnsSRVLocator.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/KDCLocator.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/ManageDomains.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/KDCLocatorTest.java 6 files changed, 38 insertions(+), 25 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11335 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14ffafec4853f23a3bfa50793febafbbf97321f2 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: updated the upgrade to use engine 'maint' mode
by juan.hernandez＠redhat.com 29 Jan '13

29 Jan '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: updated the upgrade to use engine 'maint' mode ...................................................................... packaging: updated the upgrade to use engine 'maint' mode * Updated the implementaion of the maintenance mode. Now instead of starting the engine with different ports we use the "real" maintenance mode provided by the engine and implemented with the commit [1]. * Added a use of utility that checks the system and finds zombie async tasks. If such tasks are found, we clean them, including the related compensasions. [1] SHA 9173096a1a6324045d13429ce9d704b7cc97d2ef Patch: http://gerrit.ovirt.org/#/c/8669 Bug-URL: https://bugzilla.redhat.com/868639 Bug-Url: https://bugzilla.redhat.com/868642 Change-Id: Iad4fdd62493085b7ecb4d51fb54e8057418075e1 Signed-off-by: Alex Lourie <alourie(a)redhat.com> --- M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/common_utils.py M packaging/fedora/setup/engine-upgrade.py M packaging/fedora/setup/output_messages.py 4 files changed, 67 insertions(+), 8 deletions(-) Approvals: Alex Lourie: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iad4fdd62493085b7ecb4d51fb54e8057418075e1 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: improve import conflict (for vm) dialog
by alkaplan＠redhat.com 29 Jan '13

29 Jan '13

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: improve import conflict (for vm) dialog ...................................................................... webadmin: improve import conflict (for vm) dialog The dialog which is shown when trying to import VM that already exists in the system is modified: 1. the term "clone" was replaced with a string that better express what happens during the cloning operation 2. the term "suffix:" was replaced with a string that better express that the given string will be the suffix of the cloned VMs 2. the layout was changed to better reflect that the given new name is related to the "import as cloned" option and "Suffix to add to the.." is related to the "apply to all" option Change-Id: Ia6fe863e52e9bad1d620bce75b8252bca4d0d414 Bug-Url: https://bugzilla.redhat.com/878495 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/EntityModelRadioButtonEditor.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationTemplates.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportCloneDialogPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportCloneDialogPopupView.ui.xml 6 files changed, 58 insertions(+), 25 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Arik Hadas: Verified -- To view, visit http://gerrit.ovirt.org/11435 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6fe863e52e9bad1d620bce75b8252bca4d0d414 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0