Skip to content

feat: add tenant CRD to e2e artifact collection and debug report#787

Merged
openshift-merge-bot[bot] merged 1 commit intoopendatahub-io:mainfrom
chaitanya1731:must-gather
Apr 22, 2026
Merged

feat: add tenant CRD to e2e artifact collection and debug report#787
openshift-merge-bot[bot] merged 1 commit intoopendatahub-io:mainfrom
chaitanya1731:must-gather

Conversation

@chaitanya1731
Copy link
Copy Markdown
Collaborator

Summary

Add the tenants.maas.opendatahub.io CRD to e2e must-gather artifact collection and the auth debug report.

Description

The Tenant CRD was recently introduced but was missing from the e2e debugging and artifact-collection utilities in auth_utils.sh.

  • Add tenants.maas.opendatahub.io to the MAAS_CRDS array so collect_maas_crs() dumps Tenant CR YAML to tenants.yaml.
  • Add kubectl get tenants to collect_cluster_state() alongside other MaaS CRs.
  • Add Tenant listing and status/condition detail to run_auth_debug_report() under the MaaS CRs section.
  • Update header comment to document the new tenants.yaml artifact.

How it was tested

  • Verified script syntax with bash -n.
  • Confirmed the Tenant CRD name matches deployment/base/maas-controller/crd/bases/maas.opendatahub.io_tenants.yaml.
  • Confirmed namespace usage aligns with TenantReconciler.TenantNamespace (sourced from --maas-subscription-namespace).

Made with Cursor

include tenants.maas.opendatahub.io in the MAAS_CRDS list so
collect_maas_crs() dumps tenant CR yaml, add tenant listing to
collect_cluster_state() and run_auth_debug_report() for parity
with the other maas custom resources.

Signed-off-by: Chaitanya Kulkarni <ckulkarn@redhat.com>
Signed-off-by: Chaitanya Kulkarni <chkulkar@redhat.com>
Made-with: Cursor
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026

Warning

Rate limit exceeded

@chaitanya1731 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 32 minutes and 58 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 32 minutes and 58 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 8a0f1f8f-8f07-414a-b058-c7fd252b04a0

📥 Commits

Reviewing files that changed from the base of the PR and between e746008 and 9485151.

📒 Files selected for processing (1)
  • test/e2e/scripts/auth_utils.sh

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 and usage tips.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 21, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chaitanya1731, ishitasequeira

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [chaitanya1731,ishitasequeira]

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

@chaitanya1731
Copy link
Copy Markdown
Collaborator Author

/retest

1 similar comment
@chaitanya1731
Copy link
Copy Markdown
Collaborator Author

/retest

@rhods-ci-bot
Copy link
Copy Markdown

@chaitanya1731: The following test has Succeeded:

OCI Artifact Browser URL

View in Artifact Browser

Inspecting Test Artifacts Manually

To inspect your test artifacts manually, follow these steps:

  1. Install ORAS (see the ORAS installation guide).
  2. Download artifacts with the following commands:
mkdir -p oras-artifacts
cd oras-artifacts
oras pull quay.io/opendatahub/odh-ci-artifacts:maas-group-test-8mz5j

@openshift-merge-bot openshift-merge-bot Bot merged commit fb2ea25 into opendatahub-io:main Apr 22, 2026
9 checks passed
chaitanya1731 added a commit that referenced this pull request Apr 22, 2026
Automated promotion of **9 commit(s)** from `main` to `stable`.

```
fb2ea25 feat: add tenant CRD to e2e artifact collection and debug report (#787)
1b8f212 chore: restrict rbac for db secret (#779)
e746008 docs: add/update documentation for Maas Tenant (#773)
147eaa2 fix: per-model(s) top-level values in usage dashboard (#772)
b327b34 feat: add OIDC token support for model discovery via /v1/models (#703)
dbf6d03 fix: validate token rate limits and skip invalid subs in TRLP aggregation (#752)
fae753e chore: add .worktrees/ to .gitignore (#774)
c01dc5b fix: minor updates for external model (#771)
65ca551 fix: add explicit command to v0.8.2 simulator models to prevent bash … (#765)
```
chaitanya1731 added a commit that referenced this pull request Apr 22, 2026
Automated promotion of **11 commit(s)** from `stable` to `rhoai`.

```
fb2ea25 feat: add tenant CRD to e2e artifact collection and debug report (#787)
1b8f212 chore: restrict rbac for db secret (#779)
e746008 docs: add/update documentation for Maas Tenant (#773)
147eaa2 fix: per-model(s) top-level values in usage dashboard (#772)
b327b34 feat: add OIDC token support for model discovery via /v1/models (#703)
dbf6d03 fix: validate token rate limits and skip invalid subs in TRLP aggregation (#752)
89fba29 chore: promote main to stable (#770)
fae753e chore: add .worktrees/ to .gitignore (#774)
c01dc5b fix: minor updates for external model (#771)
65ca551 fix: add explicit command to v0.8.2 simulator models to prevent bash … (#765)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants