Skip to content

values(.yaml.tmpl): Add Geneve (Class Option) to dsrDispatch paramater #40625

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 29, 2025
Merged

values(.yaml.tmpl): Add Geneve (Class Option) to dsrDispatch paramater #40625

merged 1 commit into from
Jul 29, 2025

Conversation

alagoutte
Copy link
Contributor

@alagoutte alagoutte commented Jul 21, 2025 •
edited
Loading

It is possible to use geneve for dsrDispatch (on Native and Encupaslation mode)

@alagoutte alagoutte requested review from a team as code owners July 21, 2025 18:27
@alagoutte alagoutte requested review from tommyp1ckles and squeed July 21, 2025 18:27
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 21, 2025
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Jul 21, 2025
@alagoutte
values(.yaml.tmpl): Add Geneve (Class Option) to dsrDispatch paramater
5c82479 
It is possible to use geneve for dsrDispatch (on Native and Encupaslation mode)

Signed-off-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
@alagoutte alagoutte force-pushed the helm-value-dspispatch branch from cff1991 to 5c82479 Compare July 21, 2025 18:30
tommyp1ckles
tommyp1ckles approved these changes Jul 23, 2025
View reviewed changes
Copy link
Contributor

@tommyp1ckles tommyp1ckles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@tommyp1ckles
Copy link
Contributor

/test

@squeed squeed added the release-note/misc This PR makes changes that have no direct user impact. label Jul 23, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jul 23, 2025
@joestringer joestringer added the needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch label Jul 29, 2025
@joestringer joestringer added this pull request to the merge queue Jul 29, 2025
@joestringer
Copy link
Member

Thanks! 🙏

Merged via the queue into cilium:main with commit 8d1207d Jul 29, 2025
68 checks passed
@rastislavs rastislavs mentioned this pull request Jul 31, 2025
Merged
14 tasks
@rastislavs rastislavs added backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. and removed needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Jul 31, 2025
@github-actions github-actions bot added backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. and removed backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. labels Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@squeed squeed squeed approved these changes

@tommyp1ckles tommyp1ckles tommyp1ckles approved these changes

Assignees
No one assigned
Labels
backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. kind/community-contribution This was a contribution made by a community member. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@alagoutte @tommyp1ckles @joestringer @squeed @rastislavs