Skip to content
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

ocp4: ovs conf.db: tighten file permissions #6445

Merged
merged 1 commit into from
Dec 1, 2020

Conversation

JAORMX
Copy link
Contributor

@JAORMX JAORMX commented Dec 1, 2020

It is no longer the case that the conf.db file is of mode 0644. To
support IPSec keys, it was changed upstream [1] to be 0640, and this is
now percolating down to OpenShift.

[1] https://patchwork.ozlabs.org/project/openvswitch/patch/1600894095-96196-1-git-send-email-yihung.wei@gmail.com/

It is no longer the case that the conf.db file is of mode `0644`. To
support IPSec keys, it was changed upstream [1] to be `0640`, and this is
now percolating down to OpenShift.

[1] https://patchwork.ozlabs.org/project/openvswitch/patch/1600894095-96196-1-git-send-email-yihung.wei@gmail.com/
@JAORMX
Copy link
Contributor Author

JAORMX commented Dec 1, 2020

/test e2e-aws-ocp4-cis

@openshift-merge-robot
Copy link
Collaborator

@JAORMX: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-aws-ocp4-cis 6d0b9a3 link /test e2e-aws-ocp4-cis
ci/prow/e2e-aws-ocp4-moderate 6d0b9a3 link /test e2e-aws-ocp4-moderate
ci/prow/e2e-aws-rhcos4-e8 6d0b9a3 link /test e2e-aws-rhcos4-e8

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@JAORMX JAORMX merged commit c4626d0 into ComplianceAsCode:master Dec 1, 2020
@yuumasato yuumasato added this to the 0.1.54 milestone Jan 12, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants