fix: Remove Helm deployment of NICClusterPolicy

Mellanox / network-operator

Mellanox Network Operator

Apache License 2.0

205 stars 50 forks source link

fix: Remove Helm deployment of NICClusterPolicy #1085

Open killianmuldoon opened 1 week ago

killianmuldoon commented 1 week ago

Remove the ability to create the NICClusterPolicy CR through helm.values. This feature has been deprecated for a number of releases and the documentation was updated to remove reference to this feature in the most recent release.

Users should instead use the helm package to install the Network Operator and configure the system by creating the NICClusterPolicy in a separate step.

killianmuldoon commented 1 week ago

Looks like the CI is broken due to authentication 🤔

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 11069897084

Details

0 of 0 changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 60.483%

Files with Coverage Reduction	New Missed Lines	%
pkg/state/dummy_provider.go	3	71.43%
<!--	Total:	3		-->

Totals
Change from base Build 10994266668:	0.1%
Covered Lines:	3332
Relevant Lines:	5509

💛 - Coveralls

rollandf commented 6 days ago

/retest-all