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 multi-tenancy doc #66

Merged
merged 23 commits into from
May 22, 2024
Merged

Conversation

binbin-li
Copy link
Collaborator

@binbin-li binbin-li commented May 7, 2024

  1. Adds new documentation for multi-tenancy feature.
  • Explain the basic concepts and usage for multi-tenancy.
  • List current limitations we have.
  • A quick set-up workflow to try multi-tenancy.
  1. Updates existing CRD documentations to include descriptions for new Namespaced resources.
  2. Identify the usage difference between KMP and CertStore.

Copy link

netlify bot commented May 7, 2024

Deploy Preview for ratify-dev ready!

Name Link
🔨 Latest commit 49fb508
🔍 Latest deploy log https://app.netlify.com/sites/ratify-dev/deploys/664cb2627a647a00081a2f1c
😎 Deploy Preview https://deploy-preview-66--ratify-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@binbin-li binbin-li changed the title Multi tenancy [WIP] Multi tenancy May 7, 2024
@binbin-li binbin-li force-pushed the multi-tenancy branch 4 times, most recently from 334ef19 to 3ece86e Compare May 16, 2024 15:16
@binbin-li binbin-li changed the title [WIP] Multi tenancy docs: add multi-tenancy doc May 16, 2024
@binbin-li binbin-li force-pushed the multi-tenancy branch 3 times, most recently from aa11d9c to cb9b264 Compare May 17, 2024 13:39
@binbin-li binbin-li closed this May 18, 2024
@binbin-li binbin-li reopened this May 18, 2024
Copy link
Collaborator

@susanshi susanshi left a comment

Choose a reason for hiding this comment

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

Thankyou binbin! So glad to see the multitenancy walk through. Left some comments.

docs/plugins/verifier/cosign.md Outdated Show resolved Hide resolved
docs/quickstarts/quickstart-manual.md Outdated Show resolved Hide resolved
docs/reference/multi-tenancy.md Outdated Show resolved Hide resolved
docs/reference/multi-tenancy.md Outdated Show resolved Hide resolved
docs/reference/multi-tenancy.md Outdated Show resolved Hide resolved
docs/reference/multi-tenancy.md Outdated Show resolved Hide resolved
docs/reference/multi-tenancy.md Outdated Show resolved Hide resolved
@binbin-li
Copy link
Collaborator Author

As discussed in the PR review meeting, I plan to add a short video to each setup step to help users understand more easily.

Copy link
Collaborator

@susanshi susanshi left a comment

Choose a reason for hiding this comment

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

Thanks Binbin. Let know know if you believe this is ready to merge. I think we can add the demo video as a separate PR.

@binbin-li binbin-li merged commit ac51dbc into ratify-project:main May 22, 2024
4 checks passed
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