Skip to content

CSV: set maintainers to RH support, maturity stable#2797

Open
jotak wants to merge 1 commit into
netobserv:mainfrom
jotak:csv-maintainers
Open

CSV: set maintainers to RH support, maturity stable#2797
jotak wants to merge 1 commit into
netobserv:mainfrom
jotak:csv-maintainers

Conversation

@jotak

@jotak jotak commented Jun 8, 2026

Copy link
Copy Markdown
Member

It's simpler if we just set a global RH support maintainer in the CSV description, rather than an individual list. Most other operators do that.
Also, set maturity as "stable" upstream (it was only done for downstream)

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated operator maturity status from alpha to stable
    • Consolidated maintainer contact information to Red Hat Support

@openshift-ci

openshift-ci Bot commented Jun 8, 2026

Copy link
Copy Markdown

[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 memodi 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

@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e9d6a149-3229-4c9c-b0f0-dddb2be2184b

📥 Commits

Reviewing files that changed from the base of the PR and between e5f21f6 and 9fefe49.

📒 Files selected for processing (3)
  • bundle/manifests/netobserv-operator.clusterserviceversion.yaml
  • config/csv/bases/netobserv-operator.clusterserviceversion.yaml
  • hack/patch_csv.py
💤 Files with no reviewable changes (1)
  • hack/patch_csv.py

📝 Walkthrough

Walkthrough

The PR consolidates ClusterServiceVersion maintainer metadata from multiple individual contacts to a single Red Hat Support entry, marks the operator as stable rather than alpha, and updates the CSV build script to avoid overriding the maturity field that now comes from the base configuration.

Changes

ClusterServiceVersion Metadata Update

Layer / File(s) Summary
CSV metadata consolidation
config/csv/bases/netobserv-operator.clusterserviceversion.yaml, bundle/manifests/netobserv-operator.clusterserviceversion.yaml
Maintainers list replaced with single "Red Hat Support" entry (support@redhat.com); maturity field changed from alpha to stable in both base and bundle manifest.
Patch script alignment
hack/patch_csv.py
Script no longer explicitly sets maturity to stable (now sourced from base config) and adds displayName assignment for "Network Observability".

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the rationale but lacks required template sections: no Dependencies section (though marked n/a), missing Checklist items, and no configuration/testing/QE requirements noted. Complete the PR description template by filling in the Checklist section with appropriate selections (particularly QE requirements and unit test status).
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically summarizes the main changes: updating maintainers to RH support and setting maturity to stable.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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

@jotak jotak requested review from OlivierCazade and mffiedler June 10, 2026 06:53
@mffiedler

Copy link
Copy Markdown
Contributor

/lgtm

@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (e5f21f6) to head (9fefe49).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff      @@
##   main   #2797   +/-   ##
============================
============================
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants