Skip to content

Add: [DI-29170] - Value field selection limitation in CloudPulse metrics and alerts dimension filters#13361

Open
venkymano-akamai wants to merge 14 commits intolinode:developfrom
venkymano-akamai:value_field_dimension_filter_limit
Open

Add: [DI-29170] - Value field selection limitation in CloudPulse metrics and alerts dimension filters#13361
venkymano-akamai wants to merge 14 commits intolinode:developfrom
venkymano-akamai:value_field_dimension_filter_limit

Conversation

@venkymano-akamai
Copy link
Contributor

@venkymano-akamai venkymano-akamai commented Feb 4, 2026

Description 📝

Ability to restrict the maximum selectable values in the CloudPulse metrics and alerts dimension filters.

Changes 🔄

  1. New feature flag maxDimensionFiltersValues for controlling the maximum selectable dimension filter value
  2. Disable select all and deselect all option incase of options greater than max selection limit
  3. Once max items are selected , disable the rest of the options
  4. Display a helper text 'Select up to ${max} values' for dropdown
  5. This applies only for in operator

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️ Next Release date

Preview 📷

Before After
Screenshot 2026-02-04 at 5 09 26 PM Screenshot 2026-02-04 at 5 09 45 PM
Screenshot 2026-02-04 at 5 11 47 PM Screenshot 2026-02-04 at 5 11 37 PM

How to test 🧪

  1. Login into CM and navigate to metrics under monitor section (use devcloud env login)
  2. Make sure you have for a firewall assign multiple linodes like more then 5 is associated (possibly in same region poland, since we use devcloud)
  3. Choose firewall dashboard and select all filters (make sure to select the firewall that has more linode)
  4. Inside the widget, click on dimension filter icon and add a linode dimension filter
  5. After adding 5 values, we can see the options is getting disabled.
  6. The same applies for alert while create and edit flow
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 tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@venkymano-akamai venkymano-akamai changed the title [DI-29170] - Value field selection limitation in CloudPulse metrics and alerts dimension filters Add: [DI-29170] - Value field selection limitation in CloudPulse metrics and alerts dimension filters Feb 4, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the ability to restrict the maximum number of selectable values in CloudPulse metrics and alerts dimension filters, controlled by a feature flag.

Changes:

  • Added maxDimensionFiltersValues feature flag to control selection limits
  • Implemented UI restrictions that disable options once the limit is reached and hide select/deselect all buttons when appropriate
  • Added validation schema to enforce the maximum selection limit for the 'in' operator

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
featureFlags.ts Added maxDimensionFiltersValues property to AclpAlerting interface
CloudPulseDimensionFilterRenderer.tsx Passed feature flag value to form context for validation
CloudPulseDimensionFilterFields.tsx Replaced setValue with resetField to properly reset form state
EditAlertDefinition.tsx Added feature flag to form context for alert editing
utils.ts Added helper functions to check max selections and determine option disabled state
utils.test.ts Added comprehensive test coverage for new helper functions
ValueSchemas.ts Added validation for 'in' operator to enforce max selection limit
ValueFieldRenderer.tsx Retrieved feature flag and passed maxSelections to autocomplete components
[Multiple Autocomplete files] Integrated max selection logic including helper text, disabled options, and select all control
constants.ts Added maxSelections property to DimensionFilterAutocompleteProps interface
.changeset/pr-13361-added-1770207290291.md Added changeset documenting the new feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@venkymano-akamai venkymano-akamai marked this pull request as ready for review February 5, 2026 13:04
@venkymano-akamai venkymano-akamai requested a review from a team as a code owner February 5, 2026 13:04
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #12 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing865 Passing11 Skipped37m 35s

Details

Failing Tests
SpecTest
quotas-storage.spec.tsCloud Manager Cypress Tests→Quota workflow tests→Quota storage table » Quotas and quota usages display properly

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/account/quotas-storage.spec.ts"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

4 participants