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

Allows space supporter to access GET v3/service_route_bindings #2426

Merged
merged 2 commits into from Jul 22, 2021

Conversation

galenhammond
Copy link
Contributor

Allows space supporter access to service route binding list endpoint

Closes #2422

  • 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

/v3/service_route_bindings

Closes #2422

Authored-by: Galen Hammond <galenh@vmware.com>
@sweinstein22 sweinstein22 added the space-application-supporter https://github.com/cloudfoundry/cfar-proposals/issues/22 label Jul 19, 2021
@sweinstein22 sweinstein22 self-requested a review July 19, 2021 23:38
Copy link
Contributor

@sweinstein22 sweinstein22 left a comment

Choose a reason for hiding this comment

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

Acceptance

Behavior Check

Ran cf curl against v3/service_route_bindings as both a space supporter and space developer. Manually confirmed output looks correct and checked for a diff:

$ diff space-supporter-service-route-bindings space-dev-service-route-bindings

Code Check

Looks good, pretty minimal changes that use the established patterns for this work

Copy link
Contributor

@sweinstein22 sweinstein22 left a comment

Choose a reason for hiding this comment

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

Docs Check

Looks like we missed the "Experimental" note on the docs, but otherwise this looks good to go!

Authored-by: Galen Hammond <galenh@vmware.com>
Copy link
Contributor

@sweinstein22 sweinstein22 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for adjusting the docs and for the PR!

@sweinstein22 sweinstein22 merged commit dd1394c into main Jul 22, 2021
@sweinstein22 sweinstein22 deleted the space-supporter-service-routes branch July 22, 2021 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
space-application-supporter https://github.com/cloudfoundry/cfar-proposals/issues/22
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow space application supporter to list service route bindings
3 participants