fedora-infra / ansible

Clone
Source Code
GIT

#1259 check if ipa service entry exists
Merged a year ago by praiskup. Opened a year ago by seddik.
fedora-infra/ seddik/ansible ipa-condition  into  main

file modified
+10 
@@ -9,6 +9,15 @@ 
 

    - config
 

    - krb5
 

  

+ - name: Check if service exists
 

+   delegate_to: "{{ ipa_server }}"
 

+   stat:
 

+     path: "{{ service }}/{{ host }}"
 

+   register: ipa_service_defined
 

+   tags:
 

+     - config
 

+     - krb5
 

+ 

  - name: Create service entry
 

    delegate_to: "{{ ipa_server }}"
 

    ipaservice:
 
@@ -16,6 +25,7 @@ 
 

      name: "{{ service }}/{{ host }}"
 

      principal: "{{ principal_alias | default(omit) }}"
 

      force: yes
 

+   when: not ipa_service_defined.stat.exists
 

    tags:
 

    - config
 

    - krb5

Build succeeded.

rebased onto bb9689871c7e257873ae4669336cf67d373c0c36
a year ago

rebased onto bb9689871c7e257873ae4669336cf67d373c0c36
a year ago

LGTM, but I will wait if @kevin doesn't have anything

Build succeeded.

We actually use keytab/service role that fails for Copr. Is ipa/service a transtive dep?

https://pagure.io/fedora-infra/ansible/blob/161545b81c68125f4c4a46cd31bc07a89c6f6357/f/roles/copr/frontend/tasks/httpd.yml#_86-93

Indeed, ipa/service is a transitive role triggered by keytab/service. I think this could help; I'm going to merge this and test right away. Seems relatively safe.

rebased onto 98475f6
a year ago

rebased onto 98475f6
a year ago

rebased onto 98475f6
a year ago

Pull-Request has been merged by praiskup
a year ago
Metadata
None
No Tags
Changes Summary 1
+10 -0
file changed
roles/ipa/service/tasks/main.yml
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.