Skip to content

[TEST] Retest PR #3072: ovn-kubernetes: Move MNP from ConfigMap to CLI flags - #3102

Closed
weliang1 wants to merge 1 commit into
openshift:masterfrom
weliang1:retest-pr-3072
Closed

[TEST] Retest PR #3072: ovn-kubernetes: Move MNP from ConfigMap to CLI flags#3102
weliang1 wants to merge 1 commit into
openshift:masterfrom
weliang1:retest-pr-3072

Conversation

@weliang1

@weliang1 weliang1 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

⚠️ Test PR Only - Do Not Merge

This PR is for CI validation only. All changes from PR #3072 are already merged in master.

Purpose

Trigger CI jobs to validate the MNP CLI flags changes from PR #3072.

Original PR

Summary

MultiNetworkPolicy was not being enforced on UDN secondary interfaces in OCP 5.0 because ovnkube-control-plane pods did not restart when the ConfigMap was updated with enable-multi-networkpolicy=true.

Solution: Move MNP enablement from ConfigMap to CLI flags (following the same pattern as multicast in OCPBUGS-78731).

Changes in Original PR

  • bindata/network/ovn-kubernetes/managed/004-config.yaml: Remove MNP from ConfigMaps
  • bindata/network/ovn-kubernetes/managed/ovnkube-control-plane.yaml: Add --enable-multi-networkpolicy CLI flag
  • bindata/network/ovn-kubernetes/self-hosted/004-config.yaml: Remove MNP from ConfigMap
  • bindata/network/ovn-kubernetes/self-hosted/ovnkube-control-plane.yaml: Add --enable-multi-networkpolicy CLI flag
  • pkg/network/ovn_kubernetes_test.go: Update test expectations
  • docs/ovn_node_mode.md: Document ConfigMap vs CLI flag patterns

Testing Focus

  • ✅ CI job execution and pass rates
  • ✅ MNP enforcement on UDN Layer2 and Layer3 networks
  • ✅ Upgrade paths (4.x → 5.0)
  • ✅ HyperShift managed control plane
  • ✅ DPU-host mode compatibility

Test Marker

This PR adds a trivial .retest-pr-3072 file to create a commit difference from master, allowing PR creation for CI testing.

Related

/hold
/test all

This is a test commit to trigger CI validation for the already-merged
PR openshift#3072 (commit 17f08a7).

Original PR: openshift#3072
Changes: Move MultiNetworkPolicy enablement from ConfigMap to CLI flags

Testing focus:
- CI job validation
- MNP enforcement on UDN networks
- Upgrade path testing
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

@openshift-ci openshift-ci Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Aug 4, 2026
@openshift-ci

openshift-ci Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

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: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 37f19217-d96d-4836-86d1-c027a56d6d74

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.

@openshift-ci

openshift-ci Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: weliang1
Once this PR has been reviewed and has the lgtm label, please assign tssurya 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

@weliang1

weliang1 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Closing this PR - recreating with actual MNP changes for proper CI testing.

@weliang1 weliang1 closed this Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant