| opendevreview | Steve Baker proposed openstack/tripleo-heat-templates master: Give efi_pxe_bootfile_name an accurate name https://review.opendev.org/c/openstack/tripleo-heat-templates/+/843096 | 00:40 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/tripleo-heat-templates master: DNM use pxe boot interface to try grub network boot https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848913 | 00:40 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Remove initial v1 registry entries before installing Ceph https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/848902 | 00:52 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove parameters/resources for undercloud Nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848695 | 04:24 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Get rid of RemovalPoliciesMode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848699 | 04:25 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove redundant check of undercloud Keystone https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848740 | 04:25 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Add missing hidden tags for password/credential/private keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847374 | 04:29 |
| *** ysandeep|out is now known as ysandeep|ruck | 04:32 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Neutron: Generate wrapper scripts during config file generation https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848751 | 04:40 |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 04:47 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Neutron: Generate wrapper scripts during config file generation https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848751 | 04:49 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible master: Fix ansible_facts variable validation https://review.opendev.org/c/openstack/tripleo-ansible/+/848924 | 05:11 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: container_puppet_config: Merge volumes correctly https://review.opendev.org/c/openstack/tripleo-ansible/+/848925 | 05:21 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Neutron: Generate wrapper scripts during config file generation https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848751 | 05:21 |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 05:35 | |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart master: Run featureset061 job with tempest-allow https://review.opendev.org/c/openstack/tripleo-quickstart/+/843840 | 06:27 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Ship inventory as a part of tripleo_ansible package https://review.opendev.org/c/openstack/tripleo-ansible/+/847929 | 06:30 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Ship inventory as a part of tripleo_ansible package https://review.opendev.org/c/openstack/tripleo-ansible/+/847929 | 06:36 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [WIP] initial playbook to run deploy compute node https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 06:40 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-quickstart master: Adds note to understand fs56 testing scenario https://review.opendev.org/c/openstack/tripleo-quickstart/+/848927 | 06:40 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [WIP] job to deploy additional compute nodes https://review.opendev.org/c/openstack/tripleo-ci/+/843836 | 06:45 |
| Tengu | chandankumar: hello there! do you know what can cause this CI error? https://zuul.opendev.org/t/openstack/build/c6e162901d5e4a3e8397bda106f26d83 | 06:47 |
| Tengu | it's related to the cleaning I'm proposing in the _skeleton_role_ in tripleo-ansible | 06:47 |
| chandankumar | Tengu: checking | 06:48 |
| Tengu | thanks | 06:48 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure bootstrap packages are correctly installed https://review.opendev.org/c/openstack/tripleo-ansible/+/848754 | 06:50 |
| chandankumar | Tengu: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tox.ini#L98 executes the above job | 06:52 |
| chandankumar | Tengu: I think it needs some tweaking to config_podman.yaml here https://opendev.org/openstack/tripleo-ansible/src/branch/master/tox.ini#L109 | 06:52 |
| Tengu | hmm. because there's no default Dockerfile.j2 in the role anymore? | 06:53 |
| Tengu | lemme play with git grep and friends. | 06:53 |
| chandankumar | Tengu: can you try adding --base-config .config/molecule/config_podman.yml | 06:54 |
| chandankumar | at line 98 in tox.ini | 06:54 |
| Tengu | chandankumar: 'k | 06:54 |
| Tengu | 98? that's the [testenv:role-addition] | 06:54 |
| chandankumar | yes | 06:55 |
| Tengu | err... so... [testenv:role-addition] --base-config ... ? | 06:55 |
| Tengu | doesn't sound right. | 06:55 |
| Tengu | you want it line 106 ir 109 I think? | 06:55 |
| chandankumar | sorry | 06:55 |
| chandankumar | Tengu: line 109 | 06:56 |
| Tengu | sounds better :). lemme add it | 06:56 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 06:56 |
| chandankumar | Tengu: if platform arrtibute is not found in molecule.yml then it fallback to default podman molecule config coming from podman driver https://github.com/ansible-community/molecule-podman/blob/main/src/molecule_podman/playbooks/create.yml#L47 | 06:58 |
| Tengu | hmm ok. | 06:58 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 07:02 |
| *** amoralej|off is now known as amoralej | 07:04 | |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 07:05 | |
| chandankumar | Tengu: Sorry I think we need to update the command like this molecule --base-config .config/molecule/config_podman.yml test --all; \ | 07:08 |
| Tengu | ah, wasn't sure. | 07:09 |
| Tengu | let's see. | 07:09 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 07:09 |
| Tengu | brb | 07:09 |
| chandankumar | Tengu: need to fix the path for config_podman.yml - {toxinidir}/.config/molecule/config_podman.yml | 07:31 |
| chandankumar | https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 07:31 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [WIP] initial playbook to run deploy compute node https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 07:34 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 07:36 |
| Tengu | chandankumar: done - sorry, posteman and things :) | 07:36 |
| chandankumar | :-) hope it works this time | 07:39 |
| Tengu | we'll see. | 07:39 |
| Tengu | finger crossed ;). | 07:39 |
| chandankumar | Tengu: this time it worked https://zuul.opendev.org/t/openstack/build/5ee8e55f042a40378e171689701d4cdc failed at other places, I know you can fix it | 07:50 |
| Tengu | chandankumar: not good. lemme check the logs. | 07:50 |
| Tengu | ah | 07:50 |
| Tengu | o_O what's that /opt/yum.repos.d ? | 07:50 |
| chandankumar | need to execute this playbook https://opendev.org/openstack/tripleo-ansible/src/branch/master/zuul.d/playbooks/pre.yml#L89 | 07:51 |
| Tengu | is it something expected to exist? | 07:51 |
| Tengu | ah! | 07:51 |
| chandankumar | may be we can copy this task in role-addition playbook | 07:51 |
| Tengu | nope | 07:51 |
| Tengu | that would do bad things on dev env. | 07:51 |
| chandankumar | Tengu: may be add a pre-run playbook here https://opendev.org/openstack/tripleo-ansible/src/branch/master/zuul.d/molecule.yaml#L718 ? | 07:53 |
| Tengu | adding ansible-playbook -i localhost, zuul.d/playbooks/pre.yml before the "cd {toxinidir}/...." would be OK. | 07:53 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 07:53 |
| Tengu | let's see how that one goes. | 07:54 |
| chandankumar | yup | 07:54 |
| Tengu | the role-addition.yml playbook is fine. | 07:54 |
| Tengu | that's the molecule run right after that. | 07:54 |
| chandankumar | yup | 07:54 |
| Tengu | MAYBE we can even source the script/run-local-test | 07:54 |
| Tengu | iirc, that one is faking some stuff. | 07:54 |
| Tengu | but running that pre.yaml would be OK imho. | 07:54 |
| chandankumar | yup hope pre.yaml solves the issue | 07:55 |
| ykarel | chandankumar, hi | 08:00 |
| ykarel | can you check https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/848844 | 08:00 |
| chandankumar | ykarel: done! | 08:02 |
| ykarel | thanks chandankumar | 08:03 |
| Tengu | chandankumar: so far so good... let's see. | 08:04 |
| Tengu | job's still running at least :). let's be positive. | 08:04 |
| Tengu | f*ck. | 08:06 |
| chandankumar | Failed to connect to the host via ssh: Host key verification failed | 08:08 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 08:08 |
| Tengu | nope. | 08:08 |
| Tengu | stderr: 'Error: statfs /opt/yum.repos.d: no such file or directory' | 08:08 |
| Tengu | again | 08:08 |
| Tengu | as if the "pre.yaml" wasn't launched at all. | 08:08 |
| Tengu | I've moved it earlier in the commands list. | 08:08 |
| chandankumar | ok | 08:08 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Remove rhos-17 multinode job from skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/848844 | 08:09 |
| Tengu | brb - need some water... | 08:11 |
| Tengu | humf. | 08:20 |
| Tengu | ok. -c local | 08:20 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 08:21 |
| Tengu | chandankumar: hump. I guess I'll create a new playbook in the zuul.d/playbooks in order to run that specific job. | 08:32 |
| Tengu | that will be easier to maintain anyway. | 08:32 |
| Tengu | gimme a few. | 08:32 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [WIP] initial playbook to run deploy compute node https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 08:35 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 08:36 |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 08:37 | |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/848857 | 08:43 |
| Tengu | seriously.... | 08:51 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 09:07 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 09:30 |
| Tengu | chandankumar: it's painful.... | 09:31 |
| Tengu | chandankumar: I may do some refactoring in order to get some of the tasks in a dedicated playbook, namely the ones that are just creating directories and copying some content. | 09:31 |
| chandankumar | Tengu: yes, I can understand your feeling | 09:32 |
| chandankumar | Tengu: sounds good about refactoring, may be we can do it later. | 09:33 |
| Tengu | chandankumar: I'm testing with a tag, but I'm pretty sure a proper separation will be needed. | 09:33 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 09:50 |
| opendevreview | Merged openstack/tripleo-common master: Remove unused heat-docker-agent https://review.opendev.org/c/openstack/tripleo-common/+/848799 | 09:54 |
| opendevreview | Merged openstack/tripleo-ci master: Removal of tempest_whitelist variable https://review.opendev.org/c/openstack/tripleo-ci/+/848097 | 09:54 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [WIP] initial playbook to run deploy compute node https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 09:55 |
| *** ysandeep|ruck is now known as ysandeep|ruck|brb | 10:06 | |
| *** ysandeep|ruck|brb is now known as ysandeep|ruck | 10:27 | |
| *** rlandy|out is now known as rlandy | 10:28 | |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [WIP] initial playbook to run deploy compute node https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 10:52 |
| *** ysandeep|ruck is now known as ysandeep|ruck|afk | 11:01 | |
| Tengu | chandankumar: almost there - though it's still unhappy in some OTHER jobs now -.- | 11:04 |
| Tengu | I'll find out. | 11:04 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 11:05 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 11:11 |
| Tengu | lovely. the reported error doesn't mean anything... | 11:11 |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade master: Monitor ceph update status with 'ceph orch upgrade status' cmd https://review.opendev.org/c/openstack/tripleo-upgrade/+/848944 | 11:12 |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade stable/wallaby: Monitor ceph update status with 'ceph orch upgrade status' cmd https://review.opendev.org/c/openstack/tripleo-upgrade/+/848863 | 11:13 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 11:16 |
| Tengu | 'k, got it. we can't import a playbook from within tasks or pretasks. #learning | 11:16 |
| *** pojadhav is now known as pojadhav|afk | 11:16 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 11:18 |
| Tengu | chandankumar: instead of a playbook (useless), directories are created in a dedicated tasks file. Should be cleaner anyway, especially for the inclusion. | 11:19 |
| *** dviroel|out is now known as dviroel | 11:27 | |
| Tengu | need some nap, or more coffee. | 11:31 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 11:31 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ansible stable/wallaby: WIP add new ci network template for 2 node overcloud https://review.opendev.org/c/openstack/tripleo-ansible/+/848946 | 11:31 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 11:32 |
| opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/wallaby: wip Add oslo messaging pass through template for multi stack deploy https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847962 | 11:32 |
| 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 | 11:33 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove parameters/resources for undercloud Nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848695 | 11:34 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Get rid of RemovalPoliciesMode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848699 | 11:34 |
| chandankumar | Tengu: ah, the change is getting bigger, you are cleaning it up to fix it | 11:40 |
| Tengu | chandankumar: yeah - sorry. | 11:40 |
| Tengu | should be fine now. | 11:40 |
| Tengu | at least now the syntax is correct, just need to wait for the role-addition job, but it should be just fine now. | 11:41 |
| Tengu | yessss | 11:41 |
| Tengu | we're good now! | 11:41 |
| chandankumar | Tengu: you made it https://zuul.opendev.org/t/openstack/build/65e49f5a22554cfc80d7801963a57a02 | 11:41 |
| chandankumar | yay! | 11:41 |
| chandankumar | Tengu: ++++++++++++++++ awesome, thanks :-) | 11:41 |
| Tengu | :) sorry for the huge noise generated by the different tests though... | 11:42 |
| Tengu | still - we should be just fine now. | 11:42 |
| Tengu | and it will be easier to manage imho. | 11:42 |
| chandankumar | Yes, it will be easier now :-) | 11:42 |
| Tengu | ok. now I can check https://review.opendev.org/c/openstack/tripleo-ansible/+/848754 - failed because openvswitch isn't installed for some reason. | 11:43 |
| Tengu | have to discover why. | 11:43 |
| Tengu | especially since it apparently IS installed. | 11:44 |
| Tengu | (and here, we can see how the env_data is useful :)) | 11:44 |
| Tengu | and apparently, python3-pyyaml is broken XD | 11:44 |
| chandankumar | package openvswitch is not installed it is weired | 11:45 |
| Tengu | yep | 11:45 |
| chandankumar | for python3-pyyaml docs files are missing | 11:45 |
| Tengu | fun.. I have them on my current env. | 11:47 |
| Tengu | I... well... what? :) | 11:47 |
| Tengu | chandankumar: ah, I'll do a tiny change to the env_data in order to output the package list in alphabetical order. | 11:48 |
| Tengu | right now, it's a pain in the *** to find anything in the listing. | 11:48 |
| chandankumar | ok | 11:48 |
| Tengu | afaik, we can use the |sort filter in the msg directly. | 11:49 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure package list is sorted https://review.opendev.org/c/openstack/tripleo-ansible/+/848947 | 11:49 |
| *** ysandeep|ruck|afk is now known as ysandeep|ruck | 11:50 | |
| Tengu | chandankumar: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_bootstrap/tasks/packages.yml#L51-L58 | 11:54 |
| Tengu | that's why. | 11:54 |
| Tengu | (for the "no openvswitch") | 11:54 |
| Tengu | I have to make some hacks in the same fashion. | 11:55 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 11:57 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure bootstrap packages are correctly installed https://review.opendev.org/c/openstack/tripleo-ansible/+/848754 | 11:57 |
| Tengu | chandankumar: -^ | 11:58 |
| Tengu | that update should help. though the doc things? that's not cool. | 11:58 |
| Tengu | wondering if there's some dnf/rpm settings preventing some doc install or something. Not sure if this is possible in rhel world. | 11:58 |
| marios | hjensas: o/ fyi trying with https://review.opendev.org/c/openstack/tripleo-ansible/+/848946 (& https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847962/4/ci/environments/multinode-containers-mixed-os-control.yaml for the bridge mappings) will let you know how it goes & requested new hold on that running now | 12:00 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Do not access to ansible facts by vars https://review.opendev.org/c/openstack/tripleo-ansible/+/848228 | 12:07 |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Designate: create proxy ports on external VIP to access miniDNS workers https://review.opendev.org/c/openstack/puppet-tripleo/+/848733 | 12:10 |
| Tengu | chandankumar: job seems all good now: https://zuul.opendev.org/t/openstack/status#848813 "just" the non-voting one failing, but it's known, and unrelated to the actual change this time. | 12:12 |
| Tengu | guess we'll be able to merge it once zuul's over. | 12:13 |
| *** amoralej is now known as amoralej|lunch | 12:13 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-ci master: Create containers and standalone-ipa multinode -options jobs https://review.opendev.org/c/openstack/tripleo-ci/+/848260 | 12:13 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 12:16 |
| chandankumar | Tengu: yup, good to merge that :-) | 12:23 |
| opendevreview | John Fulton proposed openstack/tripleo-docs master: Update Ceph documentation examples based on lab feedback https://review.opendev.org/c/openstack/tripleo-docs/+/848951 | 12:24 |
| opendevreview | James Slagle proposed openstack/tripleo-ci master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ci/+/848834 | 12:29 |
| *** pojadhav|afk is now known as pojadhav | 12:30 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure package list is sorted https://review.opendev.org/c/openstack/tripleo-ansible/+/848947 | 12:30 |
| Tengu | chandankumar: https://review.opendev.org/c/openstack/tripleo-ansible/+/848947 is good now. | 12:43 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Add template file for bonding + controller node without external net https://review.opendev.org/c/openstack/tripleo-ansible/+/848953 | 12:47 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [WIP] initial playbook to run deploy compute node https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/843835 | 12:51 |
| Tengu | tkajinam: uho - thought you were off today, sorry, I pushed a change onto your ansible fact related patch in tripleo-ansible without asking | 12:52 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace remaining usage of NetSoftwareConfig https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848956 | 12:52 |
| tkajinam | Tengu, np and I'm actually "off" today. thanks for updating it. It was about to drop from my todos... | 12:53 |
| Tengu | tkajinam: heh ;) | 12:54 |
| Tengu | tkajinam: the "'foo' not in bar" is a nasty one in ansible. | 12:55 |
| tkajinam | I'm always confused by "" or {{}} in ansible ... X-( | 12:56 |
| tkajinam | for that fact gather stuff, we might want to implement a generic task and use it consistently | 12:56 |
| tkajinam | currently we gather different set of facts in individual roles but that might cause a problem if two tasks require different set of facts | 12:56 |
| tkajinam | and not ensuring all of the required facts correctly | 12:57 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure bootstrap packages are correctly installed https://review.opendev.org/c/openstack/tripleo-ansible/+/848754 | 12:57 |
| tkajinam | it would not be a problem during deployment as we likely to gather "min" which covers almost all we need | 12:57 |
| Tengu | tkajinam: sooo... it may cause an issue if we're calling the roles from outside tripleo, or if we're missing fact gathering at higher level, such as the new standalone roles/playbooks/others. | 12:58 |
| *** dasm|off is now known as dasm | 12:58 | |
| Tengu | tkajinam: also, we're limiting the gathering to the essential content, in order to not lose time gathering huge data, such as the networking stuff (especially true for Controllers and Compute, iirc) | 12:59 |
| Tengu | those may expose hundreds of interfaces, and ansible will take a really long time to gether all the things :/ | 12:59 |
| opendevreview | Juan Larriba proposed openstack/tripleo-heat-templates stable/wallaby: Remove the Backup and Restore ReaR Heat templates https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848850 | 13:00 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Extract provisioned nodes - use ironic node uuid https://review.opendev.org/c/openstack/python-tripleoclient/+/848447 | 13:00 |
| tkajinam | yeah that was one of the known factors delaying puppet and ansible | 13:00 |
| tkajinam | (we solved it by fact cache in puppet and that limited scope in ansible | 13:00 |
| Tengu | yup | 13:01 |
| Tengu | and we don't want to go back on the limited scope, especially with the incomine AEE imho. | 13:02 |
| Tengu | that will be slow enough. | 13:02 |
| Tengu | THAT SAID. I was thinking about a way to keep the AEE running and just waiting for commands. That would avoid losing 10s for each playbook to bootstrap the env. | 13:02 |
| Tengu | though it's probably not supported by the ansible-runner... | 13:02 |
| Tengu | but doing so should provide a near-current perf with the AEE. | 13:03 |
| tkajinam | yeah | 13:04 |
| tkajinam | just wondering what make it take such long. AFAIK environment is created as a container so it should be more light weighted. maybe module loading is slow | 13:04 |
| tkajinam | we've seen a similar thing with openstackclient | 13:04 |
| Tengu | tkajinam: basically the container is stopped, and must start for every single playbook - and loading modules and related things isn't fast (python issue, I'd say, at this point, and agravated by ansible itself) | 13:07 |
| tkajinam | that makes sense | 13:08 |
| Tengu | ~> if we can keep the container running, with everything loaded, that would really help. I'll try to find some time to investigate. | 13:08 |
| Tengu | I discussed this with cloudnull back then, he also aggreed it was a path to follow for better perfs with AEE. | 13:08 |
| Tengu | though it would require modifications in ansible-runner, and they may be rejected for #whateverReasons | 13:09 |
| Tengu | like mitogen was rejected back then.... | 13:09 |
| tkajinam | the change sounds reasonable and generic improvement (non-specific to tripleo) though they might have their own view | 13:09 |
| Tengu | if this happens, the other solution would be to extend ansible-runner within tripleo - but that would be a terrible thing, since it would more than probably explode with each new ansible-runner release | 13:10 |
| tkajinam | yeah | 13:10 |
| Tengu | tkajinam: they may point to Tower for the playbook management thing. | 13:10 |
| tkajinam | so shall we pull AWX to tripleo ? :-) | 13:10 |
| Tengu | or something like that. Not sure. anyway. I'll try to get some time to dig into the code and see how it's done right now. | 13:10 |
| tkajinam | and run ansible tasks via AWX | 13:10 |
| Tengu | well, they may reject and advice this. | 13:11 |
| Tengu | if this is the case, it will be really, really fun. | 13:11 |
| *** amoralej|lunch is now known as amoralej | 13:14 | |
| tkajinam | fun time to play with a new stuff !!! | 13:14 |
| tkajinam | X-P | 13:15 |
| Tengu | marios, slagle I could correct the change for https://review.opendev.org/c/openstack/tripleo-ansible/+/848754 (tripleo_bootstrap validating package install) | 13:15 |
| Tengu | tkajinam: yeah :). | 13:15 |
| Tengu | now I have to understand why https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 is failing.... apparently, there's something wrong with the way I set auditd, but only for the molecule "check". Really weird. re-running the role 10 times locally doesn't show the same behavior. | 13:16 |
| marios | Tengu: o/ in call right now but will revisit that on next reviews | 13:27 |
| marios | (if it is still there ;)) | 13:27 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 13:30 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: [update] Tell subscription-manager it's not running in a container. https://review.opendev.org/c/openstack/tripleo-ansible/+/848744 | 13:32 |
| Tengu | bogdando: heya! PTAL https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 - thanks! | 13:36 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 13:37 |
| Tengu | since apparently there was a merge conflict detected... weird. | 13:38 |
| Tengu | oh. actually..... there's a bug in the gerrit dashboard :D | 13:38 |
| Tengu | when you have a patch marked as WIP, it will show the "Merge Conflict" as well in the status. | 13:38 |
| Tengu | NICE. | 13:38 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 13:40 |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade master: [WIP][update] Include tobiko test for functional testing during update. https://review.opendev.org/c/openstack/tripleo-upgrade/+/817764 | 13:51 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Allow for jinja2 templating of docker_config https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848960 | 14:04 |
| chandankumar | slagle: hello | 14:16 |
| chandankumar | slagle: I am trying to add the job to deploy compute node | 14:16 |
| chandankumar | slagle: can you check the logs https://f8212c96d08995ff4837-2c633db3240f150f6ddf18824372371d.ssl.cf5.rackcdn.com/843836/8/check/tripleo-ci-centos-9-standalone-additional-compute/b1f0730/logs/undercloud/home/zuul/overcloud_compute.log | 14:16 |
| chandankumar | I am doing it in the correct direction | 14:17 |
| marios | tkajinam: sorry :( i forgot to thank you too you helped me out with the RPC stuff | 14:18 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [WIP] job to deploy additional compute nodes https://review.opendev.org/c/openstack/tripleo-ci/+/843836 | 14:19 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Ship inventory as a part of tripleo_ansible package https://review.opendev.org/c/openstack/tripleo-ansible/+/847929 | 14:22 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [WIP] job to deploy additional compute nodes https://review.opendev.org/c/openstack/tripleo-ci/+/843836 | 14:22 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 14:37 |
| opendevreview | yatin proposed openstack/tripleo-quickstart master: [DNM] C8 OVS/OVN NFV latest test https://review.opendev.org/c/openstack/tripleo-quickstart/+/848975 | 15:16 |
| opendevreview | yatin proposed openstack/tripleo-quickstart master: [DNM] C8 OVS/OVN NFV latest test https://review.opendev.org/c/openstack/tripleo-quickstart/+/848975 | 15:18 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Nova: Start nova-compute after its NFS backend is ready https://review.opendev.org/c/openstack/tripleo-heat-templates/+/845162 | 15:41 |
| *** marios is now known as marios|out | 15:52 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace remaining usage of NetSoftwareConfig https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848956 | 15:56 |
| opendevreview | Takashi Kajinami proposed openstack/paunch stable/train: Ensure the dependent service is started in advance https://review.opendev.org/c/openstack/paunch/+/848982 | 16:09 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible stable/train: Ensure the dependent service is started in advance https://review.opendev.org/c/openstack/tripleo-ansible/+/848983 | 16:13 |
| beagles | slagle, if you get a sec https://review.opendev.org/q/topic:designate-wallaby-mdns and https://review.opendev.org/q/topic:designate-wallaby-byob ? | 16:17 |
| opendevreview | Rob Tongue proposed openstack/diskimage-builder master: Adding a feature to enable services in the post-install phase. This is good for Gentoo only currently, with systemd service units and openrc services in the default runlevel. https://review.opendev.org/c/openstack/diskimage-builder/+/848984 | 16:17 |
| *** amoralej is now known as amoralej|off | 16:22 | |
| *** ysandeep|ruck is now known as ysandeep|out | 16:28 | |
| opendevreview | Merged openstack/tripleo-validations master: Fix to get HOME environment in dhcp-provisioning validation https://review.opendev.org/c/openstack/tripleo-validations/+/846589 | 16:32 |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Fix typos in parameter descriptions, comments and tests https://review.opendev.org/c/openstack/puppet-tripleo/+/848988 | 16:35 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Fix wrong separator(,) used to list multiple dependent services https://review.opendev.org/c/openstack/tripleo-ansible/+/848989 | 16:50 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Fix wrong separator(,) used to list multiple dependent services https://review.opendev.org/c/openstack/tripleo-ansible/+/848989 | 16:56 |
| *** dviroel is now known as dviroel|lunch | 17:13 | |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Add handling for OVN Multi-RHEL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848208 | 17:28 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Add handling for OVN Multi-RHEL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848208 | 17:32 |
| opendevreview | dasm proposed openstack/openstack-tempest-skiplist master: Check if `installers` argument is provided https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/848472 | 17:41 |
| opendevreview | dasm proposed openstack/openstack-tempest-skiplist master: Check if `installers` argument is provided https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/848472 | 17:52 |
| opendevreview | Rob Tongue proposed openstack/diskimage-builder master: Adding a feature to enable services in the post-install phase. This is good for Gentoo only currently, with systemd service units and openrc services in the default runlevel. https://review.opendev.org/c/openstack/diskimage-builder/+/848984 | 18:08 |
| *** dviroel|lunch is now known as dviroel | 18:11 | |
| opendevreview | Ronelle Landy proposed openstack/openstack-tempest-skiplist master: Add featureset027 group into tempest-allow https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/843870 | 19:12 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Add featureset027 group into tempest-allow https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/843870 | 19:28 |
| opendevreview | dasm proposed openstack/tripleo-quickstart master: Run featureset030 job with tempest-allow https://review.opendev.org/c/openstack/tripleo-quickstart/+/844453 | 20:06 |
| slagle | rlandy: hi, just a heads up, i'll be out the next 2 weeks | 20:25 |
| slagle | i'll send a mail out. in case you need the ptl approval, hopefully someone in infra could do it? | 20:25 |
| *** dviroel is now known as dviroel|out | 20:42 | |
| rlandy | slagle: thanks - enjoy the time off | 20:45 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ansible/+/849043 | 20:54 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ansible/+/849044 | 20:56 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ansible/+/849045 | 20:57 |
| *** dasm is now known as dasm|off | 21:02 | |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Only run Ceph network_config_set during initial deployment https://review.opendev.org/c/openstack/tripleo-ansible/+/849048 | 22:00 |
| *** rlandy is now known as rlandy|bbl | 22:44 | |
| opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates stable/wallaby: Allow configuring the base port of the mini DNS proxy endpoints https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848878 | 23:34 |
| opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates stable/wallaby: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848895 | 23:34 |
| *** rcastillo|rover is now known as rcastillo | 23:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!