Update args for gosec, with no args it runs the -h option

[randeepdell]

Description

By default gosec expects args without it it defaults to -h option and lists the way to run gosec and does not run any actual test.

Excluded G304 for the below, which seems a valid change needed.
[/github/workspace/service/service.go:561] - G304 (CWE-22): Potential file inclusion via variable (Confidence: HIGH, Severity: MEDIUM)

561: configBytes, err := ioutil.ReadFile(isilonConfigFile)

Made changes to address the below,
Addressed other issues.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#128

Checklist:

• I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
• I have verified that new and existing unit tests pass locally with my changes
• I have not allowed coverage numbers to degenerate
• I have maintained at least 90% code coverage
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• Backward compatibility is not broken

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

• Built the image and tested creation of sc, pvc and pod.
• Testing actions via PR

[prablr79]

lgtm

[bpjain2004]

why introduce one more variable? either use config or clusterConfig, and remove one of these.

[randeepdell]

This is to address the below,
[/github/workspace/service/service.go:664] - G601 (CWE-118): Implicit memory aliasing in for loop. (Confidence: MEDIUM, Severity: MEDIUM)

664: config.isiSvc, _ = s.GetIsiService(clientCtx, &config, logLevel)