Skip to content

fix: update ibm-vpc dependencies per breaking change in 0.28.0 #485

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 8, 2025

Conversation

blackboxsw
Copy link
Collaborator

Release of ibm-vpc 0.28.0 forces providing a known API version on VpcV1 init. API versions can be determined at
https://github.com/IBM/vpc-python-sdk/releases/.

Continue bump setup.cfg contraints in coordination with an ibm-vpc API version change to avoid unexpected integration test failures when subsequent releases of ibm-vpc are published.

Description

Additional Context and Relevant Issues

Test Steps

@blackboxsw blackboxsw force-pushed the fix-ibm-vpc-version-required branch from 161520b to 4583105 Compare May 8, 2025 18:29
Release of ibm-vpc 0.28.0 forces providing a known API version on
VpcV1 __init__. API versions can be determined at
https://github.com/IBM/vpc-python-sdk/releases/.

Continue bump setup.cfg contraints in coordination with an
ibm-vpc API version change to avoid unexpected integration test
failures when subsequent releases of ibm-vpc are published.
Copy link
Member

@TheRealFalcon TheRealFalcon left a comment

Choose a reason for hiding this comment

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

LGTM!

@blackboxsw blackboxsw requested a review from uhryniuk May 8, 2025 18:31
@blackboxsw
Copy link
Collaborator Author

cc: @uhryniuk as CPC ibm runners probably would have gotten broken by this too

@blackboxsw blackboxsw merged commit 920a5a3 into canonical:main May 8, 2025
7 checks passed
@uhryniuk
Copy link
Contributor

uhryniuk commented May 8, 2025

cc: @uhryniuk as CPC ibm runners probably would have gotten broken by this too

@blackboxsw Thanks for the heads up! 🫡

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.

3 participants