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

WIP: Add article about mounting secrets as files and env; - Reviewing by mmaler #1287

Merged
merged 3 commits into from
Jun 3, 2020

Conversation

mshaposhnik
Copy link
Contributor

What does this PR do?

Add article about mounting secrets as files and env;

What issues does this PR fix or reference?

eclipse-che/che#14680

Specify the version of the product this PR applies to.

7.14

@mshaposhnik mshaposhnik removed the request for review from l0rd May 27, 2020 07:51
@MichalMaler MichalMaler changed the title Add article about mounting secrets as files and env; Add article about mounting secrets as files and env; - Reviewing by mmaler May 27, 2020
@MichalMaler MichalMaler self-assigned this May 27, 2020
@MichalMaler
Copy link
Contributor

I am rewriting it here. Please, let your comments there.
https://docs.google.com/document/d/11F8vbfoJLK4HWQxyBoIPUWY8jSwiH5-lhDBe-LbD63g/edit

@MichalMaler MichalMaler changed the title Add article about mounting secrets as files and env; - Reviewing by mmaler WIP: Add article about mounting secrets as files and env; - Reviewing by mmaler May 28, 2020
Signed-off-by: Michal Maléř <mmaler@redhat.com>
@MichalMaler MichalMaler requested a review from skabashnyuk June 1, 2020 14:35
@mshaposhnik
Copy link
Contributor Author

In general, LGTM! Minor fixes proposed.

Signed-off-by: Michal Maléř <mmaler@redhat.com>
Copy link
Member

@ScrewTSW ScrewTSW left a comment

Choose a reason for hiding this comment

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

LGTM

@mshaposhnik mshaposhnik merged commit d0537d4 into eclipse-che:master Jun 3, 2020
@mshaposhnik mshaposhnik deleted the secrets_mount branch June 3, 2020 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants