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

feat: Boosting CDE Security on k8s cluster #66

Merged
merged 8 commits into from
Oct 23, 2024
Merged

feat: Boosting CDE Security on k8s cluster #66

merged 8 commits into from
Oct 23, 2024

Conversation

tolusha
Copy link
Contributor

@tolusha tolusha commented Oct 16, 2024

Signed-off-by: Anatolii Bazko <[email protected]>
@tolusha tolusha requested review from deerskindoll and removed request for dkwon17 October 16, 2024 06:41
@ibuziuk
Copy link
Member

ibuziuk commented Oct 17, 2024

@tolusha great read, but I would reccomend adding a "Untrusted Repository Warning" section explicitly to the blog

Copy link
Contributor

@deerskindoll deerskindoll left a comment

Choose a reason for hiding this comment

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

a couple of small tweaks.

it's a well-written blog-post that's easy to follow and understand.

Signed-off-by: Anatolii Bazko <[email protected]>
@tolusha
Copy link
Contributor Author

tolusha commented Oct 18, 2024

Added a new section

Copy link
Member

@ibuziuk ibuziuk left a comment

Choose a reason for hiding this comment

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

Great read!
I believe we can change the publication date to the next week e.g. 21.08 and make it available earlier on the website

@tolusha
Copy link
Contributor Author

tolusha commented Oct 21, 2024

@ibuziuk
Should we keep 10.28 as a release date?

Copy link
Contributor

@dkwon17 dkwon17 left a comment

Choose a reason for hiding this comment

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

Looks great, I have a few suggestions

@tolusha
Copy link
Contributor Author

tolusha commented Oct 22, 2024

@deerskindoll
Please final approve

@deerskindoll
Copy link
Contributor

@deerskindoll Please final approve

looking good! approved.

Copy link
Contributor

@l0rd l0rd left a comment

Choose a reason for hiding this comment

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

Nice read. Well written and interesting.

@tolusha tolusha merged commit 1c16485 into main Oct 23, 2024
8 checks passed
@tolusha tolusha deleted the 23177 branch October 23, 2024 12:37
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.

5 participants