Closed reetika-vyas closed 2 years ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: reetika-vyas
To complete the pull request process, please assign 2uasimojo after the PR has been reviewed.
You can assign the PR to them by writing /assign @2uasimojo
in a comment when ready.
The full list of commands accepted by this bot can be found here.
In several of the places where you've added error checking, there are comments explaining why error checking was previously absent. At least in those places, suggest finding a less disruptive way around
gosec
. For example, perhaps instead of returning the error, simply log it and proceed?
I made the changes you requested
added the error handling flagged by gosec for aws-efs-operator