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

[INLONG-2028][CI] Add support for docker build on GitHub Actions #2112

Merged
merged 4 commits into from
Jan 7, 2022

Conversation

shink
Copy link
Member

@shink shink commented Jan 6, 2022

Fixes: #2028

Motivation

Add support for docker build on GitHub Actions.

Modifications

  • .github/workflows/ci_build_docker.yml

Verifying this change

Execute the following command.

mvn --batch-mode --update-snapshots -e -V clean verify -DskipTests -Pdocker

@dockerzhang
Copy link
Contributor

@shink InLong Docker Build workflow executes failed; please create an issue, I will fix it first.

@codecov-commenter
Copy link

Codecov Report

Merging #2112 (6fb2d92) into master (7b5c1ed) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #2112   +/-   ##
=========================================
  Coverage     12.28%   12.29%           
- Complexity     1159     1161    +2     
=========================================
  Files           413      413           
  Lines         35215    35215           
  Branches       5542     5542           
=========================================
+ Hits           4327     4330    +3     
+ Misses        30119    30117    -2     
+ Partials        769      768    -1     
Impacted Files Coverage Δ
.../tubemq/corebase/policies/FlowCtrlRuleHandler.java 34.51% <0.00%> (+0.44%) ⬆️
.../inlong/tubemq/corebase/policies/FlowCtrlItem.java 40.00% <0.00%> (+1.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b5c1ed...6fb2d92. Read the comment docs.

Copy link
Contributor

@dockerzhang dockerzhang left a comment

Choose a reason for hiding this comment

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

LGTM

@aloyszhang aloyszhang merged commit 7e9a7bb into apache:master Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature][CI] Add support for docker build on GitHub Actions
5 participants