New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add tests for neutron-openvswitch and neutron-gateway charms #523
Add tests for neutron-openvswitch and neutron-gateway charms #523
Conversation
Logs of a successful execution against charm-neutron-openvswitch with this patch included: After (and if) this is merged I'll be happy to update tests/tests.yaml for both charms or follow up with any other required actions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you. I'll let Alex review as well as I know he's involved in your charm-neutron-openvswitch review.
Thanks Corey! cc @ajkavanagh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see inline comment as a suggestion. Thanks.
734866b
to
cf8f8d6
Compare
* Add support for running NeutronOvsVsctlTest against any charm and not only charm-neutron-gateway. * Add new test case for charm-neutron-gateway and charm-neutron-openvswitch to verify correct handling of conflicting ext-port and data-port configurations. Signed-off-by: Przemysław Lal <przemyslaw.lal@canonical.com>
cf8f8d6
to
64fb160
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; is there a review that verifies that these work?
Not yet, I only linked output from these tests in the comment above. For neutron-openvswitch they'll actually fail without this. charm-neutron-gateway should pass right away. Should I update https://review.opendev.org/c/openstack/charm-neutron-openvswitch/+/778458 and open a new review for charm-neutron-gateway with zaza-openstack-tests in test-requirements.txt pointing to my fork/branch to prove it? Or what's the best approach? |
It just needs a review (and if an existing one is appropriate, that's great), with:
in the commit message. The amended commit message will trigger a charm-recheck which will then pull in the tests from the PR and validate them. Thanks. |
Thanks. Here's the updated neutron-openvswitch review: |
So, these tests are passing test_charm_func_full for charm-neutron-gateway, please see the last report from Canonical CI here: https://review.opendev.org/c/openstack/charm-neutron-gateway/+/780560 Called a charm-full-recheck on https://review.opendev.org/c/openstack/charm-neutron-gateway/+/780560 (previously failed on an unrelated error, but the smoke test passed this time). |
Thanks for your work on this @przemeklal ; much appreciated. |
Add reference to to the new test case that valdiates whether the charm goes in blocked state when data-port and ext-port are specified at the same time. func-test-pr: openstack-charmers/zaza-openstack-tests#523 Signed-off-by: Przemysław Lal <przemyslaw.lal@canonical.com> Change-Id: I2e1ba8ac4c09ea77c39e0337114f67690de6829c
* Update charm-neutron-gateway from branch 'master' to b679984b1fec8dce16a3435465e5cfab907906ff - Add new functional test case for ext-port config option deprecation Add reference to to the new test case that valdiates whether the charm goes in blocked state when data-port and ext-port are specified at the same time. func-test-pr: openstack-charmers/zaza-openstack-tests#523 Signed-off-by: Przemysław Lal <przemyslaw.lal@canonical.com> Change-Id: I2e1ba8ac4c09ea77c39e0337114f67690de6829c
Enable marking of OVS bridges and ports as managed by charm-neutron-openvswitch. This enables more advanced use cases like implmentation of idempotent config changes or cleanup of stale OVS configuration. Deprecate ext-port in favour of data-port and bridge-mappings. From now on ext-port value will be ignored if data-port is specified in the config. Log an error in the unit's log and display it in the unit's status so that the user is aware of misconfiguration. Replace deprecated code that was used for setting up DPDK ports and bonds with calls to charmhelpers functions. Sync charmhelpers. Update and add new unit tests. Add a new functional test case to tests/tests.yaml. Note: This patch is largely a reimplementation of corresponding change for charm-neutron-gateway: https://review.opendev.org/717074 func-test-pr: openstack-charmers/zaza-openstack-tests#523 Closes-Bug: #1917025 Partial-Bug: #1915967 Signed-off-by: Przemysław Lal <przemyslaw.lal@canonical.com> Change-Id: I343f2c8258b7b8b91417dc6efc5bfe95351392a9
Enable marking of OVS bridges and ports as managed by charm-neutron-openvswitch. This enables more advanced use cases like implmentation of idempotent config changes or cleanup of stale OVS configuration. Deprecate ext-port in favour of data-port and bridge-mappings. From now on ext-port value will be ignored if data-port is specified in the config. Log an error in the unit's log and display it in the unit's status so that the user is aware of misconfiguration. Replace deprecated code that was used for setting up DPDK ports and bonds with calls to charmhelpers functions. Sync charmhelpers. Update and add new unit tests. Add a new functional test case to tests/tests.yaml. Note: This patch is largely a reimplementation of corresponding change for charm-neutron-gateway: https://review.opendev.org/717074 func-test-pr: openstack-charmers/zaza-openstack-tests#523 Closes-Bug: #1917025 Partial-Bug: #1915967 Signed-off-by: Przemysław Lal <przemyslaw.lal@canonical.com> Change-Id: I343f2c8258b7b8b91417dc6efc5bfe95351392a9
only charm-neutron-gateway.
verify correct handling of conflicting ext-port and data-port configurations.
Signed-off-by: Przemysław Lal przemyslaw.lal@canonical.com