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

Set 'instance' in the context to be used by dashboard url. #1603

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

Pooja-08
Copy link

@Pooja-08 Pooja-08 commented Apr 4, 2022

If the plan contains dashboard url, it may refer to the sfserviceinstance for reading the name & cluster id properties. If the instance is not present it throws an exception and the GET call fails.
Fix for: https://jtrack.wdf.sap.corp/browse/NGPBUG-181387

If the plan contains dashboard url, it may refer to the sfserviceinstance
for reading the name & cluster id properties. If the instance is not present
it throws an exception and the GET call fails.
@Pooja-08 Pooja-08 requested a review from a team April 4, 2022 07:35
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/181768146

The labels on this github issue will be updated when the story is started.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/181787065

The labels on this github issue will be updated when the story is started.

@codecov
Copy link

codecov bot commented Apr 5, 2022

Codecov Report

Merging #1603 (73b458d) into master (8ed6e70) will decrease coverage by 0.11%.
The diff coverage is 60.21%.

@@            Coverage Diff             @@
##           master    #1603      +/-   ##
==========================================
- Coverage   73.06%   72.95%   -0.12%     
==========================================
  Files          45       46       +1     
  Lines        4348     4441      +93     
==========================================
+ Hits         3177     3240      +63     
- Misses        836      860      +24     
- Partials      335      341       +6     

@Pooja-08 Pooja-08 merged commit 47ef48c into cloudfoundry:master Apr 6, 2022
jintusebastian pushed a commit to jintusebastian/service-fabrik-broker that referenced this pull request May 17, 2022
If the plan contains dashboard url, it may refer to the sfserviceinstance
for reading the name & cluster id properties. If the instance is not present
it throws an exception and the GET call fails.
jintusebastian pushed a commit to jintusebastian/service-fabrik-broker that referenced this pull request Feb 13, 2023
If the plan contains dashboard url, it may refer to the sfserviceinstance
for reading the name & cluster id properties. If the instance is not present
it throws an exception and the GET call fails.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants