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

rename linuxone machines #1168

Closed
wants to merge 2 commits into from
Closed

rename linuxone machines #1168

wants to merge 2 commits into from

Conversation

gdams
Copy link
Member

@gdams gdams commented Mar 6, 2018

The Linuxone machines were all recently updated from Rhel72 to Rhel74. I imagine that the jenkins nodes will also need to be renamed

CC @nodejs/platform-s390

@mhdawson
Copy link
Member

mhdawson commented Mar 6, 2018

If/when we do this I think we should rename to Rhel7 so we don't need to do it again.

@maclover7
Copy link
Contributor

Please confirm that playbooks/jenkins/worker/create.yml works good after this change, I believe "rhel72" or some variant of that is explicitly defined somewhere in "new Ansible" playbooks/roles.

@gibfahn
Copy link
Member

gibfahn commented Mar 6, 2018

If/when we do this I think we should rename to Rhel7 so we don't need to do it again.

Good idea, who does the updates?

So when the machines were updated the hostname was changed as well?

@gibfahn
Copy link
Member

gibfahn commented Mar 6, 2018

Yep, plenty to change:

Files:

ansible/roles/java-base/vars/main.yml
ansible/roles/baselayout/vars/main.yml
ansible/roles/jenkins-worker/vars/main.yml
ansible/roles/jenkins-worker/tasks/main.yml
ansible/roles/package-upgrade/vars/main.yml
ansible/playbooks/jenkins/worker/create.yml
ansible/roles/jenkins-worker/tasks/partials/tap2junit/rhel72.yml

Hits:

ansible/roles/jenkins-worker/vars/main.yml
12:  rhel72: 'rhel72',
40:  rhel72: {

ansible/roles/baselayout/vars/main.yml
89:  rhel72: [

ansible/roles/java-base/vars/main.yml
16:  'rhel72': 'java-1.8.0-openjdk',

ansible/roles/jenkins-worker/tasks/main.yml
15:  when: "'rhel72-s390x' in inventory_hostname"
23:  when: "'rhel72-s390x' in inventory_hostname"
67:  when: "'rhel72-s390x' in inventory_hostname"
74:  when: "'rhel72-s390x' in inventory_hostname"
78:  when: "'rhel72-s390x' in inventory_hostname"

ansible/roles/package-upgrade/vars/main.yml
9:    'yum': ['centos', 'rhel72'],

ansible/playbooks/jenkins/worker/create.yml
26:      when: "'rhel72-s390x' in inventory_hostname"

ansible/roles/jenkins-worker/tasks/partials/tap2junit/rhel72.yml
4:# rhel72: python2.7 is default

@rvagg
Copy link
Member

rvagg commented Mar 8, 2018

+1 on changing to "rhel7"

If you change the hostname in Ansible I guess you should do it in Jenkins. And when you do it there you'll need to update the secret since it's tied to the name.

@gibfahn gibfahn self-assigned this Mar 8, 2018
@gibfahn
Copy link
Member

gibfahn commented Mar 8, 2018

If you change the hostname in Ansible I guess you should do it in Jenkins. And when you do it there you'll need to update the secret since it's tied to the name.

I can work with @gdams to change the Jenkins name and the secret, but I'm not sure how we change the machine name. @mhdawson is that done through the marist UI? If so I should be able to work out how to do it (probably good practice to run through it).

@mhdawson
Copy link
Member

mhdawson commented Mar 8, 2018

To be honest I don't remember how we change the machine name, may require doing it both in the UI to match as well as on the machine itself as I don't think the UI will propagate.

You do need to change at least ansible templates when you change it in jenkins otherwise you can have problems when after running ansible. I already had to figure out why the agent was not connecting when I updated the ppc release machine and I think it was because ansible had not been updated (I did not have time to investigate, I just fixed the name in the file that was starting the agent).

I'd personally prioritize getting the OSX/ccache issue resolved finishing up the OSX machines before this renaming work (unless its already half done in which case we should do the full job)

@gibfahn
Copy link
Member

gibfahn commented Mar 9, 2018

Ahh, I didn't realise that the hostnames hadn't yet changed.

$ s test-linuxonecc-rhel72-s390x-1 hostname
test-linuxonecc-rhel72-s390x-1

In that case I think this PR should directly update to rhel7 rather than rhel74.

mhdawson

This comment was marked as off-topic.

@sam-github
Copy link
Contributor

Too stale, lets close.

@sam-github sam-github closed this Jan 28, 2020
@richardlau
Copy link
Member

richardlau commented Jan 28, 2020

We had to migrate the LinuxONE hosts recently (#2080) and the new machines just have rhel7 in their names.

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.

7 participants