Skip to content

Fix typo L2_RESPONSER_MAP4_SIZE => L2_RESPONDER_ #39574

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
May 22, 2025
Merged

Fix typo L2_RESPONSER_MAP4_SIZE => L2_RESPONDER_ #39574

merged 1 commit into from
May 22, 2025

Conversation

msune
Copy link
Member

@msune msune commented May 16, 2025 •
edited
Loading

While rebasing L2 announce for v6 PR I realised there is a typo on the v4 responder map size MACRO. Probably there forever :).

@msune msune requested review from a team as code owners May 16, 2025 09:03
@msune msune requested review from ti-mo and gentoo-root May 16, 2025 09:03
@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 May 16, 2025
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label May 16, 2025
@msune
Fix typo L2_RESPONSER_MAP4_SIZE => L2_RESPONDER_
702ff78 
While rebasing L2 announce for v6 PR, I realised there is a typo on
the v4 responder map size MACRO. Probably there forever :).

Signed-off-by: Marc Suñé <marc.sune@isovalent.com>
@msune msune force-pushed the fix_responder_map_typo branch from 8443f4e to 702ff78 Compare May 16, 2025 09:04
@msune
Copy link
Member Author

msune commented May 20, 2025

Can someone hit merge? It's a simple typo fix (need to rebase). cc @ti-mo

@msune msune mentioned this pull request May 20, 2025
Merged
@msune
Copy link
Member Author

msune commented May 22, 2025

ping @joestringer @aanm

@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label May 22, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label May 22, 2025
@joestringer joestringer merged commit 9812651 into cilium:main May 22, 2025
52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gentoo-root gentoo-root gentoo-root approved these changes

@joestringer joestringer joestringer approved these changes

@ti-mo ti-mo Awaiting requested review from ti-mo ti-mo is a code owner automatically assigned from cilium/loader

Assignees
No one assigned
Labels
kind/community-contribution This was a contribution made by a community member. 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.
3 participants
@msune @gentoo-root @joestringer