Skip to content

Region adjustment #1256

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
Jan 3, 2025
Merged

Region adjustment #1256

merged 1 commit into from
Jan 3, 2025

Conversation

Fyusel
Copy link
Contributor

@Fyusel Fyusel commented Jan 2, 2025

STACKIT will move to a new way of specifying regions with v1. Instead of providing the region in the URL directly it is provided via path parameter. Once all services have migrated, the methods to specify the region in the client configuration will be removed.

@Fyusel Fyusel force-pushed the ft/region-adjustment branch from 3e8ce8e to f713b31 Compare January 3, 2025 08:14
@Fyusel Fyusel marked this pull request as ready for review January 3, 2025 08:22
@Fyusel Fyusel force-pushed the ft/region-adjustment branch 3 times, most recently from 4c80ffd to ec2541e Compare January 3, 2025 10:01
@Fyusel Fyusel force-pushed the ft/region-adjustment branch from ec2541e to 50eb3fe Compare January 3, 2025 10:30
@Fyusel Fyusel force-pushed the ft/region-adjustment branch from 50eb3fe to c5d6d96 Compare January 3, 2025 11:52
STACKIT will move to a new way of specifying regions, where the region is provided
as a function argument instead of being set in the client configuration.
Once all services have migrated, the methods to specify the region in the client
configuration will be removed.

Signed-off-by: Alexander Dahmen <[email protected]>
@Fyusel Fyusel force-pushed the ft/region-adjustment branch from c5d6d96 to b894b65 Compare January 3, 2025 11:57
@Fyusel Fyusel merged commit 0915ef4 into main Jan 3, 2025
12 checks passed
@Fyusel Fyusel deleted the ft/region-adjustment branch January 3, 2025 13:27
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