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

docs: Add tls configuration docs #176

Merged
merged 1 commit into from
Jun 30, 2022
Merged

Conversation

pvaneck
Copy link
Member

@pvaneck pvaneck commented Jun 28, 2022

Surface some internal TLS configuration documentation externally.

Co-authored-by: PRASHANT GUPTA <[email protected]>
Co-authored-by: Nick Hill <[email protected]>
Co-authored-by: JOE RUNDE <[email protected]>

Signed-off-by: Paul Van Eck <[email protected]>
@pvaneck
Copy link
Member Author

pvaneck commented Jun 28, 2022

Most original authors (from what I could tell) are listed as co-authors in the commit. Also, I think it should be fine to leave some of the OpenShift specific instructions since we do have some OpenShift users. However, we should also create instructions for using a Kubernetes Ingress resource with ModelMesh (as opposed to an OpenShift Route).

Copy link
Contributor

@chinhuang007 chinhuang007 left a comment

Choose a reason for hiding this comment

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

/lgtm

@kserve-oss-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chinhuang007, pvaneck

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kserve-oss-bot kserve-oss-bot merged commit 2d4068f into kserve:main Jun 30, 2022
pvaneck added a commit to pvaneck/modelmesh-serving that referenced this pull request Jun 30, 2022
Surface some internal TLS configuration documentation externally.

Signed-off-by: Paul Van Eck <[email protected]>

Co-authored-by: PRASHANT GUPTA <[email protected]>
Co-authored-by: Nick Hill <[email protected]>
Co-authored-by: JOE RUNDE <[email protected]>
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.

3 participants