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

chore(ci): use artifactory mirror as docker registry #6490

Closed
wants to merge 1 commit into from

Conversation

kraenhansen
Copy link
Contributor

Description

Attempt at unbreaking CI by using the artifactory mirror instead of dockerhub directly.

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@kraenhansen kraenhansen added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Nov 13, 2024
@kraenhansen kraenhansen self-assigned this Nov 13, 2024
@kraenhansen kraenhansen force-pushed the kh/artifactory-docker-mirror branch 3 times, most recently from 942af52 to af9655c Compare November 13, 2024 21:58
@kraenhansen kraenhansen force-pushed the kh/artifactory-docker-mirror branch from af9655c to e0857d2 Compare November 13, 2024 22:49
function atlascli() {
docker run \
-e MCLI_PUBLIC_API_KEY \
-e MCLI_PRIVATE_API_KEY \
-e MCLI_ORG_ID \
-e MCLI_PROJECT_ID \
-e MCLI_OPS_MANAGER_URL \
mongodb/atlas atlas $@
$DOCKER_REGISTRY/library/mongodb/atlas atlas $@
Copy link
Collaborator

Choose a reason for hiding this comment

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

Random thought – would this be easier or harder to work with if we included /library/ in the env var value, so that the default would just end up being an empty string if nothing has been explicitly configured before?

@kraenhansen
Copy link
Contributor Author

Closing as it seems we'll be taking the route of authenticated requests to Docker Hub instead 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants