Skip to content

[v2] tests: Fix TestVLANTransparentCRUD test#3465

Merged
kayrus merged 1 commit intogophercloud:v2from
kayrus:fix-vlan
Jul 18, 2025
Merged

[v2] tests: Fix TestVLANTransparentCRUD test#3465
kayrus merged 1 commit intogophercloud:v2from
kayrus:fix-vlan

Conversation

@kayrus
Copy link
Copy Markdown
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