-
Notifications
You must be signed in to change notification settings - Fork 142
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
use nightly-build wego-app image tag for nightly runs #1423
Conversation
0e0e548
to
5e8948f
Compare
0eeb5a7
to
4dac11c
Compare
705bf20
to
c9e0a0e
Compare
env: | ||
REGISTRY: ghcr.io | ||
IMAGE_NAME: weaveworks/wego-app | ||
USE_NIGHTLY_BUILD_TAG: "true" |
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.
One small nit. Can we have this as 1
and then just check that it's set? There is usually no point in doing a string "true"
in bash. :)
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.
Oh right, you are already doing this:
} else if os.Getenv("USE_NIGHTLY_BUILD_TAG") != "" {
version = "nightly-build"
}
so just set it to 1
. :)
env: | ||
REGISTRY: ghcr.io | ||
IMAGE_NAME: weaveworks/wego-app | ||
USE_NIGHTLY_BUILD_TAG: "true" |
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.
USE_NIGHTLY_BUILD_TAG: "true" | |
USE_NIGHTLY_BUILD_TAG: 1 |
c9e0a0e
to
0e69c76
Compare
closes #1421
example run: https://github.com/weaveworks/weave-gitops/runs/5138673832?check_suite_focus=true