Skip to content

Comments

Fix linode_interfaces None issue#729

Merged
yec-akamai merged 3 commits intolinode:devfrom
yec-akamai:fix-release-issue
Nov 21, 2025
Merged

Fix linode_interfaces None issue#729
yec-akamai merged 3 commits intolinode:devfrom
yec-akamai:fix-release-issue

Conversation

@yec-akamai
Copy link
Contributor

@yec-akamai yec-akamai commented Nov 21, 2025

📝 Description

linode_interfaces can be None in some cases. Adding a statement to make sure it's not None to avoid breaking the linode creation.

✔️ How to Test

make test-int TEST_SUITE=firewall_basic TEST_ARGS=-vvv

@yec-akamai yec-akamai requested a review from a team as a code owner November 21, 2025 20:59
@yec-akamai yec-akamai added the bugfix for any bug fixes in the changelog. label Nov 21, 2025
@yec-akamai yec-akamai requested review from lgarber-akamai and vshanthe and removed request for a team November 21, 2025 21:00
Copy link
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.

Looks great, thanks for fixing this!

@yec-akamai yec-akamai merged commit b5029b5 into linode:dev Nov 21, 2025
13 checks passed
@yec-akamai yec-akamai deleted the fix-release-issue branch November 21, 2025 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix for any bug fixes in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants