fluent
Bump all dependencies to latest to upgrade controller-runtime
What this PR does / why we need it:
This upgrades all dependencies of the project to their latest releases. The most notable of those is controller-runtime, which has seen some significant rework. No semantic changes are intended in this change.
Does this PR introduced a user-facing change?
None
Additional documentation, usage docs, etc.:
@markusthoemmes Thanks again for the PR. There are conflicts that need to be resolved, you'll need to rebase the master
Done. I was staging PRs and didn't notice the other one actually merged.
might need go mod tidy for this?
https://github.com/fluent/fluent-operator/actions/runs/7711728924/job/21017658151?pr=1050
Hmm. This diff is becoming much larger than intended because the PodSpec grew quiet a bit between versions, I suppose.
@benjaminhuo As mentioned above, this diff has grown quite a bit. We could try to make it smaller by upgrading controller-runtime in smaller steps or trying to break the K8s upgrades out. Let me know if you'd like me to invest some effort into trying to make the diff smaller.
Hmm. This diff is becoming much larger than intended because the PodSpec grew quiet a bit between versions, I suppose.
@benjaminhuo As mentioned above, this diff has grown quite a bit. We could try to make it smaller by upgrading controller-runtime in smaller steps or trying to break the K8s upgrades out. Let me know if you'd like me to invest some effort into trying to make the diff smaller.
@wanjunlei what do you think?
Hmm. This diff is becoming much larger than intended because the PodSpec grew quiet a bit between versions, I suppose.
@benjaminhuo As mentioned above, this diff has grown quite a bit. We could try to make it smaller by upgrading controller-runtime in smaller steps or trying to break the K8s upgrades out. Let me know if you'd like me to invest some effort into trying to make the diff smaller.
@wanjunlei what do you think?
I think it's more appropriate to upgrade the controller runtime in smaller steps.
@benjaminhuo As mentioned above, this diff has grown quite a bit. We could try to make it smaller by upgrading controller-runtime in smaller steps or trying to break the K8s upgrades out. Let me know if you'd like me to invest some effort into trying to make the diff smaller.
We might need to catch up with the upstream k8s community updates and drop supports for lower version K8s. The dependencies are quite obsolete, so maybe big changes are also acceptable as long as we add a k8s support metrics @markusthoemmes @wanjunlei
We might need to catch up with the upstream k8s community updates and drop supports for lower version K8s.
we really need to catch up with upstream now because there're quite big differences and the code cannot compile with latest k8s dependencies:
Fixed by https://github.com/fluent/fluent-operator/pull/1251, close this one