-
Notifications
You must be signed in to change notification settings - Fork 42
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
RFC-0030 - Add support to Diego for file based service bindings #100
base: main
Are you sure you want to change the base?
RFC-0030 - Add support to Diego for file based service bindings #100
Conversation
2c59134
to
2f5337e
Compare
Remove ioutil.ReadFile in favor of os.ReadFile. Remove MkdirAll in favor of Mkdir.
2f5337e
to
fa28a86
Compare
return nil | ||
} | ||
|
||
func (h *ServiceBindingRootHandler) RemoveDir(logger lager.Logger, container executor.Container) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not see this function covered with tests, or am i wrong?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello here it is the test for RemoveDir
https://github.com/cloudfoundry/executor/pull/100/files#diff-5e39d32f309883624104dc2056437db4a3d782ea689020c000a3848675ca80d8R81
Summary
This PR is continuation for the RFC-0030
Backward Compatibility
Breaking Change? No
This feature is the first PoC for RFC-0030.
Since the CAPI implementation for RFC-0030 is not done yet, the PR doesn't need to be applied immediately.