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

Fix So That SIA does NOT Write Instance Cert When Local File Backup Used #62

Merged
merged 4 commits into from
Nov 20, 2023

Conversation

mlajkim
Copy link
Contributor

@mlajkim mlajkim commented Nov 14, 2023

Background

When k8s-athenz-sia is used as sidecar, it should not try to write instance cert, when localBackup is used in the red sqaure.
image

TODOs

  • Fix the logic
  • Operation Check

@mlajkim mlajkim added the bug Something isn't working label Nov 14, 2023
@mlajkim mlajkim self-assigned this Nov 14, 2023
pkg/identity/certificated.go Outdated Show resolved Hide resolved
@mlajkim mlajkim requested a review from ctyano November 15, 2023 06:32
@mlajkim mlajkim changed the title Fix Not Write Instance Cert When Local File Backup Used Fix So That SIA does NOT Write Instance Cert When Local File Backup Used Nov 15, 2023
pkg/identity/certificated.go Outdated Show resolved Hide resolved
pkg/identity/certificated.go Outdated Show resolved Hide resolved
pkg/identity/certificated.go Show resolved Hide resolved
Co-authored-by: Windz <[email protected]>
Signed-off-by: Aaron Jeongwoo Kim <[email protected]>
@mlajkim mlajkim mentioned this pull request Nov 16, 2023
5 tasks
Signed-off-by: Aaron Jeongwoo Kim <[email protected]>
@mlajkim mlajkim requested review from WindzCUHK and ctyano November 17, 2023 03:30
@WindzCUHK WindzCUHK merged commit dafbd43 into main Nov 20, 2023
8 checks passed
@WindzCUHK WindzCUHK deleted the fix-instance-cert-writing-when-local-backup-file-used branch November 20, 2023 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants