Skip to content

[backplane-5.0] Include Hive operator NetworkPolicy in bundle/chart regeneration - #4021

Merged
openshift-merge-bot[bot] merged 2 commits into
stolostron:backplane-5.0from
openshift-cherrypick-robot:cherry-pick-4019-to-backplane-5.0
Sep 16, 2026
Merged

openshift-merge-bot[bot] merged 2 commits into
stolostron:backplane-5.0from
openshift-cherrypick-robot:cherry-pick-4019-to-backplane-5.0

Conversation

@openshift-cherrypick-robot

Copy link
Copy Markdown
Contributor

This is an automated cherry-pick of #4019

/assign dislbenn

Hive's hack/bundle-gen.sh reads config/operator/operator_netpol.yaml
only to populate CSV data in OperatorHub mode; it never emits the
NetworkPolicy as a standalone manifest in the generated bundle. As a
result the Hive operator's NetworkPolicy was never picked up when
regenerating the hive-operator Helm chart via
'make regenerate-charts-from-bundles'.

Update gen-hive-bundle.sh to copy config/operator/operator_netpol.yaml
directly from the cloned Hive repo into the bundle output directory
(renamed to hive-operator-networkpolicy.yaml) so it flows through the
existing bundle-to-chart tooling like any other manifest, picking up
the standard Helm templating (networkPolicies.enabled guard, namespace
substitution) automatically. Fails hard if the file is missing so a
future rename/removal upstream is caught immediately.

Also includes the resulting regenerated chart template.

Signed-off-by: dislbenn <dbennett@redhat.com>
@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 9fee91e1-169b-4361-8948-e6508ef2a915

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@dislbenn

Copy link
Copy Markdown
Collaborator

/lgtm

@dislbenn

Copy link
Copy Markdown
Collaborator

/retest

@red-hat-konflux

Copy link
Copy Markdown
Contributor

All PipelineRuns for this commit have already succeeded. Use /retest <pipeline-name> to re-run a specific pipeline or /test to re-run all pipelines.

@openshift-ci openshift-ci Bot removed the lgtm label Sep 16, 2026
@sonarqubecloud

Copy link
Copy Markdown

@dislbenn

Copy link
Copy Markdown
Collaborator

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Sep 16, 2026
@openshift-ci

openshift-ci Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dislbenn, openshift-cherrypick-robot

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:

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

@openshift-merge-bot
openshift-merge-bot Bot merged commit c0276ee into stolostron:backplane-5.0 Sep 16, 2026
9 checks passed
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.

2 participants