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

Fixes #12110 - updating puppet-pulp to use is_parent => true param #88

Merged
merged 1 commit into from
Oct 8, 2015

Conversation

johnpmitsch
Copy link
Contributor

No description provided.

@@ -64,4 +64,6 @@

$qpid_url = "amqp:ssl:${::fqdn}:5671"
$candlepin_event_queue = 'katello_event_queue'

$is_parent = true
Copy link
Member

Choose a reason for hiding this comment

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

Is this intended to be used somewhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

trying to update for this change theforeman/puppet-pulp#108

Copy link
Member

Choose a reason for hiding this comment

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

params.pp is only if the parameter is actually a class parameter (in this case on katello itself).

@johnpmitsch
Copy link
Contributor Author

Waiting on theforeman/puppet-pulp#108 to be merged

@stbenjam
Copy link
Member

stbenjam commented Oct 8, 2015

Looks good to me, APT

@stbenjam stbenjam closed this Oct 8, 2015
@stbenjam stbenjam reopened this Oct 8, 2015
@ehelms
Copy link
Member

ehelms commented Oct 8, 2015

Thanks @johnpmitsch ! Please do the same to puppet-katello_devel

ehelms added a commit that referenced this pull request Oct 8, 2015
Fixes #12110 - updating puppet-pulp to use is_parent => true param
@ehelms ehelms merged commit 4e8d6ee into theforeman:master Oct 8, 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.

None yet

3 participants