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

Add Docker build and push workflow #1625

Merged
merged 2 commits into from
Aug 2, 2021
Merged

Add Docker build and push workflow #1625

merged 2 commits into from
Aug 2, 2021

Conversation

rainest
Copy link
Contributor

@rainest rainest commented Jul 31, 2021

What this PR does / why we need it:
CI image builds again, in a post-cleanup world.

Since the old file is gone, see build.diff.txt for the actual job changes.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): Part of #1596

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@codecov
Copy link

codecov bot commented Jul 31, 2021

Codecov Report

Merging #1625 (a560919) into next (9842ec0) will increase coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             next    #1625      +/-   ##
==========================================
+ Coverage   62.17%   62.28%   +0.11%     
==========================================
  Files          82       82              
  Lines        7172     7172              
==========================================
+ Hits         4459     4467       +8     
+ Misses       2384     2378       -6     
+ Partials      329      327       -2     
Flag Coverage Δ
integration-test 62.28% <ø> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
internal/parser/parser.go 88.82% <0.00%> (+1.14%) ⬆️
internal/ctrlutils/ingress-status.go 62.50% <0.00%> (+1.25%) ⬆️

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 9842ec0...a560919. Read the comment docs.

@shaneutt
Copy link
Contributor

I know this is still draft, just a thought for you:

Do we potentially want to add this functionality underneath the existing release.yaml release workflow? Or do we want to otherwise make explicit interdependencies between the two?

Add a workflow to build and push Docker images for 2.x based on the new
2.x-only file structure.
@rainest rainest marked this pull request as ready for review August 2, 2021 16:35
@rainest rainest requested a review from a team as a code owner August 2, 2021 16:35
Copy link
Contributor

@ccfishk ccfishk left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@shaneutt shaneutt left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Let's just make sure we trigger some kind of test to make sure everying is OK then feel free to merge.

@rainest rainest enabled auto-merge (squash) August 2, 2021 17:38
@rainest rainest merged commit d4a3376 into next Aug 2, 2021
@rainest rainest deleted the ci/2x-buildrelease branch August 2, 2021 17:45
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.

3 participants