-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Create publish-docker.yml #11632
base: main
Are you sure you want to change the base?
Create publish-docker.yml #11632
Conversation
I was waiting for the account creation. But thanks for working on it. |
in addition to pushing the latest, it would be great to publish images tagged with specific releases (1.7/1.8/etc) |
I think many of us are eager to get the docker image in the hub to improve our integration tests in the sub project repositories. Thank you for doing the hard work @ajantha-bhat ! |
Thanks for the suggestion @kevinjqliu - I've added a new trigger for tag push events to publish the docker images with the version tag |
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.
Ran it on my fork, and it looks good: https://github.com/Fokko/iceberg/actions/runs/12007773313
run: | | ||
docker login -u ${{ secrets.DOCKERHUB_USER }} -p ${{ secrets.DOCKERHUB_TOKEN }} | ||
- name: Build Docker Image | ||
run: docker build -t apache/iceberg-rest-adapter -f docker/iceberg-rest-adapter-image/Dockerfile . |
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.
What about using apache/iceberg-rest-fixture
? If we have a consensus, I can rename apache/iceberg
.
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 am ok with the name iceberg-rest-fixture
, it is a good name 👍
Pending completion of: https://github.com/apache/iceberg/pull/11283/files