Bug 2163371
| Summary: | [OVN] Do not send geneve UDP traffic to conntrack for performance reasons | ||
|---|---|---|---|
| Product: | Red Hat OpenStack | Reporter: | Luigi Tamagnone <ltamagno> |
| Component: | openstack-tripleo-heat-templates | Assignee: | Miguel Lavalle <mlavalle> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Maor <mblue> |
| Severity: | high | Docs Contact: | |
| Priority: | medium | ||
| Version: | 16.2 (Train) | CC: | bcafarel, dalvarez, egarciar, jiehuang, jlibosva, jschluet, lsvaty, ltamagno, mblue, mburns, mlavalle, mtomaska, nlevinki, pgrist, tvignaud |
| Target Milestone: | z6 | Keywords: | Regression, Triaged |
| Target Release: | 16.2 (Train on RHEL 8.4) | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | openstack-tripleo-heat-templates-11.6.1-2.20230808225213.9adcac6.el8ost | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1851467 | Environment: | |
| Last Closed: | 2023-11-10 11:32:01 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1851467 | ||
| Bug Blocks: | |||
|
Comment 2
Luigi Tamagnone
2023-01-23 09:55:53 UTC
Hi Luigi, Thanks for looking at this. Left you a comment in the upstream patch: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/868743 Hi Miguel, I set your comment as resolved afaik we don't use puppet on Master for the firewall rules. So the code changed and state=[] is working on master but not in Train. Hi Luigi, Thank you for your response. I think you forgot to mark my comments as resolved in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/868743. Would you please do so, explaining the reason you think this is the right approach? I think the reason has to do with the fact that in Train we use puppet and we have a module that initializes state with NEW. Once you do this, I'll remove -1. The point is to document the conversation that took place during the review process. Thanks! Hi Miguel, My fault, I wrote the comment but I didn't push reply. So it was in draft... Hi Luigi, Thank you very much. I +1ed the upstream patch. Are you going to do the downstream backport? Hi Miquel, I can't I don't have the credential to work downstream. Hi Luigi, In that case, I'll monitor the upstream patch and when merged, I'll backport it downstream to the 16.2 branch Cheers Hi Miguel, The patch is available upstream. Hi Luigi, I tried to manually backport the upstream patch to branch rhos-16.2-trunk-patches, but I was getting an empty commit. Digging a little bit, I found that it was automatically backported. Please see the list of recent commits here: https://code.engineering.redhat.com/gerrit/gitweb?p=openstack-tripleo-heat-templates.git;a=shortlog;h=refs/heads/rhos-16.2-trunk-patches. This is your backported commit: https://code.engineering.redhat.com/gerrit/gitweb?p=openstack-tripleo-heat-templates.git;a=commitdiff;h=4a27d7c628cfe1f719278303c327dcae7c27adaa Checking further, I can see that your changes are in the tree: https://code.engineering.redhat.com/gerrit/gitweb?p=openstack-tripleo-heat-templates.git;a=blob;f=deployment/ovn/ovn-controller-container-puppet.yaml;h=fa065b034b89afb609054760df1f2c1d6e58cf22;hb=refs/heads/rhos-16.2-trunk-patches#l236 https://code.engineering.redhat.com/gerrit/gitweb?p=openstack-tripleo-heat-templates.git;a=blob;f=deployment/ovn/ovn-controller-container-puppet.yaml;h=fa065b034b89afb609054760df1f2c1d6e58cf22;hb=refs/heads/rhos-16.2-trunk-patches#l244 Is this what you wanted to accomplish? @ltamagno Raising severity due to the Regression keyword According to our records, this should be resolved by openstack-tripleo-heat-templates-11.6.1-2.20230808225216.el8ost. This build is available now. The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days |