Skip to content

Update nw-networkpolicy-audit-disable.adoc #50024

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from

Conversation

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 5, 2022
@Amrita42
Copy link
Contributor

Amrita42 commented Sep 5, 2022

lgtm

*Missing network prefix*

Error adding container to network "work-network": the 'address' field is expected to be in CIDR notation, got: '192.168.10.10'
@ccardenosa
Copy link
Contributor Author

Extra changes:

Missing required network prefix

Version(s):

PR applies to all versions after a specific version: 4.6+

Issue:
POD creation fails due to this:

Warning FailedCreatePodSandBox 3s kubelet Failed to create pod sandbox: rpc error: code = Unknown desc = failed to create pod network sandbox k8s_multus-training-6f8cbbbf74-vn7v2_multus-training_b397a304-a920-4cd6-97c1-203f0ca2dc33_0(1a6516c279b60da692fb0dd3fc7206b49bf8904748f0fa3e68ab3530f01d7b71): error adding pod multus-training_multus-training-6f8cbbbf74-vn7v2 to CNI network "multus-cni-network": plugin type="multus" name="multus-cni-network" failed (add): [multus-training/multus-training-6f8cbbbf74-vn7v2/b397a304-a920-4cd6-97c1-203f0ca2dc33:work-network]: error adding container to network "work-network": the 'address' field is expected to be in CIDR notation, got: '192.168.10.10'

Link to docs preview:
https://docs.openshift.com/container-platform/4.6/networking/multiple_networks/configuring-additional-network.html#nw-multus-ipvlan-config-example_configuring-additional-network
https://docs.openshift.com/container-platform/4.7/networking/multiple_networks/configuring-additional-network.html#nw-multus-ipvlan-config-example_configuring-additional-network
https://docs.openshift.com/container-platform/4.8/networking/multiple_networks/configuring-additional-network.html#nw-multus-ipvlan-config-example_configuring-additional-network
https://docs.openshift.com/container-platform/4.9/networking/multiple_networks/configuring-additional-network.html#nw-multus-ipvlan-config-example_configuring-additional-network
https://docs.openshift.com/container-platform/4.10/networking/multiple_networks/configuring-additional-network.html#nw-multus-ipvlan-config-example_configuring-additional-network
https://docs.openshift.com/container-platform/4.11/networking/multiple_networks/configuring-additional-network.html#nw-multus-ipvlan-config-example_configuring-additional-network

@mburke5678
Copy link
Contributor

mburke5678 commented Sep 6, 2022

@ccardenosa The nw-networkpolicy-audit-disable.adoc file does not exist in the enterprise-4.6 or enterprise-4.7 branches. Merging the other two modules requires a separate PR with just those two modules.

And this PR needs a squash.

@ccardenosa
Copy link
Contributor Author

ccardenosa commented Sep 7, 2022

Hello @mburke5678 .

I have created two separate PRs:

You can close this one. Sorry for the inconveniences.

BR,
Carlos.

@ccardenosa
Copy link
Contributor Author

@mburke5678 Can we close this PR?

@mburke5678
Copy link
Contributor

Closing in favor of #50079 and #50080

@mburke5678 mburke5678 closed this Sep 7, 2022
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