manager icon indicating copy to clipboard operation
manager copied to clipboard

upcoming: [DI-25385] - Added beta chip for ACLP beta service type

Open nikhagra-akamai opened this issue 6 months ago • 1 comments

Description 📝

Added beta chips in front of service types which are in beta for aclp in metrics & alerts.

Changes 🔄

List any change(s) relevant to the reviewer.

  1. Added beta chip wherever service types are mentioned based on feature flag
  2. Added aclpBetaServices flag in featureFlags.ts
  3. Changed zeroth state message for metrics
  4. Updated empty region message based on service type.

Target release date 🗓️

1st July

Preview 📷

Include a screenshot or screen recording of the change.

:lock: Use the Mask Sensitive Data setting for security.

:bulb: Use <video src="" /> tag when including recordings in table.

Before After
Screenshot 2025-06-16 at 7 48 53 PM Screenshot 2025-06-16 at 7 45 17 PM
Screenshot 2025-06-16 at 7 49 01 PM Screenshot 2025-06-16 at 7 45 00 PM
Screenshot 2025-06-16 at 7 49 06 PM Screenshot 2025-06-16 at 7 45 09 PM
Screenshot 2025-06-16 at 7 49 12 PM Screenshot 2025-06-16 at 7 45 39 PM
Screenshot 2025-06-16 at 7 49 15 PM Screenshot 2025-06-16 at 7 45 43 PM
Screenshot 2025-06-16 at 7 50 02 PM Screenshot 2025-06-16 at 7 46 33 PM

How to test 🧪

  1. Switch to mock user
  2. Control aclpBetaServices flag from dev tools
  3. Got to metrics & alerts and based on flag you'll notice beta chip in front of service type names on different pages.
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

nikhagra-akamai avatar Jun 16 '25 14:06 nikhagra-akamai

Cloud Manager UI test results

:small_red_triangle: 2 failing tests on test run #11 ↗︎

:x: Failing:white_check_mark: Passing:arrow_right_hook: Skipped:clock1: Duration
2 Failing660 Passing4 Skipped139m 18s

Details

Failing Tests
SpecTest
:x:smoke-community-stackscripts.spec.tsCloud Manager Cypress Tests→Community Stackscripts integration tests » Community Stackscripts integration tests
:x:clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » clone linode

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/stackscripts/smoke-community-stackscripts.spec.ts,cypress/e2e/core/linodes/clone-linode.spec.ts"

linode-gh-bot avatar Jun 19 '25 05:06 linode-gh-bot

Merging, as it has enough approvals & cypress failures are unrelated

nikhagra-akamai avatar Jun 19 '25 05:06 nikhagra-akamai