Skip to content

Bump neutron to fix PID clean up #1485

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
Feb 3, 2025
Merged

Bump neutron to fix PID clean up #1485

merged 1 commit into from
Feb 3, 2025

Conversation

priteau
Copy link
Member

@priteau priteau commented Feb 3, 2025

No description provided.

@priteau priteau self-assigned this Feb 3, 2025
@priteau priteau requested a review from a team as a code owner February 3, 2025 10:27
@Alex-Welsh
Copy link
Member

Does this affect OVN, OVS, or both?

@priteau
Copy link
Member Author

priteau commented Feb 3, 2025

Does this affect OVN, OVS, or both?

This is just for ML2/OVS.

Alex-Welsh
Alex-Welsh approved these changes Feb 3, 2025
View reviewed changes
Copy link
Member

@Alex-Welsh Alex-Welsh left a comment

Choose a reason for hiding this comment

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

Multinode Rocky OVS deployed just fine: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/13112438238/job/36579084813#step:1:19

If you think this is a dangerous change we can try more combinations (i.e. minor upgrade to these containers, or Ubuntu deployments) but otherwise lgtm

@priteau
Copy link
Member Author

priteau commented Feb 3, 2025

Multinode Rocky OVS deployed just fine: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/13112438238/job/36579084813#step:1:19

If you think this is a dangerous change we can try more combinations (i.e. minor upgrade to these containers, or Ubuntu deployments) but otherwise lgtm

I don't think it is dangerous, since stable/2023.1 is unmaintained there should be only few changes.

@priteau priteau merged commit 8947ef0 into stackhpc/2023.1 Feb 3, 2025
13 checks passed
@priteau priteau deleted the neutron-l3-keepalived-fix branch February 3, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alex-Welsh Alex-Welsh Alex-Welsh approved these changes

Assignees

@priteau priteau

Labels
kolla size: s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@priteau @Alex-Welsh @seunghun1ee