Skip to content

Fix actual_lrps_by_processes function#4870

Merged
jochenehret merged 1 commit intomainfrom
fix_actual_lrps_by_processes
Feb 24, 2026
Merged

Fix actual_lrps_by_processes function#4870
jochenehret merged 1 commit intomainfrom
fix_actual_lrps_by_processes

Conversation

@jochenehret
Copy link
Contributor

  • A short explanation of the proposed change:
    Fix actual_lrps_by_processes function. If an empty list of processes is passed, don't call the Diego API.

  • An explanation of the use cases your change solves
    Avoids cloud controller "UnknownError" e.g. when /v3/processes?space_guids=:guid&embed=process_instances is called on an empty space.

  • Links to any other associated PRs
    Implement GET /v3/processes/:guid/process_instances and embed=process_instances #4796

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

* check for empty processes and don't call the Diego backend (would result in process_guids must not be empty error)
@jochenehret jochenehret requested review from a team and philippthun February 23, 2026 15:43
@jochenehret jochenehret merged commit d71ab6a into main Feb 24, 2026
11 checks passed
@jochenehret jochenehret deleted the fix_actual_lrps_by_processes branch February 24, 2026 09:13
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Feb 24, 2026
Changes in cloud_controller_ng:

- Fix actual_lrps_by_processes function
    PR: cloudfoundry/cloud_controller_ng#4870
    Author: Jochen Ehret <jochen.ehret@sap.com>
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