Skip to content

add solaris2 support #21

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add solaris2 support #21

wants to merge 2 commits into from

Conversation

ccope
Copy link

@ccope ccope commented Jun 26, 2014

No description provided.

@jperry
Copy link

jperry commented Jul 31, 2014

any plans to merge this?

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

L2G commented Jan 13, 2015

@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
Copy link
Owner

L2G commented Jan 14, 2015

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
Copy link
Owner

L2G commented Mar 11, 2015

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

@L2G
Copy link
Owner

L2G commented Mar 11, 2015

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