Skip to content

Dev #218

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

Merged
merged 2 commits into from
Mar 3, 2022
Merged

Dev #218

merged 2 commits into from
Mar 3, 2022

Conversation

xiaods
Copy link
Owner

@xiaods xiaods commented Mar 3, 2022

No description provided.

xiaods added 2 commits March 3, 2022 21:32
Signed-off-by: Brian Downs [email protected]
porting by
Signed-off-by: Deshi Xiao <[email protected]>
Check and make sure the encryption-hash.json file exists when generating
secrets-encryption files.
This was breaking upgrades because previously we only had one relevant
encryption-config.json file.

Types of Changes
Bugfix

Verification
Install pre encryptions rotation version of K3s: v1.21.1+k3s1
Start K3s k3s server --secrets-encryption
Stop K3s
Upgrade to this PR
Start K3s k3s server --secrets-encryption
Run k3s secrets-encrypt status and note that it does not fail with 500
Internal Server Error

Signed-off-by: Derek Nola [email protected]
porting by
Signed-off-by: Deshi Xiao <[email protected]>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@xiaods xiaods merged commit cb89c2b into master Mar 3, 2022
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.

1 participant