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

Cloud Native Security Whitepaper submitted #452

Merged
merged 11 commits into from
Nov 18, 2020

Conversation

idvoretskyi
Copy link
Member

Signed-off-by: Ihor Dvoretskyi ihor@linux.com

@ultrasaurus
Copy link
Member

I haven't looked through content, since I know you are still working on it. Just wanted to make a couple of requests:

  • can you make filenames lowercase using dashes to separate words? I don't actually see that in CONTRIBUTING.md (which I will add) -- it's pretty consistent across the repo
  • can you please check in the source to the images? I assume we'll have future versions and this will be the canonical source files for future versions

Thanks so much for your work on the conversion! We should pull in White Paper authors when it is ready for review.

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>
Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>
@idvoretskyi idvoretskyi force-pushed the idvoretskyi-security-whitepaper branch from ffa9858 to f6e5933 Compare November 12, 2020 14:41
@TheFoxAtWork
Copy link
Contributor

@idvoretskyi a quick skim - everything looks in place. I made some alterations to the initial content in preparation for the upcoming post.

idvoretskyi and others added 2 commits November 12, 2020 17:59
Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>
Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>
@idvoretskyi
Copy link
Member Author

@TheFoxAtWork thanks!

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>
…kyi/sig-security into idvoretskyi-security-whitepaper
**Work-Group:** SIG-Security **Short self link to working doc:** https://tinyurl.com/CNCF-CNSWP | **Status** : WIP | In Review | **Approved** | Obsolete

**Created** : 2020-FEB-01 **Reviewed** : 2020-OCT-27 **Published** : 2020-NOV-18 **Release Version:** 1.0 **Approvers** : [x] @lizrice [x] @justincormack |
| **Contributors:** [aradhna.chetal@gmail.com](mailto:aradhna.chetal@gmail.com), [themoxiefoxatwork@gmail.com](mailto:themoxiefoxatwork@gmail.com), [jj@tetrate.io](mailto:jj@tetrate.io), gadi@alcide.io @lumjjb, @trishankatdatadog, [@vvenkatara@paloaltonetworks.com](mailto:vvenkatara@paloaltonetworks.com), @pushkarj-v, @whaber, @sublimino, @rowan-baker, [chase.pettet@gmail.com](mailto:chase.pettet@gmail.com), [harsingh@us.ibm.com](mailto:harsingh@us.ibm.com), jeff.lombardo@gmail.com
Copy link
Contributor

@PushkarJ PushkarJ Nov 13, 2020

Choose a reason for hiding this comment

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

@idvoretskyi : Had a recent change in my Github ID/username and the one here does not exist. Can you please replace @pushkarj-v with @pushkarj ? Thanks and sorry for the bother.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add links to the github IDs as well?

@PushkarJ
Copy link
Contributor

@idvoretskyi added a few comments, mostly nits. Thank you so much, for your excellent work :)

@idvoretskyi
Copy link
Member Author

@PushkarJ thanks a lot, will address them!

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>
@idvoretskyi idvoretskyi marked this pull request as ready for review November 17, 2020 18:44
Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>
Copy link
Contributor

@lumjjb lumjjb left a comment

Choose a reason for hiding this comment

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

Couple suggested changes to remove implementation and vendor specific details, and content rewording, and several other formatting and clarification additions.

security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
security-whitepaper/cloud-native-security-whitepaper.md Outdated Show resolved Hide resolved
Co-authored-by: Brandon Lum <lumjjb@gmail.com>
@idvoretskyi
Copy link
Member Author

@TheFoxAtWork ping for the final review :)

Copy link
Contributor

@TheFoxAtWork TheFoxAtWork left a comment

Choose a reason for hiding this comment

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

@lumjjb @pragashj @ultrasaurus @JustinCappos need a second approval on this to merge

Copy link
Contributor

@lumjjb lumjjb left a comment

Choose a reason for hiding this comment

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

LGTM!

@lumjjb lumjjb merged commit 88116a7 into cncf:master Nov 18, 2020
@idvoretskyi idvoretskyi deleted the idvoretskyi-security-whitepaper branch November 18, 2020 17:33
Copy link

@soapland-master69 soapland-master69 left a comment

Choose a reason for hiding this comment

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

micro stack issues

Michael-Susu12138 pushed a commit to Michael-Susu12138/tag-security that referenced this pull request Dec 12, 2023
* Cloud Native Security Whitepaper submitted

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>

* Cloud Native Security Whitepaper submitted

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>

* Update security-whitepaper/cloud_native_security_whitepaper.md

Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>

* Update security-whitepaper/cloud_native_security_whitepaper.md

Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>

* MD File renamed

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>

* Formatting fixes & comments addressed

Signed-off-by: Ihor Dvoretskyi <ihor@linux.com>

* Apply suggestions from code review

Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Brandon Lum <lumjjb@gmail.com>

* Update security-whitepaper/cloud-native-security-whitepaper.md

Co-authored-by: Emily Fox <33327273+TheFoxAtWork@users.noreply.github.com>
Co-authored-by: Brandon Lum <lumjjb@gmail.com>
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.

6 participants