Skip to content

TPT-4424: Use e2e firewall in all linode interface tests#685

Merged
zliang-akamai merged 1 commit intodevfrom
zhiwei/fix-firewall-test
Apr 16, 2026
Merged

TPT-4424: Use e2e firewall in all linode interface tests#685
zliang-akamai merged 1 commit intodevfrom
zhiwei/fix-firewall-test

Conversation

@zliang-akamai
Copy link
Copy Markdown
Member

@zliang-akamai zliang-akamai commented Apr 16, 2026

📝 Description

The API no longer accepts null firewall ID so we have to use explicit firewall ID now.

pytest test/integration/models/linode/interfaces/test_interfaces.py -v

@zliang-akamai zliang-akamai requested review from a team as code owners April 16, 2026 18:17
@zliang-akamai zliang-akamai requested review from ckulinsk and jbilskiAkam and removed request for a team April 16, 2026 18:17
@zliang-akamai zliang-akamai added the testing for updates to the testing suite in the changelog. label Apr 16, 2026
@zliang-akamai zliang-akamai changed the title Use e2e firewall in all linode interface tests TPT-4424: Use e2e firewall in all linode interface tests Apr 16, 2026
Copy link
Copy Markdown
Contributor

@yec-akamai yec-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test passed locally!

Copy link
Copy Markdown
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing this!

@zliang-akamai zliang-akamai merged commit 9535d63 into dev Apr 16, 2026
17 of 18 checks passed
@zliang-akamai zliang-akamai deleted the zhiwei/fix-firewall-test branch April 16, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants