bug: [STORIF-221] - corrected a bug where an empty bucket list was di…#13388
Conversation
skulpok-akamai
force-pushed
the
feature/storif-221
branch
10 times, most recently
from
df38181 to
018910a
Compare
skulpok-akamai
force-pushed
the
feature/storif-221
branch
from
018910a to
6c70a6c
Compare
skulpok-akamai
requested review from
cliu-akamai
and removed request for
a team
skulpok-akamai
force-pushed
the
feature/storif-221
branch
from
6c70a6c to
9f9b136
Compare
skulpok-akamai
force-pushed
the
feature/storif-221
branch
4 times, most recently
from
a65043a to
b571653
Compare
skulpok-akamai
force-pushed
the
feature/storif-221
branch
2 times, most recently
from
707c8d7 to
8429515
Compare
Cloud Manager UI test results🎉 0 passing tests on test run #18 ↗︎
|
skulpok-akamai
force-pushed
the
feature/storif-221
branch
from
8429515 to
e021e90
Compare
coliu-akamai
left a comment
coliu-akamai
left a comment
There was a problem hiding this comment.
thanks @skulpok-akamai, looks good!
Note: not sure why the cypress comment isn't displaying any tests, but I went through all the object storage cypress - only object-storage.e2e.spec.ts failed, specifically the first test for creating and deleting object storage buckets. Not sure if it's related especially since this PR didn't touch create/delete flows - I can't remember if this is a known flaky test or not
| </SafeTabPanel> | ||
| )} | ||
| <SafeTabPanel index={bucketsTabIndex}> | ||
| {isObjMultiClusterEnabled ? ( |
There was a problem hiding this comment.
just a quick question - since BucketLanding got removed, are there plans to remove the CreateBucketDrawer component? Seeing that we use isObjMultiClusterEnabled to check which create drawer to use in this file (lines 196-206)
There was a problem hiding this comment.
Yes, we have a separate ticket for that: https://track.akamai.com/jira/browse/STORIF-276
5 participants
…splayed; removed legacy cluster calls
Description 📝
On occasions, Bucket list was being displayed empty, even when the user had some buckets. This was caused by a race condition, in which clusters were loaded before regions or endpoints, which caused to execute the bucket query without any regions to work with.
Changes 🔄
List any change(s) relevant to the reviewer.
Scope 🚢
Upon production release, changes in this PR will be visible to:
Target release date 🗓️
End of Feb.
Preview 📷
Before
Screen.Recording.2026-02-12.at.13.57.55.mov
After
Screen.Recording.2026-02-12.at.13.49.08.mov
How to test 🧪
Prerequisites
Reproduction steps
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
As an Author, before moving this PR from Draft to Open, I confirmed ✅