Skip to content

docs: document OC_SERVER cluster API values in README - #2913

Merged
DerekRoberts merged 1 commit into
mainfrom
docs/oc-server-readme
Oct 27, 2026
Merged

DerekRoberts merged 1 commit into
mainfrom
docs/oc-server-readme

Conversation

@DerekRoberts

Copy link
Copy Markdown
Member

Expands the OC_SERVER section of README.md:

  • States it is one repository-level variable (not per environment)
  • Lists Silver (https://api.silver.devops.gov.bc.ca:6443) and Gold (https://api.gold.devops.gov.bc.ca:6443) API server values
  • Keeps gh variable set OC_SERVER --body "$(oc whoami --show-server)" as the way to set it

README only.

Closes #2912

Copilot AI balanced review requested due to automatic review settings October 27, 2026 21:08

Copilot AI left a comment

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.

🟢 Approval recommended

The documentation-only changes meet the requested scope, with no unresolved findings.

No open issues 🎉

What changed in this PR

Clarifies OpenShift cluster configuration in the README, addressing #2912 without changing workflows or setup scripts.

Changes:

  • Documents OC_SERVER as one repository-level variable shared by PR, TEST, and PROD deployments.
  • Lists the Silver and Gold API server URLs.
  • Preserves the existing setup command and clarifies that users must be logged into the correct cluster.
File Description
README.md Clarifies OC_SERVER scope, cluster URLs, and setup guidance.

Review effort: Balanced

@DerekRoberts DerekRoberts self-assigned this Oct 27, 2026
@DerekRoberts
DerekRoberts merged commit 996eb94 into main Oct 27, 2026
23 checks passed
@DerekRoberts
DerekRoberts deleted the docs/oc-server-readme branch October 27, 2026 21:24
@github-project-automation github-project-automation Bot moved this from New to Done in DevOps (NR) Oct 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

docs: document OC_SERVER variable and cluster API values in README

2 participants