Thursday, 2022-07-21

*** dviroel|afk is now known as dviroel|out00:37
*** rlandy is now known as rlandy|out01:09
opendevreviewMiro Tomaska proposed openstack/tripleo-heat-templates master: [ML2 OVN] Adding OVNDhcForBaremetalDisabled parameter  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85059401:24
opendevreviewBrendan Shephard proposed openstack/python-tripleoclient master: Use realpath for checking directories  https://review.opendev.org/c/openstack/python-tripleoclient/+/85059601:55
opendevreviewBrendan Shephard proposed openstack/python-tripleoclient master: Use realpath for checking directories  https://review.opendev.org/c/openstack/python-tripleoclient/+/85059601:55
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Manila: Remove support for Dell EMC VMAX backend  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85060103:16
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Manila: Remove implementation for Dell EMC VMAX backend  https://review.opendev.org/c/openstack/puppet-tripleo/+/85060203:18
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Manila: Remove support for Dell EMC VMAX backend  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85060103:19
*** undefined is now known as Guest559803:35
*** ysandeep|out is now known as ysandeep05:00
opendevreviewCedric Jeanneret proposed openstack/ansible-role-collect-logs master: Get a dedicated, well formatted file for dropped packets  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/85033106:40
opendevreviewCedric Jeanneret proposed openstack/ansible-role-collect-logs master: Get a dedicated, well formatted file for dropped packets  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/85033106:41
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: support tripleo_etcd ansible role  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84984407:34
*** ysandeep is now known as ysandeep|lunch07:35
opendevreviewDaniel Bengtsson proposed openstack/tripleo-upgrade master: Changed minversion in tox to 3.18.0.  https://review.opendev.org/c/openstack/tripleo-upgrade/+/83502809:10
opendevreviewMarios Andreou proposed openstack/tripleo-heat-templates stable/wallaby: [wallaby only] remove Rear from mixed-os ci environment file  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85061609:12
marioseasy one please needs votes when you have time https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850616 thank you 09:19
mariosjbadiapa: ^^ 09:19
opendevreviewCedric Jeanneret proposed openstack/ansible-role-collect-logs master: Get a dedicated, well formatted file for dropped packets  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/85033109:24
Tenguseriously.......09:24
* Tengu needs more coffee - and a COOLER place to work from -.-09:24
Tengumarios: the new log file should be working now -.-. pffff.09:24
Tenguinteresting though.09:25
TenguJul 21 08:13:50 undercloud.localdomain kernel: DROPPING: IN=lo OUT= MAC=00:00:00:00:00:00:00:00:00:00:00:00:08:00 SRC=127.0.0.1 DST=127.0.0.1 LEN=40 TOS=0x00 PREC=0x00 TTL=64 ID=0 DF PROTO=TCP SPT=42950 DPT=5001 SEQ=1659035174 ACK=0 WINDOW=0 RES=0x00 RST URGP=0 09:25
Tenguthat one doesn't seem to match the "lo" interface...09:26
Tenguand this may explain the issues I'm seeing in the testproject.09:26
* Tengu scratching his head09:26
Tenguespecially since «it's working in my lab»..... there's something fishy in that job09:27
jbadiapamarios, done...thx09:27
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Actually accept all on lo and ipv6-icmp  https://review.opendev.org/c/openstack/tripleo-ansible/+/85062009:33
Tenguthere are so many mistakes in the firewall config.... ;_;09:34
Tenguanyway.09:34
mariosthanks jbadiapa 09:39
mariosTengu: ack will revisit thank you09:39
mariosTengu: can you please vote when you have a sec @ https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850616 i'd like to merge as soon as zuul reports assuming its good09:40
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Allow ceilometer compute agent to access /run/libvirt  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84860409:58
*** ysandeep|lunch is now known as ysandeep09:59
opendevreviewchandan kumar proposed openstack/openstack-tempest-skiplist master: Skip LoadBalancerScenarioTest for octavia  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85062610:09
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Add new multinode-mixed-os job and required content provider changes  https://review.opendev.org/c/openstack/tripleo-ci/+/84158310:17
*** rlandy|out is now known as rlandy10:32
opendevreviewMerged openstack/openstack-tempest-skiplist master: Skip LoadBalancerScenarioTest for octavia  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85062610:49
*** ysandeep is now known as ysandeep|afk10:53
Tengumarios: ah, ysandeep|afk voted already :)11:01
*** dviroel|out is now known as dviroel11:24
mariosthanks Tengu 11:28
Tengumarios: the dedicated file is, at last, good. pfrrtt.11:33
Tengu(for dropped packets)11:33
Tenguuho. and... and apparently I corrected a thing with the fs035 deploy using nftables! at least, the UC deploy is running for a longer time without failure (cc ysandeep|afk )11:34
mariosTengu: great thanks please add the link on the review and can be merged (maybe add into the list for today @ https://hackmd.io/FGMoCiRfSNa8puA1BpTQ-Q?edit 11:36
mariosTengu: review time is much later today so they can check it then if not merged by then (like 1900 utc )11:37
Tengumarios: file is empty on the review itself, but I'll add the one from the other testproject once it's finished.11:37
mariosTengu: ack feel free to add supporting evidence/testproject etc into the review list (make it easy as possible for them to +2w ;O))11:38
mariosor on the review itself of course as you prefer 11:38
Tengumarios: adding to the hackmd11:39
Tengumarios: lemme know if there are enough data in the hackmd11:43
Tenguwow... did I really succeed in correcting the initial UC deploy issue?! also, why was it failing in that job, but never in the others, that's a good question. probably the ipv4/ipv6 mix, or related.11:44
mariosthanks tengu maybe add link with the file non empty if/when you have it i think they might have questions about that bit11:48
Tengumarios: waiting for it to exist :)11:49
Tenguit's in the "running testproject"11:49
TenguI have it locally, of course. but... that's local only.11:49
rlandyxek: hello ... https://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ipa-centos-9-molecule&skip=0 - this job keep failing 11:50
rlandyin the gate atm11:50
Tenguah. hm. UC deploy failed again. let's see - this time I should get the generated file.11:50
Tengumarios: ok, I have the generated file :). a pity though, because the UC failed to deploy again...11:56
xekrlandy, we fixed a major issue yesterday and I see it just succeeded 16 minutes ago on that patch12:09
xekalthough I still see it failing in random places12:09
rlandyyep - gate just died on it12:10
opendevreviewLukas Bezdicka proposed openstack/tripleo-heat-templates stable/wallaby: WIP: add tag for package setup  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85063512:16
*** amoralej is now known as amoralej|lunch12:20
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: support tripleo_etcd ansible role  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84984412:37
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: support tripleo_etcd ansible role  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84984412:42
*** abishop_ is now known as abishop12:45
*** dasm|off is now known as dasm|ruck13:05
*** Guest5598 is now known as rcastillo13:05
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Ansible_playbook to select ddp package for NFV node provision  https://review.opendev.org/c/openstack/tripleo-ansible/+/84842613:17
*** ysandeep|afk is now known as ysandeep13:20
*** amoralej|lunch is now known as amoralej13:28
Tenguysandeep: I modified the testproject, removing some of the config you pushed to make it build the containers, and the UC is deployed, it's starting now to deploy the OC. I therefore should get, at last, the needed infos related to "what is blocked by nftables when runninf fs035 job"13:37
ysandeepTengu, ack o/13:45
Tenguysandeep: o_____O it's passing!!!14:17
Tenguso the correction I made for another reason seems to be actually the thing we needed.... it's hitting tempest now :D14:18
Tenguysandeep: meaning: fs035 is apparently working, provided we get the 3 patches I've pushed in tripleo-ansible (actually, one, but since it's the last of the triplet... ;))14:18
ysandeepTengu, so this one is the real fix: https://review.opendev.org/c/openstack/tripleo-ansible/+/850620/1/tripleo_ansible/roles/tripleo_nftables/defaults/main.yml ?14:24
Tenguysandeep: yep.... *apparently*. Let's see if the job ends properly.14:25
Tengubut I wouldn't be THAT surprised actually.14:25
Tenguas said in the commit message, only the NEW were allowed, while I saw some RST and other status being dropped for lo and ipv6-dhcp.14:25
ysandeepIf its tempest we already passed the previous step so YAY14:25
Tenguso I thought, "heh, we're not doing what the rule description says, let's correct that"14:26
Tenguysandeep: #tooEasy !14:26
ysandeepTengu++ great catch! :D14:28
Tenguysandeep: correcting the issue while trying to debug it: #checked14:30
ysandeepFortune always favors the brave :D14:33
Tengutkajinam, bshephar, ramishra care to have a look at those 3 patches? https://review.opendev.org/q/topic:nftables%252Flogging   they are fixing issues while improving logging for future debugging :)14:34
Tenguysandeep: indeed! I apparently was brave enough to own this while jumping on nftables.14:34
Tengubtw - switching to nftables will also close "some" holes... 14:35
opendevreviewMerged openstack/puppet-tripleo master: Gnocchi: Clean up unused parameters  https://review.opendev.org/c/openstack/puppet-tripleo/+/84015814:36
opendevreviewMerged openstack/python-tripleoclient stable/wallaby: Generate inventory with default ansible_ssh_user  https://review.opendev.org/c/openstack/python-tripleoclient/+/85047114:36
opendevreviewMerged openstack/python-tripleoclient stable/wallaby: Stop using heat-admin as ansible_ssh_user  https://review.opendev.org/c/openstack/python-tripleoclient/+/85038914:36
opendevreviewyatin proposed openstack/tripleo-heat-templates master: Switch default ML2 plugin in Undercloud to OVN  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85015814:38
ysandeepTengu, do you plan to backport this to wallaby also?14:38
opendevreviewyatin proposed openstack/tripleo-heat-templates master: Switch default ML2 plugin in Undercloud to OVN  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85015814:39
Tenguysandeep: might be cool for 17.1 indeed.14:40
Tenguysandeep: I'll list the needed patches to get things working - iirc I started in wallaby already.14:40
ysandeep+1, I can test rest of the featuresets tomorrow and we can plan to switch the default soon.14:41
Tenguysandeep: perfect! note that I'll be on PTO next week, back on Aug 2nd14:42
ysandeepokay, we can switch the default when you are back then.(just incase something breaks)14:42
Tenguthat would be better, yes.14:42
Tenguah. nope. master only. wokay.14:43
Tenguthere will be a fair amount of backports then.14:44
Tengulet's prep them already.14:44
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: Supports nftables instead of iptables  https://review.opendev.org/c/openstack/tripleo-ansible/+/85065914:45
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: Ensure we don't duplicate lines  https://review.opendev.org/c/openstack/tripleo-ansible/+/85066014:45
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: Add missing "inet nat" table  https://review.opendev.org/c/openstack/tripleo-ansible/+/85066114:45
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: Better logging management for nftables rules  https://review.opendev.org/c/openstack/tripleo-ansible/+/85066214:45
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: Actually accept all on lo and ipv6-icmp  https://review.opendev.org/c/openstack/tripleo-ansible/+/85066314:45
Tenguysandeep: ouch.... there's also the masquerade via ansible that is needed.14:47
Tenguthat's a bit more annoying.14:47
Tenguhmmm... ysandeep nah. that's too much. There's also all the frontend work by tkajinam to import... it's a bad idea.14:51
Tenguthat's about 10+ patches. Even more with the tripleo-heat-templates. I'm not doing it.14:51
ysandeepokay so no cherry-pick to wallaby then14:51
Tenguyeah. Lemme abandon the one I started.14:52
ade_leeTengu, hey - can we get +2 on https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84915215:18
ade_leetkajinam, ramishra ^^15:19
*** dviroel_ is now known as dviroel15:26
*** rlandy is now known as rlandy|afk15:28
*** ysandeep is now known as ysandeep|out15:30
opendevreviewMerged openstack/python-tripleoclient stable/train: [train-only] Don't always generate keys for ssh  https://review.opendev.org/c/openstack/python-tripleoclient/+/85023615:30
*** marios is now known as marios|out15:40
Tenguade_lee: woops, sorry, already off and away from work laptop..15:44
*** amoralej is now known as amoralej|off15:57
opendevreviewMiro Tomaska proposed openstack/tripleo-heat-templates master: [ML2 OVN] Adding DisableOvnDhcpForBaremetalPorts parameter  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85059416:05
opendevreviewJaganathan Palanisamy proposed openstack/python-tripleoclient stable/train: Deprecate derive parameters  https://review.opendev.org/c/openstack/python-tripleoclient/+/85067416:25
opendevreviewTakashi Kajinami proposed openstack/tripleo-common master: Stop downgrading pacemaker  https://review.opendev.org/c/openstack/tripleo-common/+/85067617:14
opendevreviewTakashi Kajinami proposed openstack/tripleo-common master: Remove unused ironic-inspector-client  https://review.opendev.org/c/openstack/tripleo-common/+/85067917:22
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: Stop mocking workflow client  https://review.opendev.org/c/openstack/python-tripleoclient/+/85068017:29
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: [wallaby only] remove Rear from mixed-os ci environment file  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85061619:10
opendevreviewMerged openstack/tripleo-common stable/wallaby: Split off Ceph related container images  https://review.opendev.org/c/openstack/tripleo-common/+/84635619:11
opendevreviewMerged openstack/tripleo-validations master: Removing undercloud-heat-purge-deleted from validation groups  https://review.opendev.org/c/openstack/tripleo-validations/+/85049519:11
opendevreviewMerged openstack/tripleo-ansible master: Filter out nftables keys when using iptables  https://review.opendev.org/c/openstack/tripleo-ansible/+/85022119:11
opendevreviewMerged openstack/tripleo-quickstart-extras master: Adds toci playbooks and updates for new multinode mixed OS jobs  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/84176419:11
opendevreviewMerged openstack/puppet-tripleo master: Gnocchi: Replace hiera by lookup  https://review.opendev.org/c/openstack/puppet-tripleo/+/84015619:11
opendevreviewMerged openstack/tripleo-heat-templates master: Add credentials cache for novajoin user  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84915219:55
opendevreviewDouglas Viroel proposed openstack/tripleo-ci master: WIP - Add fips jobs for centos 9  https://review.opendev.org/c/openstack/tripleo-ci/+/82447920:35
*** dasm|ruck is now known as dasm|off21:04
opendevreviewDouglas Viroel proposed openstack/tripleo-ci master: Add more scenario jobs with fips enabled  https://review.opendev.org/c/openstack/tripleo-ci/+/85068921:05
*** dviroel is now known as dviroel|out21:07
*** abishop_ is now known as abishop22:30

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!