Skip to content

Fix missing label on OpenShift Logging with multitenant isolation mode#33193

Closed
eramoto wants to merge 1 commit intoopenshift:masterfrom
eramoto:fix-missing-label-with-logging
Closed

Fix missing label on OpenShift Logging with multitenant isolation mode#33193
eramoto wants to merge 1 commit intoopenshift:masterfrom
eramoto:fix-missing-label-with-logging

Conversation

@eramoto
Copy link

@eramoto eramoto commented Jun 8, 2021

When OpenShift Logging with multitenant isolation using NetworkPolicy mode was deployed,
network traffic from the openshift-operators-redhat namespace to the openshift-logging namespace was not allowed
because a label was used in the network policy object but was not set on the openshift-operators-redhat namespace.

When OpenShift Logging with multitenant isolation using NetworkPolicy mode was
deployed, network traffic from the `openshift-operators-redhat` namespace to
the `openshift-logging` namespace was not allowed because a label was used in
the network policy object but was not set on the `openshift-operators-redhat`
namespace.
@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 8, 2021
@netlify
Copy link

netlify bot commented Jun 8, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: 4a9d709

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/60beb7c12672b70008a20722

😎 Browse the preview: https://deploy-preview-33193--osdocs.netlify.app/openshift-enterprise/latest/logging/cluster-logging-deploying

@Preeticp
Copy link
Contributor

Preeticp commented Jun 8, 2021

@rolfedh can you PTAL and get the necessary LGTMs, please?

@rolfedh
Copy link
Contributor

rolfedh commented Jun 8, 2021

Thanks for making this update, @eramoto. I've duplicated them, made some additional changes, and submitted them as #33202.
@Preeticp Please close this PR.

@rolfedh
Copy link
Contributor

rolfedh commented Jun 9, 2021

Closing this PR because I have included it in #33202

@rolfedh
Copy link
Contributor

rolfedh commented Jun 9, 2021

/close

@openshift-ci
Copy link

openshift-ci bot commented Jun 9, 2021

@rolfedh: Closed this PR.

Details

In response to this:

/close

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/test-infra repository.

@openshift-ci openshift-ci bot closed this Jun 9, 2021
@eramoto eramoto deleted the fix-missing-label-with-logging branch June 10, 2021 01:03
neal-timpe added a commit that referenced this pull request Jun 15, 2021
RHDEVDOCS-3053: Tracker for PR #33193, Fix missing label on OpenShift …
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants