Skip to content

Fix managed_service_instances dataset for spaces#4481

Merged
jochenehret merged 1 commit intomainfrom
fix_space_managed_instances
Jul 30, 2025
Merged

Fix managed_service_instances dataset for spaces#4481
jochenehret merged 1 commit intomainfrom
fix_space_managed_instances

Conversation

@jochenehret
Copy link
Copy Markdown
Contributor

  • A short explanation of the proposed change:
    The managed_service_instances dataset for a space should only include managed services (where "is_gateway_service" is true) and no user-provided services.

  • An explanation of the use cases your change solves
    Consistent behavior with the organization managed_service_instances dataset.

  • Links to any other associated PRs

  • 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

* the dataset should only include managed services (where "is_gateway_service" is true) and no user-provided services
* this behaviour is consistent with the organization managed_service_instances dataset
@jochenehret jochenehret merged commit 3fc5eca into main Jul 30, 2025
18 of 19 checks passed
@jochenehret jochenehret deleted the fix_space_managed_instances branch July 30, 2025 08:13
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Jul 30, 2025
Changes in cloud_controller_ng:

- Fix space usage summary
    PR: cloudfoundry/cloud_controller_ng#4482
    Author: Jochen Ehret <jochen.ehret@sap.com>

- Fix managed_service_instances dataset for spaces
    PR: cloudfoundry/cloud_controller_ng#4481
    Author: Jochen Ehret <jochen.ehret@sap.com>
@kathap kathap mentioned this pull request Aug 18, 2025
5 tasks
@johha
Copy link
Copy Markdown
Contributor

johha commented Aug 18, 2025

With this change the total number of all service instances is shown in the usage summary endpoint as the spaces filter is missing. Fix: #4520

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