Skip to content

change: [DPS-41857] - Logs: Add new statuses - deactivating, failed#13551

Open
bbanucha-akamai wants to merge 1 commit intolinode:developfrom
bbanucha-akamai:feature/DPS-41857-add-failed-and-deactivating-status
Open

change: [DPS-41857] - Logs: Add new statuses - deactivating, failed#13551
bbanucha-akamai wants to merge 1 commit intolinode:developfrom
bbanucha-akamai:feature/DPS-41857-add-failed-and-deactivating-status

Conversation

@bbanucha-akamai
Copy link
Copy Markdown
Contributor

@bbanucha-akamai bbanucha-akamai commented Mar 31, 2026

Description 📝

Two new statuses (Delivery Logs Streams status) have been added in the UI: Failed and Deactivating.
It also prevents users from performing edit or activate actions when a stream is in either of these statuses.

Changes 🔄

  • Added Failed and Deactivating statuses to the UI.
  • Disabled edit and activate actions when the stream status is Failed or Deactivating.
  • Updated UI logic to handle these statuses consistently.

Scope 🚢

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

  • No customers / Not applicable

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

Target release date 🗓️

April 2026

Preview 📷

Screenshot 2026-03-31 at 18 26 15 Screenshot 2026-03-31 at 18 27 06

How to test 🧪

  • Application running locally
  • Having a stream that returns Failed or Deactivating status from the API (or mocked in packages/manager/src/factories/delivery.ts)

Verification steps

  • Verify Failed status is displayed correctly in the UI.
  • Verify Deactivating status is displayed correctly in the UI.
  • Verify Edit action is disabled when status is Failed.
  • Verify Edit action is disabled when status is Deactivating.
  • Verify the Activate action is disabled when the status is Failed or Deactivating.
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
📱 Providing mobile 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

@bbanucha-akamai bbanucha-akamai force-pushed the feature/DPS-41857-add-failed-and-deactivating-status branch from 765d145 to 0c8d1c3 Compare March 31, 2026 16:37
@bbanucha-akamai bbanucha-akamai self-assigned this Mar 31, 2026
@bbanucha-akamai bbanucha-akamai marked this pull request as ready for review March 31, 2026 16:39
@bbanucha-akamai bbanucha-akamai requested a review from a team as a code owner March 31, 2026 16:39
"@linode/manager": Added
---

Add new stream statuses - deactivating, failed ([#13551](https://github.com/linode/manager/pull/13551))
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.

https://linode.github.io/manager/CONTRIBUTING.html

Avoid starting a changeset with the verb "Add", "Remove", "Change" or "Fix" when listed under that respective Added, Removed, Changed or Fixed section. It is unnecessary repetition.

{
value: streamStatus.Failed,
label: 'Failed',
pendoId: 'Logs Delivery Streams-Status Failed',
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.

? `You cannot save changes while the stream status is ${selectedStreamStatus}`
: undefined,
[selectedStreamStatus]
[isStreamStatusBlocking, selectedStreamStatus]
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.

[selectedStreamStatus] should be enough.
isStreamStatusBlocking depends on selectedStreamStatus change.

},
title: 'Edit',
pendoId: 'Logs Delivery Streams-Edit',
disabled:
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.

remove disabled prop. Edit in action menu stays enabled, same as for 'provisioning' status.

@linode-gh-bot
Copy link
Copy Markdown
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #6 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing887 Passing11 Skipped40m 50s

Details

Failing Tests
SpecTest
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can update bucket access

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts"

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

Labels

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

3 participants