-
Notifications
You must be signed in to change notification settings - Fork 385
Conversation
Welcome @zulandar! |
/assign @jberkhahn |
mentioned a couple things, other than that this looks good |
Adjusted per PR recommendations
Made the recommended adjustments. |
/retest No code changes was performed to cause test failures. Here is the failure from the test.
|
/test pull-service-catalog-xbuild |
@zulandar currently is a known issue that the pull-service-catalog-xbuild is sometimes failing with such
to execute that test one more time. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jberkhahn, zulandar The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR is a
What this PR does / why we need it:
Requested change from UPS Broker to Minibroker within walkthrough.
Which issue(s) this PR fixes
Fixes #2658
Please leave this checklist in the PR comment so that maintainers can ensure a good PR.
Merge Checklist:
breaking the chart release and existing clients who provide a
flag that will get an error when they try to update