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

ansible: add requireio-osx1010-x64-1 to inventory #753

Merged
merged 1 commit into from
Jun 12, 2017

Conversation

gibfahn
Copy link
Member

@gibfahn gibfahn commented Jun 10, 2017

Fixes: #752

Not really sure why the user is iojs rather than root here, is that intended?

@rvagg
Copy link
Member

rvagg commented Jun 12, 2017

yeah, the root account isn't active on that machine, we could fix it up but for now let's just leave it as is cause we're migrating to MacStadium anyway. There's a user account "rvagg" that has admin privs, if need be I could probably stick the test key in there too. For full admin though you really need remote desktop, which you could do via SSH tunnel but you then need a password to log in and do anything useful. We're going to have to figure out an answer for this problem, it'll probably mean assigning the same password to an admin account for all of the test OSX machines. /cc @jbergstroem @mhdawson

@gibfahn while you're in here, could you add another entry in there, under release, same name, IP 192.168.2.211 please

@gibfahn
Copy link
Member Author

gibfahn commented Jun 12, 2017

could you add another entry in there, under release, same name, IP 192.168.2.211 please

Done

@gibfahn gibfahn requested a review from rvagg June 12, 2017 10:42
@rvagg
Copy link
Member

rvagg commented Jun 12, 2017

lgtm

@gibfahn gibfahn merged commit 114edd8 into nodejs:master Jun 12, 2017
@gibfahn gibfahn deleted the invent-osx branch June 12, 2017 13:54
@jbergstroem
Copy link
Member

jbergstroem commented Jun 12, 2017

It might be worth mentioning that the create playbooks with fail now seeing how we don't have any support for macos builders. Hopefully this will not be the case any longer in the coming week or two, seeing how we now have a few machines up and running at macstadium we will fully ansible-fy (if possible).

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.

3 participants