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

Container image building and publishing #23

Closed

Conversation

jacobweinstock
Copy link
Member

Description

This enables building and publishing container images to quay.io/tinkerbell. GitHub actions and Goreleaser are set up to create and publish container images based on Git tags.

This could potentially be an example repo for how we could handle versioning. See proposal 0002.

CC: @displague

Why is this needed

Fixes: #22

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

@codecov
Copy link

codecov bot commented Jan 5, 2022

Codecov Report

Merging #23 (a76d900) into main (7548eaf) will decrease coverage by 1.10%.
The diff coverage is 100.00%.

❗ Current head a76d900 differs from pull request most recent head 4632a43. Consider uploading reports for the commit 4632a43 to get more accurate results

@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   97.10%   96.00%   -1.11%     
==========================================
  Files           4        4              
  Lines         415      425      +10     
==========================================
+ Hits          403      408       +5     
- Misses          8       11       +3     
- Partials        4        6       +2     
Impacted Files Coverage Δ
cmd.go 96.10% <100.00%> (ø)
ipxedust.go 93.52% <0.00%> (-2.48%) ⬇️
itftp/itftp.go 96.84% <0.00%> (-0.07%) ⬇️
ihttp/ihttp.go 100.00% <0.00%> (ø)

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 7548eaf...4632a43. Read the comment docs.

@jacobweinstock jacobweinstock added the kind/feature New feature or request label Jan 5, 2022
Makefile Show resolved Hide resolved
Allows users to run ipxedust in a container.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
This uses goreleaser to build and push container
images to quay.io for multiple architectures.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@@ -16,7 +16,7 @@ jobs:
- name: Install Go
uses: actions/setup-go@v2
with:
go-version: "1.17"
go-version: "1.17.5"
Copy link
Member

Choose a reason for hiding this comment

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

Do we need to lock this?

@chrisdoherty4
Copy link
Member

This is super old. Is it still relevant?

@jacobweinstock
Copy link
Member Author

This is super old. Is it still relevant?

It is definitely old. It needs a revisit code wise (merge main, possibly update versions, etc) and i'd still like to see it incorporated.

@chrisdoherty4 chrisdoherty4 marked this pull request as draft November 18, 2022 18:25
@jacobweinstock
Copy link
Member Author

no cycles for this. will re-open if/when i do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Container image build and push to quay.io/tinkerbell
3 participants