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.

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

OVN bug fixes backports (stable-5.21) #14966

Merged
merged 11 commits into from
Feb 12, 2025
Merged

OVN bug fixes backports (stable-5.21) #14966

merged 11 commits into from
Feb 12, 2025
+101 −47

Conversation

tomponline
Copy link
Member

escabo and others added 9 commits February 12, 2025 12:23
@escabo @tomponline
Modified ChassisGroupChassisDelete to convert chassis_name to _uuid t…
64fc2c1 
…o fix later comparison

Signed-off-by: Eric Gelinas <[email protected]>

better implementation for parsing a74125a8-b580-4763-b389-11ce2c8c5509,node2

Signed-off-by: Eric Gelinas <[email protected]>
(cherry picked from commit 0ece95e)
@tomponline
lxd/network/openvswitch/ovn: Fix escaping of values in ChassisGroupCh…
3af3719 
…assisDelete

Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 104e893)
@tomponline
test: Remove unnecessary sudo from ovn tests
5ba8b1c 
Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 664f10b)
@tomponline
test: Check for ha_chassis removal on lxd shutdown
293728b 
Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 5dbecf2)
@tomponline
lxd/network/zone/reverse: move to shared/dnsutil/reverse
0c8de15 
To make available to other packages and to avoid duplicate constant definitions for PTR suffixes.

Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 31f1b0e)
@tomponline
lxd/network/zone/zone: Switch to dnsutil package's Reverse function
9c2d62f 
And use dnsutil.IsReverse function.

Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit b4fc2c2)
@tomponline
lxd/network/openvswitch/ovn: Add PTR record generation for instance N…
7f3da9f 
…IC IPs

Fixes canonical#1481

Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit f1bbfd9)
@tomponline
lxd/network/openvswitch/ovn: Ignore prealloc warnings for complex loop
0a7a5be 
Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 2d39250)
@tomponline
test: Add tests for ovn DNS
c36f338 
Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 7ef4b6e)
@tomponline tomponline self-assigned this Feb 12, 2025
escabo and others added 2 commits February 12, 2025 12:29
@escabo @tomponline
Linter fixes
7ab73df 
Signed-off-by: Eric Gelinas <[email protected]>

nolint instead of 10

Signed-off-by: Eric Gelinas <[email protected]>

empty lines

Signed-off-by: Eric Gelinas <[email protected]>
(cherry picked from commit d395708)
@tomponline
lxd/network/openvswitch/ovn: Don't initialise slices to empty capacity
b18f1c3 
Leave as default nil value.

Signed-off-by: Thomas Parrott <[email protected]>
(cherry picked from commit 6798a66)
@tomponline tomponline marked this pull request as ready for review February 12, 2025 12:46
@tomponline tomponline merged commit 0585483 into canonical:stable-5.21 Feb 12, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@escabo escabo escabo approved these changes

@MusicDin MusicDin Awaiting requested review from MusicDin

@markylaing markylaing Awaiting requested review from markylaing

Assignees

@tomponline tomponline

Labels
5.21 LTS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tomponline @escabo