Skip to content

[OSDOCS#17270]: Document dnsName allow vs 0.0.0.0/0 deny behavior - #119036

Open
harsh21nama wants to merge 1 commit into
openshift:mainfrom
harsh21nama:cursor/osdocs-17270-egress-firewall-dnsname
Open

[OSDOCS#17270]: Document dnsName allow vs 0.0.0.0/0 deny behavior#119036
harsh21nama wants to merge 1 commit into
openshift:mainfrom
harsh21nama:cursor/osdocs-17270-egress-firewall-dnsname

Conversation

@harsh21nama

Copy link
Copy Markdown

Summary

  • Documents that an IPv4 cidrSelector: 0.0.0.0/0 deny rule denies all egress traffic because connections are sent to IP addresses.
  • Clarifies that an Allow rule with dnsName does not permit that traffic: after FQDN resolution, the connection uses the resolved IP and matches the deny-all CIDR rule.
  • Shows the working pattern: allow specific destinations with cidrSelector, plus a note on matching order and the example CR.

Test plan

  • Confirm the limitation, matching-order note, and DNS restriction read correctly on the configuring-egress-firewall-ovn preview.
  • Confirm the IMPORTANT admonition appears next to the example EgressFirewall CR that uses cidrSelector + 0.0.0.0/0 deny.

Version(s):
main (in-development); cherry-pick to 4.18+ as needed

Issue:
https://redhat.atlassian.net/browse/OSDOCS-17270

Link to docs preview:
(add after preview build)

QE review:

  • QE has approved this change.

A cidrSelector 0.0.0.0/0 deny matches resolved destination IPs, so dnsName allow rules do not permit that traffic.
@openshift-ci openshift-ci Bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 31, 2026
@openshift-ci

openshift-ci Bot commented Aug 31, 2026

Copy link
Copy Markdown

Hi @harsh21nama. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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

Labels

needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant