Skip to content

TPT-4280: Add integration tests for Reserved IP for IPv4#886

Merged
mawilk90 merged 22 commits into
linode:devfrom
mawilk90:feature/TPT-4280-linode-cli-add-integration-tests-for-reserved-ip-for-ipv4
Jul 14, 2026
Merged

TPT-4280: Add integration tests for Reserved IP for IPv4#886
mawilk90 merged 22 commits into
linode:devfrom
mawilk90:feature/TPT-4280-linode-cli-add-integration-tests-for-reserved-ip-for-ipv4

Conversation

@mawilk90

@mawilk90 mawilk90 commented May 8, 2026

Copy link
Copy Markdown
Contributor

📝 Description

Integration tests verifying Reserved IPs CLI commands

NOTE:
PR with openapi.json for Reserved IPs is already merged, however openapi.json file is not updated yet in linode-api-openapi repo so CLI can be build locally only using the latest openapi.json taken from the PR mentioned above

✔️ How to Test

make test-int TEST_SUITE=networking TEST_ARGS="-k reserved"
make test-int TEST_SUITE=tags TEST_ARGS="-k reserved"

@mawilk90 mawilk90 added testing for updates to the testing suite in the changelog. new-feature for new features in the changelog. do-not-merge PRs that should not be merged until the commented issue is resolved labels May 8, 2026
@mawilk90 mawilk90 force-pushed the feature/TPT-4280-linode-cli-add-integration-tests-for-reserved-ip-for-ipv4 branch from e719cab to cb4cd17 Compare May 12, 2026 10:27
@mawilk90 mawilk90 force-pushed the feature/TPT-4280-linode-cli-add-integration-tests-for-reserved-ip-for-ipv4 branch from 600bc51 to ae58218 Compare June 22, 2026 15:12
@mawilk90 mawilk90 marked this pull request as ready for review July 1, 2026 14:01
@mawilk90 mawilk90 requested review from a team as code owners July 1, 2026 14:01
@mawilk90 mawilk90 requested review from ezilber-akamai and psnoch-akamai and removed request for a team July 1, 2026 14:01
Comment thread tests/integration/linodes/helpers.py Outdated

@ezilber-akamai ezilber-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tests look good and pass locally! One small issue though, left a comment about it.

Co-authored-by: Erik Zilber <ezilber@akamai.com>
Comment thread tests/integration/networking/fixtures.py Outdated

@psnoch-akamai psnoch-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, tests work locally

@mawilk90 mawilk90 merged commit 35fb1d8 into linode:dev Jul 14, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge PRs that should not be merged until the commented issue is resolved new-feature for new features in the changelog. 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