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

Add CA cert to keystore as a trustedcert #245

Merged
merged 1 commit into from
Feb 28, 2019

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Feb 20, 2019

No description provided.

@ekohl
Copy link
Member

ekohl commented Feb 20, 2019

#173 should do this as well. Shall we revisit that?

@ehelms
Copy link
Member Author

ehelms commented Feb 20, 2019

#173 Does not actually do what this does. I'm happy to re-visit that if you are signing on to keep that PR up to date as I test and updates are needed.

This part is also needed -- https://github.com/theforeman/puppet-certs/pull/245/files#diff-0a129cc5a540f220caade2da4fc3229bR113

manifests/candlepin.pp Outdated Show resolved Hide resolved
@ehelms ehelms force-pushed the update-candlepin-certs branch 3 times, most recently from c01c061 to f764aa2 Compare February 26, 2019 18:14
@ehelms
Copy link
Member Author

ehelms commented Feb 26, 2019

Updated and passing tests

manifests/candlepin.pp Outdated Show resolved Hide resolved
@ehelms ehelms force-pushed the update-candlepin-certs branch from f764aa2 to fa2d4e9 Compare February 28, 2019 13:32
@ehelms ehelms force-pushed the update-candlepin-certs branch from fa2d4e9 to 3a186e1 Compare February 28, 2019 13:44
@ehelms ehelms merged commit f71ab88 into theforeman:master Feb 28, 2019
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.

3 participants