-
Notifications
You must be signed in to change notification settings - Fork 0
Add CD deploy to docker hub & k8s #23
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.
Minior updates to make the hidden knowledge more explicity in favour for Opensource development.
"auto_merge": false, | ||
"repository": "oursky/puttyimages-web", | ||
"environments": ["pandawork"], | ||
"allowed_rooms": ["deployment", "likecoin-internal"] |
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.
What is the usage of allowed_rooms?
Sorry, May I know where will the payload is used?
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.
The payload maybe not necessary, but I cannot trigger deploy with name only.
env.shell = '/bin/sh -l -c' | ||
env.user = 'unimportant' | ||
env.roledefs.update({ | ||
'pandawork': ['puttyimage.pandawork.com'] # this is also unimportant |
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.
Let update the comments to explain why this is unimportant. Instead of leaving it as magic unimportant
.
We need to document to tell people about the settings... |
I am working on it and will close this PR for now. |
Connects likecoin#23