Skip to content

add solaris2 support - #21

Open
ccope wants to merge 2 commits into
L2G:masterfrom
ccope:solaris-support
Open

add solaris2 support#21
ccope wants to merge 2 commits into
L2G:masterfrom
ccope:solaris-support

Conversation

@ccope

@ccope ccope commented Jun 26, 2014

Copy link
Copy Markdown

No description provided.

@jperry

jperry commented Jul 31, 2014

Copy link
Copy Markdown

any plans to merge this?

@L2G L2G changed the title add solaris support add solaris2 support Jan 13, 2015
@L2G

L2G commented Jan 13, 2015

Copy link
Copy Markdown
Owner

@ccope, thanks very much for doing this. I'm adding some ChefSpec tests and polishing it a bit.

@jperry, yes. 😸

@L2G L2G self-assigned this Jan 13, 2015
@L2G

L2G commented Jan 14, 2015

Copy link
Copy Markdown
Owner

I'm trying to add Solaris into Test Kitchen, but I'm running into trouble because the syntax of the shell script currently being served out of https://www.chef.io/chef/install.sh is not a pure Bourne-shell script, so it fails with a syntax error in the Vagrant box I'm using.

The Chef folks appear to have fixed it, except the fix hasn't been uploaded to the web server. I've reported the latter (chef/omnitruck#75).

@L2G

L2G commented Mar 11, 2015

Copy link
Copy Markdown
Owner

Chef 12.1.1 seems to be working with this now, so I'm back on this.

@L2G

L2G commented Mar 11, 2015

Copy link
Copy Markdown
Owner

The sticky wicket with the SSH key may become moot with the next Vagrant release. See hashicorp/vagrant#5290.

L2G added a commit that referenced this pull request Mar 11, 2015
Add config.ssh.insert_key = false because Vagrant does not support
replacing the SSH key on Solaris yet.
L2G pushed a commit that referenced this pull request Mar 11, 2015
L2G pushed a commit that referenced this pull request Mar 11, 2015
L2G added a commit that referenced this pull request Mar 11, 2015
L2G added a commit that referenced this pull request Mar 11, 2015
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