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

Remove creation of static node bootstrapToken #4824

Merged
merged 3 commits into from
Oct 21, 2021

Conversation

BeckerMax
Copy link
Contributor

How to categorize this PR?

/area security
/kind cleanup

What this PR does / why we need it:

This commit is part of issue #3898
which replaces the long-valid bootstrap-token shared between nodes with a
short-lived token unique for each node.

The new flow of using smaller-scoped, short-lived tokens was already active
once you updated to compatible versions of the infrastructure-extension,
operatingsystem-extension and of gardener/gardener.

With this commit we are now removing the old secret from the Shoot
which means you need to run supported versions of the os-extensions
and the infrastructure-provider-extensions when
upgrading gardener to this version.

A compatibility matrix is added to the documentation which includes the required versions needed.

Which issue(s) this PR fixes:
Fixes #3898

Release note:

Remove creation of shared and long-valid node bootstrapTokens. Instead use short-lived tokens unique for each node. You are required to update to a version of the operatingsystem-extension and the infrastructure extension that support creating bootstrap-tokens. Please see the [compatibility matrix](https://github.com/gardener/gardener/blob/master/docs/extensions/operatingsystemconfig.md#compatibility-matrix-for-node-bootstrap-token) before updating gardener.

@BeckerMax BeckerMax requested a review from a team as a code owner October 12, 2021 06:04
@gardener-robot gardener-robot added needs/review size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Oct 12, 2021
@rfranzke
Copy link
Member

/assign

Copy link
Member

@rfranzke rfranzke left a comment

Choose a reason for hiding this comment

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

Nice, thanks! A few nit improvements for the docs, otherwise lgtm

docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
@BeckerMax
Copy link
Contributor Author

/merge squash

@BeckerMax
Copy link
Contributor Author

BeckerMax commented Oct 18, 2021

Thank you very much for the review. I applied your suggestions.

@timebertt
Copy link
Member

/rebase
ref #4822

Max Becker and others added 2 commits October 18, 2021 10:56
This commit is part of issue [gardener#3898](gardener#3898)
which replaces the long-valid bootstrap-token shared between nodes with a
short-lived token unique for each node.

The new flow of using smaller-scoped, short-lived tokens was already active
once you updated to compatible versions of the infrastructure-extension,
operatingsystem-extension and of gardener/gardener as specified in:
gardener#3898

With this commit we are now removing the old secret from the Shoot
which means you need to run supported versions of the os-extensions
and the infrastructure-provider-extensions when
upgrading gardener to this version.
Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>
rfranzke
rfranzke previously approved these changes Oct 18, 2021
Copy link
Member

@rfranzke rfranzke left a comment

Choose a reason for hiding this comment

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

/lgtm

docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
docs/extensions/operatingsystemconfig.md Outdated Show resolved Hide resolved
Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>
@rfranzke rfranzke merged commit 9551bf5 into gardener:master Oct 21, 2021
@rfranzke rfranzke deleted the cleanup/node-bootstrap-token branch October 21, 2021 15:39
krgostev pushed a commit to krgostev/gardener that referenced this pull request Apr 21, 2022
* Remove creation of static node bootstrapToken

This commit is part of issue [gardener#3898](gardener#3898)
which replaces the long-valid bootstrap-token shared between nodes with a
short-lived token unique for each node.

The new flow of using smaller-scoped, short-lived tokens was already active
once you updated to compatible versions of the infrastructure-extension,
operatingsystem-extension and of gardener/gardener as specified in:
gardener#3898

With this commit we are now removing the old secret from the Shoot
which means you need to run supported versions of the os-extensions
and the infrastructure-provider-extensions when
upgrading gardener to this version.

* Apply suggestions from rfranzke

Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>

* Change shipped feature to v1.35

Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>

Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>
krgostev pushed a commit to krgostev/gardener that referenced this pull request Jul 5, 2022
* Remove creation of static node bootstrapToken

This commit is part of issue [gardener#3898](gardener#3898)
which replaces the long-valid bootstrap-token shared between nodes with a
short-lived token unique for each node.

The new flow of using smaller-scoped, short-lived tokens was already active
once you updated to compatible versions of the infrastructure-extension,
operatingsystem-extension and of gardener/gardener as specified in:
gardener#3898

With this commit we are now removing the old secret from the Shoot
which means you need to run supported versions of the os-extensions
and the infrastructure-provider-extensions when
upgrading gardener to this version.

* Apply suggestions from rfranzke

Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>

* Change shipped feature to v1.35

Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>

Co-authored-by: Rafael Franzke <rafael.franzke@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

☂️ issue: Use dedicated bootstrap-tokens per shoot worker machine (instead of a long-valid shared token)
7 participants