Skip to content
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

Make svc bypass code dual-stack aware #470

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Make svc bypass code dual-stack aware #470

merged 1 commit into from
Oct 10, 2023

Conversation

mchalla
Copy link
Contributor

@mchalla mchalla commented Sep 26, 2023

It was added long back for some openstack customer and only handled V4. This was causing a mess up
of dual-stack service flows. Fixed that.

It was added long back for some openstack customer
and only handled V4. This was causing a mess up
of dual-stack service flows. Fixed that.

Signed-off-by: Madhu Challa <challa@gmail.com>
@mchalla
Copy link
Contributor Author

mchalla commented Sep 26, 2023

Pl do not merge. Sumit is doing a new release this Thursday and wants to put minimal changes in it.

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 68.89% (-10.4%) from 79.267% when pulling 3028684 on dual-stack into b676dd9 on master.

Copy link
Contributor

@shastrinator shastrinator left a comment

Choose a reason for hiding this comment

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

LGTM

@mchalla mchalla merged commit c3b2ec8 into master Oct 10, 2023
5 checks passed
@mchalla mchalla deleted the dual-stack branch October 10, 2023 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants