Skip to content

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
-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.

3 participants