Skip to content

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

Merged
gibfahn merged 1 commit into
nodejs:masterfrom
gibfahn:invent-osx
Jun 12, 2017
Merged

gibfahn merged 1 commit into
nodejs:masterfrom
gibfahn:invent-osx

Conversation

@gibfahn

@gibfahn gibfahn commented Jun 10, 2017 •

Copy link
Copy Markdown
Member

Fixes: #752

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

@rvagg

rvagg commented Jun 12, 2017

Copy link
Copy Markdown
Member

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

gibfahn commented Jun 12, 2017

Copy link
Copy Markdown
Member Author

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

rvagg commented Jun 12, 2017

Copy link
Copy Markdown
Member

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

jbergstroem commented Jun 12, 2017 •

Copy link
Copy Markdown
Member

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