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

Backup addon certs in admin node #848

Merged
merged 6 commits into from
Jun 4, 2020
Merged

Backup addon certs in admin node #848

merged 6 commits into from
Jun 4, 2020

Conversation

jenting
Copy link

@jenting jenting commented May 27, 2020

Make sure it's tested

Technical writers will not always be able to verify the implementation!

Any PR opened is assumed to have been verified by QA if not designated by comments or labels otherwise. Please make sure you have tested the changes and included any information that a user would require to use the documentation.

Describe your changes

Back up the addon certificates in the admin node.

Related Issues / Projects

Please provide links to Bugzilla and other GitHub projects with your description if they are related to the changes.

Fix #840

Enable maintainer updates

Please enable maintainer updates so we can push commits into your branch to make collaboration and reviews easier.

Do not force push your branch

Please avoid force pushing to branches that are subject of pull requests. Force pushing breaks maintainer commits in many cases and is very hard (if not impossible) to untangle for backporting.

Labels

Please set any (and all) appropriate labels that describe the status of the PR.

Label Description
P1 PR should be worked on and merged as soon as possible
Blocked Work can not proceed because other work has not been completed, PR can not be merged (code has not been merged but documentation is ready)
On-Hold Underlying work is completed but the PR should not be merged
ReleaseNotes User interaction is required after the introduction of this change and the change must be mentioned in the release notes
v3/v4/v4.x Which version of the release the PR should be merged into, this can be multiple versions, please set the "Backport" label if it needs to go into a previous release
Needs Review Some details of the PR are known to be incomplete and must be discussed with other engineers before merging (if possible assign reviewers or cc mention in comments), PR can not be merged

Signed-off-by: JenTing Hsiao jenting.hsiao@suse.com

Markus Napp and others added 4 commits May 20, 2020 14:38
Copy link
Contributor

@innobead innobead left a comment

Choose a reason for hiding this comment

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

LGTM, typo to fix

Co-authored-by: David Ko <dko@suse.com>
@jenting jenting requested a review from innobead May 28, 2020 07:53
@nkoranova nkoranova self-assigned this May 28, 2020
@jenting jenting changed the base branch from master to maintenance/CaaS4 June 1, 2020 02:07
@jenting
Copy link
Author

jenting commented Jun 1, 2020

Merge remote branch maintenance/CaaS4.

@r0ckarong r0ckarong added this to the Sprint 31 milestone Jun 2, 2020
@nkoranova nkoranova merged commit 1693672 into SUSE:maintenance/CaaS4 Jun 4, 2020
nkoranova pushed a commit that referenced this pull request Jun 4, 2020
* Bump release version to 5.0

* Make LICENSE symlink valid again

* Typo fixes

* Backup addon certs in admin node

Signed-off-by: JenTing Hsiao <jenting.hsiao@suse.com>

* Update adoc/admin-security-certificates.adoc

Co-authored-by: David Ko <dko@suse.com>

Co-authored-by: Markus Napp <mnapp@suse.com>
Co-authored-by: Stefan Knorr <sknorr@suse.de>
Co-authored-by: David Ko <dko@suse.com>
@jenting jenting deleted the issue-840 branch June 5, 2020 00:21
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.

[doc] 4.10.7.2 Renewing Certificates Created -> why /etc/kubernetes/pki.bak ?
4 participants