Moving rbac resources to Kubernetes 1.22+

[aravol]

Fixes #119

[mschneider82]

thanks, verified it works

@JoelSpeed can you please merge an release a new version?

[JoelSpeed]

I'm no longer involved in this project, CC @icelynjennings

[aravol]

@gargath ?

[aravol]

Still seeing this as an issue on our side

[mcc138]

Looks good and is needed now

[aravol]

Tests are still having a problem loading up dependencies for checks on this PR - looks like an environment issue?

[mwmahlberg]

@aravol My 2 cents: This should be a conditional and not hard coded. Something in the line of

{{ if .Capabilities.KubeVersion.Minor >= 22 }}
apiVersion: rbac.authorization.k8s.io/v1
{{ else }}
apiVersion: rbac.authorization.k8s.io/v1beta1
{{ end }}

This way, the helm chart will work on "older" versions. Keep in mind that many enterprise kubernetes distributions are usually several versions behind.

[aravol]

That was a choice not to use a conditional statement; the rbac api enetered general availability with Kubernetes 1.8.1, which has been available for nearly five years now and which has already been deprecated by most hosts

[mwmahlberg]

@aravol Well, checks fail anyway -- guess it would be worth digging into that. So much to do, so little time. ;)

[nifr]

Related #125

[aaron-trout]

Bump @icelynjennings can you take a look at this? I can step in to help maintain this project if needs be; let me know!

[tw]

@icelynjennings sorry to tag you again, just wondered if this will be being merged before I make an adjustment locally.

[aravol]

Ended up making the changes locally to get this working

[mschneider82]

Consider migrate to https://github.com/stakater/Reloader

Wave is not maintained anymore

[toelke]

Thank you for the contribution. I will merge #112 instead of this, as it is the earlier contribution.