Monday, 2022-08-08

opendevreviewBrendan Shephard proposed openstack/python-tripleoclient master: Add timeout arg to node delete commands  https://review.opendev.org/c/openstack/python-tripleoclient/+/85230600:10
*** rlandy_ is now known as rlandy|out00:43
opendevreviewMerged openstack/python-tripleoclient master: Return Undercloud role instead of Controller  https://review.opendev.org/c/openstack/python-tripleoclient/+/84960102:16
*** pojadhav is now known as pojadhav|ruck02:36
opendevreviewMerged openstack/python-tripleoclient master: Remove leftover of minion  https://review.opendev.org/c/openstack/python-tripleoclient/+/84347502:59
*** lbragstad8 is now known as lbragstad04:48
*** raukadah is now known as chandankumar04:51
opendevreviewchandan kumar proposed openstack/tripleo-common master: Add an Ansible execution environment to our Build  https://review.opendev.org/c/openstack/tripleo-common/+/77667405:07
opendevreviewRabi Mishra proposed openstack/tripleo-heat-templates stable/train: [train-only] Don't update hash when using paunch  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85237405:11
opendevreviewchandan kumar proposed openstack/tripleo-common master: [DNM] Build tripleo-ansible-ee container  https://review.opendev.org/c/openstack/tripleo-common/+/85045805:12
*** marios is now known as marios|ruck05:12
*** pojadhav|ruck is now known as pojadhav|rover05:26
opendevreviewchandan kumar proposed openstack/tripleo-ci master: Move tripleo-tox-molecule job definition to base-upstream.yaml  https://review.opendev.org/c/openstack/tripleo-ci/+/84985605:38
*** ysandeep|PTO is now known as ysandeep05:38
opendevreviewchandan kumar proposed openstack/tripleo-ci master: Move tripleo-tox-molecule job definition to base-upstream.yaml  https://review.opendev.org/c/openstack/tripleo-ci/+/84985605:38
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove stdout logging support (8): Nova  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84586005:42
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd  https://review.opendev.org/c/openstack/tripleo-ansible/+/84875805:57
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove stdout logging support (9): Rsyslog sidecar  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84586106:19
opendevreviewRabi Mishra proposed openstack/tripleo-heat-templates stable/train: [train-only] Don't update hash when using paunch  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85237406:25
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: Add cell export to overcloud deploy  https://review.opendev.org/c/openstack/python-tripleoclient/+/85185306:29
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates stable/wallaby: Fix unit test discovery  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85215906:30
opendevreviewMerged openstack/tripleo-heat-templates master: Ceilometer: Remove snmpd_* parameters  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84649306:36
opendevreviewCedric Jeanneret proposed openstack/ansible-role-collect-logs master: Change the output format for journalctl  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/85237506:41
opendevreviewSandeep Yadav proposed openstack/tripleo-quickstart-extras master: Add custom CI rules to the Undercloud Firewall  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85216006:47
opendevreviewSandeep Yadav proposed openstack/tripleo-quickstart-extras master: Add custom CI rules to the Undercloud Firewall  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85216006:50
*** ysandeep is now known as ysandeep|afk06:52
*** amoralej|off is now known as amoralej07:01
opendevreviewTakashi Kajinami proposed openstack/tripleo-ci master: Collect config files of audutd  https://review.opendev.org/c/openstack/tripleo-ci/+/85208107:13
opendevreviewTakashi Kajinami proposed openstack/tripleo-ci master: Collect config files of auditd  https://review.opendev.org/c/openstack/tripleo-ci/+/85208107:13
Tenguhjensas_: heya! quick question: while working on porting tripleo to nftables, we detected a connection attempt from the OC nodes to the UC. "weird" thing: the SPT (source port) was always the same: 9999. After some searches, it seems to be related to the ironic-python-agent. Would this make any sens? weird thing is, the attempt is actually an RST, aka RESET07:16
Tenguhjensas_: if it's really the ironic agent, would it make sens to edit the provisionning role in order to manage specific rules to allow this connection (and, if needed, some more) ?07:16
Tengunote that nftables has a terrible way to manage things when it comes to "remove a rule". So I'd rather be 100% sure about the actual need.07:17
Tengu(and when I say terrible: https://wiki.nftables.org/wiki-nftables/index.php/Simple_rule_management#Removing_rules )07:22
*** jpena|off is now known as jpena07:35
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: Remove migration from undercloud-passwords.conf  https://review.opendev.org/c/openstack/python-tripleoclient/+/85237607:42
opendevreviewKe Niu proposed openstack/tripleo-puppet-elements master: remove unicode prefix from code  https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/85236107:42
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates master: DNM - QUINCY - Test Pending Ceph daemons in CI  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/77891507:44
opendevreviewKe Niu proposed openstack/tripleo-puppet-elements master: remove unicode prefix from code  https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/85236107:45
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: tripleo deoloy: Use Standalone role by default  https://review.opendev.org/c/openstack/python-tripleoclient/+/85237907:53
*** undefined is now known as Guest767407:59
opendevreviewJiri Podivin proposed openstack/validations-common master: Adding rsyslog check in general validation tests  https://review.opendev.org/c/openstack/validations-common/+/85238208:16
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: Remove migration from undercloud-passwords.conf  https://review.opendev.org/c/openstack/python-tripleoclient/+/85237608:24
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove stdout logging support (8): Nova  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84586008:27
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove stdout logging support (9): Rsyslog sidecar  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84586108:27
Tengumarios|ruck: ack on the "firewall management not a blocker anymore, BUT" - added a comment/question on the LP.09:13
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238609:17
Tenguysandeep|afk: -^^  might be a good addition if we were to add temporary rules09:18
marios|ruckthanks Tengu will check 09:19
opendevreviewJiri Podivin proposed openstack/validations-common master: Changing filter pattern on setup module call  https://review.opendev.org/c/openstack/validations-common/+/85238209:30
*** ysandeep|afk is now known as ysandeep09:33
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: tripleo deoloy: Use Standalone role by default  https://review.opendev.org/c/openstack/python-tripleoclient/+/85237909:48
opendevreviewMilana Levy proposed openstack/tripleo-validations master: Added a new tripleo validation  https://review.opendev.org/c/openstack/tripleo-validations/+/85238910:06
opendevreviewVijayalakshmi proposed openstack/tripleo-ansible master: Restart tuned service to apply tuned boot options  https://review.opendev.org/c/openstack/tripleo-ansible/+/85173810:09
*** rlandy|out is now known as rlandy10:29
tkajinamTengu, chandankumar do you mind checking https://review.opendev.org/c/openstack/tripleo-ci/+/852081 ? I've removed depends-on so that we can merge it right now.10:32
opendevreviewRajesh Tailor proposed openstack/puppet-tripleo master: Fix typos in parameter descriptions, comments and tests  https://review.opendev.org/c/openstack/puppet-tripleo/+/84898810:53
chandankumartkajinam: done!10:54
Tengutkajinam: while at it, care to check https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/852375 ?11:05
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238611:07
tkajinamchandankumar Tengu thanks !11:10
tkajinamTengu, checking11:10
tkajinamTengu, +2ed11:12
Tenguthanks!11:12
Tenguchandankumar, marios|ruck care to have a quick check on https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/852375 ?11:13
Tenguthere's an example file in a comment to show the new output.11:14
ysandeepTengu, ^^ +wed11:14
Tenguysandeep: oh, thanks :)11:15
marios|ruckthanks Tengu you had enough votes but yeah lgtm 11:19
Tengumarios|ruck: :)11:19
opendevreviewMerged openstack/tripleo-validations master: Fix to check the result of undercloud_debug validation  https://review.opendev.org/c/openstack/tripleo-validations/+/84852911:27
Tengutkajinam, slagle, ramishra heya! any other concerns on https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 ? I think I addressed all the comments. Shall we see that one nudget to gate? :)11:33
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238611:35
*** dviroel_ is now known as dviroel11:38
slagleTengu: lgtm!11:42
slaglethanks for making those changes11:42
Tenguslagle: thanks :)11:42
Tengunp - if we can converge toward a naming convention, I'm all for it.11:43
TenguI overlooked that specific thing though.11:43
Tenguand I may rename the "service.yml" in the tripleo_nftables as well I guess.11:43
Tenguthough in that specific case, I don't really think it's valid.11:43
slaglei'd like to update the skeleton role with these conventions. i can do that11:44
Tenguslagle: might be a really good start for new roles, yeah11:44
slagleand just use yaml instead of yml. i dont think we need to support windows 95 3 letter file extensions constraints11:45
Tenguok :). Well, it was in the doc you pointed.11:45
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238611:45
slaglei don't remember which I put in there, but I'd prefer yaml honestly11:46
Tenguyml apparently, if I renamed the files11:46
Tenguoh. sorry. yaml11:46
Tenguok.11:46
Tenguyeah.... I use yml because it's shorter to type XD11:46
Tengubut I really has no opinion.11:47
Tengu*have11:47
hjensas_Tengu: Not sure about that port 9999 request, may want to send an e-mail to either upstream list or downstream (rhos-hardprov) list with bit more details? I can bring it up with the wider team.11:55
*** dhill is now known as Guest769611:55
Tenguhjensas: sure - I'm working on another thing, I'll gather the details shortly.11:56
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238612:05
opendevreviewTakashi Kajinami proposed openstack/python-tripleoclient master: tripleo deoloy: Use Standalone role by default  https://review.opendev.org/c/openstack/python-tripleoclient/+/85237912:21
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238612:28
*** Guest7674 is now known as rcastillo12:35
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238612:45
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238612:58
Tengubleh.12:58
*** amoralej is now known as amoralej|lunch13:01
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: Use the new tripleo_auditd ansible role instead of puppet  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85195813:03
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: [DNM] Enable auditd in standalone role  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85196913:04
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238613:18
opendevreviewchandan kumar proposed openstack/tripleo-ci master: [DNM] tcib_args  https://review.opendev.org/c/openstack/tripleo-ci/+/85073613:22
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238613:28
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238613:30
* Tengu needs some more coffee.13:30
*** pojadhav is now known as pojadhav|rover13:30
*** dasm|off is now known as dasm13:31
opendevreviewMerged openstack/tripleo-ci master: Collect config files of auditd  https://review.opendev.org/c/openstack/tripleo-ci/+/85208113:36
tkajinamMay I get 2nd review for https://review.opendev.org/c/openstack/puppet-tripleo/+/838955 ,  https://review.opendev.org/c/openstack/puppet-tripleo/+/839222 and  https://review.opendev.org/c/openstack/puppet-tripleo/+/839288 ?13:44
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Neutron: Replace hiera by lookup  https://review.opendev.org/c/openstack/puppet-tripleo/+/84734113:49
opendevreviewVanou Ishii proposed openstack/diskimage-builder master: Fix wrong yum.conf name of CentOS 9 Stream  https://review.opendev.org/c/openstack/diskimage-builder/+/85240813:53
*** amoralej|lunch is now known as amoralej13:59
*** hjensas is now known as hjensas|afk14:07
opendevreviewLukas Bezdicka proposed openstack/tripleo-heat-templates stable/wallaby: WIP: FFWD3: Add handling for OVN Multi-RHEL  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85243514:33
opendevreviewLukas Bezdicka proposed openstack/tripleo-heat-templates stable/wallaby: WIP: FFWD3: Allow for jinja2 templating of docker_config  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85243614:33
opendevreviewLukas Bezdicka proposed openstack/tripleo-common stable/wallaby: WIP: FFWD3: Add multi-rhel support for OvnController  https://review.opendev.org/c/openstack/tripleo-common/+/85243714:35
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Remove ctlplane-subnet from example file.  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85230514:45
opendevreviewMerged openstack/ansible-role-collect-logs master: Change the output format for journalctl  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/85237514:45
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Simplify the pattern to exclude puppet header from cron files  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85243914:48
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Simplify the pattern to exclude puppet header from cron files  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85243914:49
*** marios|ruck is now known as marios|ruck|call14:49
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238614:50
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove reference to OpenDaylight  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85244014:52
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove reference to Contrail  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85244114:52
*** marios|ruck|call is now known as marios|ruck15:15
*** amoralej is now known as amoralej|off15:15
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment  https://review.opendev.org/c/openstack/tripleo-ansible/+/85238615:18
opendevreviewMerged openstack/tripleo-ansible master: New role, replacing puppet-auditd  https://review.opendev.org/c/openstack/tripleo-ansible/+/84875815:19
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Drop incomplete implementation to deploy netcontrold  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85244415:19
opendevreviewRafael Castillo proposed openstack/tripleo-ci master: Add role to run virt jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/85160515:26
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Enable post-copy by setting unprivileged_userfaultfd  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85244615:28
*** dviroel is now known as dviroel|lunch15:29
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove resource template to deploy ovndb at host  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85244715:29
*** ysandeep is now known as ysandeep|out15:30
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Remove manifest to deploy OVNDB at host  https://review.opendev.org/c/openstack/puppet-tripleo/+/85244915:31
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove resource template to deploy OVNDB at host  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85244715:33
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Remove manifest to deploy OVNDB at host  https://review.opendev.org/c/openstack/puppet-tripleo/+/85244915:33
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Neutron: Replace hiera by lookup  https://review.opendev.org/c/openstack/puppet-tripleo/+/84734115:39
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Neutron: Replace hiera by lookup  https://review.opendev.org/c/openstack/puppet-tripleo/+/84734115:40
*** marios|ruck is now known as marios|out15:55
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Bind mount swtpm logs in libvirt virtqemud container  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85242015:57
opendevreviewMerged openstack/tripleo-quickstart-extras master: Add custom CI rules to the Undercloud Firewall  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85216016:11
*** jpena is now known as jpena|off16:34
*** dviroel|lunch is now known as dviroel16:38
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Do not override CephClientConfigVars in standalone scenarios  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85004717:23
opendevreviewMerged openstack/tripleo-heat-templates master: Fix TRIPLEO_CONFIG_HASH generation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85224217:23
*** tosky_ is now known as tosky19:25
*** undefined is now known as Guest773319:30
*** Guest7733 is now known as rcastillo19:31
*** rlandy is now known as rlandy|brb20:09
*** rlandy|brb is now known as rlandy20:55
*** dviroel is now known as dviroel|out21:22
*** dasm is now known as dasm|off22:03
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Fix libvirt-guests handling post virt-guest-shutdown  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84315922:21
*** rlandy is now known as rlandy|bbl22:24
opendevreviewBrendan Shephard proposed openstack/tripleo-ansible master: Add standalone ovn-controller role  https://review.opendev.org/c/openstack/tripleo-ansible/+/84565323:52

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