Skip to content

Add COO product to PRODUCT_NAMESPACE_MAP and PRODUCT_KUBECONFIG_MAP - #3220

Open
jan--f wants to merge 1 commit into
openshift-eng:mainfrom
jan--f:add-coo-product
Open

Add COO product to PRODUCT_NAMESPACE_MAP and PRODUCT_KUBECONFIG_MAP#3220
jan--f wants to merge 1 commit into
openshift-eng:mainfrom
jan--f:add-coo-product

Conversation

@jan--f

@jan--f jan--f commented Aug 3, 2026

Copy link
Copy Markdown

Add coo (Cluster Observability Operator) entries:

  • PRODUCT_NAMESPACE_MAP: "coo": "art-coo-tenant"
  • PRODUCT_KUBECONFIG_MAP: "coo": "COO_KONFLUX_SA_KUBECONFIG"

Summary by CodeRabbit

  • New Features
    • Added Konflux configuration support for the COO product.
    • COO workflows can now use the art-coo-tenant namespace and corresponding service-account kubeconfig.

…_MAP

Add 'coo' (Cluster Observability Operator) entries mapping to
art-coo-tenant namespace and COO_KONFLUX_SA_KUBECONFIG env var.

Assisted-by: claude-opus-4-6@default
Signed-off-by: Jan Fajerski <jfajersk@redhat.com>
@openshift-ci

openshift-ci Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign locriandev for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Aug 3, 2026
@openshift-ci

openshift-ci Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Hi @jan--f. Thanks for your PR.

I'm waiting for a openshift-eng member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: openshift-eng/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 3b79ab2a-190e-4f69-82e5-d413f0db6c49

📥 Commits

Reviewing files that changed from the base of the PR and between b3f2c06 and bcf99a6.

📒 Files selected for processing (1)
  • artcommon/artcommonlib/constants.py

Walkthrough

The shared product constants now map coo to the art-coo-tenant namespace and COO_KONFLUX_SA_KUBECONFIG kubeconfig variable.

Changes

COO Konflux mappings

Layer / File(s) Summary
Add COO product mappings
artcommon/artcommonlib/constants.py
Adds the COO namespace and kubeconfig variable to the exported product mappings.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: ashwindasr, lgarciaaco, thegreyd

🚥 Pre-merge checks | ✅ 11
✅ Passed checks (11 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the addition of COO mappings to both product maps.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No-Weak-Crypto ✅ Passed The commit only adds two COO namespace and kubeconfig mappings; it introduces no weak-crypto algorithm, custom crypto, or secret comparison.
Container-Privileges ✅ Passed The PR changes only constants.py and adds two mapping strings; no container/Kubernetes manifest or privilege declaration appears in the commit.
No-Sensitive-Data-In-Logs ✅ Passed The commit adds only two product mapping constants; it adds no logging, passwords, tokens, API keys, PII, hostnames, or customer data.
No-Hardcoded-Secrets ✅ Passed The added values are a tenant namespace and an environment-variable name; they contain no API key, token, password, credential, private key, or base64 secret.
No-Injection-Vectors ✅ Passed The diff adds only two literal string mappings, and the added-line scan found no SQL concatenation, shell=True, eval/exec, pickle, unsafe YAML, os.system, or HTML injection.
Ai-Attribution ✅ Passed The HEAD commit mentions Claude and includes an Assisted-by: trailer; it does not use Co-Authored-By for the AI tool.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Labels

needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant