opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: Designate: specify source address when using rndc https://review.opendev.org/c/openstack/tripleo-ansible/+/841456 | 00:09 |
---|---|---|
*** rlandy|bbl is now known as rlandy|out | 00:45 | |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Octavia: Include separate networking class https://review.opendev.org/c/openstack/puppet-tripleo/+/842448 | 01:30 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Octavia: Include separate networking class https://review.opendev.org/c/openstack/puppet-tripleo/+/842448 | 01:32 |
opendevreview | Merged openstack/tripleo-docs master: Replace existing virt-customize usage with tripleo-mount-image https://review.opendev.org/c/openstack/tripleo-docs/+/836259 | 01:43 |
opendevreview | yatin proposed openstack/tripleo-quickstart master: Switch to c9-stream image in standalone libvirt env https://review.opendev.org/c/openstack/tripleo-quickstart/+/842357 | 04:01 |
opendevreview | Lewis 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/+/842044 | 04:42 |
opendevreview | Lewis 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/+/842044 | 04:46 |
*** ykarel_ is now known as ykarel | 04:51 | |
*** ysandeep|out is now known as ysandeep|rover | 04:52 | |
*** pojadhav|afk is now known as pojadhav | 05:10 | |
*** ysandeep|rover is now known as ysandeep|rover|brb | 05:51 | |
*** ysandeep|rover|brb is now known as ysandeep|rover | 05:56 | |
Tengu | slagle: looks nice - just tox unhappy with the depends-on. Didn't check the t-h-t one yet | 06:01 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Rename molecule jobs to centos-stream https://review.opendev.org/c/openstack/tripleo-ansible/+/842036 | 06:22 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Rename molecule jobs to centos-stream https://review.opendev.org/c/openstack/tripleo-ansible/+/842036 | 06:26 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Use cs9 image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/839688 | 06:27 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Use cs9 image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/839688 | 06: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 issues | 06: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 one | 06:48 |
ysandeep|rover | jm1, 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|rover | hjensas: 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 jpena | 07:36 | |
stevebaker[m] | I'm working tomorrow, but yeah revert | 07:39 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible master: Revert "Use instance_info.display_name for existing hostname matching" https://review.opendev.org/c/openstack/tripleo-ansible/+/842399 | 07:45 |
ysandeep|rover | stevebaker[m]: thanks https://review.opendev.org/c/openstack/tripleo-ansible/+/842399 | 07:45 |
ykarel | ysandeep|rover, may be can attempt fix in parallel | 07:50 |
ykarel | seems just need to change node.uuid to node.id | 07:50 |
ykarel | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/baremetal/v1/node.py#L133 | 07:51 |
ykarel | if it works can avoid revert, else merge revert | 07:51 |
ysandeep|rover | ykarel, at https://review.opendev.org/c/openstack/tripleo-ansible/+/840262/3/tripleo_ansible/ansible_plugins/module_utils/baremetal_deploy.py#426 and #428 | 07:51 |
ykarel | ysandeep|rover, yes | 07:52 |
ysandeep|rover | ykarel, let me try the fix locally | 07:53 |
ykarel | ack should be easy there | 07:54 |
ysandeep|rover | issue reproduced locally, let try the fix now | 08:02 |
ysandeep|rover | ykarel++ passed that failing step | 08:09 |
opendevreview | Merged openstack/tripleo-ci master: Add keystone deployment files filter for ipa job https://review.opendev.org/c/openstack/tripleo-ci/+/842356 | 08:09 |
ysandeep|rover | ykarel, 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 |
opendevreview | Jiri Podivin proposed openstack/tripleo-quickstart-extras master: DNM Adjusting DRLN builds for alternative branch names https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/842273 | 08:10 |
ykarel | ysandeep|rover, i think that is fine, you seeing issue with it? | 08:10 |
ysandeep|rover | I tested after changing with fields=['id' , let me test with uuid also | 08:11 |
ysandeep|rover | In 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 |
opendevreview | yatin proposed openstack/tripleo-quickstart master: Switch to c9-stream image in standalone libvirt env https://review.opendev.org/c/openstack/tripleo-quickstart/+/842357 | 08:18 |
ysandeep|rover | ykarel, uuid worked as well | 08:19 |
ysandeep|rover | for #425 | 08:20 |
ykarel | ysandeep|rover, ack | 08:21 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: bnr added cli-undercloud-backup-db molecule job https://review.opendev.org/c/openstack/tripleo-ansible/+/836845 | 08: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 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: bnr added cli-undercloud-backup-db molecule job https://review.opendev.org/c/openstack/tripleo-ansible/+/836845 | 09:15 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible master: Correcting attribute for 'Node' object https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 | 09:24 |
ysandeep|rover | stevebaker[m], hjensas ^^ https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 (worked locally) - please review if this looks okay | 09:26 |
ysandeep|rover | thanks ykarel++ for the help | 09:26 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible master: Correcting attribute for 'Node' object https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 | 09:29 |
*** ysandeep|rover is now known as ysandeep|rover|lunch | 09:31 | |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: WIP add new featureset66 for mixed deployment jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/841582 | 09:36 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: WIP add new mixed mode nodeset and job https://review.opendev.org/c/openstack/tripleo-ci/+/841583 | 09:37 |
opendevreview | Merged openstack/puppet-tripleo stable/train: haproxy: Enable httplog for all http endpoints https://review.opendev.org/c/openstack/puppet-tripleo/+/828447 | 09:44 |
*** ysandeep|rover|lunch is now known as ysandeep|rover | 10:09 | |
opendevreview | Merged openstack/paunch stable/train: Ensure systemd services are activated https://review.opendev.org/c/openstack/paunch/+/837591 | 10:14 |
opendevreview | yatin proposed openstack/tripleo-quickstart master: Switch to c9-stream image in standalone libvirt env https://review.opendev.org/c/openstack/tripleo-quickstart/+/842357 | 10:19 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Use cs9 image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/839688 | 10:20 |
*** rlandy|out is now known as rlandy | 10:27 | |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: WIP add new mixed mode nodeset and job https://review.opendev.org/c/openstack/tripleo-ci/+/841583 | 10:47 |
opendevreview | Damien Ciabrini proposed openstack/puppet-tripleo master: Support mariabackup as a galera SST method https://review.opendev.org/c/openstack/puppet-tripleo/+/836189 | 10: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 that | 11:00 |
lecris[m] | Would that be CephAll? Does it include Cinder Swift etc? | 11:04 |
bshephar | lecris[m]: The default Controller role has both. You can create a custom role to include both if you wanted to. | 11:08 |
bshephar | Cinder: 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-L177 | 11: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 otherwise | 11:10 |
lecris[m] | If I use CephAll, would that do the trick? | 11:11 |
bshephar | You would probably need to add the pure related services to it, but I guess it should have everything else you're after | 11:12 |
bshephar | https://github.com/openstack/tripleo-heat-templates/blob/master/roles_data.yaml#L79 | 11: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 it | 11:14 |
bshephar | As long as you have all the cinder, swift and pure related services in the role, it should be fine. | 11:15 |
bshephar | https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/features/custom_roles.html | 11:15 |
lecris[m] | Thanks for the advice | 11:16 |
bshephar | np | 11:20 |
opendevreview | Jiri 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/+/841713 | 11:22 |
*** dviroel|out is now known as dviroel | 11:26 | |
*** rlandy is now known as rlandy|mtg | 11:26 | |
*** ysandeep|rover is now known as ysandeep|rover|afk | 11:33 | |
opendevreview | Bogdan 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/+/842530 | 11:35 |
opendevreview | Merged openstack/python-tripleoclient stable/train: Exposes the community validation init command https://review.opendev.org/c/openstack/python-tripleoclient/+/825736 | 11:46 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart master: DNM : Testing tempest tests for intergration/component jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/842531 | 11:48 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart master: DNM : Testing tempest tests for intergration/component jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/842531 | 11:57 |
opendevreview | Bogdan 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/+/842530 | 11:58 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart master: DNM : Testing tempest tests for intergration/component jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/842531 | 11:59 |
*** rlandy|mtg is now known as rlandy | 12:02 | |
opendevreview | Bogdan 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/+/842530 | 12:09 |
opendevreview | Merged openstack/tripleo-ci master: Ussuri tear down as EOL https://review.opendev.org/c/openstack/tripleo-ci/+/835460 | 12:10 |
opendevreview | Merged openstack/tripleo-upgrade stable/wallaby: Install pkgs required to apply workarounds https://review.opendev.org/c/openstack/tripleo-upgrade/+/840165 | 12:10 |
opendevreview | Jiri Podivin proposed openstack/tripleo-ci master: Fixing typo in validations-* branch overrides https://review.opendev.org/c/openstack/tripleo-ci/+/841762 | 12:23 |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Added comment to revert in a future https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834036 | 12:24 |
*** ysandeep|rover|afk is now known as ysandeep|rover | 12:26 | |
ysandeep|rover | could someone please +w https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 | 12:27 |
fultonj | ysandeep|rover: done | 12:28 |
opendevreview | Khomesh 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/+/842075 | 12:29 |
ysandeep|rover | fultonj: thanks :) | 12:29 |
fultonj | hjensas: have you seen Property error: Networks.resources.NetworkIpVersionMap.properties.value: 'ip_version' ? https://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/ | 12:37 |
jm1 | stevebaker[m]: hello, could you have a look at this patch please? :) https://review.opendev.org/c/openstack/tripleo-ci/+/841992 | 12:38 |
hjensas | fultonj: 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-L49 | 12:44 |
hjensas | fultonj: can you find out what was in CtlplaneNetworkAttributes ? | 12:44 |
hjensas | fultonj: https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v1/tripleo_deploy.py | 12:45 |
hjensas | fultonj: sorry, better with a line number https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v1/tripleo_deploy.py#L687 | 12:45 |
hjensas | fultonj: I think the issue that ip_version is not set in subnets for CtlplaneNetworkAttributes in tripleo_deploy.py. | 12:46 |
fultonj | I'm not overriding CtlplaneNetworkAttributes so I assume it's using the default | 12:47 |
fultonj | if "not set in subnets", should I not use subnets syntax in DeployedNetworkEnvironment ? | 12:48 |
hjensas | fultonj: I think its a bug, or rather we never tried composable network with standalone before. | 12:53 |
hjensas | fultonj: 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 |
fultonj | http://ix.io/3Y8Y | 12:53 |
fultonj | ^ shows what's in CtlplaneNetworkAttributes | 12:54 |
fultonj | no ip_version | 12:54 |
fultonj | could i just not have ip_version in my deployed_network_env? | 12:54 |
hjensas | fultonj: 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#L649 | 12:54 |
fultonj | so it might be an easy fix, i'll try it | 12:55 |
fultonj | thx | 12:55 |
opendevreview | Bogdan 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/+/842530 | 12:55 |
hjensas | fultonj: add me on review if you propose a patch to tripleoclient | 12:55 |
odyssey4me | slagle 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 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Add ip_version to CtlplaneNetworkAttributes https://review.opendev.org/c/openstack/python-tripleoclient/+/842537 | 13:03 |
fultonj | hjensas: is this what you meant ^ ? | 13:03 |
fultonj | I 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 |
fultonj | I think I need to pass the new ip_version variable tmp_env.update | 13:10 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Add ip_version to CtlplaneNetworkAttributes https://review.opendev.org/c/openstack/python-tripleoclient/+/842537 | 13:15 |
fultonj | hjensas: I seem to be passed that error now using PS2 of ^ thanks | 13:15 |
fultonj | and now it fails with https://paste.opendev.org/show/bU7bszlLsFBsVD1X18LP/ | 13:19 |
fultonj | 08: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 standalone | 13:21 | |
*** dasm|off is now known as dasm | 13:21 | |
opendevreview | Brent 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/+/839973 | 13:21 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: Designate: remove unnecessary transitional code for moving extra IP https://review.opendev.org/c/openstack/tripleo-ansible/+/840451 | 13:21 |
hjensas | fultonj: 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 |
opendevreview | Bogdan 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/+/842530 | 13:29 |
fultonj | ok, thanks hjensas | 13:30 |
hjensas | fultonj: hm, StandaloneServiceChain - Error in 19 output role_data: The values of the "for_each" argument to "repeat" must be lists or maps | 13:31 |
hjensas | fultonj: 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 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: WIP Supports nftables instead of iptables https://review.opendev.org/c/openstack/tripleo-ansible/+/841414 | 13:35 |
opendevreview | Cedric 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/+/841445 | 13:40 |
opendevreview | Bogdan Dobrelya proposed openstack/puppet-tripleo stable/train: [train-only] Fix certmonger QEMU key file data race https://review.opendev.org/c/openstack/puppet-tripleo/+/840904 | 13:44 |
opendevreview | Bogdan 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/+/842530 | 13:44 |
fultonj | hjensas: $ cat overcloud.yaml | curl -F 'f:1=<-' ix.io | 13:51 |
fultonj | http://ix.io/3Y9i | 13:51 |
fultonj | contains my StandaloneServiceChain | 13:51 |
opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: WIP Add a new parameter for the undercloud.conf https://review.opendev.org/c/openstack/python-tripleoclient/+/842546 | 13:55 |
fultonj | I assume this line did it: "value: {get_attr: [StandaloneServiceChain, role_data]}" | 13:55 |
fultonj | i'm using -r https://github.com/openstack/tripleo-heat-templates/blob/master/roles/Standalone.yaml | 13:57 |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Use timezone UTC on undercloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/842218 | 14:11 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: WIP Supports nftables instead of iptables https://review.opendev.org/c/openstack/tripleo-ansible/+/841414 | 14:17 |
opendevreview | Jiri 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/+/841713 | 14:17 |
opendevreview | Jiri 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/+/842048 | 14:18 |
bogdando | hjensas: 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 |
bogdando | IIRC I saw something similar in tripleo-ansible to extract things from ServiceNetMap... can't recall what and how | 14:20 |
bogdando | s/case/cast* :) | 14:20 |
bogdando | perhaps I can use a group_var for it, just passing that value from t-h-t | 14:21 |
bogdando | slagle: ^^ | 14:21 |
opendevreview | Jiri 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/+/842048 | 14:22 |
bogdando | hjensas, slagle: but I still can't get then where that {{fqdn_nova_libvirt_network}} should come from into that group var | 14:23 |
bogdando | I see it's defined in puppet/role.role.j2.yaml | 14: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 | |
bogdando | ahaa! so it's end up in AnsibleHostVars dict | 14:25 |
hjensas | bogdando: we have service_net_map in global_vars, dns: "fqdn_{{service_net_map['nova_libvirt_network']}} ? | 14:25 |
bogdando | well right, but how to access that from AnsibleHostVars, in a standalone ansible role? | 14:25 |
fultonj | in http://ix.io/3Y9i roles_data was rendered to a list | 14:26 |
bogdando | hjensas: ok, I get it | 14:26 |
bogdando | thanks | 14:26 |
hjensas | bogdando: 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#L318 | 14:29 |
bogdando | slagle: ^^ | 14:29 |
bogdando | hjensas: right, but standalone ansible service roles know nothing about tripleo roles | 14:30 |
bogdando | iiuc | 14:30 |
slagle | bogdando: 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 constructs | 14:30 |
bogdando | and should not use that abstraction | 14:30 |
bogdando | yea | 14:30 |
slagle | what you likely are after is a network name, so you can get an IP? | 14:31 |
bogdando | now just asking myself how to pass that magic into a group_var | 14:31 |
slagle | just use a new ansible var, tripleo_libvirt_network_ip, or whatever | 14:31 |
bogdando | ok, and in tht, as a group var, I just pass the needed value?. | 14:31 |
slagle | the 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 facts | 14:31 |
bogdando | https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-modular-libvirt-container-puppet.yaml#L772-L781 - my case | 14:32 |
fultonj | hjensas: 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 |
fultonj | rendered output is http://ix.io/3Y9i and heat error is http://ix.io/3Y9m | 14:33 |
hjensas | fultonj: I thin role_data is correct - https://github.com/openstack/tripleo-heat-templates/blob/31cfa8cd84ec5c20b6dca202ff6ce83a8128e3d2/common/services/role.role.j2.yaml#L392 | 14:34 |
opendevreview | Jiri 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/+/842048 | 14:35 |
hjensas | fultonj: 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 |
opendevreview | Jiri Podivin proposed openstack/tripleo-quickstart-extras master: Propagating branch_override variable into DLRN build role https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/842273 | 14:39 |
hjensas | I think SearviceChain.$NUMBER (i.e 19) links to one of those service templates. Not sure how to figure out which one. | 14:39 |
hjensas | fultonj: 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 |
fultonj | thanks hjensas | 14:41 |
opendevreview | Jiri 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/+/842048 | 14:41 |
hjensas | fultonj: does heat logs have more verbosity? | 14:42 |
fultonj | $ cat heat.log | curl -F 'f:1=<-' ix.io | 14:43 |
fultonj | http://ix.io/3Y9w | 14:43 |
slagle | fultonj: are you testing some changes? or what roles are you using? | 14:43 |
fultonj | standalone w/ composable network | 14:43 |
fultonj | https://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/ | 14:43 |
slagle | the error just means there's a problem somewhere on one of the service's role_data outputs | 14:43 |
fultonj | i got past the ip_version issue w/ https://review.opendev.org/c/openstack/python-tripleoclient/+/842537 | 14:44 |
slagle | fultonj: the " Error in 19 output" means it's index 19 in the service chain | 14:45 |
slagle | look at the heat stack output to see which service that is | 14:45 |
hjensas | fultonj: 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 to | 14:46 | |
* fultonj starting heat pod | 14:46 | |
slagle | you can with --keep-running and there is also tripleo launch heat --restore-db | 14:46 |
opendevreview | Ananya proposed openstack/tripleo-ci master: [wip] Changing container registry in testenv to quay https://review.opendev.org/c/openstack/tripleo-ci/+/842551 | 14:46 |
slagle | but you should be able to tell service 19 in just the deploy log | 14:47 |
slagle | if this is really standalone, then ephemeral heat is not new here. and it uses heat-all process, not a pod | 14:48 |
fultonj | ack | 14:49 |
fultonj | using --keep-running | 14:49 |
fultonj | ack | 14:50 |
fultonj | standalone.StandaloneServiceChain.ServiceChain.13.GlanceLogging | 14:50 |
* fultonj waits for 19 | 14:50 | |
fultonj | 2022-05-19 14:50:30Z [standalone.StandaloneServiceChain.ServiceChain.19.ContainersCommon]: CREATE_IN_PROGRESS state changed | 14:50 |
slagle | it might not be 19 on a new deployment. don't think that order is guaranteed | 14:50 |
fultonj | ack | 14:50 |
slagle | but, see what the error says this time | 14:50 |
* fultonj waits for heat failure | 14:50 | |
fultonj | 2022-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 maps | 14:52 |
opendevreview | Manoj Katari proposed openstack/puppet-tripleo master: Support configuring multiple NFS cinder backends https://review.opendev.org/c/openstack/puppet-tripleo/+/842380 | 14:55 |
slagle | fultonj: i'd stack show on ServiceChain.19, whatever that stack uuid is | 14:55 |
slagle | or stack resource list the StandaloneServiceChain.ServiceChain stack | 14:56 |
slagle | 19.ContainersCommon is just referring to the ContainersCommon resource in stack 19. whatever that service is | 14:58 |
* fultonj slowing getting it | 15:01 | |
fultonj | $ openstack stack resource list standalone | curl -F 'f:1=<-' ix.io | 15:01 |
fultonj | http://ix.io/3Y9K | 15:01 |
slagle | pass that -n 7 | 15:03 |
slagle | to see nested stacks | 15:03 |
fultonj | $ openstack stack resource list standalone -n 7 | curl -F 'f:1=<-' ix.io | 15:04 |
fultonj | http://ix.io/3Y9M | 15:04 |
slagle | OS::TripleO::Services::Memcached | 15:06 |
slagle | problem with that <%net_cidr%> dict most likely | 15:06 |
slagle | stack show e3a6a2b9-c72e-49c4-ac92-640f1e1cd276 and see what gets passed in for ServiceData and ServiceNetMap | 15:07 |
fultonj | https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/memcached/memcached-container-puppet.yaml#L157 | 15:08 |
fultonj | $ openstack stack show e3a6a2b9-c72e-49c4-ac92-640f1e1cd276 | curl -F 'f:1=<-' ix.io | 15:09 |
fultonj | http://ix.io/3Y9P | 15:09 |
slagle | "net_cidr_map": {"ctlplane": ["192.168.24.0/24"], "storage": ["192.168.42.0/24"]} | 15:09 |
slagle | but... | 15:10 |
slagle | "MemcachedNetwork": "internal_api" | 15:10 |
slagle | so you get no result there | 15:10 |
fultonj | oh | 15:10 |
slagle | something is off. you've enable net-iso or something for just storage? but no internal_api network? | 15:10 |
opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: Use node UUID for bridge_mapping agent check https://review.opendev.org/c/openstack/python-tripleoclient/+/834978 | 15:10 |
fultonj | yes | 15:10 |
fultonj | i didn't set internal_api | 15:10 |
fultonj | hjensas: ^ | 15:11 |
fultonj | deployed_network_env.yaml only has storage | 15:11 |
fultonj | network_data.yaml only has storage | 15:11 |
fultonj | i'll set them all | 15:11 |
* fultonj was trying to add just one network | 15:11 | |
fultonj | ctlplane + storage | 15:11 |
fultonj | since I didn't need the others | 15:11 |
fultonj | you've enable net-iso or something for just storage? --> yes! | 15:12 |
slagle | you might need to use a corresponding network_data that only defines storage? | 15:12 |
slagle | that way servicenetmap would default memcachednetwork to ctlplane | 15:13 |
fultonj | i did | 15:13 |
fultonj | https://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/ | 15:13 |
fultonj | line 67 | 15:13 |
fultonj | and line 35 is my deployed_network_env.yaml | 15:13 |
slagle | ok, well. something in the servicenetmap jinja2 is thinking internal_api is defined/enabled | 15:13 |
fultonj | I suppose i could trick it | 15:14 |
fultonj | add internal_api etc and set them to ctl plane | 15:14 |
fultonj | i'll try that | 15:14 |
fultonj | thanks for finding the issue slagle | 15:15 |
slagle | memcachednetwork is set in overcloud-resource-registry-puppet.j2.yaml from _service_nets which is defined at the very top of that file | 15:15 |
slagle | so it definitely thinks internal_api is enabled for some reason | 15:16 |
opendevreview | Merged openstack/tripleo-ansible master: Correcting attribute for 'Node' object https://review.opendev.org/c/openstack/tripleo-ansible/+/842473 | 15:18 |
* fultonj had passed environments/network-environment.yaml | 15:20 | |
fultonj | saw similar issue without it though | 15:20 |
*** dviroel is now known as dviroel|lunch | 15:26 | |
slagle | fultonj: 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 MemcachedNetwork | 15:27 |
slagle | it's almost like it didnt use your networks file with only storage defined | 15:27 |
opendevreview | Ananya proposed openstack/tripleo-ci master: [wip] Changing container registry in testenv to quay https://review.opendev.org/c/openstack/tripleo-ci/+/842551 | 15:28 |
*** ysandeep|rover is now known as ysandeep|out | 15:29 | |
opendevreview | Ananya proposed openstack/tripleo-ci master: [wip] Changing container registry in testenv to quay https://review.opendev.org/c/openstack/tripleo-ci/+/842551 | 15:29 |
opendevreview | Ananya proposed openstack/tripleo-ci master: Changing container registry in testenv to quay https://review.opendev.org/c/openstack/tripleo-ci/+/842551 | 15:30 |
opendevreview | Manoj Katari proposed openstack/tripleo-heat-templates master: Support for multiple Cinder NFS storage backends https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842370 | 15:37 |
opendevreview | Bogdan 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/+/842554 | 15:41 |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: WIP add new featureset66 for mixed deployment jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/841582 | 16:11 |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart-extras master: WIP add some new plays for mixed mode https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/841764 | 16:11 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: WIP add new mixed mode nodeset and job https://review.opendev.org/c/openstack/tripleo-ci/+/841583 | 16:11 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/wallaby: DNM testing tripleo-ci/+/841583 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842120 | 16:12 |
*** marios is now known as marios|out | 16:14 | |
*** dviroel|lunch is now known as dviroel | 16:25 | |
opendevreview | Goutham Pacha Ravi proposed openstack/tripleo-heat-templates master: Add service OS::TripleO::Services::ManilaBackendPowerMax https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842112 | 16:27 |
opendevreview | Goutham Pacha Ravi proposed openstack/tripleo-heat-templates stable/wallaby: Add service OS::TripleO::Services::ManilaBackendPowerMax https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842315 | 16:28 |
opendevreview | Goutham Pacha Ravi proposed openstack/tripleo-heat-templates stable/train: Add service OS::TripleO::Services::ManilaBackendPowerMax https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842418 | 16:31 |
*** jpena is now known as jpena|off | 16:31 | |
opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: WIP Add standalone Nova Libvirt role https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 16:46 |
opendevreview | Merged openstack/tripleo-validations master: Normalizing Tripleo Validations metadata https://review.opendev.org/c/openstack/tripleo-validations/+/840311 | 17:30 |
opendevreview | Merged openstack/validations-libs master: Dockerfile updated to eliminate dependency conflicts https://review.opendev.org/c/openstack/validations-libs/+/836747 | 17:30 |
hjensas | fultonj: 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#L8 | 17:33 |
hjensas | fultonj: with only storage in network-data that should not be the case. | 17:34 |
fultonj | hjensas: sure, i was afk for a bit | 17:34 |
hjensas | fultonj: 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 |
fultonj | http://ix.io/3Yaq | 17:38 |
fultonj | ^ has MemcachedNetwork: internal_api | 17:39 |
fultonj | slagle: ^ fyi | 17:39 |
hjensas | fultonj: that is strange, it has all the networks - CephClusterNetwork: storage_mgmt, DesignateBindNetwork: external etc. | 17:39 |
fultonj | my deploy command and the conents of deployed_netwprk_env.yaml and network_data.yaml are in https://paste.opendev.org/show/b0Ya0utB9v7EA8UnSxPf/ | 17:41 |
hjensas | fultonj: 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 path | 17:43 | |
fultonj | 2022-05-19 17:48:11Z [standalone]: CREATE_COMPLETE Stack CREATE completed successfully | 17:48 |
fultonj | hjensas: the only thing i changed was using absolute path | 17:48 |
fultonj | hjensas: it's now running ansible :) | 17:49 |
hjensas | fultonj: \o/ | 17:49 |
hjensas | fultonj: did you add storage network in ~/templates/roles/Standalone.yaml as well? | 17:50 |
fultonj | no i didn't do that... | 17:51 |
hjensas | might need that for the "for network in networks if network in role_networks" j2 conditions in THT. | 17:51 |
fultonj | https://paste.opendev.org/show/bRjpmjxXXQlNJITukdKL/ | 17:52 |
fultonj | hjensas: 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 |
fultonj | do i need to change it to just this? | 17:54 |
fultonj | networks: | 17:54 |
fultonj | Storage: | 17:54 |
fultonj | subnet: storage_subnet | 17:54 |
opendevreview | Manoj Katari proposed openstack/puppet-tripleo master: Support configuring multiple NFS cinder backends https://review.opendev.org/c/openstack/puppet-tripleo/+/842380 | 17:54 |
opendevreview | Merged openstack/tripleo-heat-templates master: Set tls-verify false when using insecure reg https://review.opendev.org/c/openstack/tripleo-heat-templates/+/840555 | 17:54 |
hjensas | fultonj: ah, ok I figured standalone would have no networks. No need to change it in this case. | 17:54 |
fultonj | slagle: FYI changing the path to --network-files seems to have fixed it https://paste.opendev.org/show/bRjpmjxXXQlNJITukdKL/ | 17:56 |
fultonj | thank you for your help hjensas | 17:56 |
hjensas | yw | 17:56 |
* fultonj going to see if ansible finishes OK and then go update my CI patch | 17:56 | |
slagle | fultonj: sounds like a bug | 17:58 |
fultonj | i can file it in launch pad | 17:58 |
fultonj | and see if I can see why it missed it | 17:59 |
fultonj | Deployment successful! | 18:11 |
opendevreview | Merged openstack/tripleo-ansible master: Migrate tripleo_ovn host_prep_tasks to tripleo-ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/842041 | 18:18 |
opendevreview | Brent 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/+/839973 | 18:27 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: Designate: remove unnecessary transitional code for moving extra IP https://review.opendev.org/c/openstack/tripleo-ansible/+/840451 | 18:27 |
fultonj | https://bugs.launchpad.net/tripleo/+bug/1974232 | 20:00 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Fix setting volumes for internal tls https://review.opendev.org/c/openstack/tripleo-ansible/+/842436 | 20:32 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node https://review.opendev.org/c/openstack/tripleo-ansible/+/840509 | 20:32 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_bootstrap role to operating system install phase https://review.opendev.org/c/openstack/tripleo-ansible/+/840675 | 20:33 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_ssh_known_hosts role to OpenStack configure phase https://review.opendev.org/c/openstack/tripleo-ansible/+/840676 | 20:33 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_kernel role to operating system install phase https://review.opendev.org/c/openstack/tripleo-ansible/+/841795 | 20:33 |
opendevreview | James 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/+/841996 | 20:33 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_logrotate_crond role https://review.opendev.org/c/openstack/tripleo-ansible/+/841997 | 20:33 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_sshd role to operating system playbooks https://review.opendev.org/c/openstack/tripleo-ansible/+/842150 | 20:33 |
opendevreview | James 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/+/842224 | 20:33 |
opendevreview | James 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/+/842234 | 20:33 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add iscsid role to OpenStack install and run https://review.opendev.org/c/openstack/tripleo-ansible/+/842437 | 20:33 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Use ansible.netcommon.ipaddr filter in tripleo_mysql_client https://review.opendev.org/c/openstack/tripleo-ansible/+/842618 | 20:33 |
*** dviroel is now known as dviroel|out | 20:45 | |
stevebaker[m] | ysandeep|rover: Thanks for fixing that Node.id thing | 21:01 |
*** dasm is now known as dasm|off | 21:26 | |
rlandy | cloudnull: 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=0 | 21:52 |
rlandy | 2022-05-19 18:32:48.401386 | ubuntu-bionic | tripleo-common 16.4.0 depends on oslo.log>=3.36.0 | 21:52 |
rlandy | 2022-05-19 18:32:48.401412 | ubuntu-bionic | The user requested (constraint) oslo-log===5.0.0 | 21:52 |
rlandy | gate blocker | 21:52 |
rlandy | https://github.com/openstack/tripleo-heat-templates/blob/master/requirements.txt#L7 | 21:54 |
rlandy | can we up this? | 21:55 |
* rlandy creates bug | 21:55 | |
rlandy | dasm|off: ^^ if you are in tomorrow | 21:55 |
rlandy | chandankumar: ^^ | 21:55 |
opendevreview | Steve Baker proposed openstack/openstack-virtual-baremetal master: Update the letsencrypt root cert on centos7 https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/842638 | 21:57 |
opendevreview | Steve 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/+/842441 | 22:02 |
opendevreview | Steve Baker proposed openstack/openstack-virtual-baremetal stable/2.0: Simplify bmc repo setup and installation https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/842442 | 22:02 |
opendevreview | Steve 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/+/842443 | 22:02 |
rlandy | dasm|off: chandankumar: https://bugs.launchpad.net/tripleo/+bug/1974244 - pls check if this is still an issue tomorrow - or just a mirror issue atm | 22:08 |
rlandy | https://pypi.org/project/oslo.log/ - released 12 hours ago - maybe just mirror | 22:10 |
cloudnull | rlandy just saw your ping | 22:16 |
cloudnull | do we have a patch in place for that? | 22:16 |
opendevreview | Steve Baker proposed openstack/tripleo-ansible stable/wallaby: Use instance_info.display_name for existing hostname matching https://review.opendev.org/c/openstack/tripleo-ansible/+/842434 | 22:22 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: kernelargs play, gather facts after waiting for pre_tasks https://review.opendev.org/c/openstack/tripleo-ansible/+/840384 | 22:32 |
opendevreview | Merged openstack/tripleo-ansible master: Modify cert gen to make it work under fips https://review.opendev.org/c/openstack/tripleo-ansible/+/830308 | 22:32 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Horizon: Fix wrong tag used in puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842644 | 23:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!