Skip to content
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

[Workflow]: Auto build Official Docker Image #2136

Merged
merged 29 commits into from
Oct 18, 2023
Merged

[Workflow]: Auto build Official Docker Image #2136

merged 29 commits into from
Oct 18, 2023

Conversation

baiyangtx
Copy link
Contributor

Why are the changes needed?

Auto build official docker image by github workflow actions.

Brief change log

  • Add workflow to build official image Automated.
  1. When commits merged into master or tags pushed, amoro, optimizer-flink, quickdemo will be built automated.
  2. When a PR is merged into the master branch, it triggers the building of a "master-snapshot" image. When a tag is pushed, it triggers the building of a "release" image, and the image tag matches the corresponding GitHub tag.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@baiyangtx baiyangtx marked this pull request as ready for review October 18, 2023 02:06
@shidayang
Copy link
Contributor

LGTM

@GOODBOY008
Copy link
Member

@baiyangtx Should add concurrency limit .

- "v*"


workflow_dispatch:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the on:workflow_dispatch used for?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed.

on:
push:
branches:
- "master"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build docker triggered by master push is too frequently,we should discuss a better trigger event to build latest docker image.

Copy link
Contributor

@zhoujinsong zhoujinsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@zhoujinsong zhoujinsong merged commit 177609c into master Oct 18, 2023
@zhoujinsong zhoujinsong deleted the docker-image branch October 18, 2023 07:58
@baiyangtx baiyangtx mentioned this pull request Oct 18, 2023
3 tasks
@shidayang shidayang mentioned this pull request Oct 24, 2023
70 tasks
ShawHee pushed a commit to ShawHee/arctic that referenced this pull request Dec 29, 2023
* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test docker ci

* test quickdemo docker ci

* test quickdemo docker ci

* test quickdemo docker ci

* test quickdemo docker ci

* test quickdemo docker ci

* test quickdemo docker ci

* add concurrency.

* add latest label.

* add latest label.

* add latest label.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants