Skip to content
This repository has been archived by the owner on May 4, 2021. It is now read-only.

Switch to using uber-container-tools project for image registry #305

Merged
merged 2 commits into from
Feb 11, 2020

Conversation

yiranwang52
Copy link
Contributor

As title.

sema
sema previously approved these changes Feb 11, 2020
Copy link
Contributor

@sema sema left a comment

Choose a reason for hiding this comment

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

I'm OK with this. Should we treat the change to the registry in which we publish the images as a breaking change?

imagePullPolicy: IfNotPresent
args:
- build
- --push=gcr.io
- --modifyfs=true
- -t=makisu-project/example-github:v0.1.11
Copy link
Contributor

Choose a reason for hiding this comment

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

We should update the version numbers in the examples to match images that do exist in the new repository. Alternatively, we should do a backfill.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do both.

@yiranwang52
Copy link
Contributor Author

It is breaking. I hope editing readme and including that in release note would be good enough.

@yiranwang52 yiranwang52 merged commit 354ac02 into master Feb 11, 2020
@yiranwang52 yiranwang52 deleted the yiran/registry branch March 12, 2020 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants