Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixes #11609: Update to use Pulp module 1.0 #77

Merged
merged 1 commit into from
Sep 8, 2015

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Aug 28, 2015

No description provided.

messaging_ca_cert => $certs::pulp_parent::messaging_ca_cert,
messaging_client_cert => $certs::pulp_parent::messaging_client_cert,
consumers_ca_cert => $certs::ca_cert,
consumers_ca_key => $certs::ca_key,
consumers_crl => $candlepin::crl_file,
proxy_url => $proxy_url,
proxy_port => $proxy_port,
proxy_username => $proxy_username,
proxy_password => $proxy_password,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The =>'s should be aligned one space to the right of the longest parameter

@stbenjam
Copy link
Member

Is 1.0 ready?

@ehelms
Copy link
Member Author

ehelms commented Aug 28, 2015

I believe so functional parity wise - I wanted to do this part and do a test round with the installer, however, that is hard to do right now in the middle of TFM migration. This PR does remove an input parameter from this module.

@ehelms ehelms force-pushed the fixes-11609 branch 2 times, most recently from 0a22003 to 8ea7c81 Compare September 8, 2015 17:19
@ehelms
Copy link
Member Author

ehelms commented Sep 8, 2015

@stbenjam pushed updates to what we need to pass to Pulp and deploy, my local testing says this should be good to go along with -- theforeman/puppet-pulp#97

@stbenjam
Copy link
Member

stbenjam commented Sep 8, 2015

ACK

ehelms added a commit that referenced this pull request Sep 8, 2015
Fixes #11609: Update to use Pulp module 1.0
@ehelms ehelms merged commit 1d55eae into theforeman:master Sep 8, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants