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

Solaris 11.1 -- OpenCSW #12

Merged
merged 3 commits into from
Sep 3, 2015
Merged

Solaris 11.1 -- OpenCSW #12

merged 3 commits into from
Sep 3, 2015

Conversation

jaymell
Copy link

@jaymell jaymell commented Aug 27, 2015

Add support for installation using OpenCSW -- assumes you already have OpenCSW repositories enabled

@jloh jloh self-assigned this Aug 27, 2015
@jloh
Copy link
Owner

jloh commented Aug 31, 2015

Awesome, thanks for the addition. You've confirmed this works on Solaris systems? I don't have any to test on myself.

@jaymell
Copy link
Author

jaymell commented Aug 31, 2015

Good question!

So, it worked on the systems I deployed to in my environment. Just to cover
all my bases, I spun up a fresh VM this morning and am running into
issues. I didn't install the package we use to manage privileged commands
(our equivalent of 'sudo') and did have problems until I removed all the
references to sudo in the tasks and handlers. This kind of goes back to the
issue you asked about previously, so I will do some additional testing to
see if I can work around it without changing the rest of the files in the
role.

But, once I get past the sudo issue and make sure that CSW repositories are
enabled, things work great. Regarding that latter point (CSW packages),
the role will likely fail if people haven't previously set the repo up, so
I should probably go ahead and add a specific task to enable this
repository if not already done.

I'll get back to you once I've gotten some time do that.

--James

On Mon, Aug 31, 2015 at 7:22 AM, James notifications@github.com wrote:

Awesome, thanks for the addition. You've confirmed this works on Solaris
systems? I don't have any to test on myself.


Reply to this email directly or view it on GitHub
#12 (comment)
.

@jloh
Copy link
Owner

jloh commented Aug 31, 2015

Please feel free to remove the sudo dependency as I'm planning on removing it from the whole role after this merge.

In terms of the repo, could you please set that as a default? It'll make overriding it easier for people that don't use it, I'm assuming it isn't the only way to get the package on solaris and that some people may setup internal repos with the package?

. Solaris tasks now include installing, if necessary,
CSWpkgutil
@jaymell
Copy link
Author

jaymell commented Sep 3, 2015

I made an additional commit, adding the installation of CSWpkgutil if not already present on the system. I've verified that it works perfectly on freshly deployed Solaris 11.1 VMs.

Also, I removed all the references to sudo throughout the role and verified that it successfully deploys.

Let me know if this looks good to you, or if you'd like to see any additional changes.

Cheers,
James

jloh pushed a commit that referenced this pull request Sep 3, 2015
Solaris 11.1 -- OpenCSW. Closes #11
@jloh jloh merged commit 89d6f8f into jloh:master Sep 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants