Filter regions based on account availabilities in get_regions function#625
Filter regions based on account availabilities in get_regions function#625zliang-akamai merged 5 commits intolinode:devlinode/linode_api4-python:devfrom zliang-akamai:zhiwei/filter-regions-with-account-availabilitieszliang-akamai/linode_api4-python:zhiwei/filter-regions-with-account-availabilitiesCopy head branch name to clipboard
Conversation
|
Should be bypassed if the token doesn't have account API access. Converting to draft to work on it. |
lgarber-akamai
left a comment
There was a problem hiding this comment.
LGTM other than the comment above, great work!
jriddle-linode
left a comment
There was a problem hiding this comment.
Works locally, looks good!
There was a problem hiding this comment.
Pull request overview
This PR enhances the get_regions function to filter regions based on account-level availability constraints, helping to reduce intermittent errors during integration tests by ensuring selected regions have the necessary capabilities enabled for the account.
Key Changes:
- Added account availability checking to filter regions based on account-specific capabilities
- Added error handling with logging for cases where account availability retrieval fails
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lgarber-akamai
left a comment
There was a problem hiding this comment.
LGTM, great work!
📝 Description
This may be helpful to reduce intermittent errors during integration tests.
✔️ How to Test
Run any test you like, for example: