| ianw | hi, could i get some eyes on https://review.opendev.org/c/openstack/openstack-ansible/+/803404 and 803405 to remove debian stable from openstack-ansible on train/stein | 00:31 |
|---|---|---|
| ianw | one complication is that the train gate seems to have issues with pep8 currently | 00:31 |
| opendevreview | Merged openstack/openstack-ansible master: set non-voting for broken tox-doc test https://review.opendev.org/c/openstack/openstack-ansible/+/803371 | 01:18 |
| *** rpittau|afk is now known as rpittau | 07:23 | |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-rabbitmq_server master: Fix erlang upgrades for rabbitmq external_repo installs https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/803711 | 08:55 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible master: haproxy: decrease check interval for letsencrypt back ends https://review.opendev.org/c/openstack/openstack-ansible/+/802716 | 09:14 |
| *** rpittau is now known as rpittau|afk | 14:43 | |
| spatel | anskiy hey! i am submitting patch to fix OVN for CentOS-8-Stream | 14:51 |
| spatel | if you want to give it a try | 14:52 |
| spatel | jrosser how to break nested loop in CI job, because job A depend on B and B depend on A :( | 14:58 |
| anskiy | spatel: for ovn-central? Sure! I can test it on a weekend. | 14:58 |
| spatel | yes ovn-central bug.. i tested and it working for me | 14:59 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Change OVN metadata protocol to https https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802135 | 15:15 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Set openvswitch version for centos 8.4 https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802402 | 15:19 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Set openvswitch version for centos 8.4 https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802402 | 15:24 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Add CentOS-8-Stream OVN support https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/803798 | 18:04 |
| spatel | anskiy try this patch and let me know if any issue with c8-stream - https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/803798 | 18:05 |
| spatel | I have tested for metal and it works. also going to try with lxc and see how it goes. | 18:05 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Add CentOS-8-Stream OVN support https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/803798 | 18:07 |
| *** odyssey4me is now known as Guest3489 | 19:03 | |
| jrosser | spatel: to break a loop either set the job to NV, merge one then set it back to voting, merge the second - you should be able to stack all those patches up together | 19:15 |
| jrosser | alternatively combine some of the changes together | 19:15 |
| jrosser | if they are in the same repo | 19:15 |
| spatel | jrosser hey! | 19:36 |
| spatel | That is what i am thinking to create big patch for os-neutron | 19:37 |
| jrosser | a list of them surrounded by NV is good though to preserve the intent/commit history | 19:38 |
| spatel | could you bump this up because this one is causing more issue - https://review.opendev.org/c/openstack/openstack-ansible/+/803041 | 19:39 |
| spatel | This is easy and safe merge | 19:39 |
| jrosser | it’s my own patch :/ | 19:40 |
| jrosser | need to prod mgariepy / spotz_ for that I guess | 19:41 |
| spatel | Oh! okay i thought you can vote on your patch :) | 19:42 |
| jrosser | nope | 19:42 |
| jrosser | if it’s a backport then yes | 19:42 |
| spatel | ok.. | 19:44 |
| spatel | jrosser how do i remove or set NV on centos-8.4 so this patch can get merge - openstack-ansible-deploy-aio_ovs_lxc-centos-8 | 19:45 |
| spatel | https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802134 | 19:45 |
| spatel | It has 8.4 dependency and that is failing | 19:46 |
| spatel | jrosser can i remove or set non-voting of all centos-8 job? https://opendev.org/openstack/openstack-ansible/src/branch/master/zuul.d/project-templates.yaml | 19:47 |
| spatel | oh wait i can set NV in os-neutron/zuul.d also right? or not | 19:48 |
| jrosser | yes, make a patch which does only that | 19:49 |
| jrosser | then rebase the others onto the top of that | 19:49 |
| jrosser | then make a revert of the NV patch which goes last | 19:49 |
| spatel | why don't i just set NV here https://review.opendev.org/plugins/gitiles/openstack/openstack-ansible-os_neutron/+/refs/heads/master/zuul.d/project.yaml | 19:49 |
| jrosser | I think that’s what you mean before? | 19:50 |
| spatel | couple of job failing because of centos-8 OVN failing and i can't add them dependency (because of loop). If i set NV here then it should work right? https://review.opendev.org/plugins/gitiles/openstack/openstack-ansible-os_neutron/+/refs/heads/master/zuul.d/project.yaml | 19:51 |
| spatel | OVN/OVS* | 19:51 |
| spatel | Anyway we are going to remove centos-8 so trying to not do any work there | 19:52 |
| jrosser | yes make a patch in os_neutron which sets the failing job to NV | 19:53 |
| jrosser | Rebase the rest onto it | 19:53 |
| jrosser | then create a revert of the first one to put the jobs back to voting | 19:54 |
| jrosser | use the rebase function in gerrit to stack them all up in order | 19:54 |
| spatel | jrosser i did that here, remove all centos-8 and replace with stream - https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802701 | 19:54 |
| spatel | its not going to merge because dependency is failing | 19:55 |
| spatel | its kind of chicken egg here | 19:55 |
| jrosser | right, so like I say you need to make the failing job non voting? | 20:04 |
| jrosser | sorry I only have a mobile, no laptop I can’t look at any of this directly ina useful way | 20:05 |
| spatel | changing that in project.yaml | 20:05 |
| spatel | jrosser no worry, let me see what i can do | 20:05 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Change OVN metadata protocol to https https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802135 | 20:09 |
| spatel | done - https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802135/5/zuul.d/project.yaml | 20:10 |
| spotz_ | spatel jrosser - +2d. And generally I only vote on my own patches if someone patched on top. Or if I patched a spelling/grammar fix on someone else's larger patch | 20:11 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Set openvswitch version for centos 8.4 https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/802402 | 20:13 |
| spatel | spotz_ no worry! lets wait for others to chime in, anyway its Friday | 20:19 |
| *** spotz_ is now known as spotz | 20:32 | |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Add CentOS-8-Stream OVN support https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/803798 | 20:41 |
| opendevreview | Satish Patel proposed openstack/openstack-ansible-os_neutron master: Add CentOS-8-Stream OVN support https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/803798 | 21:23 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!