Thursday, 2022-05-19

opendevreviewBrent Eagles proposed openstack/tripleo-ansible master: Designate: specify source address when using rndc  https://review.opendev.org/c/openstack/tripleo-ansible/+/84145600:09
*** rlandy|bbl is now known as rlandy|out00:45
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Octavia: Include separate networking class  https://review.opendev.org/c/openstack/puppet-tripleo/+/84244801:30
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Octavia: Include separate networking class  https://review.opendev.org/c/openstack/puppet-tripleo/+/84244801:32
opendevreviewMerged openstack/tripleo-docs master: Replace existing virt-customize usage with tripleo-mount-image  https://review.opendev.org/c/openstack/tripleo-docs/+/83625901:43
opendevreviewyatin proposed openstack/tripleo-quickstart master: Switch to c9-stream image in standalone libvirt env  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84235704:01
opendevreviewLewis Denny proposed openstack/tripleo-heat-templates master: Use tripleo_ovn standalone role for host_prep_tasks  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84204404:42
opendevreviewLewis Denny proposed openstack/tripleo-heat-templates master: Use tripleo_ovn standalone role for host_prep_tasks  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84204404:46
*** ykarel_ is now known as ykarel04:51
*** ysandeep|out is now known as ysandeep|rover04:52
*** pojadhav|afk is now known as pojadhav05:10
*** ysandeep|rover is now known as ysandeep|rover|brb05:51
*** ysandeep|rover|brb is now known as ysandeep|rover05:56
Tenguslagle: looks nice - just tox unhappy with the depends-on. Didn't check the t-h-t one yet06:01
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: Rename molecule jobs to centos-stream  https://review.opendev.org/c/openstack/tripleo-ansible/+/84203606:22
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: Rename molecule jobs to centos-stream  https://review.opendev.org/c/openstack/tripleo-ansible/+/84203606:26
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: Use cs9 image as base  https://review.opendev.org/c/openstack/tripleo-ansible/+/83968806:27
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: Use cs9 image as base  https://review.opendev.org/c/openstack/tripleo-ansible/+/83968806:31
lecris[m]dciabrin: About the deployment you've sent last time. I have tried to get your to work on the baremetal, but there were various networking issues, NTP server configurations etc. So I went through each settings on mine to configure to a working state as close to yours, but it still gets me pacemaker issues06:47
lecris[m]One thing to note is that I never see any containers in `podman container list`, and your `containers-prepare-parameter.yaml` is significantly different than the default one06:48
ysandeep|roverjm1, stevebaker[m] hjensas new blocking bug on master https://bugs.launchpad.net/tripleo/+bug/1974134 , I think https://review.opendev.org/c/openstack/tripleo-ansible/+/840262/ caused this.07:23
ysandeep|roverhjensas: As tomorrow is recharge day I am reverting ^^ for stevebaker[m] to properly fix this issue because this issue is blocking master integration line. 07:34
*** jpena|off is now known as jpena07:36
stevebaker[m]I'm working tomorrow,  but yeah revert07:39
opendevreviewSandeep Yadav proposed openstack/tripleo-ansible master: Revert "Use instance_info.display_name for existing hostname matching"  https://review.opendev.org/c/openstack/tripleo-ansible/+/84239907:45
ysandeep|roverstevebaker[m]: thanks https://review.opendev.org/c/openstack/tripleo-ansible/+/842399 07:45
ykarelysandeep|rover, may be can attempt fix in parallel07:50
ykarelseems just need to change node.uuid to node.id07:50
ykarelhttps://opendev.org/openstack/openstacksdk/src/branch/master/openstack/baremetal/v1/node.py#L13307:51
ykarelif it works can avoid revert, else merge revert07:51
ysandeep|roverykarel, at https://review.opendev.org/c/openstack/tripleo-ansible/+/840262/3/tripleo_ansible/ansible_plugins/module_utils/baremetal_deploy.py#426 and #42807:51
ykarelysandeep|rover, yes07:52
ysandeep|roverykarel, let me try the fix locally 07:53
ykarelack should be easy there07:54
ysandeep|roverissue reproduced locally, let try the fix now08:02
ysandeep|roverykarel++ passed that failing step08:09
opendevreviewMerged openstack/tripleo-ci master: Add keystone deployment files filter for ipa job  https://review.opendev.org/c/openstack/tripleo-ci/+/84235608:09
ysandeep|roverykarel, i think we also need changes in line#425 , https://review.opendev.org/c/openstack/tripleo-ansible/+/840262/3/tripleo_ansible/ansible_plugins/module_utils/baremetal_deploy.py#425 08:10
ysandeep|rover fields=['uuid', to  fields=['id',08:10
opendevreviewJiri Podivin proposed openstack/tripleo-quickstart-extras master: DNM Adjusting DRLN builds for alternative branch names  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/84227308:10
ykarelysandeep|rover, i think that is fine, you seeing issue with it?08:10
ysandeep|roverI tested after changing with  fields=['id' , let me test with uuid also08:11
ysandeep|roverIn first look these were looking like field to me https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/baremetal/v1/node.py#L100-L246 08:13
opendevreviewyatin proposed openstack/tripleo-quickstart master: Switch to c9-stream image in standalone libvirt env  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84235708:18
ysandeep|roverykarel, uuid worked as well08:19
ysandeep|roverfor #42508:20
ykarelysandeep|rover, ack08:21
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible master: bnr added cli-undercloud-backup-db molecule job  https://review.opendev.org/c/openstack/tripleo-ansible/+/83684508:55
lecris[m]bshephar: Thanks to dciabrin, we've found the issue is that my node hostnames were FQDN. Can we patch the `overcloud node provision` to check for bad naming of form `node.domain` so it's more idiot-proof?09:10
opendevreviewJuan Badia Payno proposed openstack/tripleo-ansible master: bnr added cli-undercloud-backup-db molecule job  https://review.opendev.org/c/openstack/tripleo-ansible/+/83684509:15
opendevreviewSandeep Yadav proposed openstack/tripleo-ansible master: Correcting attribute for 'Node' object  https://review.opendev.org/c/openstack/tripleo-ansible/+/84247309:24
ysandeep|roverstevebaker[m], hjensas ^^ https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 (worked locally) - please review if this looks okay09:26
ysandeep|roverthanks ykarel++ for the help09:26
opendevreviewSandeep Yadav proposed openstack/tripleo-ansible master: Correcting attribute for 'Node' object  https://review.opendev.org/c/openstack/tripleo-ansible/+/84247309:29
*** ysandeep|rover is now known as ysandeep|rover|lunch09:31
opendevreviewMarios Andreou proposed openstack/tripleo-quickstart master: WIP add new featureset66 for mixed deployment jobs  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84158209:36
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: WIP add new mixed mode nodeset and job  https://review.opendev.org/c/openstack/tripleo-ci/+/84158309:37
opendevreviewMerged openstack/puppet-tripleo stable/train: haproxy: Enable httplog for all http endpoints  https://review.opendev.org/c/openstack/puppet-tripleo/+/82844709:44
*** ysandeep|rover|lunch is now known as ysandeep|rover10:09
opendevreviewMerged openstack/paunch stable/train: Ensure systemd services are activated  https://review.opendev.org/c/openstack/paunch/+/83759110:14
opendevreviewyatin proposed openstack/tripleo-quickstart master: Switch to c9-stream image in standalone libvirt env  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84235710:19
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: Use cs9 image as base  https://review.opendev.org/c/openstack/tripleo-ansible/+/83968810:20
*** rlandy|out is now known as rlandy10:27
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: WIP add new mixed mode nodeset and job  https://review.opendev.org/c/openstack/tripleo-ci/+/84158310:47
opendevreviewDamien Ciabrini proposed openstack/puppet-tripleo master: Support mariabackup as a galera SST method  https://review.opendev.org/c/openstack/puppet-tripleo/+/83618910:50
lecris[m]Question, is it possible to mix BlockStorage and ObjectStorage on the same baremetal node? I do not see a role that does that11:00
lecris[m]Would that be CephAll? Does  it include Cinder Swift etc?11:04
bshepharlecris[m]: The default Controller role has both. You can create a custom role to include both if you wanted to.11:08
bshepharCinder: https://github.com/openstack/tripleo-heat-templates/blob/master/roles_data.yaml#L83  Swift: https://github.com/openstack/tripleo-heat-templates/blob/master/roles_data.yaml#L174-L17711:09
lecris[m]I have a node that will be pure storage, but I want to use it for both object and volume, and I don't want it to be a controller otherwise11:10
lecris[m]If I use CephAll, would that do the trick?11:11
bshepharYou would probably need to add the pure related services to it, but I guess it should have everything else you're after11:12
bshepharhttps://github.com/openstack/tripleo-heat-templates/blob/master/roles_data.yaml#L7911:13
lecris[m]Ok, so if I merge the two BlockStorage and ObjectStorage roles together, you reckon it would not be an issue?11:14
lecris[m]Like if I setup software raid as well on it11:14
bshepharAs long as you have all the cinder, swift and pure related services in the role, it should be fine.11:15
bshepharhttps://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/features/custom_roles.html11:15
lecris[m]Thanks for the advice11:16
bshepharnp11:20
opendevreviewJiri Podivin proposed openstack/validations-libs stable/1.6: [Stable-Only] Refactoring the stable/1.6 CI pipeline  https://review.opendev.org/c/openstack/validations-libs/+/84171311:22
*** dviroel|out is now known as dviroel11:26
*** rlandy is now known as rlandy|mtg11:26
*** ysandeep|rover is now known as ysandeep|rover|afk11:33
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger novnc_proxy key file data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84253011:35
opendevreviewMerged openstack/python-tripleoclient stable/train: Exposes the community validation init command  https://review.opendev.org/c/openstack/python-tripleoclient/+/82573611:46
opendevreviewPooja Jadhav proposed openstack/tripleo-quickstart master: DNM : Testing tempest tests for intergration/component jobs  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84253111:48
opendevreviewPooja Jadhav proposed openstack/tripleo-quickstart master: DNM : Testing tempest tests for intergration/component jobs  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84253111:57
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger novnc_proxy key file data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84253011:58
opendevreviewPooja Jadhav proposed openstack/tripleo-quickstart master: DNM : Testing tempest tests for intergration/component jobs  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84253111:59
*** rlandy|mtg is now known as rlandy12:02
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger novnc_proxy key file data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84253012:09
opendevreviewMerged openstack/tripleo-ci master: Ussuri tear down as EOL  https://review.opendev.org/c/openstack/tripleo-ci/+/83546012:10
opendevreviewMerged openstack/tripleo-upgrade stable/wallaby: Install pkgs required to apply workarounds  https://review.opendev.org/c/openstack/tripleo-upgrade/+/84016512:10
opendevreviewJiri Podivin proposed openstack/tripleo-ci master: Fixing typo in validations-* branch overrides  https://review.opendev.org/c/openstack/tripleo-ci/+/84176212:23
opendevreviewMerged openstack/tripleo-quickstart-extras master: Added comment to revert in a future  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/83403612:24
*** ysandeep|rover|afk is now known as ysandeep|rover12:26
ysandeep|rovercould someone please +w https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 12:27
fultonjysandeep|rover: done12:28
opendevreviewKhomesh Thakre proposed openstack/tripleo-heat-templates stable/wallaby: Skip 'ensure dnf modules' for CS9 on overcloud-upgrade  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84207512:29
ysandeep|roverfultonj: thanks :)12:29
fultonjhjensas: have you seen Property error: Networks.resources.NetworkIpVersionMap.properties.value: 'ip_version' ? https://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/ 12:37
jm1stevebaker[m]: hello, could you have a look at this patch please? :) https://review.opendev.org/c/openstack/tripleo-ci/+/84199212:38
hjensasfultonj: hm, not seen before. Wonder if it is the yaql here that fail? https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/network/deployed_networks.yaml#L48-L4912:44
hjensasfultonj: can you find out what was in CtlplaneNetworkAttributes ?12:44
hjensasfultonj: https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v1/tripleo_deploy.py12:45
hjensasfultonj: sorry, better with a line number https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v1/tripleo_deploy.py#L68712:45
hjensasfultonj: I think the issue that ip_version is not set in subnets for CtlplaneNetworkAttributes in tripleo_deploy.py.12:46
fultonjI'm not overriding CtlplaneNetworkAttributes so I assume it's using the default12:47
fultonjif "not set in subnets", should I not use subnets syntax in DeployedNetworkEnvironment ?12:48
hjensasfultonj: I think its a bug, or rather we never tried composable network with standalone before.12:53
hjensasfultonj: we set the CtlplaneNetworkAttributes in tripleo_deploy.py, it is missing the ip_version.12:53
fultonj# cat tripleoclient-hosts-portmaps.yaml | curl -F 'f:1=<-' ix.io 12:53
fultonjhttp://ix.io/3Y8Y12:53
fultonj^ shows what's in CtlplaneNetworkAttributes12:54
fultonjno ip_version12:54
fultonjcould i just not have ip_version in my deployed_network_env?12:54
hjensasfultonj: yes, so add 'ip_version': ip_nw.version after L649 in  https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v1/tripleo_deploy.py#L64912:54
fultonjso it might be an easy fix, i'll try it12:55
fultonjthx12:55
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger novnc_proxy key file data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84253012:55
hjensasfultonj: add me on review if you propose a patch to tripleoclient12:55
odyssey4meslagle is +w missing from https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839789 on purpose to hold it back for some reason?12:56
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Add ip_version to CtlplaneNetworkAttributes  https://review.opendev.org/c/openstack/python-tripleoclient/+/84253713:03
fultonjhjensas: is this what you meant ^ ?13:03
fultonjI installed the client with the patch and re-ran. i saw tripleo-heat-installer-templates was recreated but tripleoclient-hosts-portmaps.yaml doesn't have the ip_version in CtlplaneNetworkAttributes 13:05
fultonjI think I need to pass the new ip_version variable  tmp_env.update13:10
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Add ip_version to CtlplaneNetworkAttributes  https://review.opendev.org/c/openstack/python-tripleoclient/+/84253713:15
fultonjhjensas: I seem to be passed that error now using PS2 of ^ thanks13:15
fultonjand now it fails with https://paste.opendev.org/show/bU7bszlLsFBsVD1X18LP/ 13:19
fultonj08:53 <hjensas> fultonj: I think its a bug, or rather we never tried composable network with standalone before.13:20
fultonj"we never tried composable network with standalone before"13:20
* fultonj was hoping to get ceph running without adding composalbe networks to standalone13:21
*** dasm|off is now known as dasm13:21
opendevreviewBrent Eagles proposed openstack/tripleo-ansible master: Designate: use extra IP for bind/unbound colocation on the bind9 backend  https://review.opendev.org/c/openstack/tripleo-ansible/+/83997313:21
opendevreviewBrent Eagles proposed openstack/tripleo-ansible master: Designate: remove unnecessary transitional code for moving extra IP  https://review.opendev.org/c/openstack/tripleo-ansible/+/84045113:21
hjensasfultonj: yes, patch exactly what I was thinking. (I +1, I think there is a couple unit tests that need to be tweaked as well.)13:29
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger novnc_proxy key file data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84253013:29
fultonjok, thanks hjensas 13:30
hjensasfultonj: hm, StandaloneServiceChain -  Error in 19 output role_data: The values of the "for_each" argument to "repeat" must be lists or maps13:31
hjensasfultonj: Error in 19 - I think we need to find out what service "standalone.StandaloneServiceChain.ServiceChain.19" is? And look at the service template.13:35
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: WIP Supports nftables instead of iptables  https://review.opendev.org/c/openstack/tripleo-ansible/+/84141413:35
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: WIP Add new parameter in order to switch firewall engine  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84144513:40
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger QEMU key file data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84090413:44
opendevreviewBogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger novnc_proxy key files data race  https://review.opendev.org/c/openstack/puppet-tripleo/+/84253013:44
fultonjhjensas: $ cat overcloud.yaml | curl -F 'f:1=<-' ix.io 13:51
fultonjhttp://ix.io/3Y9i13:51
fultonjcontains my StandaloneServiceChain 13:51
opendevreviewCedric Jeanneret proposed openstack/python-tripleoclient master: WIP Add a new parameter for the undercloud.conf  https://review.opendev.org/c/openstack/python-tripleoclient/+/84254613:55
fultonjI assume this line did it: "value: {get_attr: [StandaloneServiceChain, role_data]}"13:55
fultonji'm using -r https://github.com/openstack/tripleo-heat-templates/blob/master/roles/Standalone.yaml 13:57
opendevreviewMerged openstack/tripleo-quickstart-extras master: Use timezone UTC on undercloud  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/84221814:11
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: WIP Supports nftables instead of iptables  https://review.opendev.org/c/openstack/tripleo-ansible/+/84141414:17
opendevreviewJiri Podivin proposed openstack/validations-libs stable/1.6: [Stable-Only] Refactoring the stable/1.6 CI pipeline  https://review.opendev.org/c/openstack/validations-libs/+/84171314:17
opendevreviewJiri Podivin proposed openstack/validations-common stable/1.6: [Stable-Only] Refactoring validations-common stable branch CI pipeliene  https://review.opendev.org/c/openstack/validations-common/+/84204814:18
bogdandohjensas: hi, do you have ideas how to case that https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-modular-libvirt-container-puppet.yaml#L772-L781 into ansible vars ?14:19
bogdandoIIRC I saw something similar in tripleo-ansible to extract things from ServiceNetMap... can't recall what and how14:20
bogdandos/case/cast* :)14:20
bogdandoperhaps I can use a group_var for it, just passing that value from t-h-t14:21
bogdandoslagle: ^^14:21
opendevreviewJiri Podivin proposed openstack/validations-common stable/1.6: [Stable-Only] Refactoring validations-common stable branch CI pipeliene  https://review.opendev.org/c/openstack/validations-common/+/84204814:22
bogdandohjensas, slagle: but I still can't get then where that {{fqdn_nova_libvirt_network}} should come from into that group var14:23
bogdandoI see it's defined in puppet/role.role.j2.yaml14:24
* fultonj confused about role_data vs roles_data (with an 's') in http://ix.io/3Y9i in resposne to "CREATE FAILED (standalone-StandaloneServiceChain-lrtro5hmpdvm): Error in 19 output role_data: The values of the "for_each" argument to "repeat" must be lists or maps"14:24
bogdandoahaa! so it's end up in AnsibleHostVars dict14:25
hjensasbogdando: we have service_net_map in global_vars, dns: "fqdn_{{service_net_map['nova_libvirt_network']}} ?14:25
bogdandowell right, but how to access that from AnsibleHostVars, in a standalone ansible role?14:25
fultonjin http://ix.io/3Y9i roles_data was rendered to a list 14:26
bogdandohjensas: ok, I get it14:26
bogdandothanks14:26
hjensasbogdando: I guess a standalone ansible role need to rely on the service_net_map being defined in inventory? btw, we do some per-role ServiceNetMap stuff in THT so would make sense to have in group_vars - https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/overcloud.j2.yaml#L31814:29
bogdandoslagle: ^^14:29
bogdandohjensas: right, but standalone ansible service roles know nothing about tripleo roles14:30
bogdandoiiuc14:30
slaglebogdando: the new standalone-roles shouldn't rely on ServiceNetMap imo. we need to make this stuff native ansible, not rely on legacy t-h-t constructs14:30
bogdandoand should not use that abstraction14:30
bogdandoyea14:30
slaglewhat you likely are after is a network name, so you can get an IP?14:31
bogdandonow just asking myself how to pass that magic into a group_var14:31
slaglejust use a new ansible var, tripleo_libvirt_network_ip, or whatever14:31
bogdandook, and in tht, as a group var, I just pass the needed value?.14:31
slaglethe ansible role should likely configure this with just a var that is set to an IP, or maybe an interface name. and you can rely entirely on ansible gathered facts14:31
bogdandohttps://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-modular-libvirt-container-puppet.yaml#L772-L781 - my case14:32
fultonjhjensas: do you think tht has a bug w/ role_data vs roles_data in https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.j2.yaml#L665 14:32
fultonjrendered output is http://ix.io/3Y9i and heat error is http://ix.io/3Y9m 14:33
hjensasfultonj: I thin role_data is correct - https://github.com/openstack/tripleo-heat-templates/blob/31cfa8cd84ec5c20b6dca202ff6ce83a8128e3d2/common/services/role.role.j2.yaml#L39214:34
opendevreviewJiri Podivin proposed openstack/validations-common stable/1.6: [Stable-Only] Refactoring validations-common stable branch CI pipeliene  https://review.opendev.org/c/openstack/validations-common/+/84204814:35
hjensasfultonj: I think the problem is the input for one ore more of the for_each loops in these files: https://paste.opendev.org/show/bUYrO8IqgrD2IZEqp6YF/14:38
opendevreviewJiri Podivin proposed openstack/tripleo-quickstart-extras master: Propagating branch_override variable into DLRN build role  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/84227314:39
hjensasI think SearviceChain.$NUMBER (i.e 19) links to one of those service templates. Not sure how to figure out which one.14:39
hjensasfultonj: we need rabi. :) Maybe slagle or bshephar knows? ^ fultonj is getting error -  Error in 19 output role_data: The values of the "for_each" argument to "repeat" must be lists or maps"14:40
fultonjthanks hjensas 14:41
opendevreviewJiri Podivin proposed openstack/validations-common stable/1.6: [Stable-Only] Refactoring validations-common stable branch CI pipeliene  https://review.opendev.org/c/openstack/validations-common/+/84204814:41
hjensasfultonj: does heat logs have more verbosity?14:42
fultonj$ cat heat.log  | curl -F 'f:1=<-' ix.io14:43
fultonjhttp://ix.io/3Y9w14:43
slaglefultonj: are you testing some changes? or what roles are you using?14:43
fultonjstandalone w/ composable network14:43
fultonjhttps://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/14:43
slaglethe error just means there's a problem somewhere on one of the service's role_data outputs14:43
fultonji got past the ip_version issue w/ https://review.opendev.org/c/openstack/python-tripleoclient/+/842537 14:44
slaglefultonj: the " Error in 19 output" means it's index 19 in the service chain14:45
slaglelook at the heat stack output to see which service that is14:45
hjensasfultonj: Stack CREATE FAILED (standalone-StandaloneServiceChain-lrtro5hmpdvm) - pre ephemeral heat we could do stack show|resource list etc on that stack. If we could start the heat pod and inspect?14:46
* slagle slightly guessing. but i don't know what else 19 would refer to14:46
* fultonj starting heat pod14:46
slagleyou can with --keep-running and there is also tripleo launch heat --restore-db14:46
opendevreviewAnanya proposed openstack/tripleo-ci master: [wip] Changing container registry in testenv to quay  https://review.opendev.org/c/openstack/tripleo-ci/+/84255114:46
slaglebut you should be able to tell service 19 in just the deploy log14:47
slagleif this is really standalone, then ephemeral heat is not new here. and it uses heat-all process, not a pod14:48
fultonjack14:49
fultonjusing --keep-running14:49
fultonjack14:50
fultonjstandalone.StandaloneServiceChain.ServiceChain.13.GlanceLogging14:50
* fultonj waits for 1914:50
fultonj2022-05-19 14:50:30Z [standalone.StandaloneServiceChain.ServiceChain.19.ContainersCommon]: CREATE_IN_PROGRESS  state changed14:50
slagleit might not be 19 on a new deployment. don't think that order is guaranteed14:50
fultonjack14:50
slaglebut, see what the error says this time14:50
* fultonj waits for heat failure14:50
fultonj2022-05-19 14:52:35Z [standalone.StandaloneServiceChain]: CREATE_FAILED  Error in 19 output role_data: The values of the "for_each" argument to "repeat" must be lists or maps14:52
opendevreviewManoj Katari proposed openstack/puppet-tripleo master: Support configuring multiple NFS cinder backends  https://review.opendev.org/c/openstack/puppet-tripleo/+/84238014:55
slaglefultonj: i'd stack show on ServiceChain.19, whatever that stack uuid is14:55
slagleor stack resource list the StandaloneServiceChain.ServiceChain stack14:56
slagle19.ContainersCommon is just referring to the ContainersCommon resource in stack 19. whatever that service is14:58
* fultonj slowing getting it15:01
fultonj$ openstack stack resource list standalone | curl -F 'f:1=<-' ix.io15:01
fultonjhttp://ix.io/3Y9K15:01
slaglepass that -n 715:03
slagleto see nested stacks15:03
fultonj$ openstack stack resource list standalone -n 7 | curl -F 'f:1=<-' ix.io15:04
fultonjhttp://ix.io/3Y9M15:04
slagleOS::TripleO::Services::Memcached15:06
slagleproblem with that <%net_cidr%> dict most likely15:06
slaglestack show e3a6a2b9-c72e-49c4-ac92-640f1e1cd276 and see what gets passed in for ServiceData and ServiceNetMap15:07
fultonjhttps://github.com/openstack/tripleo-heat-templates/blob/master/deployment/memcached/memcached-container-puppet.yaml#L15715:08
fultonj$ openstack stack show e3a6a2b9-c72e-49c4-ac92-640f1e1cd276 | curl -F 'f:1=<-' ix.io15:09
fultonjhttp://ix.io/3Y9P15:09
slagle"net_cidr_map": {"ctlplane": ["192.168.24.0/24"], "storage": ["192.168.42.0/24"]}15:09
slaglebut...15:10
slagle"MemcachedNetwork": "internal_api"15:10
slagleso you get no result there15:10
fultonjoh15:10
slaglesomething is off. you've enable net-iso or something for just storage? but no internal_api network?15:10
opendevreviewBrendan Shephard proposed openstack/python-tripleoclient master: Use node UUID for bridge_mapping agent check  https://review.opendev.org/c/openstack/python-tripleoclient/+/83497815:10
fultonjyes15:10
fultonji didn't set internal_api15:10
fultonjhjensas: ^ 15:11
fultonjdeployed_network_env.yaml only has storage15:11
fultonjnetwork_data.yaml only has storage15:11
fultonji'll set them all15:11
* fultonj was trying to add just one network15:11
fultonjctlplane + storage 15:11
fultonjsince I didn't need the others15:11
fultonjyou've enable net-iso or something for just storage? --> yes!15:12
slagleyou might need to use a corresponding network_data that only defines storage?15:12
slaglethat way servicenetmap would default memcachednetwork to ctlplane15:13
fultonji did15:13
fultonjhttps://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/15:13
fultonjline 6715:13
fultonjand line 35 is my deployed_network_env.yaml 15:13
slagleok, well. something in the servicenetmap jinja2 is thinking internal_api is defined/enabled15:13
fultonjI suppose i could trick it15:14
fultonjadd internal_api etc and set them to ctl plane15:14
fultonji'll try that15:14
fultonjthanks for finding the issue slagle 15:15
slaglememcachednetwork is set in overcloud-resource-registry-puppet.j2.yaml from _service_nets which is defined at the very top of that file15:15
slagleso it definitely thinks internal_api is enabled for some reason15:16
opendevreviewMerged openstack/tripleo-ansible master: Correcting attribute for 'Node' object  https://review.opendev.org/c/openstack/tripleo-ansible/+/84247315:18
* fultonj had passed environments/network-environment.yaml15:20
fultonjsaw similar issue without it though15:20
*** dviroel is now known as dviroel|lunch15:26
slaglefultonj: double check the rendered overcloud-resource-registry-puppet.yaml in the working dir. and check the _service_nets dict at the top and value for MemcachedNetwork15:27
slagleit's almost like it didnt use your networks file with only storage defined15:27
opendevreviewAnanya proposed openstack/tripleo-ci master: [wip] Changing container registry in testenv to quay  https://review.opendev.org/c/openstack/tripleo-ci/+/84255115:28
*** ysandeep|rover is now known as ysandeep|out15:29
opendevreviewAnanya proposed openstack/tripleo-ci master: [wip] Changing container registry in testenv to quay  https://review.opendev.org/c/openstack/tripleo-ci/+/84255115:29
opendevreviewAnanya proposed openstack/tripleo-ci master: Changing container registry in testenv to quay  https://review.opendev.org/c/openstack/tripleo-ci/+/84255115:30
opendevreviewManoj Katari proposed openstack/tripleo-heat-templates master: Support for multiple Cinder NFS storage backends  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84237015:37
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates master: Fix run_after of certs_requests in modular libvirt  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84255415:41
opendevreviewMarios Andreou proposed openstack/tripleo-quickstart master: WIP add new featureset66 for mixed deployment jobs  https://review.opendev.org/c/openstack/tripleo-quickstart/+/84158216:11
opendevreviewMarios Andreou proposed openstack/tripleo-quickstart-extras master: WIP add some new plays for mixed mode  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/84176416:11
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: WIP add new mixed mode nodeset and job  https://review.opendev.org/c/openstack/tripleo-ci/+/84158316:11
opendevreviewMarios Andreou proposed openstack/tripleo-heat-templates stable/wallaby: DNM testing tripleo-ci/+/841583  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84212016:12
*** marios is now known as marios|out16:14
*** dviroel|lunch is now known as dviroel16:25
opendevreviewGoutham Pacha Ravi proposed openstack/tripleo-heat-templates master: Add service OS::TripleO::Services::ManilaBackendPowerMax  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84211216:27
opendevreviewGoutham Pacha Ravi proposed openstack/tripleo-heat-templates stable/wallaby: Add service OS::TripleO::Services::ManilaBackendPowerMax  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84231516:28
opendevreviewGoutham Pacha Ravi proposed openstack/tripleo-heat-templates stable/train: Add service OS::TripleO::Services::ManilaBackendPowerMax  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84241816:31
*** jpena is now known as jpena|off16:31
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: WIP Add standalone Nova Libvirt role  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215216:46
opendevreviewMerged openstack/tripleo-validations master: Normalizing Tripleo Validations metadata  https://review.opendev.org/c/openstack/tripleo-validations/+/84031117:30
opendevreviewMerged openstack/validations-libs master: Dockerfile updated to eliminate dependency conflicts  https://review.opendev.org/c/openstack/validations-libs/+/83674717:30
hjensasfultonj: sorry I got distracted earlier, but can you try to add something to debug why internal_api seem to be in _service_nets? https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/overcloud-resource-registry-puppet.j2.yaml#L817:33
hjensasfultonj: with only storage in network-data that should not be the case.17:34
fultonjhjensas: sure, i was afk for a bit17:34
hjensasfultonj: I get MemcachedNetwork: ctlplane - in /tmp/rendered/overcloud-resource-registry-puppet.yaml when I try with ./tools/process-templates.py -p /home/hjensas/code/tripleo-heat-templates -r /home/hjensas/code/tripleo-heat-templates/roles/Standalone.yaml -n /tmp/net-data.yaml -o /tmp/rendered/17:37
fultonj$ cat ./tripleo-heat-installer-templates/overcloud-resource-registry-puppet.yaml | curl -F 'f:1=<-' ix.io 17:38
fultonjhttp://ix.io/3Yaq17:38
fultonj^ has MemcachedNetwork: internal_api17:39
fultonjslagle: ^ fyi17:39
hjensasfultonj: that is strange, it has all the networks - CephClusterNetwork: storage_mgmt, DesignateBindNetwork: external etc. 17:39
fultonjmy deploy command and the conents of deployed_netwprk_env.yaml and network_data.yaml are in https://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/ 17:41
hjensasfultonj: can you try a full path to --networks-file network_data.yaml \ - I wonder if it picking the one relative to THT dir.17:42
* fultonj trying abs path17:43
fultonj2022-05-19 17:48:11Z [standalone]: CREATE_COMPLETE  Stack CREATE completed successfully17:48
fultonjhjensas: the only thing i changed was using absolute path17:48
fultonjhjensas: it's now running ansible :)17:49
hjensasfultonj: \o/17:49
hjensasfultonj: did you add storage network in ~/templates/roles/Standalone.yaml as well?17:50
fultonjno i didn't do that...17:51
hjensasmight need that for the "for network in networks if network in role_networks" j2 conditions in THT.17:51
fultonjhttps://paste.opendev.org/show/bRjpmjxXXQlNJITukdKL/17:52
fultonjhjensas: my Standalone.yaml is the default with these lines https://github.com/openstack/tripleo-heat-templates/blob/master/roles/Standalone.yaml#L19-L31 17:54
fultonjdo i need to change it to just this?17:54
fultonj  networks:17:54
fultonj    Storage:17:54
fultonj      subnet: storage_subnet17:54
opendevreviewManoj Katari proposed openstack/puppet-tripleo master: Support configuring multiple NFS cinder backends  https://review.opendev.org/c/openstack/puppet-tripleo/+/84238017:54
opendevreviewMerged openstack/tripleo-heat-templates master: Set tls-verify false when using insecure reg  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84055517:54
hjensasfultonj: ah, ok I figured standalone would have no networks. No need to change it in this case.17:54
fultonjslagle: FYI changing the path to --network-files seems to have fixed it https://paste.opendev.org/show/bRjpmjxXXQlNJITukdKL/ 17:56
fultonjthank you for your help hjensas17:56
hjensasyw17:56
* fultonj going to see if ansible finishes OK and then go update my CI patch17:56
slaglefultonj: sounds like a bug17:58
fultonji can file it in launch pad17:58
fultonjand see if I can see why it missed it17:59
fultonjDeployment successful!18:11
opendevreviewMerged openstack/tripleo-ansible master: Migrate tripleo_ovn host_prep_tasks to tripleo-ansible  https://review.opendev.org/c/openstack/tripleo-ansible/+/84204118:18
opendevreviewBrent Eagles proposed openstack/tripleo-ansible master: Designate: use extra IP for bind/unbound colocation on the bind9 backend  https://review.opendev.org/c/openstack/tripleo-ansible/+/83997318:27
opendevreviewBrent Eagles proposed openstack/tripleo-ansible master: Designate: remove unnecessary transitional code for moving extra IP  https://review.opendev.org/c/openstack/tripleo-ansible/+/84045118:27
fultonjhttps://bugs.launchpad.net/tripleo/+bug/197423220:00
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Fix setting volumes for internal tls  https://review.opendev.org/c/openstack/tripleo-ansible/+/84243620:32
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node  https://review.opendev.org/c/openstack/tripleo-ansible/+/84050920:32
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_bootstrap role to operating system install phase  https://review.opendev.org/c/openstack/tripleo-ansible/+/84067520:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_ssh_known_hosts role to OpenStack configure phase  https://review.opendev.org/c/openstack/tripleo-ansible/+/84067620:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_kernel role to operating system install phase  https://review.opendev.org/c/openstack/tripleo-ansible/+/84179520:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_podman role to operating system configure and install playbooks  https://review.opendev.org/c/openstack/tripleo-ansible/+/84199620:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_logrotate_crond role  https://review.opendev.org/c/openstack/tripleo-ansible/+/84199720:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_sshd role to operating system playbooks  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215020:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add chrony role to operating system install, configure, and run playbooks  https://review.opendev.org/c/openstack/tripleo-ansible/+/84222420:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add tripleo_timezone role to operating system install and run playbooks  https://review.opendev.org/c/openstack/tripleo-ansible/+/84223420:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add iscsid role to OpenStack install and run  https://review.opendev.org/c/openstack/tripleo-ansible/+/84243720:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Use ansible.netcommon.ipaddr filter in tripleo_mysql_client  https://review.opendev.org/c/openstack/tripleo-ansible/+/84261820:33
*** dviroel is now known as dviroel|out20:45
stevebaker[m]ysandeep|rover: Thanks for fixing that Node.id thing21:01
*** dasm is now known as dasm|off21:26
rlandycloudnull: hmmm ... hate to be the barer of bad news before recharge day: https://zuul.openstack.org/builds?job_name=openstack-tox-py36&project=openstack%2Ftripleo-heat-templates&skip=021:52
rlandy2022-05-19 18:32:48.401386 | ubuntu-bionic |     tripleo-common 16.4.0 depends on oslo.log>=3.36.021:52
rlandy2022-05-19 18:32:48.401412 | ubuntu-bionic |     The user requested (constraint) oslo-log===5.0.021:52
rlandygate blocker21:52
rlandyhttps://github.com/openstack/tripleo-heat-templates/blob/master/requirements.txt#L721:54
rlandycan we up this?21:55
* rlandy creates bug21:55
rlandydasm|off: ^^ if you are in tomorrow21:55
rlandychandankumar: ^^21:55
opendevreviewSteve Baker proposed openstack/openstack-virtual-baremetal master: Update the letsencrypt root cert on centos7  https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/84263821:57
opendevreviewSteve Baker proposed openstack/openstack-virtual-baremetal stable/2.0: Explicitly support only centos-7 on the bmc host  https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/84244122:02
opendevreviewSteve Baker proposed openstack/openstack-virtual-baremetal stable/2.0: Simplify bmc repo setup and installation  https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/84244222:02
opendevreviewSteve Baker proposed openstack/openstack-virtual-baremetal stable/2.0: Add bmc host support for centos-9-stream  https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/84244322:02
rlandydasm|off: chandankumar: https://bugs.launchpad.net/tripleo/+bug/1974244 - pls check if this is still an issue tomorrow - or just a mirror issue atm22:08
rlandyhttps://pypi.org/project/oslo.log/ - released 12 hours ago - maybe just mirror22:10
cloudnullrlandy just saw your ping 22:16
cloudnulldo we have a patch in place for that?22:16
opendevreviewSteve Baker proposed openstack/tripleo-ansible stable/wallaby: Use instance_info.display_name for existing hostname matching  https://review.opendev.org/c/openstack/tripleo-ansible/+/84243422:22
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: kernelargs play, gather facts after waiting for pre_tasks  https://review.opendev.org/c/openstack/tripleo-ansible/+/84038422:32
opendevreviewMerged openstack/tripleo-ansible master: Modify cert gen to make it work under fips  https://review.opendev.org/c/openstack/tripleo-ansible/+/83030822:32
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Horizon: Fix wrong tag used in puppet  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84264423:18

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