-
Notifications
You must be signed in to change notification settings - Fork 258
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
host-deploy: condition for adding fapolicy rules #256
Conversation
remove ansible_distribution == 'RedHat' since we already check if the service is running. fapolicyd < 1.1 doesn't have /etc/fapolicyd/rules.d/,add check if the directory exists.
/ost |
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.
+1
@@ -64,6 +64,11 @@ | |||
command: vdsm-tool configure --force | |||
changed_when: True | |||
|
|||
- name: Verify fapolicyd.rules file | |||
stat: | |||
path: /etc/fapolicyd/fapolicyd.rules |
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.
You should check if /etc/fapolicyd/rules.d
exists since you add the new rule to this directory.
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.
Crap, I've completely missed that, sorry :-(
Fixed in #259
Fixes fapolicy configuration path introduced in oVirt#256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in oVirt#256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in oVirt#256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in #256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in oVirt#256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in oVirt#256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in oVirt#256 Signed-off-by: Martin Perina <mperina@redhat.com>
Fixes fapolicy configuration path introduced in #256 Signed-off-by: Martin Perina <mperina@redhat.com>
remove ansible_distribution == 'RedHat' since we already check if the service is running.
fapolicyd < 1.1 doesn't have /etc/fapolicyd/rules.d/,add check if the directory exists.