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.

Sign up for GitHub

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

Jump to bottom

[RELEASE] Operator v5.0.7 doc updates #943

Closed
5 of 6 tasks
djwfyi opened this issue Aug 9, 2023 · 4 comments · Fixed by #1060
Closed
5 of 6 tasks

[RELEASE] Operator v5.0.7 doc updates #943

djwfyi opened this issue Aug 9, 2023 · 4 comments · Fixed by #1060
Assignees
@feorlen @ravindk89

Comments

@djwfyi
Copy link
Collaborator

djwfyi commented Aug 9, 2023 •
edited by feorlen
Loading

Operator v5.0.7 has changes that impact the docs:

  • kubectl minio tenant info now includes tenant secret info (PR #1704)
  • Can now mount volumes to the Operator pod (PR #1700)
  • Add SFTP support during tenant creation (PR #1685 and PR #1692) (Existing Tenants also)
  • kubectl minio delete now has a --force flag (PR #1687)

Helm changes:

  • Support digest for OpenShift PR #1706
  • The helm chart has been split into separate files as per best practices PR #1677
@feorlen feorlen self-assigned this Aug 29, 2023
@feorlen
Copy link
Collaborator

feorlen commented Aug 30, 2023

minio/operator#1677 doesn't require doc updates at this time.

Specifics of chart structure/contents will get covered in a future section about how to customize. For now we link to the relevant Helm docs and presume anyone needing this level of customization already knows how charts work.

@feorlen
Copy link
Collaborator

feorlen commented Aug 30, 2023

Similar to previous, no doc updates needed for minio/operator#1706

The new option is documented with a comment in operator/values.yaml.

@feorlen feorlen mentioned this issue Aug 30, 2023
Merged
2 tasks
feorlen added a commit that referenced this issue Aug 31, 2023
@feorlen
Partial updates for Operator v5.0.7 docs (#978)
33b2882 
The easy bits from #943: 

- [x] `kubectl minio tenant info` now includes tenant secret info ([PR
#1704](minio/operator#1704))
- [x] `kubectl minio delete` now has a `--force` flag ([PR
#1687](minio/operator#1687))

Staged:

http://192.241.195.202:9000/staging/DOCS-943-1/k8s/reference/kubectl-minio-plugin/kubectl-minio-delete.html

http://192.241.195.202:9000/staging/DOCS-943-1/k8s/reference/kubectl-minio-plugin/kubectl-minio-tenant-info.html

http://192.241.195.202:9000/staging/DOCS-943-1/k8s/operations/install-deploy-manage/deploy-minio-tenant.html#persistent-volumes
@feorlen feorlen mentioned this issue Sep 6, 2023
Merged
feorlen added a commit that referenced this issue Sep 8, 2023
@feorlen
SFTP access for Operator Tenants (#993)
1aea92b 
Document SFTP access for Operator Tenants, from Operator v5.0.7.
minio/operator#1685
minio/operator#1692

The new Operator SFTP text and the not-Operator version it is based on
were converted to includes rather than have a mess of conditionals in a
single file.

Staged

http://192.241.195.202:9000/staging/DOCS-943-2/k8s/developers/file-transfer-protocol.html

http://192.241.195.202:9000/staging/DOCS-943-2/linux/developers/file-transfer-protocol.html

Partially addresses #943
@djwfyi
Copy link
Collaborator Author

djwfyi commented Oct 3, 2023

Last item relates to Helm charts, which are not documented.
Also waiting for feedback on the PR 1700 about the use case.

@feorlen
Copy link
Collaborator

feorlen commented Oct 19, 2023

Requested use case details for minio/operator#1700. We are unclear about their intended purpose that needed this functionality.

@ravindk89 ravindk89 mentioned this issue Nov 3, 2023
Merged
@ravindk89 ravindk89 self-assigned this Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@feorlen feorlen

@ravindk89 ravindk89

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

DOCS-1021: Adding helm information minio/docs
3 participants
@feorlen @ravindk89 @djwfyi