Minor change to clusterrole

[abhi16394]

Description

removing the object from error statement

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#159

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
• I have maintained backward compatibility

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

• Test A
• Test B

[bharathsreekanth]

@abhi16394 Please update checklist items above.

[rensyct]

@abhi16394, please help understand why namespace is being added in operator.yaml? We had specifically removed namespace from the manifest to fix the warning message that was thrown when installing the operator

@rensyct can you change the script to work if namespace exists, this way we can deploy without scripts

[rensyct]

@abhi16394, please help understand why namespace is being added in operator.yaml? We had specifically removed namespace from the manifest to fix the warning message that was thrown when installing the operator

@rensyct can you change the script to work if namespace exists, this way we can deploy without scripts

We dont want to make any code changes now. We can discuss this (whether install script should create the namespace or should fail if it does not exist ) in the next meet and take it forward in the next release

[bharathsreekanth]

@abhi16394 - Can you help answer @rensyct question? Is this change absolutely necesary at this time, and if yes, will this create changes in other areas? What happens if we dont set the namespace?

added comment