-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[github action] Fix a bug which can cover build image #10647
Conversation
@@ -37,13 +37,19 @@ jobs: | |||
with: | |||
distribution: zulu | |||
java-version: "11" | |||
- name: Leeway build | |||
- name: Change build hash | |||
- run: | |
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.
-
before run should be removed
- name: Leeway build | ||
- name: Change build hash | ||
- run: | | ||
echo Jetbrains nightly build |
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.
echo into README.md?
2f92401
to
91e24ed
Compare
91e24ed
to
249ae58
Compare
249ae58
to
ca46e65
Compare
/hold |
3981b93
to
c12b122
Compare
c12b122
to
4ae1db0
Compare
After discuss with @akosyakov , we change
and
This way the latest version and the stable version do not overwrite each other, i.e. they do not have same image tag Another thing is to remove the @mustard-mh Could you have a look again? If it is good, we should merge this to main branch. |
/werft run 👍 started the job as gitpod-build-pd-fix-build-task.8 |
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.
Test good, but have some problem about cache
gcloud auth configure-docker --quiet | ||
export LEEWAY_WORKSPACE_ROOT=$(pwd) | ||
data=$(curl -sSL "https://data.services.jetbrains.com/products?code=${{ inputs.productCode }}&fields=distributions%2Clink%2Cname%2Creleases&_=$(date +%s)000") | ||
link=$(echo "$data" | jq -r '.[0].releases[0].downloads.linux.link') | ||
cd components/ide/jetbrains/image | ||
leeway build -Dversion=latest -DimageRepoBase=eu.gcr.io/gitpod-core-dev/build -DjetbrainsBackendQualifier=latest -D${{ inputs.productId }}DownloadUrl=$link .:${{ inputs.productId }} | ||
leeway build -Dversion=latest -DimageRepoBase=eu.gcr.io/gitpod-core-dev/build -D${{ inputs.productId }}DownloadUrl=$link .:${{ inputs.productId }}-latest |
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.
I look at leeway package with suffix -latest
, it doesn't use ${{ inputs.productId }}DownloadUrl
, it is a fixed url, so If we want to make sure this always latest version, we should not use cache
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.
🤦♂️ Ah... didn't notice this
4ae1db0
to
f19b065
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
/unhold |
Description
This PR fix a problem which can cover build image
This is because leeway always upload image using commit-{commitID}
gitpod/components/ide/jetbrains/image/BUILD.yaml
Line 38 in 2e55f9c
At before, we only checkout main branch and run
leeway
command, it will cause cover image build with same tagThis PR add a commit before runleeway
, so it never effect main branchThis PR using suffix
-latest
as build target, so it never affect stable branchRelated Issue(s)
Fixes #
How to test
Release Notes
Documentation