-
Notifications
You must be signed in to change notification settings - Fork 268
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
Features/readme release branch #568
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to add a few lines explaining the issue with the release symptoms when you are not signed in Docker Hub.
Makefile
Outdated
@@ -58,13 +58,13 @@ include tasks/*.mk | |||
############################################################################### | |||
|
|||
## Porcelain ################################################################## | |||
install: network-create vault-create assets-create app-create sandbox-create diversity-create proxy-create | |||
install: network-create vault-create assets-create app-create sandbox-create proxy-create |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you remove the diversity tasks from the normal flow, are you going to provide another task, say make diversity-install
to be able to deploy updates on that?
tasks/release.mk
Outdated
@@ -40,3 +41,6 @@ release-push-snapshot: | |||
$(DOCKER) push $(app_image) | |||
$(DOCKER) push $(assets_image) | |||
$(DOCKER) push $(sandbox_image) | |||
|
|||
## Build new images with a tag name, e.g. VERSION=auto, then push to docker hub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be good to explain the use case for this task. Perhaps with an example situation?
48c4b8f
to
0819218
Compare
@arnau Here is the added release-branch task.
Also, just note, I removed diversity image tasks from the capture-all porcelain tasks. I don't see it necessary to be a part of that install process. Seem ok to you?