Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update test region capability for LDE#530

Merged
ykim-akamai merged 6 commits intolinode:devlinode/linode_api4-python:devfrom
ykim-akamai:test/update_test_region_ldeykim-akamai/linode_api4-python:test/update_test_region_ldeCopy head branch name to clipboard
Apr 15, 2025
Merged

Update test region capability for LDE#530
ykim-akamai merged 6 commits intolinode:devlinode/linode_api4-python:devfrom
ykim-akamai:test/update_test_region_ldeykim-akamai/linode_api4-python:test/update_test_region_ldeCopy head branch name to clipboard

Conversation

@ykim-akamai
Copy link
Contributor

@ykim-akamai ykim-akamai commented Apr 11, 2025

📝 Description

  • Update test region capability to "LA Disk Encryption"
  • Remove filter condition in test_vpc_create
  • Add Skip to expire Oauth token test (Not sure if it is deprecated but showing a 404 error now)

✔️ How to Test

make test-int TEST_SUITE=lke

PR run - https://github.com/linode/linode_api4-python/actions/runs/14409542816

📷 Preview

If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.

@ykim-akamai ykim-akamai added the testing for updates to the testing suite in the changelog. label Apr 11, 2025
@ykim-akamai ykim-akamai requested a review from a team as a code owner April 11, 2025 16:17
@ykim-akamai ykim-akamai requested review from jriddle-linode and lgarber-akamai and removed request for a team April 11, 2025 16:17
.github/workflows/e2e-test.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@ezilber-akamai ezilber-akamai left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines -26 to -27
assert res

Copy link
Member

Choose a reason for hiding this comment

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

Is this removal intentional?

Copy link
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

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

LGTM other than Zhiwei's feedback. Thanks for fixing this!

@ykim-akamai ykim-akamai merged commit 0ecf79a into linode:dev Apr 15, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.