Skip to content

refactor: Update workspace new to produce logs describing when a workspace has been created but failed to populate its state - #39192

Draft
SarahFrench wants to merge 2 commits into
refactor-workspace-new-use-viewsfrom
refactor-workspace-new-log-failure-to-populate-state
Draft

SarahFrench wants to merge 2 commits into
refactor-workspace-new-use-viewsfrom
refactor-workspace-new-log-failure-to-populate-state

Conversation

@SarahFrench

Copy link
Copy Markdown
Member

This PR is intended to prevent a scenario where this command that declares success when creating the (empty) workspace but then logs additional errors that show the -state flag wasn't able to be used successfully. The change in output makes it clearer to users about the impacts of the command before it experienced an error.

Target Release

1.18.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@SarahFrench SarahFrench added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Sep 11, 2026
@SarahFrench
SarahFrench changed the base branch from main to refactor-workspace-new-use-views September 11, 2026 14:10
@SarahFrench
SarahFrench force-pushed the refactor-workspace-new-use-views branch from aa3cc9d to 24901e9 Compare September 11, 2026 14:14
@SarahFrench
SarahFrench force-pushed the refactor-workspace-new-log-failure-to-populate-state branch from 35b9f34 to aad4b94 Compare September 11, 2026 14:14
@SarahFrench
SarahFrench force-pushed the refactor-workspace-new-use-views branch 3 times, most recently from dd77deb to cf77dff Compare October 1, 2026 16:30
@SarahFrench
SarahFrench force-pushed the refactor-workspace-new-log-failure-to-populate-state branch from aad4b94 to 953ad2b Compare October 1, 2026 16:40
@SarahFrench

Copy link
Copy Markdown
Member Author

Rebased as the base branch has changed

This branch has not been deployed

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

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant