Skip to content
This repository was archived by the owner on Jan 9, 2023. It is now read-only.

RHEL8 mode and backup jobs #64

Merged
merged 1 commit into from
Jun 18, 2019
Merged

Conversation

jan-cerny
Copy link
Member

I have fixed the RHEL 8 node and then I tried to reflect the
changes on the node in our Playbook. But I don't know how
to generate a Playbook from a node, so I have updated the Playbook
manually. Also, I have enabled RHEL 8 node in some jobs.
I backup the jobs configuration to the git repository.

Copy link
Member

@yuumasato yuumasato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested the playbook in check mode?

@@ -25,6 +33,7 @@
key: "{{ lookup('file', 'generated_bits/master_id_rsa.pub') }}"
exclusive: yes # only need one key for "jenkins" user, users can still connect as ec2-user
state: present
when: False
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this intentional? Why?

name:
- PyYAML # needed for the new SSG yaml port
state: installed
when: ansible_distribution != 'RedHat' or ansible_distribution_version.split(".")[0] != "8"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is assume this task RHEL8 only, as next task is Fedora only.
So shouldn't this be ansible_distribution == 'RedHat' and ansible_distribution_version.split(".")[0] != "8"?

state: installed
when: ansible_distribution == 'Fedora' or (ansible_distribution == 'RedHat' and ansible_distribution_version.split(".")[0] == "8")

- name: Ensure OpenSCAP Anaconda dependencies are installed (Fedora only)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would change name of task to explicitly mention zanata.
It is confusing as it is now, because there is two tasks with same name, one Fedora and RHEL8 only, and the other Fedora only.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that we install the Zanata Python client for RHEL7 and for Fedora, but not for RHEL8. Is this expected?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matejak yum search zanata doesn't find anything on our RHEL 8 worker

I have fixed the RHEL 8 node and then I tried to reflect the
changes on the node in our Playbook. But I don't know how
to generate a Playbook from a node, so I have updated the Playbook
manually. Also, I have enabled RHEL 8 node in some jobs.
I backup the jobs configuration to the git repository.
@jan-cerny
Copy link
Member Author

I have run ansible-playbook -i inventory.ini --check --diff workers.yml but I have to disable the task Authorize master's jenkins user public key because this task is using output of running master setup. That's also why I forget the False in the original commit. :-)

@matejak matejak mentioned this pull request Jun 17, 2019
@ggbecker ggbecker self-assigned this Jun 18, 2019
@ggbecker
Copy link
Member

LGTM.

Note: Please backup new rhel8 jobs from jenkins and include missing jobs into ansible/master.yml file (as pointed out by #66).

@ggbecker
Copy link
Member

@yuumasato PR author has addressed requested modifications and tests. Merging PR

@ggbecker ggbecker merged commit d7640ad into OpenSCAP:master Jun 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants