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

Setup Che in TLS mode with self-signed certificates #1007

Merged
merged 17 commits into from
Jan 29, 2020
Merged

Setup Che in TLS mode with self-signed certificates #1007

merged 17 commits into from
Jan 29, 2020

Conversation

mshaposhnik
Copy link
Contributor

@mshaposhnik mshaposhnik commented Jan 2, 2020

What does this PR do?

Documenting how to setup Che with TLS with self-signed certificate for development.

What issues does this PR fix or reference?

eclipse-che/che#15301

@skabashnyuk
Copy link
Contributor

CC @tolusha

@tolusha
Copy link
Contributor

tolusha commented Jan 3, 2020

/cc @mmorhun

mshaposhnik and others added 4 commits January 3, 2020 09:31
Signed-off-by: Michal Vala <mvala@redhat.com>
* add crc domain

Signed-off-by: Michal Vala <mvala@redhat.com>

* link to browser import

Signed-off-by: Michal Vala <mvala@redhat.com>
* add crc domain

Signed-off-by: Michal Vala <mvala@redhat.com>

* link to browser import

Signed-off-by: Michal Vala <mvala@redhat.com>

* added some links to better orientation and minor tweaks

Signed-off-by: Michal Vala <mvala@redhat.com>
@sparkoo
Copy link
Member

sparkoo commented Jan 3, 2020

@mshaposhnik I'm ok with current state. Can we switch PR from draft?

@mshaposhnik mshaposhnik marked this pull request as ready for review January 3, 2020 14:34
* add crc domain

Signed-off-by: Michal Vala <mvala@redhat.com>

* link to browser import

Signed-off-by: Michal Vala <mvala@redhat.com>

* added some links to better orientation and minor tweaks

Signed-off-by: Michal Vala <mvala@redhat.com>

* added OSX example

Signed-off-by: Michal Vala <mvala@redhat.com>

* system-wide note

Signed-off-by: Michal Vala <mvala@redhat.com>
* add crc domain

Signed-off-by: Michal Vala <mvala@redhat.com>

* link to browser import

Signed-off-by: Michal Vala <mvala@redhat.com>

* added some links to better orientation and minor tweaks

Signed-off-by: Michal Vala <mvala@redhat.com>

* added OSX example

Signed-off-by: Michal Vala <mvala@redhat.com>

* system-wide note

Signed-off-by: Michal Vala <mvala@redhat.com>

* Revert "system-wide note"

This reverts commit 00ed96c.
Signed-off-by: Michal Vala <mvala@redhat.com>
* add crc domain

Signed-off-by: Michal Vala <mvala@redhat.com>

* link to browser import

Signed-off-by: Michal Vala <mvala@redhat.com>

* added some links to better orientation and minor tweaks

Signed-off-by: Michal Vala <mvala@redhat.com>

* added OSX example

Signed-off-by: Michal Vala <mvala@redhat.com>

* system-wide note

Signed-off-by: Michal Vala <mvala@redhat.com>

* Revert "system-wide note"

This reverts commit 00ed96c.

* revert system-wide note

Signed-off-by: Michal Vala <mvala@redhat.com>

* OSX rootCA import

Signed-off-by: Michal Vala <mvala@redhat.com>
@sparkoo sparkoo mentioned this pull request Jan 6, 2020
* sha256

Signed-off-by: Michal Vala <mvala@redhat.com>
* update docs for crc
* update navigation

Signed-off-by: Michal Vala <mvala@redhat.com>
@sparkoo sparkoo self-assigned this Jan 8, 2020
Copy link
Contributor

@mmorhun mmorhun left a comment

Choose a reason for hiding this comment

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

LGTM

* add keywords to tls docs

Signed-off-by: Michal Vala <mvala@redhat.com>
@sparkoo sparkoo requested review from sparkoo and removed request for sparkoo January 8, 2020 08:27
@skabashnyuk
Copy link
Contributor

@rkratky asked me about since it's for devel purposes, can we find a place for it in the Contributor Guide?

@sparkoo for TLS docs https://github.com/eclipse/che/wiki this is imho not the right place, there is basically nothing technical about development. Another candidate is this https://github.com/eclipse/che/blob/master/CONTRIBUTING.md also this Contributor guide on che-docs https://github.com/eclipse/che-docs/tree/master/src/main/pages/che-7/contributor-guide

@tolusha For me it is part of installation guide

@sparkoo
Copy link
Member

sparkoo commented Jan 9, 2020

It would be nice to have a place where we can share knowledge about development (setup, deploy, build, test, debug, various tips etc.) without any formal ceremonies like reviews. Pure informal knowledge base.

For now, I'm +1 for Contributor guide in che-docs https://github.com/eclipse/che-docs/tree/master/src/main/pages/che-7/contributor-guide

@skabashnyuk
Copy link
Contributor

skabashnyuk commented Jan 9, 2020

@rkratky @themr0c @slemeur @tolusha @l0rd wdyt about to move this content to https://github.com/eclipse/che-docs/tree/master/src/main/pages/che-7/contributor-guide ?

@themr0c
Copy link
Contributor

themr0c commented Jan 9, 2020

One good point for the Contribution guide is: it is not used downstream.

We will end up having documents explaining how to install Che in: Overview > Che quick-starts, Installation guide, and Contribution guide, but I believe it is quite OK as each user story has specific requirements. I wonder if some parts could be reused.

@sparkoo
Copy link
Member

sparkoo commented Jan 15, 2020

From our side, content is complete and we're not currently actively working on this. Remaining question is what is proper place for this doc? Can someone from docs team please comment on this and review the changeset?
cc: @themr0c @rkratky

@sparkoo
Copy link
Member

sparkoo commented Jan 27, 2020

@themr0c I've moved the doc under the contributor's guide. Any other comment?

@themr0c themr0c merged commit c1866ff into master Jan 29, 2020
@themr0c themr0c deleted the tls_docs branch March 12, 2020 07:47
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.

7 participants