| *** dviroel|out is now known as dviroel | 00:05 | |
| opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart-extras master: Include dependency vars on subnodes https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834972 | 00:06 |
|---|---|---|
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Adds compose-repos pre playbook on base jobs https://review.opendev.org/c/openstack/tripleo-ci/+/834515 | 00:15 |
| opendevreview | Merged openstack/tripleo-ci master: Call container-registry role at required job level https://review.opendev.org/c/openstack/tripleo-ci/+/835101 | 00:17 |
| *** dviroel is now known as dviroel|out | 00:20 | |
| *** rlandy|bbl is now known as rlandy|out | 01:17 | |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Format [keystone_authtoken] memcache_servers when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834142 | 01:39 |
| *** soniya29 is now known as soniya29|rover | 04:19 | |
| *** marios is now known as marios|ruck | 04:58 | |
| *** ysandeep|out is now known as ysandeep | 05:05 | |
| *** pojadhav|afk is now known as pojadhav | 05:11 | |
| opendevreview | Harald Jensås proposed openstack/puppet-tripleo stable/train: Format [oslo_cache] memcache_server when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834884 | 06:02 |
| opendevreview | Rajesh Tailor proposed openstack/tripleo-heat-templates master: Deprecate `NovaVGPUTypesDeviceAddressesMapping parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835440 | 06:04 |
| opendevreview | Harald Jensås proposed openstack/puppet-tripleo stable/train: Format [oslo_cache] memcache_server when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834884 | 06:04 |
| opendevreview | Harald Jensås proposed openstack/puppet-tripleo stable/train: Format [oslo_cache] memcache_server when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834884 | 06:06 |
| opendevreview | yatin proposed openstack/tripleo-quickstart-extras master: [DNM] enable ovn debug https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835597 | 06:07 |
| opendevreview | Harald Jensås proposed openstack/puppet-tripleo stable/train: Format [keystone_authtoken] memcache_servers when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834883 | 06:17 |
| opendevreview | Harald Jensås proposed openstack/puppet-tripleo stable/train: Format [oslo_cache] memcache_server when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834884 | 06:18 |
| opendevreview | Harald Jensås proposed openstack/puppet-tripleo stable/train: Format [oslo_cache] memcache_server when IPv6 is used https://review.opendev.org/c/openstack/puppet-tripleo/+/834884 | 06:21 |
| opendevreview | Merged openstack/tripleo-docs master: Add initial doc for compose pinning dependency jobs https://review.opendev.org/c/openstack/tripleo-docs/+/834957 | 06:37 |
| ysandeep | marios|ruck soniya29|rover: Seeing some NODE_FAILURE issues today: https://zuul.opendev.org/t/openstack/builds?result=NODE_FAILURE | 06:38 |
| * ysandeep not sure what percentage of jobs are hitting issue, but worth to keep an eye incase we hit more NODE_FAILURE. | 06:39 | |
| ysandeep | seeing some more node_failure in running jobs, for ex: https://zuul.opendev.org/t/openstack/status#835500 | 06:43 |
| marios|ruck | ysandeep: looking thanks | 06:46 |
| marios|ruck | unfortunately no logs ysandeep cant even see which cloud | 06:49 |
| ysandeep | marios|ruck: thanks! I see you have reached out to infra already, lets see if they can find anything. | 06:53 |
| *** amoralej|off is now known as amoralej | 07:13 | |
| *** ysandeep is now known as ysandeep|lunch | 07:23 | |
| *** jpena|off is now known as jpena | 07:34 | |
| Tengu | uho.... RuntimeError: This module requires ansible-collections-openstack when trying to provision OC node today. Is this known already? | 08:09 |
| *** gthiemon1e is now known as gthiemonge | 08:10 | |
| Tengu | marios|ruck: -^^does it ring any bell? | 08:12 |
| Tengu | uho. maybe a missing (new) dependency on tripleo-ansible? | 08:16 |
| Tengu | ah. better.... it's installed already. | 08:18 |
| Tengu | hmmm. | 08:18 |
| marios|ruck | Tengu: not seen that one around | 08:19 |
| Tengu | weird. collection IS installed, but something is masking it. | 08:19 |
| hjensas | tkajinam: do we need to backport https://review.opendev.org/c/openstack/puppet-tripleo/+/835589 for the memcache backports to land? | 08:22 |
| Tengu | and the affected file (/usr/share/ansible/plugins/modules/metalsmith_instances.py) wasn't change for ages..... | 08:22 |
| tkajinam | hjensas, I believe we don't need that. setup.py stuff only affects ubuntu job but the job has setup.py pinned by global constraints | 08:25 |
| tkajinam | release note job in ubuntu | 08:25 |
| hjensas | tkajinam: ok, then I won't propose a backport. Thanks! | 08:26 |
| tkajinam | hjensas, ack :-) | 08:28 |
| opendevreview | Slawek Kaplonski proposed openstack/tripleo-heat-templates stable/wallaby: Don't add conntrack entries for vxlan https://review.opendev.org/c/openstack/tripleo-heat-templates/+/831589 | 08:30 |
| opendevreview | Slawek Kaplonski proposed openstack/tripleo-heat-templates stable/victoria: Don't add conntrack entries for vxlan https://review.opendev.org/c/openstack/tripleo-heat-templates/+/831750 | 08:31 |
| opendevreview | Slawek Kaplonski proposed openstack/tripleo-heat-templates stable/train: Don't add conntrack entries for vxlan https://review.opendev.org/c/openstack/tripleo-heat-templates/+/831734 | 08:33 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 09:20 |
| *** ysandeep|lunch is now known as ysandeep | 10:07 | |
| *** soniya29|rover is now known as soniya29|rover|lunch | 10:08 | |
| opendevreview | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [train-only] Use state file for workaround 1925078 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834285 | 10:17 |
| *** rlandy|out is now known as rlandy | 10:25 | |
| *** soniya29|rover|lunch is now known as soniya29|rover | 10:27 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Memcached: Allow puppet to be aware IPv6 is used https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834597 | 10:37 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Memcached: Allow puppet to be aware IPv6 is used https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834596 | 10:37 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Include dependency vars on subnodes https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834972 | 10:37 |
| marios|ruck | tkajinam: o/ hey can you please check when you have time https://bugs.launchpad.net/tripleo/+bug/1966625 this blocks tripleo component lines | 11:03 |
| marios|ruck | tkajinam: mainly what is the latest/something we can help/push please? | 11:03 |
| opendevreview | Saravanan KR proposed openstack/tripleo-heat-templates stable/wallaby: Add nvidia firstboot yaml to tripleo-heat-template https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834946 | 11:06 |
| *** marios|ruck is now known as marios|ruck|lunch | 11:09 | |
| *** bhagyashris_ is now known as bhagyashris | 11:09 | |
| paramite | Greetings guys, can somebody please push remaining backports of https://review.opendev.org/q/Ie7ee7c90107cfe5bff08f5c778a6273ae9ffcc76 through? | 11:12 |
| *** marios|ruck|lunch is now known as marios|ruck | 11:15 | |
| *** pojadhav is now known as pojadhav|afk | 11:17 | |
| *** soniya29|rover is now known as soniya29|rover|afk | 11:18 | |
| rlandy | hjensas: hi - I think you (maybe steve?)are the only one left with +2 rights on https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/834813 | 11:20 |
| rlandy | can we add ysandeep to that list? | 11:21 |
| rlandy | so he can w+ | 11:21 |
| rlandy | also can you w+^^ | 11:21 |
| *** dviroel_ is now known as dviroel | 11:22 | |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible master: [RFE] Add a new Snapshot And Revert feature to Backup and Restore https://review.opendev.org/c/openstack/tripleo-ansible/+/835961 | 11:23 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Capping validations-libs at 1.6.* https://review.opendev.org/c/openstack/python-tripleoclient/+/835962 | 11:35 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: [Train-only] Capping validations-libs at 1.6.* https://review.opendev.org/c/openstack/python-tripleoclient/+/835962 | 11:36 |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/victoria: Remove additional dependency from pre-commit https://review.opendev.org/c/openstack/tripleo-upgrade/+/835972 | 11:44 |
| *** ysandeep is now known as ysandeep|brb | 11:49 | |
| *** amoralej is now known as amoralej|lunch | 11:54 | |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible master: [RFE] Add a new Snapshot And Revert feature to Backup and Restore https://review.opendev.org/c/openstack/tripleo-ansible/+/835961 | 11:57 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Adds compose-repos pre playbook on base jobs https://review.opendev.org/c/openstack/tripleo-ci/+/834515 | 12:01 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Adds compose-repos pre playbook on base jobs https://review.opendev.org/c/openstack/tripleo-ci/+/834515 | 12:02 |
| *** ysandeep|brb is now known as ysandeep|afk | 12:05 | |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible master: [RFE] Add a new Snapshot And Revert feature to Backup and Restore https://review.opendev.org/c/openstack/tripleo-ansible/+/835961 | 12:07 |
| *** soniya29 is now known as soniya29|rover | 12:07 | |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible master: [RFE] Add a new Snapshot And Revert feature to Backup and Restore https://review.opendev.org/c/openstack/tripleo-ansible/+/835961 | 12:12 |
| *** chem is now known as Guest852 | 12:32 | |
| *** pojadhav|afk is now known as pojadhav | 12:33 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835678 | 12:38 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835796 | 12:38 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835680 | 12:38 |
| opendevreview | Merged openstack/validations-common master: Removing leftover coverchange reference https://review.opendev.org/c/openstack/validations-common/+/834837 | 12:38 |
| beagles | slagle: I proposed a "designate update" to the ptg etherpad if there's room on the schedule | 12:40 |
| *** amoralej|lunch is now known as amoralej | 13:06 | |
| *** ysandeep|afk is now known as ysandeep | 13:14 | |
| opendevreview | Rabi Mishra proposed openstack/tripleo-heat-templates master: Disabling container validation when running redhat-subscription https://review.opendev.org/c/openstack/tripleo-heat-templates/+/831813 | 13:15 |
| slagle | beagles: wednesday ok? | 13:24 |
| beagles | slagle: yes, thanks | 13:30 |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade master: [DNM] to see if we hit job freeze https://review.opendev.org/c/openstack/tripleo-upgrade/+/836021 | 13:47 |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/victoria: Remove additional dependency from pre-commit https://review.opendev.org/c/openstack/tripleo-upgrade/+/835972 | 13:54 |
| *** dasm|off is now known as dasm | 14:00 | |
| *** marios|ruck is now known as marios | 15:00 | |
| *** marios is now known as marios|ruck | 15:01 | |
| *** ysandeep is now known as ysandeep|out | 15:40 | |
| *** dviroel_ is now known as dviroel | 15:54 | |
| marios|ruck | bogdando: o/ hey can you please check https://bugs.launchpad.net/tripleo/+bug/1966625/comments/17 why is wf -1 @ https://review.opendev.org/c/openstack/puppet-tripleo/+/835448 | 15:56 |
| marios|ruck | rlandy: dasm: ^^ | 15:56 |
| dasm | ack | 15:56 |
| bogdando | marios|ruck: because there is no 8.0.1 affected version in that job | 16:02 |
| bogdando | and it fails with puppet-ssh 8.0.0 | 16:02 |
| bogdando | 8.0.1 was not yet promoted (at least by the writing time) | 16:02 |
| marios|ruck | bogdando: ok thanks can you write something in the bug about what we're waiting for (we were discussing during a ruck|rover handoff call we switch shifts tomorrow) thanks bogdando | 16:03 |
| bogdando | I would update it so it should pass | 16:04 |
| opendevreview | Bogdan Dobrelya proposed openstack/puppet-tripleo master: Fix double declaration of ssh::server class https://review.opendev.org/c/openstack/puppet-tripleo/+/835448 | 16:07 |
| opendevreview | Bogdan Dobrelya proposed openstack/puppet-tripleo master: Fix double declaration of ssh::server class https://review.opendev.org/c/openstack/puppet-tripleo/+/835448 | 16:07 |
| *** rlandy is now known as rlandy|rover | 16:09 | |
| *** marios|ruck is now known as marios|out | 16:12 | |
| *** dviroel is now known as dviroel|lunch | 16:14 | |
| opendevreview | Dariusz Smigiel proposed openstack/openstack-tempest-skiplist master: Revert "Skip failing test 'test_image_web_download_import'" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/835974 | 16:23 |
| opendevreview | Dariusz Smigiel proposed openstack/openstack-tempest-skiplist master: Revert "Skip failing test 'test_image_web_download_import'" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/835974 | 16:26 |
| opendevreview | Dariusz Smigiel proposed openstack/openstack-tempest-skiplist master: Revert "Skip failing test 'test_image_web_download_import'" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/835974 | 16:28 |
| *** dasm is now known as dasm|ruck | 16:38 | |
| *** jpena is now known as jpena|off | 16:39 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Add CephNfs service on roles providing "external" network connectivity https://review.opendev.org/c/openstack/tripleo-heat-templates/+/833332 | 16:45 |
| *** amoralej is now known as amoralej|off | 17:15 | |
| *** dviroel|lunch is now known as dviroel | 17:18 | |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade stable/wallaby: Run control plane update first, next compute and cephstorage last https://review.opendev.org/c/openstack/tripleo-upgrade/+/833743 | 17:23 |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade stable/ussuri: Run control plane update first, next compute and cephstorage last https://review.opendev.org/c/openstack/tripleo-upgrade/+/834145 | 17:25 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 17:26 |
| opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352 | 17:30 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Async has no value here https://review.opendev.org/c/openstack/tripleo-ansible/+/836049 | 17:31 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Async has no value here https://review.opendev.org/c/openstack/tripleo-ansible/+/836049 | 17:35 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Async has no value here https://review.opendev.org/c/openstack/tripleo-ansible/+/836049 | 17:37 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Async has no value here https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836051 | 17:40 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Async has no value here https://review.opendev.org/c/openstack/tripleo-ansible/+/836049 | 17:49 |
| opendevreview | Merged openstack/puppet-tripleo stable/ussuri: Add powermax support https://review.opendev.org/c/openstack/puppet-tripleo/+/834936 | 17:50 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Async has no value here https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836051 | 17:51 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 18:11 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-common master: Async has no value here https://review.opendev.org/c/openstack/tripleo-common/+/836076 | 19:09 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Adds compose-repos pre playbook on base jobs https://review.opendev.org/c/openstack/tripleo-ci/+/834515 | 19:17 |
| opendevreview | Merged openstack/tripleo-heat-templates master: libvirt: Fix invalid dependencies when virtlogd wrapper is enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835768 | 19:25 |
| opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352 | 19:48 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Async has no value here https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836051 | 20:14 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Async has no value here https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836051 | 20:20 |
| *** dviroel is now known as dviroel|afk | 20:24 | |
| rlandy|rover | cloudnull: hello ... looks like you have +2 rights on openstack-ansible-os_tempest/ | 20:41 |
| rlandy|rover | dasm|ruck: ^^ | 20:41 |
| rlandy|rover | https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835977 | 20:42 |
| cloudnull | done | 20:42 |
| rlandy|rover | cloudnull: https://bugs.launchpad.net/tripleo/+bug/1967430 | 20:42 |
| rlandy|rover | going to take all our jobs out | 20:42 |
| rlandy|rover | cloudnull: thank you | 20:43 |
| rlandy|rover | we will need a w+ on that when zuul gets done | 20:45 |
| rlandy|rover | looking for other cores | 20:45 |
| cloudnull | rlandy|rover let me know | 20:47 |
| cloudnull | i can merge it if needed | 20:47 |
| rlandy|rover | if I don;t find another core will ping | 20:47 |
| rlandy|rover | not sure it helped | 20:50 |
| rlandy|rover | this may have to be rebuilt | 20:52 |
| rlandy|rover | cloudnull: ^^ would depends-on testing work in this case? | 20:53 |
| rlandy|rover | https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/build/1b28ac882f024d29b6fdd949bd8ef889 failed same way | 20:53 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Support ceph_conf_overrides for multiple_external_ceph_clusters https://review.opendev.org/c/openstack/tripleo-ansible/+/836087 | 20:55 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-heat-templates master: DNM Testing os-tempest https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836088 | 20:57 |
| rlandy|rover | guess that is a no show | 20:59 |
| rlandy|rover | waiting on other fix | 20:59 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: cephadm support for 'openstack overcloud export ceph' https://review.opendev.org/c/openstack/python-tripleoclient/+/835511 | 20:59 |
| rlandy|rover | dasm|ruck: emailing openstack-discuss | 20:59 |
| slagle | cloudnull: hey, for the async thing, if we had 1,000 nodes, we'd start all 1,000 in parallel | 21:02 |
| slagle | and then the poll is based on forks, so if forks=25, we poll 25 at a time, but 1,000 are still running in parallel. so the polls all end "about" the same time | 21:03 |
| slagle | it means these tasks run in ~ constant time, regardless of # of nodes | 21:04 |
| dasm|ruck | rlandy|rover: ack, thx | 21:21 |
| cloudnull | slagle it still starts using forks | 21:24 |
| cloudnull | it just doesn't check the return until the async_status call | 21:24 |
| cloudnull | from the docs "Using a higher value for --forks will result in kicking off asynchronous tasks even faster. This also increases the efficiency of polling." | 21:24 |
| slagle | cloudnull: yes, it uses forks, but it's going to fire and forget 1,000 just as fast as it can | 21:25 |
| slagle | so then 1,000 are running in parallel, for all practical purposes | 21:26 |
| cloudnull | true, it will run then one by one in the background as fast as possible. then block and poll for the results. | 21:27 |
| slagle | right in groups of "forks", so say 25 if forks=25. as soon as the first is done, it will move on to polling the next, which is also likely now done | 21:28 |
| slagle | for a task that takes 5 minutes, instead of taking (nodes/forks * 5) minutes, it takes ~ 5 minutes | 21:31 |
| cloudnull | I believe its still governed by forks even when executing in async, so we're not actually running all tasks simultaneously. but I guess we should time it. | 21:40 |
| cloudnull | async just backgrounds the task so you can run other tasks then poll before moving on to the next roll or playbook | 21:40 |
| cloudnull | async isn't a task level replacement for the free strategy | 21:41 |
| slagle | yea it would be easy to test with 2 nodes, a sleep task and forks=1 | 21:41 |
| slagle | it does run them in parallel, https://paste.opendev.org/show/bKip9R9oer2qvGrNUgeW/ | 21:48 |
| slagle | i didn't poll them, but both were running despite forks=1 | 21:48 |
| dasm|ruck | cloudnull: you +2-ed earlier. Could you do again? https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835977 | 21:53 |
| dasm|ruck | And if you have super powers for: https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/836091 that would be nice to +2 too | 21:53 |
| cloudnull | slagle++ fair enough. local test https://paste.opendev.org/show/b15er52VEAgvuiMWgTwy/ it was faster, albeit not by much. | 21:56 |
| * cloudnull will try and figure out how to get more info out of those tasks | 21:57 | |
| slagle | what was in your tests-async.yaml? | 21:58 |
| dasm|ruck | thx! | 21:58 |
| cloudnull | test playbook https://paste.opendev.org/show/berMaNtPmpKkHCIjuc0q/ | 21:58 |
| cloudnull | the test without that async was just the one command | 21:59 |
| slagle | right, b/c the poll is subject to forks=1 | 21:59 |
| cloudnull | polling added using params similar to the pr | 21:59 |
| slagle | and delay | 21:59 |
| slagle | but yea, at scale, it would approach constant time | 22:00 |
| cloudnull | that said, with no async and forks 2 (my 2 hosts) the no-async example is faster. | 22:01 |
| cloudnull | https://paste.opendev.org/show/b9Q8Z9bvs4YYoDTgACXe/ | 22:02 |
| slagle | well, right :). if forks is always >= nodes, then no reason to use async here | 22:03 |
| cloudnull | and with our default forks (4xCPU or 100 whatever is less) I wonder if it matters ? | 22:03 |
| cloudnull | issue: async is hiding task output | 22:03 |
| cloudnull | assuming we're running an undercloud on 16 core machines == 64 default forks. | 22:04 |
| * cloudnull something to think about | 22:04 | |
| slagle | don't we get the output from the next debug task? | 22:05 |
| slagle | puppet_host_outputs? | 22:05 |
| cloudnull | nope its skipped | 22:07 |
| cloudnull | and its worse in async tasks with loops | 22:08 |
| slagle | which condition causes the skip? the check for rc? maybe we could default that | 22:10 |
| cloudnull | one of the three conditions is faulting, it doesn't say which one. I'll rerun it and see if I can produce something more. | 22:13 |
| opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352 | 22:25 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Override image.http_image on tempestconf https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835475 | 22:47 |
| *** dasm|ruck is now known as dasm|off | 22:59 | |
| opendevreview | Rafael Castillo proposed openstack/tripleo-quickstart master: Install roles with ansible-galaxy https://review.opendev.org/c/openstack/tripleo-quickstart/+/836104 | 23:01 |
| *** rlandy|rover is now known as rlandy|out | 23:08 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!