We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Bug:
During unbind job we remove any credentials we stored. However this is also being done in the unbind subscriber ubind logic https://github.com/openshift/ansible-service-broker/blob/master/pkg/broker/unbinding_subscriber.go#L89
unbind job https://github.com/openshift/ansible-service-broker/blob/master/pkg/apb/unbind.go#L74
Happy to have this assigned to me if we agree it is something that should be fixed
The text was updated successfully, but these errors were encountered:
Hey,
This was supposed to moved to the APB package when we moved to secrets for extracted credentials. https://github.com/openshift/ansible-service-broker/blob/master/pkg/apb/deprovision.go#L77
In this case, I think the bug is that unbinding_subscriber should not deal with the extracted credential.
Sorry, something went wrong.
ahh ok will update this issue based on that. So really the delete in the unbind subscriber should be removed
shawn-hurley
Successfully merging a pull request may close this issue.
During unbind job we remove any credentials we stored. However this is also being done in the unbind subscriber
ubind logic https://github.com/openshift/ansible-service-broker/blob/master/pkg/broker/unbinding_subscriber.go#L89
unbind job https://github.com/openshift/ansible-service-broker/blob/master/pkg/apb/unbind.go#L74
Happy to have this assigned to me if we agree it is something that should be fixed
The text was updated successfully, but these errors were encountered: