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

Feat: Add cpu entitlement to app process table (v9) #2850

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

acrmp
Copy link
Member

@acrmp acrmp commented Apr 5, 2024

Where this PR should be backported?

  • main - all changes should by default start here
  • v8
  • v7

Description of the Change

Extend the v9 cf CLI to support showing the new CPU Entitlement metric. This is implemented as a breaking change with cpu entitlement replacing the cpu column from earlier versions:

     state     since                  cpu entitlement   memory        disk           logging             details
#0   running   2024-04-05T17:46:17Z   12.5%             47.4M of 1G   130.1M of 1G   0B/s of unlimited
#1   running   2024-04-05T17:46:17Z   12.8%             47.8M of 1G   130.1M of 1G   0B/s of unlimited

If the cpu entitlement stats are not available (e.g. deployment does not support it, app is stopped, etc.) then the cf CLI will show an empty value for the column.

Why Is This PR Valuable?

CPU Entitlement is a more useful metric than the older CPU metric.

Applicable Issues

#2812

How Urgent Is The Change?

Slightly less than urgent.

Other Relevant Parties

@rroberts2222

@a-b a-b merged commit 2d9b726 into cloudfoundry:main Apr 30, 2024
10 of 13 checks passed
@acrmp acrmp deleted the feat/main/cpu-entitlement-only branch May 2, 2024 22:58
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.

3 participants