Change in vdsm[master]: net: dhclient.is_active result caching due to slowdown
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: net: dhclient.is_active result caching due to slowdown
......................................................................
Patch Set 1:
(4 comments)
https://gerrit.ovirt.org/#/c/67544/1/lib/vdsm/utils.py
File lib/vdsm/utils.py:
PS1, Line 500: backoff
could you point me to where this name is used?
PS1, Line 503: leave
live
PS1, Line 503: time passed
expired
PS1, Line 504: from
since
--
To view, visit https://gerrit.ovirt.org/67544
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I30cf0bf24ec204245e1f7ec70317fdfbded08099
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes
6 years, 6 months
Change in vdsm[refs/meta/config]: Modify Permissions for VDSM project
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: Modify Permissions for VDSM project
......................................................................
Patch Set 6: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/66573
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I62c7be095056102c0701f1c6017dbae53dcd0a17
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: refs/meta/config
Gerrit-Owner: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
6 years, 6 months
Change in vdsm[master]: autoconf: make data-center location customizable
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: autoconf: make data-center location customizable
......................................................................
autoconf: make data-center location customizable
configure.ac now accepts a parameter (--with-data-center)
which sets the location of data-center.
changing the default data-center location will break backward
compatibility with existing ovirt installation, and is not supported.
We are adding this to be compatible with fedora file system layout.
Change-Id: If2498e473a11a180254326c6ad9ed9b117bdaec7
Signed-off-by: Leon Goldberg <lgoldber(a)redhat.com>
Bug-Id: https://bugzilla.redhat.com/1260743
---
M configure.ac
M contrib/koji_build.sh
M vdsm.spec.in
3 files changed, 14 insertions(+), 6 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Leon Goldberg: Verified
--
To view, visit https://gerrit.ovirt.org/67248
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If2498e473a11a180254326c6ad9ed9b117bdaec7
Gerrit-PatchSet: 38
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
6 years, 6 months
Change in vdsm[master]: autoconf: make data-center location customizable
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: autoconf: make data-center location customizable
......................................................................
Patch Set 38: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/67248
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If2498e473a11a180254326c6ad9ed9b117bdaec7
Gerrit-PatchSet: 38
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
6 years, 6 months
Change in vdsm[master]: virt: Don't crash on missing <memory> in MutableDomainDescri...
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: Don't crash on missing <memory> in MutableDomainDescriptor
......................................................................
virt: Don't crash on missing <memory> in MutableDomainDescriptor
During minidom->etree conversion changes and rebases, <memory> element stopped
being optional in MutableDomainDescriptor.get_memory_size. This patch restores
the original behavior and adds a test for it.
Change-Id: I338956c5f27f7b670e6ef7854ede1f393e66713f
Signed-off-by: Milan Zamazal <mzamazal(a)redhat.com>
---
M tests/domainDescriptorTests.py
M vdsm/virt/domain_descriptor.py
2 files changed, 25 insertions(+), 2 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Francesco Romani: Looks good to me, approved
Milan Zamazal: Verified
--
To view, visit https://gerrit.ovirt.org/67180
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I338956c5f27f7b670e6ef7854ede1f393e66713f
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
6 years, 6 months
Change in vdsm[master]: tests: Avoid global state modification using forked
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: tests: Avoid global state modification using forked
......................................................................
Patch Set 1:
(2 comments)
https://gerrit.ovirt.org/#/c/67338/1/tests/moduleloader_test.py
File tests/moduleloader_test.py:
Line 39: for f in files:
Line 40: utils.touchFile(os.path.join(path, f))
Line 41: utils.touchFile(os.path.join(path, '__init__.py'))
Line 42: sys.path.append(os.path.dirname(path))
Line 43: yield importlib.import_module(os.path.basename(path))
> If we run each test in a new child process, we don't need to clean anything
this is true for all tests. Still, we care to use MonkeyPatch to clean after ourselves!
Line 44:
Line 45: @permutations(
Line 46: [(('a.py', 'b.py'), ('a', 'b')),
Line 47: (('a.py', 'b.py', 'a.pyc', 'a.pyioas'), ('a', 'b')),
Line 47: (('a.py', 'b.py', 'a.pyc', 'a.pyioas'), ('a', 'b')),
Line 48: (('a.py', 'b.py', 'a.pyc', 'a.py'), ('a', 'b')),
Line 49: (('a.py', 'b.py', 'a.pyc', '_my.py'), ('a', 'b', '_my'))]
Line 50: )
Line 51: @forked
> Importing stuff into python leave traces in sys.modules that are not easy (
I'm anxious to see Travis green, and thank you for all these cleanups. I simply don't understand how the traceback that we see is related to a stale sys.path, so this patch seems a bit of black magic to me...
Yes, I'd love to mark that test as broken (but the patch doing it is high on a chain of other fixes)
Line 52: def test_import_modules(self, files, expected_modules):
Line 53: with self._setup_test_modules(files) as module_name:
Line 54: result = moduleloader.load_modules(module_name)
Line 55:
--
To view, visit https://gerrit.ovirt.org/67338
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2130bcf39918c3c70ed9f8eb9c436c5c8f9c6c09
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Irit Goihman <igoihman(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes
6 years, 6 months
Change in vdsm[master]: containers: tests: reduce duplication
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: containers: tests: reduce duplication
......................................................................
containers: tests: reduce duplication
Kill some leftovers from the time convirt was an external package.
Instead of borrowing code, let's just use facilities from testlib.
Change-Id: I237fd36f9e4246f76afe53fb301b4d27172d96ea
Signed-off-by: Francesco Romani <fromani(a)redhat.com>
---
M tests/containers/conttestlib.py
1 file changed, 5 insertions(+), 21 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Verified
Milan Zamazal: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/67124
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I237fd36f9e4246f76afe53fb301b4d27172d96ea
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
6 years, 6 months
Change in vdsm[master]: containers: tests: reduce duplication
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: containers: tests: reduce duplication
......................................................................
Patch Set 10: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/67124
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I237fd36f9e4246f76afe53fb301b4d27172d96ea
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
6 years, 6 months
Change in vdsm[master]: containers: tests: reduce duplication
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: containers: tests: reduce duplication
......................................................................
Patch Set 10:
raising
--
To view, visit https://gerrit.ovirt.org/67124
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I237fd36f9e4246f76afe53fb301b4d27172d96ea
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
6 years, 6 months
Change in vdsm[master]: tests: deeper libvirtconnection test
by Code Review
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: tests: deeper libvirtconnection test
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/67494/2/tests/libvirtconnectionTests.py
File tests/libvirtconnectionTests.py:
Line 123: conn = libvirtconnection.get()
Line 124: self.assertRaises(libvirt.libvirtError,
Line 125: conn.getLibVersion)
Line 126: finally:
Line 127: libvirtconnection._clear()
we need to test that
* we attempt to commit suicide if libvirt dies
* target receives events
* ? the underlying libvirt is called ?
--
To view, visit https://gerrit.ovirt.org/67494
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Icca111caa25fdc568b7f188a503c8faa83848f4c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes
6 years, 6 months