Add consolelink permissions to cluster role

[rubenvp8510]

Signed-off-by: Ruben Vargas [email protected]

[codecov]

Codecov Report

Merging #1177 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1177   +/-   ##
=======================================
  Coverage   88.30%   88.30%           
=======================================
  Files          89       89           
  Lines        5506     5506           
=======================================
  Hits         4862     4862           
  Misses        474      474           
  Partials      170      170           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f394fea...84998f1. Read the comment docs.

[jpkrohling]

Could you run the "generate CSV" command and see if the next CSV will include this change? The release script has a step where it generates the CSV for the new version, you can use that as reference.

[rubenvp8510]

I ran this command:

operator-sdk generate csv --csv-channel=stable --csv-version="1.19.1" --from-version="1.19.0"

And it seems like the change is included on the manifest.

[jpkrohling]

@kevinearls, @rubenvp8510, there's a failing test. Is it a known issue?

[jpkrohling]

    TestDaemonSetSuite/TestDaemonSet: daemonset_test.go:151: 
        	Error Trace:	daemonset_test.go:151
        	Error:      	Received unexpected error:
        	            	timed out waiting for the condition
        	Test:       	TestDaemonSetSuite/TestDaemonSet
        	Messages:   	Failed waiting for expected content

@rubenvp8510, could you rebase the PR? This test failure has been fixed in the latest master.

[rubenvp8510]

Rebased, is green now