August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-iso-uploader[ovirt-iso-uploader-4.0]: ovirt-iso-uploader: added --with-kerberos engine option
by rmartins＠redhat.com 05 Aug '16

05 Aug '16

Rafael Martins has submitted this change and it was merged. Change subject: ovirt-iso-uploader: added --with-kerberos engine option ...................................................................... ovirt-iso-uploader: added --with-kerberos engine option Change-Id: I3a3f9ab278e2f0f2feebc92de9804d7c93f06f1a Bug-Url: https://bugzilla.redhat.com/1363681 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> (cherry picked from commit 29b667f16f58dcfa4b52df260f2a2ee90501a71c) --- M src/__main__.py M src/ovirt-iso-uploader.8 2 files changed, 30 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61908 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a3f9ab278e2f0f2feebc92de9804d7c93f06f1a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-4.0 Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-log-collector[ovirt-log-collector-4.0]: ovirt-log-collector: added --with-kerberos engine option
by rmartins＠redhat.com 05 Aug '16

05 Aug '16

Rafael Martins has submitted this change and it was merged. Change subject: ovirt-log-collector: added --with-kerberos engine option ...................................................................... ovirt-log-collector: added --with-kerberos engine option Change-Id: I6960a9a7bde8944f2884f944ef170dccf059bc27 Bug-Url: https://bugzilla.redhat.com/1363684 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> (cherry picked from commit dfaf35675bee3da1c53b4fd74b816efafa13d070) --- M src/__main__.py M src/helper/hypervisors.py M src/ovirt-log-collector.8 3 files changed, 36 insertions(+), 11 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61906 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6960a9a7bde8944f2884f944ef170dccf059bc27 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-4.0 Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in otopi[master]: minidnf: drop fill_sack on clean
by didi＠redhat.com 05 Aug '16

05 Aug '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: minidnf: drop fill_sack on clean ...................................................................... minidnf: drop fill_sack on clean In clean method we called fill_sack right after expiring metadata. This caused a SIGABRT to be raised by one of the C libraries used by DNF. We've not completely understood why this happen but we can safely drop the call to fill_sack here. The clear operation is called within a minidnf.transaction so the sack will be destroyed right after the cleaning. Next operations are still within another minidnf.transaction so the sack will be filled again in a fresh new base object. Change-Id: I05755de3406a811434236e020061bdf3ad1680ff Bug-Url: https://bugzilla.redhat.com/1361888 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/otopi/minidnf.py 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved; Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61975 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I05755de3406a811434236e020061bdf3ad1680ff Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: packaging: drop FIX_RELEASE handling
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: drop FIX_RELEASE handling ...................................................................... packaging: drop FIX_RELEASE handling When FIX_RELEASE was added to version.mak, the intention was to allow to release an ovirt-engine package without any modification to the java code, allowing to avoid to bump the version in the pom files. The intention was good but as per package maintenance we keep aligned pom version and package version anyway when we release. FIX_RELEASE has become meaningless and source of human errors, therefore we're dropping it. That done, POM_VERSION, APP_VERSION and RPM_VERSION became redundant, dropped all of them and using the unique value ENGINE_VERSION. Kept only PACKAGE_VERSION which differ by the MILESTONE suffix. Change-Id: I4ff946551dc52d4a278976fa1cd049697c7496a3 Bug-Url: https://bugzilla.redhat.com/1354180 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M Makefile M version.mak 2 files changed, 4 insertions(+), 13 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61949 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ff946551dc52d4a278976fa1cd049697c7496a3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Ido Rosenzwig <irosenzw(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: automation: use 4.0 repos
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: use 4.0 repos ...................................................................... automation: use 4.0 repos Change-Id: I86698e34eeba0590078f351310d67969df41ade9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.repos M automation/check-merged.repos M automation/check-patch.repos 3 files changed, 6 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61968 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86698e34eeba0590078f351310d67969df41ade9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: Updated README.environment
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Updated README.environment ...................................................................... Updated README.environment Update README.environment with keys added up to 1.5.1 Updated some modules doc-text to align with existing moduels documentation style Change-Id: Id6ba44b62406dbc42ae0b7cc732b7e26639a03e1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1332586 (cherry picked from commit 6cd75db76065967147828c7a9545b6b4925b9c55) --- M README.environment M src/plugins/ovirt-host-common/hosted-engine/packages.py M src/plugins/ovirt-host-deploy/gluster/nrpe.py M src/plugins/ovirt-host-deploy/tune/tuned.py 4 files changed, 112 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61962 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6ba44b62406dbc42ae0b7cc732b7e26639a03e1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: doc: update README.environment with vmconsole env
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: doc: update README.environment with vmconsole env ...................................................................... doc: update README.environment with vmconsole env Added VM Console environment documentation to README.environment. Bug-Url: https://bugzilla.redhat.com/1332586 Change-Id: I3505ddb3cde6585391f841a3b2a2a7d556ffdaee Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit b2700485fd8992dd770fc808ff8c201ad5893897) --- M README.environment 1 file changed, 27 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61961 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3505ddb3cde6585391f841a3b2a2a7d556ffdaee Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: vmconsole: pki: initialize VMCONSOLE/caKey
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: vmconsole: pki: initialize VMCONSOLE/caKey ...................................................................... vmconsole: pki: initialize VMCONSOLE/caKey Initialize to None the env VMCONSOLE/caKey. In standard execution it's not an issue since the variable is used only if initialized, but the missing initialization is formally incorrect. Change-Id: I5a625f3774c49c0ef7efc8beba69ded0333a064c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 8b02cef1ae0e2ae076ac60591b3ff315f59c713c) --- M src/plugins/ovirt-host-common/vmconsole/pki.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61960 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a625f3774c49c0ef7efc8beba69ded0333a064c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: automation: use 4.0 repos
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: use 4.0 repos ...................................................................... automation: use 4.0 repos Change-Id: Ieef6a4f3387ffa7305d8f988c8dfc7fd61d00575 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.repos M automation/check-merged.repos M automation/check-patch.repos 3 files changed, 6 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61963 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieef6a4f3387ffa7305d8f988c8dfc7fd61d00575 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Added the ability to specify the distro of the chroot for te...
by sbonazzo＠redhat.com 05 Aug '16

05 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added the ability to specify the distro of the chroot for test-repo_ovirt_experimental jobs ...................................................................... Added the ability to specify the distro of the chroot for test-repo_ovirt_experimental jobs Change-Id: I7fd5be14a869f96303f6e0c3cef5e12a90c1f362 Signed-off-by: gbenhaim <galbh2(a)gmail.com> --- M jobs/confs/groovy-scripts/test-ovirt-repo.groovy M jobs/confs/projects/ovirt/test-experimental-latest-repo.yaml 2 files changed, 10 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Gal Ben Haim: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61958 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fd5be14a869f96303f6e0c3cef5e12a90c1f362 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gal Ben Haim <galbh2(a)gmail.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gal Ben Haim <galbh2(a)gmail.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0