Skip to content

CNV#82024: formula correction#111405

Open
ousleyp wants to merge 1 commit intoopenshift:mainfrom
ousleyp:cnv-82024
Open

CNV#82024: formula correction#111405
ousleyp wants to merge 1 commit intoopenshift:mainfrom
ousleyp:cnv-82024

Conversation

@ousleyp
Copy link
Copy Markdown
Member

@ousleyp ousleyp commented May 7, 2026

Version(s): likely all supported versions

Issue: CNV-82024

Link to docs preview: https://111405--ocpdocs-pr.netlify.app/openshift-enterprise/latest/virt/support/virt-collecting-virt-data.html

QE review:

  • QE has approved this change.

Additional information:

@ousleyp ousleyp requested a review from germanovm May 7, 2026 20:55
@openshift-ci openshift-ci Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 7, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 7, 2026

@ousleyp: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

** The PVC must be large enough to contain the memory dump.
+
The formula for calculating the PVC size is `(VMMemorySize + 100Mi) * FileSystemOverhead`, where `100Mi` is the memory dump overhead, and `FileSystemOverhead` is defined in the `HCO` object.
The formula for calculating the PVC size is `(VMMemorySize + 100Mi) * (1 + FileSystemOverhead)`, where `100Mi` is the memory dump overhead, and `FileSystemOverhead` is defined in the `HCO` object.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Correct, so the VM memory size is multiplied by (1+0.xxx) and not by 0.xxx.

Copy link
Copy Markdown

@germanovm germanovm left a comment

Choose a reason for hiding this comment

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

LGTM

@ousleyp ousleyp requested a review from stu-gott May 8, 2026 14:47
@ousleyp
Copy link
Copy Markdown
Member Author

ousleyp commented May 8, 2026

Hey @stu-gott , are you around to QE this fix? Thanks :)

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

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants