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 mongo 5.0 debian / ubuntu apt key #615

Merged
merged 1 commit into from
Oct 21, 2021

Conversation

xepa
Copy link

@xepa xepa commented Jul 28, 2021

Pull Request (PR) description

Adds 5.0 apt key for debian/ubuntu mongo repo

This Pull Request (PR) fixes the following issues

Resolves #614

@xepa xepa changed the title Resolves voxpupuli/puppet-mongodb#614 Add mongo 5.0 debian / ubuntu apt key [Resolves voxpupuli/puppet-mongodb#614] Jul 28, 2021
@kenyon kenyon changed the title Add mongo 5.0 debian / ubuntu apt key [Resolves voxpupuli/puppet-mongodb#614] Add mongo 5.0 debian / ubuntu apt key Aug 2, 2021
@bastelfreak bastelfreak added the enhancement New feature or request label Aug 13, 2021
@bastelfreak
Copy link
Member

hi @xepa , thanks for the PR. Can you add a unit test for this?

1 similar comment
@bastelfreak
Copy link
Member

hi @xepa , thanks for the PR. Can you add a unit test for this?

@root-expert
Copy link
Member

Hello! Can you rebase with our master branch?

@root-expert
Copy link
Member

Can you instead perform a rebase instead of a merge commit? Also can you squash the rest of the commits?

@xepa
Copy link
Author

xepa commented Sep 4, 2021

Sorry for the late response I was on vacation, it was rebased and squashed.

@xepa
Copy link
Author

xepa commented Sep 28, 2021

I rebased again

@root-expert
Copy link
Member

@xepa We need some acceptance tests to verify MongoDB 5.0 works as expected.

@xepa xepa force-pushed the issue.upstream#614 branch from 8e90d14 to 4f02462 Compare September 30, 2021 09:04
@xepa xepa force-pushed the issue.upstream#614 branch from 4f02462 to 9e5c471 Compare September 30, 2021 09:16
@xepa
Copy link
Author

xepa commented Sep 30, 2021

I added some static unit tests for 26, 4 and 5 @root-expert if this is not what you were looking for please advise on how to proceed.

Please note: I am a ruby newbie .. so any tips to improve the unit tests are welcome ;)

@bastelfreak bastelfreak merged commit 61b00f2 into voxpupuli:master Oct 21, 2021
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.

Missing key for Apt for installing 5.0 mongo
5 participants