Skip to content

Fix issue with displaying Tasks that no longer have a Droplet#4469

Merged
johha merged 1 commit intomainfrom
fix-task-user-listing
Jul 23, 2025
Merged

Fix issue with displaying Tasks that no longer have a Droplet#4469
johha merged 1 commit intomainfrom
fix-task-user-listing

Conversation

@tcdowney
Copy link
Member

  • Although a Task must have a Droplet associated with it when it is created, these Droplets will be eventually cleaned up as the app is pushed with newer code over time. This change updates the presenter to return null when a Task's user can no longer be looked up because its droplet has been deleted.
  • Fixes GET /v3/tasks?app_guids=<guid> fails with An unknown error occurred #4467
  • 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

- Although a Task must have a Droplet associated with it when it is
  created, these Droplets will be eventually cleaned up as the app is
  pushed with newer code over time. This change updates the presenter
  to return null when a Task's user can no longer be looked up because
  its droplet has been deleted.
- Fixes #4467
@tcdowney tcdowney requested review from Gerg, johha and stephanme July 22, 2025 17:52
Copy link
Contributor

@johha johha left a comment

Choose a reason for hiding this comment

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

LGTM

@johha johha merged commit 1cf84a0 into main Jul 23, 2025
17 of 19 checks passed
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Jul 23, 2025
Changes in cloud_controller_ng:

- Fix issue with displaying Tasks that no longer have a Droplet
    PR: cloudfoundry/cloud_controller_ng#4469
    Author: Tim Downey <tcdowney@users.noreply.github.com>
@moleske moleske deleted the fix-task-user-listing branch July 23, 2025 15:12
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.

GET /v3/tasks?app_guids=<guid> fails with An unknown error occurred

2 participants