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

Improve cf apps with --no-stats and better error handling [V8] #2796

Merged
merged 2 commits into from
Feb 22, 2024
Merged

Improve cf apps with --no-stats and better error handling [V8] #2796

merged 2 commits into from
Feb 22, 2024

Commits on Feb 20, 2024

  1. Introduce --no-stats for cf apps 

    Fetching process stats requires a `/stats` request for each process.
If a space contains a large number apps or processes this slows down `cf apps`.
In case a user is not interested in stats the `--no-stats` flag can be helpful.
    @johha
    johha committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    b083ace View commit details
    Browse the repository at this point in the history

  2. Handle ProcessNotFoundError in cf apps 

    As a user calls `cf apps` in a large space it might happen that a app
was deleted while the CLI is putting all the different information (processes, stats & routes) together.
This leads to a failing `cf apps` command.
This change prevents this by catching `ProcessNotFoundError` errors and returning `nil` instead.
Related to cloudfoundry#2734
    @johha
    johha committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    2f0bce4 View commit details
    Browse the repository at this point in the history