Skip to content

test [M3-9569]: Fix for OBJ multicluster delete test app crash in DevCloud - #12085

Merged
dmcintyr-akamai merged 2 commits into
linode:developfrom
dmcintyr-akamai:M3-9569_multicluster_delete_devcloud
Apr 22, 2025
Merged

test [M3-9569]: Fix for OBJ multicluster delete test app crash in DevCloud#12085
dmcintyr-akamai merged 2 commits into
linode:developfrom
dmcintyr-akamai:M3-9569_multicluster_delete_devcloud

Conversation

@dmcintyr-akamai

@dmcintyr-akamai dmcintyr-akamai commented Apr 22, 2025

Copy link
Copy Markdown
Contributor

Description 📝

Fix OBJ multicluster delete test app crash in cypress/e2e/core/objectStorageMulticluster/bucket-delete-multicluster.spec.ts by ensuring an available region is used in the created cluster. Test fn objectStorageBucketFactory.build() defaults to using hardcoded region id of 'us-east' which is not available in all test environments.

Changes 🔄

List any change(s) relevant to the reviewer.

Ensure that objectStorageBucketFactory.build() uses an available region.

How to test 🧪

pnpm cy:run -s "cypress/e2e/core/objectStorageMulticluster/bucket-delete-multicluster.spec.ts" 
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@dmcintyr-akamai
dmcintyr-akamai requested a review from a team as a code owner April 22, 2025 15:50
@dmcintyr-akamai
dmcintyr-akamai requested review from jdamore-linode and removed request for a team April 22, 2025 15:50
@dmcintyr-akamai
dmcintyr-akamai requested a review from a team as a code owner April 22, 2025 15:50
@dmcintyr-akamai
dmcintyr-akamai requested review from abailly-akamai and mjac0bs and removed request for a team April 22, 2025 15:50
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 565 passing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing565 Passing5 Skipped127m 47s

@mjac0bs mjac0bs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks 👍🏼

Screenshot 2025-04-22 at 9 51 14 AM

@mjac0bs mjac0bs added the Add'tl Approval Needed Waiting on another approval! label Apr 22, 2025

@alioso alioso left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great!

@abailly-akamai abailly-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Now for a proper GH user approval ✅

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Apr 22, 2025
@dmcintyr-akamai
dmcintyr-akamai merged commit 1e33017 into linode:develop Apr 22, 2025
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Apr 22, 2025
@mjac0bs mjac0bs added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge!

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants