[TEST] Retest PR #3072: ovn-kubernetes: Move MNP from ConfigMap to CLI flags - #3102
[TEST] Retest PR #3072: ovn-kubernetes: Move MNP from ConfigMap to CLI flags#3102weliang1 wants to merge 1 commit into
Conversation
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
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Skipping CI for Draft Pull Request. |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: weliang1 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Closing this PR - recreating with actual MNP changes for proper CI testing. |
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
--enable-multi-networkpolicyCLI flag--enable-multi-networkpolicyCLI flagTesting Focus
Test Marker
This PR adds a trivial
.retest-pr-3072file to create a commit difference from master, allowing PR creation for CI testing.Related
/hold
/test all