Skip to content

upcoming: [DPS-35488] Limit Logs feature based on Account Capability#13062

Merged
mikeZuber-akamai merged 2 commits intolinode:developfrom
mduda-akamai:feature/DPS-35488-limit-logs-feature-based-on-account-capability
Nov 10, 2025
Merged

upcoming: [DPS-35488] Limit Logs feature based on Account Capability#13062
mikeZuber-akamai merged 2 commits intolinode:developfrom
mduda-akamai:feature/DPS-35488-limit-logs-feature-based-on-account-capability

Conversation

@mduda-akamai
Copy link
Contributor

@mduda-akamai mduda-akamai commented Nov 6, 2025

Description 📝

Check for the Akamai Cloud Pulse Logs Account Capability when determining when to enable the Logs menu item and routing.
Akamai Cloud Pulse Logs Account Capability is depending on https://bits.linode.com/LinodeAPI/apinext/pull/9098

Changes 🔄

  • Add Akamai Cloud Pulse Logs Account Capability
  • Add useIsACLPLogsEnabled hook
  • Use useIsACLPLogsEnabled in PrimaryNav and DeliveryRoute

Scope 🚢

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

  • Some customers (e.g. in Beta or Limited Availability)

Target release date 🗓️

18 November 2025

Verification steps

Check if the Logs menu item is hidden and logs/delivery/* routes are blocked when aclpLogs.bypassAccountCapabilities flag is not set to true or the user doesn't have Akamai Cloud Pulse Logs capability

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
🚩 Using a feature flag to protect the release


  • 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

@mduda-akamai mduda-akamai force-pushed the feature/DPS-35488-limit-logs-feature-based-on-account-capability branch from adad32d to 6171024 Compare November 6, 2025 13:38
@mduda-akamai mduda-akamai marked this pull request as ready for review November 6, 2025 13:39
@mduda-akamai mduda-akamai requested a review from a team as a code owner November 6, 2025 13:39
@mduda-akamai mduda-akamai requested a review from a team as a code owner November 7, 2025 07:15
@mduda-akamai mduda-akamai requested review from cliu-akamai and removed request for a team November 7, 2025 07:15
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 4 failing tests on test run #3 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
4 Failing857 Passing11 Skipped44m 55s

Details

Failing Tests
SpecTest
clone-linode.spec.tsCloud Manager Cypress Tests→clone linode » can clone a Linode from Linode details page
update-stackscripts.spec.tsCloud Manager Cypress Tests→Update stackscripts » updates a StackScript
create-stackscripts.spec.tsCloud Manager Cypress Tests→Create stackscripts » creates a StackScript and deploys a Linode with it
create-stackscripts.spec.tsCloud Manager Cypress Tests→Create stackscripts » creates a StackScript with Any/All target image

Troubleshooting

Use this command to re-run the failing tests:

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

Copy link
Contributor

@abailly-akamai abailly-akamai left a comment

Choose a reason for hiding this comment

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

ok.. I am not sure why you'd want to bypass capabilities - we have them as a source of truth to dictate an API driven behavior. You're introducing unnecessary complexity in the flag management

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Nov 10, 2025
@mikeZuber-akamai mikeZuber-akamai merged commit ba8e70d into linode:develop Nov 10, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants