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
build: Simplify builds #3478
build: Simplify builds #3478
Conversation
I've re-enabled up-to-date with master check. |
mkdir -p /tmp/log/argo-e2e | ||
KUBECONFIG=~/.kube/config make start PROFILE=$PROFILE E2E_EXECUTOR=docker DEV_IMAGE=true > /tmp/log/argo-e2e/argo.log & | ||
git fetch --tags |
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.
this was a key fix
mkdir -p /tmp/log/argo-e2e | ||
KUBECONFIG=~/.kube/config make start PROFILE=$PROFILE E2E_EXECUTOR=docker DEV_IMAGE=true > /tmp/log/argo-e2e/argo.log & | ||
git fetch --tags | ||
KUBECONFIG=~/.kube/config make start PROFILE=$PROFILE E2E_EXECUTOR=docker DEV_IMAGE=true 2>&1 > /tmp/log/argo-e2e/argo.log & |
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.
also pipe stderr
@@ -102,14 +99,10 @@ jobs: | |||
codegen: | |||
name: Codegen | |||
runs-on: ubuntu-latest | |||
timeout-minutes: 10 | |||
timeout-minutes: 15 |
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.
occasionally this takes longer than 10m
@@ -21,10 +21,6 @@ jobs: | |||
steps: | |||
- name: Checkout code | |||
uses: actions/checkout@v2 | |||
with: |
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.
@simster7 thoughts on this change?
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.
All good with me as long as it plays nice with GIT_BRANCH = $(shell git branch --show-current)
, which was the original reason for the code being this way
@@ -1,4 +1,4 @@ | |||
# Resume The Template | |||
# Resume A Workflow |
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.
error
@@ -21,10 +21,6 @@ jobs: | |||
steps: | |||
- name: Checkout code | |||
uses: actions/checkout@v2 | |||
with: |
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.
All good with me as long as it plays nice with GIT_BRANCH = $(shell git branch --show-current)
, which was the original reason for the code being this way
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.@simster7 @sarabala1979 I've been spending too much time fixing builds recently and want to simplify them.
This change will mean that we no longer build branch images (i.e. no images name
argoproj/argocli:my-branch
instead, they will be latest. You must runmake clean
whenever you change branch to get new images.