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

update deployment guide with FTR changes #917

Merged
merged 1 commit into from Oct 29, 2022

Conversation

rksharma95
Copy link
Collaborator

ref: #891
Signed-off-by: rk ramakant@accuknox.com

@rksharma95 rksharma95 force-pushed the aws-ftr-doc branch 2 times, most recently from 92e771b to 1216239 Compare September 27, 2022 10:55
@rksharma95 rksharma95 force-pushed the aws-ftr-doc branch 2 times, most recently from 7b49c7d to 4c7e7e0 Compare October 12, 2022 17:36
@rksharma95 rksharma95 force-pushed the aws-ftr-doc branch 8 times, most recently from 7d58005 to eea818e Compare October 25, 2022 05:13
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Ankurk99 Ankurk99 requested a review from nyrahul October 26, 2022 16:43
@rksharma95 rksharma95 force-pushed the aws-ftr-doc branch 2 times, most recently from 492158c to 729ea50 Compare October 26, 2022 17:15
Copy link
Member

@Ankurk99 Ankurk99 left a comment

Choose a reason for hiding this comment

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

Hey @rksharma95 Can you please remove the changes made in the main README.md file. We decided that we will have the "use-cases" as a part of a document and submit that separately for the FTR. These changes are adding complexity to the main README page, which we are trying to be as brief as possible.

Signed-off-by: rk <ramakant@accuknox.com>
@nyrahul nyrahul merged commit 5c78962 into kubearmor:main Oct 29, 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.

None yet

3 participants