-
Notifications
You must be signed in to change notification settings - Fork 116
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
Publish Docker images for the GC tool #8055
Changes from 5 commits
0845f01
49738a2
2349ca0
27c674b
6b87872
9d3e4e5
508e48a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -134,12 +134,19 @@ jobs: | |
-a "${ARTIFACTS}" \ | ||
-g ":nessie-quarkus" \ | ||
-p "servers/quarkus-server" \ | ||
-d "Dockerfile-server" \ | ||
ghcr.io/projectnessie/nessie | ||
|
||
# Add version to the openapi file name | ||
cp api/model/build/generated/openapi/META-INF/openapi/openapi.yaml api/model/build/nessie-openapi-${RELEASE_VERSION}.yaml | ||
|
||
cp gc/gc-tool/build/executable/nessie-gc gc/gc-tool/build/executable/nessie-gc-${RELEASE_VERSION} | ||
tools/dockerbuild/build-push-images.sh \ | ||
-a "${ARTIFACTS}" \ | ||
-g ":nessie-gc-tool" \ | ||
-p "gc/gc-tool" \ | ||
-d "Dockerfile-gctool" \ | ||
ghcr.io/projectnessie/nessie-gc | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @snazy This would end up as a new repo in Quay and DockerHub as well. Do we need to create those repos upfront? Do we need to change regsync settings? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The I think we don't need to create repos upfront. Let's see whether the snapshot-publish job has something to complain - I guess not. |
||
|
||
echo "QUARKUS_UBER_JAR=${ARTIFACTS}/nessie-quarkus-${RELEASE_VERSION}-runner.jar" >> ${GITHUB_ENV} | ||
echo "CLI_UBER_JAR=${ARTIFACTS}/nessie-quarkus-cli-${RELEASE_VERSION}-runner.jar" >> ${GITHUB_ENV} | ||
|
@@ -219,6 +226,8 @@ jobs: | |
(\`chmod 744 nessie-gc-${RELEASE_VERSION}\` after download.) | ||
Can also be run using \`java -jar nessie-gc-${RELEASE_VERSION}\`, because it is actually a Java archive. | ||
Shell completion can be generated from the \`nessie-gc\` tool. | ||
Nessie GC tool is also available as Docker image: | ||
adutra marked this conversation as resolved.
Show resolved
Hide resolved
|
||
\`docker run --rm ghcr.io/projectnessie/nessie-gc:${RELEASE_VERSION} --help\`. | ||
|
||
The attached [\`nessie-helm-${RELEASE_VERSION}.tgz\`](${Q_HELM_CHART_URL}) is a packaged Helm chart, which can be downloaded and installed via Helm. | ||
There is also the [Nessie Helm chart repo](https://charts.projectnessie.org/), which can be added and used to install the Nessie Helm chart. | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,3 @@ | ||
# Nessie GC | ||
|
||
See [here](../site/docs/features/gc-internals.md). | ||
|
||
|
||
```shell | ||
docker run --rm -e POSTGRES_USER=pguser -e POSTGRES_PASSWORD=mysecretpassword -e POSTGRES_DB=nessie_gc -p 5432:5432 postgres:14 | ||
gc/gc-tool/build/executable/nessie-gc create-sql-schema --jdbc-url jdbc:postgresql://127.0.0.1:5432/nessie_gc --jdbc-user pguser --jdbc-password mysecretpassword | ||
``` | ||
See [here](../site/docs/features/gc). |
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.
You can omit the tags here. We needed those for Nessie to not mirror super-old versions.
Since you're already touching this file - can you change the regex for Nessie server above to something like
0[.][789]\\d[.].*|[123][.].*
- to include everything like0.7[0-9]....
and newer - and1.xxx
as well.