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
Enh/transmit unencoded #29
Conversation
If you want to test this PR here is a image with this version: |
/rebase |
a0dc6ac
to
473c2bf
Compare
473c2bf
to
2777c97
Compare
2777c97
to
2a68e7f
Compare
2d727ea
to
b6eb204
Compare
#3902 got merged, please revendor it and mark this PR as 'ready for review' |
/invite @vpnachev |
b6eb204
to
d87d016
Compare
/squash |
/ready-for-review |
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.
/lgtm
I guess we wait before cutting a new version of the os-extension provider until we released a new version of g/g ? So the go.mod points to a proper version and not a commit? |
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.
/lgtm
I guess we wait before cutting a new version of the os-extension provider until we released a new version of g/g ? So the go.mod points to a proper version and not a commit?
I do not see a problem when a commit is used.
There is no problem, but we agreed to use only released tags when an extension is released. |
@BeckerMax Please open similar PRs for the other OS extensions. |
Accepts the flag
Transmit Unencoded
.Needed for creation of bootstrap-tokens for machines introduced here: gardener/gardener#3902
How to categorize this PR?
/area security
/kind enhancement
/os garden-linux
What this PR does / why we need it:
See gardener/gardener#3898 and gardener/gardener#3902
Release note: