Skip to content

TPT-3501: update existing tests to validate python sdk support nodebalancer lke cluster and type fields - #716

Open
psnoch-akamai wants to merge 5 commits into
linode:proj/nb-front-end-ip-in-vpcfrom
psnoch-akamai:test/TPT-3501-python-sdk-support-nodebalancer-lke_cluster-and-type-fields
Open

TPT-3501: update existing tests to validate python sdk support nodebalancer lke cluster and type fields#716
psnoch-akamai wants to merge 5 commits into
linode:proj/nb-front-end-ip-in-vpcfrom
psnoch-akamai:test/TPT-3501-python-sdk-support-nodebalancer-lke_cluster-and-type-fields

Conversation

@psnoch-akamai

Copy link
Copy Markdown
Contributor

✔️ How to Test

make test-int TEST_ARGS="-k test_get_nb"
make test-int TEST_ARGS="-k test_update_nb"

@psnoch-akamai
psnoch-akamai requested review from a team as code owners July 14, 2026 08:15
@psnoch-akamai
psnoch-akamai requested review from jriddle-linode and mawilk90 and removed request for a team July 14, 2026 08:15
@psnoch-akamai
psnoch-akamai changed the base branch from dev to proj/nb-front-end-ip-in-vpc July 14, 2026 08:18
@mawilk90

Copy link
Copy Markdown
Contributor

Tests test_get_nb_node and test_update_nb_node fail for me, because there is no nodes in nb config (list index error). Could you please check?

Comment thread test/integration/filters/fixtures.py Outdated


@pytest.fixture(scope="package")
def lke_cluster(test_linode_client):

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.

Could you please add a description to make it easier to distinguish lke_cluster and lke_cluster_instance ? Moreover, lke_cluster fixture needs to be imported in test_nodebalancers.py, otherwise the test test_get_nb_with_lke_cluster will fail

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point, thanks! I added import and rename function to be more descriptive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants