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

Reprovision s390x machines #2083

Merged
merged 7 commits into from
Dec 18, 2019
Merged

Reprovision s390x machines #2083

merged 7 commits into from
Dec 18, 2019

Conversation

AshCripps
Copy link
Member

@AshCripps AshCripps commented Dec 9, 2019

READY FOR REVIEW

This is to replace the current s390x machines with 5 machines in the new datacentre - 1 release, 4 test

See: #2080

@AshCripps
Copy link
Member Author

Part of this PR is to update every reference from rhel72 to rhel7 because its just plain wrong otherwise, Let me know if ive missed anything or if thats likely to cause problems.

@richardlau
Copy link
Member

richardlau commented Dec 10, 2019

Part of this PR is to update every reference from rhel72 to rhel7 because its just plain wrong otherwise, Let me know if ive missed anything or if thats likely to cause problems.

We'd need to go through all of the jobs on Jenkins (both test and release) and update (which we'll have to do regardless if we're adding new machine names (i.e. the switch from linuxonecc to ibm as the provider) rather than replacing the existing ones).

SelectCompiler.sh and VersionSelectorScript.groovy should be okay as they just match on s390x:

*s390x* ) SELECT_ARCH=S390X ;;

[ /s390x/, anyType, lt(6) ],

@AshCripps AshCripps changed the title [WIP] - Reprovision s390x machines Reprovision s390x machines Dec 13, 2019
@AshCripps
Copy link
Member Author

all five machines are ansibled and this pr is ready to be reviewed

Copy link
Contributor

@sam-github sam-github left a comment

Choose a reason for hiding this comment

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

a couple small comments, this is close to landing.

ansible/roles/baselayout/tasks/main.yml Outdated Show resolved Hide resolved
ansible/roles/jenkins-worker/meta/main.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@sam-github sam-github left a comment

Choose a reason for hiding this comment

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

@nodejs/build-infra This requires firewall changes. There is some urgency to this, it would be helpful to have it done in the next few days, to give us time to test the replacements. We may have the existing machines shut off (along with their datacentre) by January 1st 2020 (though I hope that won't happen so quickly).

ansible/inventory.yml Show resolved Hide resolved
ansible/inventory.yml Show resolved Hide resolved
@sam-github
Copy link
Contributor

@richardlau / @nodejs/build-wg Any reason for me not to squash merge this right now? It doesn't touch select-compiler.sh or the groovy files, just ansible, so I can't see it interfering with the ongoing security release, it looks entirely benign.

@richardlau
Copy link
Member

@richardlau / @nodejs/build-wg Any reason for me not to squash merge this right now? It doesn't touch select-compiler.sh or the groovy files, just ansible, so I can't see it interfering with the ongoing security release, it looks entirely benign.

No objections from me.

cc @nodejs/build

@mhdawson
Copy link
Member

@sam-github IPs added to firewalls.

@sam-github sam-github merged commit 9df1fc4 into nodejs:master Dec 18, 2019
@AshCripps AshCripps deleted the new-linuxone branch December 24, 2019 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants