-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v1.14 Backports 2023-12-07 #29689
v1.14 Backports 2023-12-07 #29689
Conversation
/test-backport-1.14 |
Why is my commit #24836 part of this backport? I can see that it's already part of the latest v1.14.4 see https://github.com/cilium/cilium/blob/main/install/kubernetes/cilium/values.yaml#L74 Is it because the commits by @ayuspin where done on top of my commit? |
The changes in #24836 are only available in main, but not in 1.14 branch and older branches. |
[ upstream commit 4a5cbba ] Signed-off-by: Andrii Iuspin <yuspin@gmail.com> Signed-off-by: Tam Mach <tam.mach@cilium.io>
[ upstream commit 6f54448 ] This commit is to add automountServiceAccountToken, extraVolumes and extraVolumeMounts field to preflight workload. Signed-off-by: Andrii Iuspin <yuspin@gmail.com> Signed-off-by: Tam Mach <tam.mach@cilium.io>
[ upstream commit 22a4d1d ] Signed-off-by: Andrii Iuspin <yuspin@gmail.com> Signed-off-by: Tam Mach <tam.mach@cilium.io>
[ upstream commit 76126b7 ] Signed-off-by: Andrii Iuspin <yuspin@gmail.com> Signed-off-by: Tam Mach <tam.mach@cilium.io>
[ upstream commit de788fa ] Signed-off-by: Andrii Iuspin <yuspin@gmail.com> Signed-off-by: Tam Mach <tam.mach@cilium.io> Signed-off-by: Tam Mach <tam.mach@cilium.io>
fb47da6
to
7705546
Compare
/test-backport-1.14 |
You are right, the mentioned changes are already part of 1.14, I have dropped the commit accordingly. Thanks for highlighting 👍 |
helm: Add SA to nodeinit ds #24836 (@darox)already part of 1.14 as highlighted below.Once this PR is merged, a GitHub action will update the labels of these PRs: