Update k8s deps for 1.21.0 release

[sunjayBhatia]

No kind image yet so no update to tested k8s versions yet

[codecov]

Codecov Report

Merging #3581 (ef93ceb) into main (3e59de8) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3581      +/-   ##
==========================================
- Coverage   76.70%   76.66%   -0.05%     
==========================================
  Files         100      100              
  Lines        7066     7066              
==========================================
- Hits         5420     5417       -3     
- Misses       1530     1533       +3     
  Partials      116      116              

Impacted Files	Coverage Δ
internal/k8s/log.go	63.04% <0.00%> (-6.53%)	⬇️

[stevesloka]

Could you also update the support matrix bits as well?

[skriss]

LGTM; if we want to add a main entry to the compat matrix that's fine with me, otherwise can update when we release 1.15 IMHO.

[sunjayBhatia]

I'll add an Issue for the 1.15.0 release, since we're not actually testing against 1.21.0 yet (no kind image) I would hold off on changing the matrix

[sunjayBhatia]

#3583

[youngnick]

I think we should add a 'main` line to the support matrix, that makes keeping everything up to date much easier. In addition, we probably need to remove really old versions.

[sunjayBhatia]

I think we should add a 'main` line to the support matrix, that makes keeping everything up to date much easier. In addition, we probably need to remove really old versions.

#3605

[jroper]

client-go v1.21.0 appears to be incompatible with controller-runtime v0.8.3:

vendor/sigs.k8s.io/controller-runtime/pkg/metrics/client_go_adapter.go:134:3: cannot use &latencyAdapter{...} (type *latencyAdapter) as type metrics.LatencyMetric in field value:
	*latencyAdapter does not implement metrics.LatencyMetric (wrong type for Observe method)
		have Observe(string, url.URL, time.Duration)
		want Observe(context.Context, string, url.URL, time.Duration)
vendor/sigs.k8s.io/controller-runtime/pkg/metrics/client_go_adapter.go:135:3: cannot use &resultAdapter{...} (type *resultAdapter) as type metrics.ResultMetric in field value:
	*resultAdapter does not implement metrics.ResultMetric (wrong type for Increment method)
		have Increment(string, string, string)
		want Increment(context.Context, string, string, string)

I get this compilation error when I depend on contour v1.15.0.

[skriss]

client-go v1.21.0 appears to be incompatible with controller-runtime v0.8.3:

vendor/sigs.k8s.io/controller-runtime/pkg/metrics/client_go_adapter.go:134:3: cannot use &latencyAdapter{...} (type *latencyAdapter) as type metrics.LatencyMetric in field value:
	*latencyAdapter does not implement metrics.LatencyMetric (wrong type for Observe method)
		have Observe(string, url.URL, time.Duration)
		want Observe(context.Context, string, url.URL, time.Duration)
vendor/sigs.k8s.io/controller-runtime/pkg/metrics/client_go_adapter.go:135:3: cannot use &resultAdapter{...} (type *resultAdapter) as type metrics.ResultMetric in field value:
	*resultAdapter does not implement metrics.ResultMetric (wrong type for Increment method)
		have Increment(string, string, string)
		want Increment(context.Context, string, string, string)

I get this compilation error when I depend on contour v1.15.0.

Yep, see kubernetes-sigs/controller-runtime#1473. Contour PR #3510 updates to the latest controller-runtime 0.9.0 beta and we'll update to the GA release as soon as it's out.