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

Change apt_key_url to avoid redirection and possible play failure #681

Merged

Conversation

4SH-gaupee
Copy link
Contributor

@4SH-gaupee 4SH-gaupee commented Oct 18, 2024

EDIT : Fix typo in summary to avoid misunderstanding to anyone looking at this PR in the future

SUMMARY

Current url cause breaking when installing the repo (here's a result after beeing redirected)
image

This change set the proper url to retreive key files (https://pgp.mongodb.com)

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

roles/mongodb_repository/defaults/main.yml

ADDITIONAL INFORMATION

Running an install will fail because the retrieved apt key is broken.

Copy link

codecov bot commented Oct 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.40%. Comparing base (42d35bd) to head (1175278).
Report is 22 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #681      +/-   ##
==========================================
- Coverage   73.68%   73.40%   -0.29%     
==========================================
  Files           6        6              
  Lines         935      940       +5     
  Branches      144      145       +1     
==========================================
+ Hits          689      690       +1     
- Misses        210      211       +1     
- Partials       36       39       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rhysmeister rhysmeister merged commit 7cdd7b8 into ansible-collections:master Oct 19, 2024
203 of 204 checks passed
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.

2 participants