Skip to content

[v2] tests: Fix TestVLANTransparentCRUD test #3465

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

Merged
merged 1 commit into from
Jul 18, 2025
Merged

Conversation

kayrus
Copy link
Contributor

@kayrus kayrus commented Jul 16, 2025

The 'vlan-transparent' extension is now enabled by default [1]. This is highlighting a bug in the aforementioned test: namely, that we are trying to update an attribute which is read-only [2]. Remove the update step of the job and fix the test.

[1] openstack/neutron@11ff4f2
[2] https://github.com/openstack/neutron-lib/blob/fd011c955dfae1072555c69b6ba742b85f041736/neutron_lib/api/definitions/vlantransparent.py#L49

The 'vlan-transparent' extension is now enabled by default [1]. This is
highlighting a bug in the aforementioned test: namely, that we are
trying to update an attribute which is read-only [2]. Remove the update
step of the job and fix the test.

[1] openstack/neutron@11ff4f2
[2] https://github.com/openstack/neutron-lib/blob/fd011c955dfae1072555c69b6ba742b85f041736/neutron_lib/api/definitions/vlantransparent.py#L49

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
@github-actions github-actions bot added edit:networking This PR updates networking code v2 This PR targets v2 semver:patch No API change labels Jul 16, 2025
@kayrus kayrus requested a review from stephenfin July 16, 2025 20:40
@kayrus kayrus merged commit b2cff9d into gophercloud:v2 Jul 18, 2025
13 of 16 checks passed
@kayrus kayrus deleted the fix-vlan branch July 18, 2025 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
edit:networking This PR updates networking code semver:patch No API change v2 This PR targets v2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants