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

Air gap support #2299

Merged
merged 3 commits into from
Nov 10, 2022
Merged

Air gap support #2299

merged 3 commits into from
Nov 10, 2022

Conversation

vaikas
Copy link
Contributor

@vaikas vaikas commented Oct 1, 2022

Summary

Use newer version of theupdateframework/go-tuf, sigstore/sigstore
theupdateframework/go-tuf#397
sigstore/sigstore#715

This allows one to specify a local filesystem as a TUF remote so there's no network calls related to TUF.

Release Note

  • Add support for air gap scenarios by being able to initialize TUF root where remote is local filesystem.

Documentation

@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2022

Codecov Report

Merging #2299 (0528e19) into main (35595e8) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main    #2299   +/-   ##
=======================================
  Coverage   30.48%   30.48%           
=======================================
  Files         136      136           
  Lines        8384     8384           
=======================================
  Hits         2556     2556           
  Misses       5494     5494           
  Partials      334      334           
Impacted Files Coverage Δ
cmd/cosign/cli/options/initialize.go 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@dlorenc
Copy link
Member

dlorenc commented Oct 15, 2022

Is this ready for review?

@vaikas
Copy link
Contributor Author

vaikas commented Oct 17, 2022

I want to add the e2e tests for it.

@vaikas vaikas changed the title Air gap support [WIP]: Air gap support Oct 17, 2022
@vaikas vaikas changed the title [WIP]: Air gap support Air gap support Oct 21, 2022
@znewman01 znewman01 self-requested a review October 21, 2022 20:18
@vaikas
Copy link
Contributor Author

vaikas commented Oct 21, 2022

ho hum, I see we need to add the required tests because I added a new matrix segment for this to be considered passing.

znewman01
znewman01 previously approved these changes Nov 7, 2022
Copy link
Contributor

@znewman01 znewman01 left a comment

Choose a reason for hiding this comment

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

Sorry this took so long, I assumed it was going to be a lot more complicated to review than it was :)

@vaikas
Copy link
Contributor Author

vaikas commented Nov 7, 2022

thanks @znewman01 ! Yeah, all the "real" work was in upstreams :) Here's just tests.

dlorenc
dlorenc previously approved these changes Nov 8, 2022
@vaikas vaikas dismissed stale reviews from dlorenc and znewman01 via 691ef06 November 9, 2022 23:48
Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
Signed-off-by: Ville Aikas <vaikas@chainguard.dev>
@dlorenc dlorenc merged commit 18e4d2b into sigstore:main Nov 10, 2022
@github-actions github-actions bot added this to the v1.14.0 milestone Nov 10, 2022
@vaikas vaikas deleted the air-gap branch November 10, 2022 16:03
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.

6 participants