-
Notifications
You must be signed in to change notification settings - Fork 166
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
ansible: update Joyent hosted IP addresses #2568
Conversation
It was never added to inventory; perhaps add it as part of the list? |
done. |
In terms of test-joyent-ubuntu1604_arm_cross-x64-1 and release-joyent-ubuntu1604_arm_cross-x64-1 I noticed that after the fact. I think we should just confirm with @rvagg and then spin those down. For |
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.
LGTM
This host was not migrated with the recent Joyent datacenter closure.
I've removed |
Refs: #2552
cc @mhdawson @jbergstroem
Some oddities:
I can't find a replacement forRemoved in this PR.test-joyent-ubuntu1804_docker-x64-1
in the table in Replacing machines from Joyent data center #2552. It also looks like all the hosts in https://ci.nodejs.org/search/?q=test-joyent that are containers (i.e. have the word "container" in their node names) are offline.test-joyent-ubuntu1604_arm_cross-x64-1
andrelease-joyent-ubuntu1604_arm_cross-x64-1
were removed from the inventory in https://github.com/nodejs/build/pull/2290/files#diff-6faf346b88ef5cbd5f6733c35ef37f0884f3fcc946a55c5b811843b652421f6d and possibly did not need to be migrated as we now cross compile in docker containers. The build history for https://ci.nodejs.org/computer/test-joyent-ubuntu1604_arm_cross-x64-1/builds is empty (i.e. no recent builds).Added in this PR.infra-joyent-debian10-x64-1/grafana
(from the table) doesn't appear to match anything in the inventory.