Skip to content

docs: mention DWO_NAMESPACE env var in backup docs#3059

Merged
dkwon17 merged 2 commits intoeclipse-che:mainfrom
dkwon17:backup-namespace
Apr 7, 2026
Merged

docs: mention DWO_NAMESPACE env var in backup docs#3059
dkwon17 merged 2 commits intoeclipse-che:mainfrom
dkwon17:backup-namespace

Conversation

@dkwon17
Copy link
Copy Markdown
Contributor

@dkwon17 dkwon17 commented Mar 31, 2026

What does this pull request change?

Adds a notice about the DWO_NAMESPACE env var which is needed for the dashboard to detect backup status. This is needed only when the DWO install namespace is not openshift-operators.

What issues does this pull request fix or reference?

Specify the version of the product this pull request applies to

DS 3.27.

Pull Request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
    • The page contains a redirection for the previous URL.
    • Propagate the URL change in:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

@dkwon17 dkwon17 requested a review from l0rd March 31, 2026 20:01
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 31, 2026

Click here to review and test in web IDE: Contribute

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 31, 2026

🎊 Navigate the preview: https://69ce8301716c0e38b98eb68c--eclipse-che-docs-pr.netlify.app 🎊

Signed-off-by: dkwon17 <dakwon@redhat.com>
- env:
- name: DWO_NAMESPACE
value: $OPERATOR_INSTALL_NAMESPACE
name: che-dashboard
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This value is the same for both upstream and downstream.

Copy link
Copy Markdown
Contributor

@gtrivedi88 gtrivedi88 left a comment

Choose a reason for hiding this comment

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

Thanks for adding the DWO_NAMESPACE documentation — this is useful context for non-default operator installations.

A few CQA/style issues to address:

  1. Bold pseudo-heading**Note:** must be a proper [NOTE] admonition block (DITA compatibility, Vale BlockTitle rule)
  2. Bare placeholder$OPERATOR_INSTALL_NAMESPACE in the snippet has no explanation; the reader needs guidance on what value to substitute
  3. Snippet placement — In the regular OCI registry file, the include breaks the flow between the callout and authSecret explanation; consider moving it to the end
  4. Missing final newline — The snippet file should end with a newline

Signed-off-by: dkwon17 <dakwon@redhat.com>
@dkwon17 dkwon17 merged commit 6ec9374 into eclipse-che:main Apr 7, 2026
5 checks passed
dkwon17 added a commit to dkwon17/che-docs that referenced this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants