Skip to content
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

chore: fix e2e ci build on release branch #1161

Merged
merged 2 commits into from
Apr 5, 2021

Conversation

whynowy
Copy link
Member

@whynowy whynowy commented Apr 5, 2021

Signed-off-by: Derek Wang whynowy@gmail.com

On release branch, the image tags in the manifests are not latest but the real version like v1.3.0, this makes make start fail like https://github.com/argoproj/argo-events/runs/2272839939?check_suite_focus=true. This PR is used to fix this issue.

Checklist:

whynowy added 2 commits April 5, 2021 14:43
Signed-off-by: Derek Wang <whynowy@gmail.com>
Signed-off-by: Derek Wang <whynowy@gmail.com>
@@ -14,6 +14,7 @@ GIT_TREE_STATE=$(shell if [ -z "`git status --porcelain`" ]; then echo "clean" ;
DOCKER_PUSH?=false
IMAGE_NAMESPACE?=argoproj
VERSION?=latest
BASE_VERSION:=latest
Copy link
Member Author

Choose a reason for hiding this comment

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

To clarify:

BASE_VERSION is the original image tag in the manifests.
VERSION is the new image tag that could be passed in during image build.

@whynowy whynowy changed the title chore: fix e2e ci build in release branch chore: fix e2e ci build on release branch Apr 5, 2021
@whynowy whynowy requested review from alexec and VaibhavPage April 5, 2021 22:03
@whynowy whynowy merged commit b97a84d into argoproj:master Apr 5, 2021
@whynowy whynowy deleted the release-fix branch April 5, 2021 23:40
whynowy added a commit that referenced this pull request Apr 5, 2021
* chore: fix e2e ci build in release branch

Signed-off-by: Derek Wang <whynowy@gmail.com>
juliev0 pushed a commit to juliev0/argo-events that referenced this pull request Mar 29, 2022
* chore: fix e2e ci build in release branch

Signed-off-by: Derek Wang <whynowy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants