Skip to content

Add aptkey_options parameter to mongodb::repo class#524

Merged
bastelfreak merged 3 commits intovoxpupuli:masterfrom
itshosted:aptkey_options
Jan 28, 2019
Merged

Add aptkey_options parameter to mongodb::repo class#524
bastelfreak merged 3 commits intovoxpupuli:masterfrom
itshosted:aptkey_options

Conversation

@JvGinkel
Copy link
Copy Markdown
Contributor

@JvGinkel JvGinkel commented Jan 8, 2019

Pull Request (PR) description

Adding an aptkey_options parameter so we can pass for example proxy settings to the apt::key resource via hiera:

mongodb::repo::aptkey_options: 'http-proxy="http://proxyuser:proxypass@example.org:3128"'

Adding this parameter so we can pass proxy settings for example
Comment thread manifests/repo.pp Outdated
@bastelfreak bastelfreak added the enhancement New feature or request label Jan 16, 2019
Comment thread manifests/repo.pp Outdated
}

contain mongodb::repo::apt
contain ::mongodb::repo::apt
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please don't add the leading ::. This is a deprecated syntax, required for Puppet 3.

@bastelfreak
Copy link
Copy Markdown
Member

The CI jobs are currently failing because of puppetlabs/beaker-puppet#102. We need to rerun them after the fix got merged and released.

@bastelfreak
Copy link
Copy Markdown
Member

To clarify, we don't support puppet 3, so the :: can be removed. They are required for Puppet 3.

@JvGinkel
Copy link
Copy Markdown
Contributor Author

Ah, after a quick google I understand what you mean. I also have updated my local gem set and removed the puppet-lint-absolute_classname-check and puppet-lint-classes_and_types_beginning_with_digits-check gem

@bastelfreak
Copy link
Copy Markdown
Member

Thanks for the PR!

@bastelfreak bastelfreak merged commit 5133782 into voxpupuli:master Jan 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants