DEVPROD-34208: use DevProd ECR for container images instead of Artifactory - #1208
DEVPROD-34208: use DevProd ECR for container images instead of Artifactory#1208fetsko wants to merge 1 commit into
Conversation
|
@kevinAlbs @vector-of-bool Do you mind reviewing this PR when you get a chance? Because of security concerns, artifactory.corp.mongodb.com has been shutdown so any builds using it would fail. |
vector-of-bool
left a comment
vector-of-bool
left a comment
There was a problem hiding this comment.
The changes LGTM other than the comment about the earthly script changes, which I think are mostly unnecessary complication.
Also may want more input from the people that maintain the Python bindings' scripts.
There was a problem hiding this comment.
The bindings/python/ subdirectory is owned by Python devs and may require some input from a Python dev member. @kevinAlbs do you know who best to look at this?
There was a problem hiding this comment.
Not sure this change to the earthly script is necessary, and it assumes Docker when some of us use Podman. These tasks aren't run manually very often and I think we can leave the authentication as a manual step for now.
|
Kevin and I will be absent on Friday, July 31. I don't think this is urgent for us as the tasks in question aren't mandatory until we need to do a release. Recommend consulting the Python team about the Python bindings' scripts. |
3 participants
No description provided.