Skip to content

Remove 10gen repository support#497

Merged
ekohl merged 2 commits intovoxpupuli:masterfrom
ekohl:remove-10gen
Oct 14, 2018
Merged

Remove 10gen repository support#497
ekohl merged 2 commits intovoxpupuli:masterfrom
ekohl:remove-10gen

Conversation

@ekohl
Copy link
Copy Markdown
Member

@ekohl ekohl commented Oct 7, 2018

There's still a reference in init.pp but #493 removes that file altogether. There's also some in the acceptance tests but I'll fix those later.

Closes GH-416

@ekohl ekohl added the backwards-incompatible This change will lead to a major version bump for the next release label Oct 7, 2018
@ekohl ekohl changed the title Rip out 10gen repository support Remove 10gen repository support Oct 7, 2018
Comment thread manifests/repo.pp
$proxy = undef,
$proxy_username = undef,
$proxy_password = undef,
Optional[String] $repo_location = undef,
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.

datatypes \o/

Comment thread manifests/repo/yum.pp
# http://docs.mongodb.org/manual/tutorial/install-mongodb-on-red-hat-centos-or-fedora-linux/

if($mongodb::repo::ensure == 'present' or $mongodb::repo::ensure == true) {
if $mongodb::repo::ensure == 'present' or $mongodb::repo::ensure == true {
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.

we should probably add that to the styleguide later

@ekohl ekohl merged commit 9c0ef87 into voxpupuli:master Oct 14, 2018
@ekohl ekohl deleted the remove-10gen branch October 14, 2018 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backwards-incompatible This change will lead to a major version bump for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate (and then remove) 10gen requirement/pieces

2 participants