Martin Kennelly

Do we want a stable tag?

4

Now that #348 is merged.. ## What would you like to be added? Add ```stable-${major version}-${arch}``` where arch is amd64, arm64 and ppc64le. ## How? We need to change deployments...

Can we add an explicitly named amd64 release tag?

5

Now that #348 is merged. ## What would you like to be added? Add ```${GitHub Tag}-amd64``` to ensure consistency in our current release tags and aid readability in customer yamls....

Build, test and static analysis for each PR/push and time period using GitHub actions

1

Hey folks, I think the experiment for NRI seems to work fine. I propose adding build, test and static analysis GitHub actions for each PR / push and also build...

Live migration: dont rollback within the same test as migration

11

After merging the code and applying the jobs for migration and rollback, we noticed that monitors have a toleration for disruption. This toleration was configured for upgrade. For our job,...

When I try a replace via iproute2, it works as expected. I can replace fields that arent the address like for example preferred_lft and valid_lft.

Any reason why? Seems like a bug. ``` func (r Route) Equal(x Route) bool { return r.LinkIndex == x.LinkIndex && r.ILinkIndex == x.ILinkIndex && r.Scope == x.Scope && ipNetEqual(r.Dst, x.Dst)...

Flake: ANP: `AdminNetworkPolicyIngressSCTP`

6

Test: ``` 2024-02-07T13:29:53.3214245Z --- FAIL: TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressSCTP/Should_support_an_'deny-ingress'_policy_for_SCTP_protocol;_ensure_rule_ordering_is_respected (0.19s) ``` https://github.com/ovn-org/ovn-kubernetes/actions/runs/7814875289/job/21318044260?pr=4061 ``` === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressSCTP/Should_support_an_'allow-ingress'_policy_for_SCTP_protocol;_ensure_rule_ordering_is_respected 2024-02-07T13:26:29.9926003Z === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressSCTP/Should_support_an_'allow-ingress'_policy_for_SCTP_protocol_at_the_specified_port 2024-02-07T13:26:33.1522114Z === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressSCTP/Should_support_an_'deny-ingress'_policy_for_SCTP_protocol;_ensure_rule_ordering_is_respected 2024-02-07T13:26:33.2574606Z admin-network-policy-core-ingress-sctp-rules.go:118: FAILED Command was [/agnhost connect --timeout=3s --protocol=sctp...

Seen here: https://github.com/ovn-org/ovn-kubernetes/actions/runs/8647390432/job/23709942317?pr=4271 https://github.com/ovn-org/ovn-kubernetes/pull/4271 ``` 2024-04-11T14:05:56.6845475Z admin-network-policy-core-egress-tcp-rules.go:118: FAILED Command was [/agnhost connect --timeout=3s --protocol=tcp 10.244.2.148:80] 2024-04-11T14:05:56.6848218Z admin-network-policy-core-egress-tcp-rules.go:118: Expected connection to fail from network-policy-conformance-gryffindor/harry-potter-0 to 10.244.2.148, but instead it successfully connected....

``` 2024-03-12T15:26:10.8723587Z === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressUDP 2024-03-12T15:26:10.8728131Z suite.go:187: Applying base/admin_network_policy/core-ingress-udp-rules.yaml 2024-03-12T15:26:10.8839261Z apply.go:124: Creating ingress-udp AdminNetworkPolicy 2024-03-12T15:26:10.8893134Z === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressUDP/Should_support_an_'allow-ingress'_policy_for_UDP_protocol;_ensure_rule_ordering_is_respected 2024-03-12T15:26:11.0462320Z === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressUDP/Should_support_an_'allow-ingress'_policy_for_UDP_protocol_at_the_specified_port 2024-03-12T15:26:14.1921073Z === RUN TestNetworkPolicyV2Conformance/AdminNetworkPolicyIngressUDP/Should_support_an_'deny-ingress'_policy_for_UDP_protocol;_ensure_rule_ordering_is_respected 2024-03-12T15:26:14.2764159Z admin-network-policy-core-ingress-udp-rules.go:118: FAILED Command...

Today, we do not remove `chassis_template_var` variables that arent being used. Steps to reproduce: 1. Add IP to breth0 2. Delete IP from breth0 3. `ovn-nbctl list chassis_template_var` and see...