Dan Kenigsberg has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 3: Code-Review+2
copying score and raising
--
To view, visit https://gerrit.ovirt.org/56449
To
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: OVS driver - bond_slave commands
..
net: OVS driver - bond_slave commands
The *bond_slave commands are a bit special, they are composed of two
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 4:
* #1195208::Update tracker: OK
* Set MODIFIED::bug 1195208#1195208IGNORE, not all related patches
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 21:
* #1195208::Update tracker: OK
* Set MODIFIED::bug 1195208#1195208IGNORE, not all related patches are
closed, check
Dan Kenigsberg has submitted this change and it was merged.
Change subject: net: OVS driver
..
net: OVS driver
This patch introduces OVS driver, its API and ovs-vsctl implementation.
The openvswitch is added as a build and
Dan Kenigsberg has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 20: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
Edward Haas has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 3:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 20: Verified+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 20:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/55958/19/tests/network/ovs_driver_test.py
File tests/network/ovs_driver_test.py:
PS19,
Dan Kenigsberg has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe, visit
Dan Kenigsberg has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 19:
(1 comment)
https://gerrit.ovirt.org/#/c/55958/19/tests/network/ovs_driver_test.py
File tests/network/ovs_driver_test.py:
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 19: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 19: Verified+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 19:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 18: -Verified
(4 comments)
https://gerrit.ovirt.org/#/c/55958/18/lib/vdsm/network/ovs/driver/vsctl.py
File
Dan Kenigsberg has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 18: Code-Review-1
(4 comments)
https://gerrit.ovirt.org/#/c/55958/18/lib/vdsm/network/ovs/driver/vsctl.py
File
Edward Haas has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe, visit
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe, visit
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 2: Verified+1
Passed ovs_driver_test.py
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe,
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 2:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 1: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/56449/1/lib/vdsm/network/ovs/driver/vsctl.py
File
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 17:
(1 comment)
https://gerrit.ovirt.org/#/c/55958/17/automation/check-patch.repos.el7
File automation/check-patch.repos.el7:
Line
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe, visit
Edward Haas has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/56449
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver - bond_slave commands
..
Patch Set 1:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check
Edward Haas has uploaded a new change for review.
Change subject: net: OVS driver - bond_slave commands
..
net: OVS driver - bond_slave commands
The *bond_slave commands are a bit special, they are composed of two
sub-commands.
Yaniv Bronhaim has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 17:
(1 comment)
https://gerrit.ovirt.org/#/c/55958/17/automation/check-patch.repos.el7
File automation/check-patch.repos.el7:
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 18: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 18: Verified+1
(1 comment)
https://gerrit.ovirt.org/#/c/55958/17/lib/vdsm/network/ovs/driver/vsctl.py
File
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 18:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 17:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 16: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/55958/16/lib/vdsm/network/ovs/driver/vsctl.py
File
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 16: Code-Review+1 Verified+1
Passed ovs_driver_tests.py and manual testing.
--
To view, visit https://gerrit.ovirt.org/55958
To
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 16:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 15:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 14:
* #1195208::Update tracker: OK
* Check Bug-Url::OK
* Check Public Bug::#1195208::OK, public bug
* Check Product::#1195208::OK,
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 13: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 13:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 12:
I need to test simple OVS patch somehow. For example, I need to test, that OVS
configuration rejects usage of non-existing
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 12:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 11: Code-Review-1
This last change is a bit out of scope for this patch. We need the setup and
teardown preparations for integration
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 11:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 10: Code-Review-1
I think we should create OvsCaseBase. The service mechanics and bridge removal
would be needed also in other
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 10: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 10: Code-Review+1 Verified+1
(2 comments)
Passed ovs_driver_test.py
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 10:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 9:
(2 comments)
https://gerrit.ovirt.org/#/c/55958/9/lib/vdsm/network/ovs/driver/__init__.py
File
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 9: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/55958/9/lib/vdsm/network/ovs/driver/vsctl.py
File
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 9: Verified+1
Passed ovs_driver_test.py
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 9:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 8:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 7:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 6: Code-Review-1
(1 comment)
https://gerrit.ovirt.org/#/c/55958/6/lib/vdsm/network/ovs/driver/vsctl.py
File
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 6: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit https://gerrit.ovirt.org/settings
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 6: Verified+1
Integration tests passed
--
To view, visit https://gerrit.ovirt.org/55958
To unsubscribe, visit
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 6:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 5:
(1 comment)
https://gerrit.ovirt.org/#/c/55958/5/tests/network/ovs_driver_test.py
File tests/network/ovs_driver_test.py:
Line
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/55958/2/lib/vdsm/network/ovs/configurators/vsctl.py
File
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 5:
(10 comments)
https://gerrit.ovirt.org/#/c/55958/5//COMMIT_MSG
Commit Message:
PS5, Line 11: build
> If it is a build
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 5: Code-Review-1
(4 comments)
tiny notes
https://gerrit.ovirt.org/#/c/55958/5/lib/vdsm/network/ovs/driver/__init__.py
File
Petr Horáček has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 2:
(7 comments)
https://gerrit.ovirt.org/#/c/55958/5//COMMIT_MSG
Commit Message:
PS5, Line 11:
If it is a build dependency, why
Edward Haas has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 5: Verified+1
Integration tests are passing and Functional tests have not been affected.
--
To view, visit
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 5:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 4:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
gerrit-hooks has posted comments on this change.
Change subject: net: OVS driver
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: '
and is
67 matches
Mail list logo
