Skip to content

Conversation

@jaalah-akamai
Copy link
Contributor

Cloud Manager

Fixed:

  • Filter available regions in Object Gen2 Create Drawer and Access Keys List based on endpoint capabilities (#11432)
  • Region name display in Gen2 warning notices when regions are unavailable due to format mismatch (#11432)

@jaalah-akamai jaalah-akamai added the Release Cloud Manager releases label Dec 19, 2024
@jaalah-akamai jaalah-akamai self-assigned this Dec 19, 2024
@jaalah-akamai jaalah-akamai requested a review from a team as a code owner December 19, 2024 15:12
Co-authored-by: Banks Nussman <115251059+bnussman-akamai@users.noreply.github.com>
hkhalil-akamai
hkhalil-akamai previously approved these changes Dec 19, 2024
@github-actions
Copy link

github-actions bot commented Dec 19, 2024

Coverage Report:
Base Coverage: 86.85%
Current Coverage: 86.85%

cpathipa
cpathipa previously approved these changes Dec 19, 2024
@cypress
Copy link

cypress bot commented Dec 19, 2024

Cloud Manager E2E    Run #7005

Run Properties:  status check failed Failed #7005  •  git commit b00ed18d0f: Merge pull request #11441 from linode/staging
Project Cloud Manager E2E
Branch Review master
Run status status check failed Failed #7005
Run duration 30m 36s
Commit git commit b00ed18d0f: Merge pull request #11441 from linode/staging
Committer Jaalah Ramos
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 2
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 465
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/core/linodes/clone-linode.spec.ts • 1 failed test

View Output Video

Test Artifacts
clone linode > can clone a Linode from Linode details page Screenshots Video
Flakiness  cypress/e2e/core/linodes/linode-config.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Linode Config management > End-to-End > Clones a config Screenshots Video

Copy link
Contributor

@coliu-akamai coliu-akamai left a comment

Choose a reason for hiding this comment

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

re-sanity checked a bunch of things:

  • bucket creating with legacy/gen2 (as mentioned/explained by the notice did have trouble having gen2 buckets appear, but was able to create them)
  • access key creation with legacy/gen2
  • show all access key with legacy/gen2
  • gen2 regions do not show up for account w/o gen2

things are looking good on my end - thanks Jaalah!

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #4 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing465 Passing2 Skipped101m 51s

Details

Failing Tests
SpecTest
switch-linode-state.spec.tsswitch linode state » powers off a linode from details page

Troubleshooting

Use this command to re-run the failing tests:

yarn cy:run -s "cypress/e2e/core/linodes/switch-linode-state.spec.ts"

@jaalah-akamai
Copy link
Contributor Author

There may have been updates to the availability of some of our regions on the backend, Cypress just chooses a region at random for these tests and it’s getting these. These E2E test failures are irrelevant to these changes but I still manually tested that linodes can still be rebooted and powered on. 👍

@jaalah-akamai jaalah-akamai merged commit b00ed18 into master Dec 19, 2024
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release Cloud Manager releases

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

8 participants