-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[1.5 backport] set the proper label of /var/lib/etcd directory #4690
[1.5 backport] set the proper label of /var/lib/etcd directory #4690
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be backported to 1.4?
Saw your comment on the BZ and I reply to myself: no since in 1.4 we were using etcdctl from the host. |
[test] |
@ingvagabund @ashcrow, is it worth creating a card to track upstream changes required to make the official ansible module work? We should at least file a bug or track this. |
@kwoodson I like that. Us using the commands directly is tech debt. @ingvagabund please create a card in the backlog for this. |
aos-ci-test |
[test] |
7c045ef
to
58de9fc
Compare
aos-ci-test |
Evaluated for openshift ansible test up to 58de9fc |
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/329/) (Base Commit: aa1f0a0) (PR Branch Commit: 58de9fc) |
flake openshift/origin#10162 |
aos-ci-test |
Cause there is no guarantee the /var/lib/etcd will keep its
svirt_sandbox_file_t
label between installation and upgrade.The /var/lib/etcd will have incorrect
var_lib_t
label until theetcd_container
service is restarted. Once restarted, the directory is correctly re-labeled.