-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
Signed-off-by: Raphael Ludwig <[email protected]>
Signed-off-by: Raphael Ludwig <[email protected]>
Signed-off-by: Raphael Ludwig <[email protected]>
Signed-off-by: Raphael Ludwig <[email protected]>
Signed-off-by: Raphael Ludwig <[email protected]>
a1f43db
to
97eee7d
Compare
Signed-off-by: Raphael Ludwig <[email protected]>
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.
Left a comment.
Also I saw that chart/charts/common-0.1.0.tgz is in the PR... not sure if that is on purpose?
Signed-off-by: Raphael Ludwig <[email protected]>
It was like this in the original repository, I removed the common chart and used the similar helpers that we have in the job-executor-service Chart. |
Signed-off-by: Raphael Ludwig <[email protected]>
Signed-off-by: Raphael Ludwig <[email protected]>
Signed-off-by: Raphael Ludwig <[email protected]>
8f2f269
to
0d6c3a2
Compare
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.
LGTM
The following Docker Images have been built: |
This PR