Skip to content

Conversation

@mburke5678
Copy link
Contributor

@mburke5678 mburke5678 commented Nov 11, 2025

https://issues.redhat.com/browse/OSDOCS-17247

Link to docs preview:
Automatically adjust pod resource levels with the vertical pod autoscaler -- Added additional reference links to About using the Vertical Pod Autoscaler Operator after About the in place or recreate mode.

Automatically adjust pod resource levels with the vertical pod autoscaler -> About the Vertical Pod Autoscaler Operator -- Updated text in a few places, new About the Vertical Pod Autoscaler Operator modes section.

Automatically adjust pod resource levels with the vertical pod autoscaler -> Using the Vertical Pod Autoscaler Operator -- Replaced call-out 3, switched Auto (Removed from API) to InPlaceOrRecreate. Updated the call-out text.

Automatically adjust pod resource levels with the vertical pod autoscaler -> Example custom resources for the Vertical Pod Autoscaler -- Replaced call-out 3, switched Auto (Removed from API) to InPlaceOrRecreate.

Automatically adjust pod resource levels with the vertical pod autoscaler -> About the in place or recreate mode -- New module.

Automatically adjust pod resource levels with the vertical pod autoscaler -> About using the Vertical Pod Autoscaler Operator -- Switched Auto (Removed from API) to InPlaceOrRecreate. Updated text. Added link to Adjust pod resource levels without pod disruption.

QE review:

  • QE has approved this change.

@mburke5678 mburke5678 added this to the Continuous Release milestone Nov 11, 2025
@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 11, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Nov 11, 2025

🤖 Tue Nov 25 21:20:35 - Prow CI generated the docs preview:

https://102322--ocpdocs-pr.netlify.app/openshift-enterprise/latest/nodes/pods/nodes-pods-vertical-autoscaler.html

@mburke5678
Copy link
Contributor Author

@maxcao13 PTAL

@maxcao13
Copy link
Member

maxcao13 commented Nov 17, 2025

Thanks for the ping!

General note, but I think we might want to have a notice that the Auto update mode is deprecated? It seems in this PR we are removing all references to it altogether as if it's removed in the latest 4.21 version, but it is not. User's can still reference it, and Auto still is an alias for Recreate.

If users continue to use it, they will see warnings from the API, but eventually when upstream goes through the deprecation process, the mode will be removed from the API, (I don't think there is a clear plan about when, but it is not removed as of this release, or the next few I believe).

@maxcao13
Copy link
Member

Looks pretty good to me :-)

One last thing, is I think this section we should replace the mention of Auto with Recreate.
https://github.com/openshift/openshift-docs/pull/102322/files#diff-762491c08640b5d799f45ef181732f27ea79b9b87021ea56bf442a2e1bfab8abR26

@mburke5678 mburke5678 force-pushed the autoscale-vpa-in-place-updates branch 3 times, most recently from fcfbc7b to f2d6a18 Compare November 25, 2025 20:28
@mburke5678 mburke5678 force-pushed the autoscale-vpa-in-place-updates branch from f2d6a18 to a1500a8 Compare November 25, 2025 21:12
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 25, 2025
@mburke5678 mburke5678 added the merge-review-needed Signifies that the merge review team needs to review this PR label Nov 25, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 25, 2025

@mburke5678: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.21 merge-review-needed Signifies that the merge review team needs to review this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants