Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9:
Any other suggestion besides renaming export_xml for the upcoming rebase?
--
To view, visit https:/
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/9/vdsm/virt/domain_descriptor.py
File vdsm/virt/domain_d
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9: Code-Review+1
let's not hide other score.
--
To view, visit https://gerrit.ovirt.org/55769
To un
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/9/vdsm/virt/domain_descriptor.py
File vdsm/virt/domai
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9:
(1 comment)
Very partial review, will look at it again next week.
https://gerrit.ovirt.org/#/c/55769/9
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9: Code-Review+2
*Really* nice improvements. I like this.
--
To view, visit https://gerrit.ovirt.or
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 9:
(10 comments)
I tried to address all the suggestions for improvements, so hopefully we can
move for
Milan Zamazal has uploaded a new change for review.
Change subject: virt: Make DomainDescriptor use XML helpers
..
virt: Make DomainDescriptor use XML helpers
We are going to stop using xml.dom.minidom and to use xml.etree inste
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(7 comments)
Thank you all for the comments and suggestions, I hope things get clarified and
we can
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(9 comments)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 78:
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Lin
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
> Not having a value is an exceptional situation,
No, it's not an exceptional situation when we know
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 7
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 78:
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 7
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 78:
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 7
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(2 comments)
https://gerrit.ovirt.org/#/c/55769/8/vdsm/virt/vmxml.py
File vdsm/virt/vmxml.py:
Line 78:
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(2 comments)
https://gerrit.ovirt.org/#/c/55769/8//COMMIT_MSG
Commit Message:
Line 33: DomainDescri
Nir Soffer has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
(7 comments)
https://gerrit.ovirt.org/#/c/55769/8//COMMIT_MSG
Commit Message:
Line 33: DomainDescripto
Martin Polednik has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
Something makes me feel uneasy about this particular patch (not series as a
whole) and it's that i
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 7:
(2 comments)
https://gerrit.ovirt.org/#/c/55769/7/tests/vmXmlTests.py
File tests/vmXmlTests.py:
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55769
To unsubscribe, visit https://gerr
gerrit-hooks has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 8:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header m
Milan Zamazal has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 7:
(2 comments)
https://gerrit.ovirt.org/#/c/55769/7/tests/vmXmlTests.py
File tests/vmXmlTests.py:
Lin
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 7:
(1 comment)
https://gerrit.ovirt.org/#/c/55769/7/tests/vmXmlTests.py
File tests/vmXmlTests.py:
L
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 7: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/55769/7/vdsm/virt/vmxml.py
File vdsm/virt/
gerrit-hooks has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 7:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header m
gerrit-hooks has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 6:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header m
Francesco Romani has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 3:
(1 comment)
I'm convinced that adding a (thin) wrapper layer is a good direction, probably
our b
gerrit-hooks has posted comments on this change.
Change subject: virt: Make DomainDescriptor use XML helpers
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header m
32 matches
Mail list logo