-
Notifications
You must be signed in to change notification settings - Fork 71
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
[launcher] Fix image pulling in launcher #282
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jkl73
force-pushed
the
miscfixfeb
branch
3 times, most recently
from
February 8, 2023 22:45
27bd624
to
53ba115
Compare
alexmwu
reviewed
Feb 13, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add a test that pulls a noauth dockerhub image?
Update token refresh end log message. Now will only supply access token when pulling from GCP gcr.io or artifact registries. Also can pull images without a token from the public docker.io registry. Signed-off-by: Jiankun Lu <jiankun@google.com>
alexmwu
approved these changes
Feb 15, 2023
alexmwu
added a commit
to alexmwu/go-tpm-tools
that referenced
this pull request
Apr 7, 2023
Use region in spec to create attestation service rest client google#281 Parse EFI App state from the TCG event log google#277 Increase default systemd wait timeout to 900s google#276 Use same env var formatting logic on the launcher as server google#253 Fix image pulling in launcher google#282 Bump version and fix a kernel cmd issue google#291 Return the actual number of bytes written to through command buffer google#287 Fix lint issues after using golangci-lint-1.52.2 google#296 Add image tests and test automation google#275 Update go-sev-guest to v0.4.2 google#278 Update to go-sev-guest v0.4.5 google#279 Add proper debug license and logging to launcher google#280 Upgrade to go-sev-guest v0.5.0 google#283 Import go-sev-guest v0.5.2 google#284 Add override test for workload env vars and cmd google#286 Add test workload code, check OIDC claims, and validate launch policy checks google#288 Bump golang.org/x/net in /launcher google#290 Add RELEASING instructions google#187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
alexmwu
added a commit
to alexmwu/go-tpm-tools
that referenced
this pull request
Apr 7, 2023
Breaking Changes: New Features: Use region in spec to create attestation service rest client google#281 Parse EFI App state from the TCG event log google#277 Bug Fixes: Increase default systemd wait timeout to 900s google#276 Use same env var formatting logic on the launcher as server google#253 Fix image pulling in launcher google#282 Bump version and fix a kernel cmd issue google#291 Return the actual number of bytes written to through command buffer google#287 Fix lint issues after using golangci-lint-1.52.2 google#296 Other Changes: Add image tests and test automation google#275 Update go-sev-guest to v0.4.2 google#278 Update to go-sev-guest v0.4.5 google#279 Add proper debug license and logging to launcher google#280 Upgrade to go-sev-guest v0.5.0 google#283 Import go-sev-guest v0.5.2 google#284 Add override test for workload env vars and cmd google#286 Add test workload code, check OIDC claims, and validate launch policy checks google#288 Bump golang.org/x/net in /launcher google#290 Add RELEASING instructions google#187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
Merged
alexmwu
added a commit
that referenced
this pull request
Apr 7, 2023
Breaking Changes: New Features: Use region in spec to create attestation service rest client #281 Parse EFI App state from the TCG event log #277 Bug Fixes: Increase default systemd wait timeout to 900s #276 Use same env var formatting logic on the launcher as server #253 Fix image pulling in launcher #282 Bump version and fix a kernel cmd issue #291 Return the actual number of bytes written to through command buffer #287 Fix lint issues after using golangci-lint-1.52.2 #296 Other Changes: Add image tests and test automation #275 Update go-sev-guest to v0.4.2 #278 Update to go-sev-guest v0.4.5 #279 Add proper debug license and logging to launcher #280 Upgrade to go-sev-guest v0.5.0 #283 Import go-sev-guest v0.5.2 #284 Add override test for workload env vars and cmd #286 Add test workload code, check OIDC claims, and validate launch policy checks #288 Bump golang.org/x/net in /launcher #290 Add RELEASING instructions #187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
alexmwu
added a commit
to alexmwu/go-tpm-tools
that referenced
this pull request
Apr 7, 2023
Breaking Changes: New Features: Use region in spec to create attestation service rest client google#281 Parse EFI App state from the TCG event log google#277 Bug Fixes: Increase default systemd wait timeout to 900s google#276 Use same env var formatting logic on the launcher as server google#253 Fix image pulling in launcher google#282 Bump version and fix a kernel cmd issue google#291 Return the actual number of bytes written to through command buffer google#287 Fix lint issues after using golangci-lint-1.52.2 google#296 Other Changes: Add image tests and test automation google#275 Update go-sev-guest to v0.4.2 google#278 Update to go-sev-guest v0.4.5 google#279 Add proper debug license and logging to launcher google#280 Upgrade to go-sev-guest v0.5.0 google#283 Import go-sev-guest v0.5.2 google#284 Add override test for workload env vars and cmd google#286 Add test workload code, check OIDC claims, and validate launch policy checks google#288 Bump golang.org/x/net in /launcher google#290 Add RELEASING instructions google#187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
michael-pregman
pushed a commit
that referenced
this pull request
Apr 10, 2023
Breaking Changes: New Features: Use region in spec to create attestation service rest client #281 Parse EFI App state from the TCG event log #277 Bug Fixes: Increase default systemd wait timeout to 900s #276 Use same env var formatting logic on the launcher as server #253 Fix image pulling in launcher #282 Bump version and fix a kernel cmd issue #291 Return the actual number of bytes written to through command buffer #287 Fix lint issues after using golangci-lint-1.52.2 #296 Other Changes: Add image tests and test automation #275 Update go-sev-guest to v0.4.2 #278 Update to go-sev-guest v0.4.5 #279 Add proper debug license and logging to launcher #280 Upgrade to go-sev-guest v0.5.0 #283 Import go-sev-guest v0.5.2 #284 Add override test for workload env vars and cmd #286 Add test workload code, check OIDC claims, and validate launch policy checks #288 Bump golang.org/x/net in /launcher #290 Add RELEASING instructions #187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
michael-pregman
pushed a commit
that referenced
this pull request
Apr 11, 2023
Breaking Changes: New Features: Use region in spec to create attestation service rest client #281 Parse EFI App state from the TCG event log #277 Bug Fixes: Increase default systemd wait timeout to 900s #276 Use same env var formatting logic on the launcher as server #253 Fix image pulling in launcher #282 Bump version and fix a kernel cmd issue #291 Return the actual number of bytes written to through command buffer #287 Fix lint issues after using golangci-lint-1.52.2 #296 Other Changes: Add image tests and test automation #275 Update go-sev-guest to v0.4.2 #278 Update to go-sev-guest v0.4.5 #279 Add proper debug license and logging to launcher #280 Upgrade to go-sev-guest v0.5.0 #283 Import go-sev-guest v0.5.2 #284 Add override test for workload env vars and cmd #286 Add test workload code, check OIDC claims, and validate launch policy checks #288 Bump golang.org/x/net in /launcher #290 Add RELEASING instructions #187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
Pranjali-2501
pushed a commit
to Pranjali-2501/go-tpm-tools
that referenced
this pull request
Apr 13, 2023
Breaking Changes: New Features: Use region in spec to create attestation service rest client google#281 Parse EFI App state from the TCG event log google#277 Bug Fixes: Increase default systemd wait timeout to 900s google#276 Use same env var formatting logic on the launcher as server google#253 Fix image pulling in launcher google#282 Bump version and fix a kernel cmd issue google#291 Return the actual number of bytes written to through command buffer google#287 Fix lint issues after using golangci-lint-1.52.2 google#296 Other Changes: Add image tests and test automation google#275 Update go-sev-guest to v0.4.2 google#278 Update to go-sev-guest v0.4.5 google#279 Add proper debug license and logging to launcher google#280 Upgrade to go-sev-guest v0.5.0 google#283 Import go-sev-guest v0.5.2 google#284 Add override test for workload env vars and cmd google#286 Add test workload code, check OIDC claims, and validate launch policy checks google#288 Bump golang.org/x/net in /launcher google#290 Add RELEASING instructions google#187 https://github.com/google/go-tpm-tools/compare/53cab1a...5dd1056?expand=1
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Now can pull images with an access token from GCP gcr.io or artifact registries.
Also can pull images without a token from the public docker.io registry.
Signed-off-by: Jiankun Lu jiankun@google.com