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-690, DOCS-707: Release updates #720

Merged
merged 2 commits into from
Feb 3, 2023
Merged

DOCS-690, DOCS-707: Release updates #720

merged 2 commits into from
Feb 3, 2023

Conversation

@ravindk89 ravindk89 added priority: medium Review Assigned Reviewers Must LGTM To Close labels Feb 2, 2023
@ravindk89 ravindk89 requested a review from djwfyi February 2, 2023 23:06
@ravindk89 ravindk89 self-assigned this Feb 2, 2023
Copy link
Collaborator

@djwfyi djwfyi left a comment

Choose a reason for hiding this comment

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

LGTM, mod nits


The MinIO Tenant Console Audit Log feature is scheduled for deprecation and removal in an upcoming release.

You can instead use any webhook-capable database or logging service to capture :ref:`audit logs <minio-logging-publish-audit-logs>` from the Tenant.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I suggest a different opening phrase:

As an alternative, use ...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done throughout.

The MinIO Tenant Console Audit Log feature is scheduled for deprecation and removal in an upcoming release.
MinIO recommends disabling this feature in preparation for this change.

You can instead use any webhook-capable database or logging service to capture :ref:`audit logs <minio-logging-publish-audit-logs>` from the Tenant.
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you change it in managing-deployment, use similar phrasing here.

The Tenant Prometheus pod feature is scheduled for deprecation and removal in an upcoming release.
MinIO recommends setting this value to ``false`` in preparation for this change.

You can instead use any Prometheus service deployed within the Kubernetes cluster or externally to :ref:`capture Tenant metrics <minio-metrics-collect-using-prometheus>`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same as the other two similar sentences. Be consistent in the the phrasing if you change them.


.. important::

The Tenant Prometheus pod feature is scheduled for deprecation and removal in an upcoming release.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is probably fine in passive, but maybe:

MinIO plans to deprecate and remove the Tenant Prometheus pod feature in an upcoming release.

or

We plan to ...

- Select the storage class and requested capacity associated to the PVC generated to support Prometheus.

* - Storage Size
- Specify the amount of size of storage to make available for Prometheus.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Either amount of or size of, but not both.

- Specify the amount of size of storage to make available for Prometheus.

* - :guilabel:`SecurityContext`
- The MinIO Operator assigns this Security Context for the Prometheus pod
Copy link
Collaborator

Choose a reason for hiding this comment

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

Need a . at the end.

- The MinIO Operator assigns this Security Context for the Prometheus pod

You can modify the Security Context to run the associated pod commands using a different User, Group, FsGroup, or FSGroupChangePolicy.
You can also direct the pod to not run commands as the Root user.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Consider monotyping Root?
Maybe also the User, Group ... list in the previous line?

@ravindk89 ravindk89 merged commit da88409 into main Feb 3, 2023
@ravindk89 ravindk89 deleted the k8s-fixups branch February 3, 2023 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Review Assigned Reviewers Must LGTM To Close
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RELEASE] Console release v0.22.5 [RELEASE] Operator v4.5.6 Doc Impacts
2 participants