| *** dviroel|afk is now known as dviroel | 00:08 | |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: [TRAIN-ONLY] Passing NovaComputeOpt{Volumes,EnvVars} to hybrid containers https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866238 | 00:51 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Add limit to container_config data https://review.opendev.org/c/openstack/tripleo-ansible/+/857593 | 01:22 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Use ansible variable to detect bootstrap node https://review.opendev.org/c/openstack/tripleo-heat-templates/+/857714 | 01:24 |
| *** dviroel is now known as dviroel|out | 01:33 | |
| opendevreview | Merged openstack/tripleo-ansible master: tripleo_container_manage: Add retry mechanism https://review.opendev.org/c/openstack/tripleo-ansible/+/858854 | 01:48 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/zed: Fix logs permissions for swtpm https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866015 | 02:26 |
| *** pojadhav|out is now known as pojadhav|ruck | 03:00 | |
| *** yadnesh|away is now known as yadnesh | 04:00 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: Remove CephHciOsdType and CephHciOsdCount parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866164 | 05:29 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-common stable/train: Fix token cache expires_at check https://review.opendev.org/c/openstack/tripleo-common/+/865248 | 05:30 |
| mrjoshi | Hi, looking for reviews on the patch to get merged https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865898 | 06:07 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_ironic: Make wsgi process/thread tunable https://review.opendev.org/c/openstack/tripleo-ansible/+/866290 | 06:10 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_ironic: Make wsgi process/thread tunable https://review.opendev.org/c/openstack/tripleo-ansible/+/866290 | 06:13 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ironic-api: Fix ignored IronicWorkers https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866292 | 06:13 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ci master: Trigger more standalone scenarios when haproxy template is updated https://review.opendev.org/c/openstack/tripleo-ci/+/866294 | 06:28 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ci master: Run more standalone scenarios when haproxy service is updated https://review.opendev.org/c/openstack/tripleo-ci/+/866294 | 06:29 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_ironic: Make wsgi processes/threads tunable https://review.opendev.org/c/openstack/tripleo-ansible/+/866290 | 06:37 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ironic-api: Fix ignored IronicWorkers https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866292 | 06:38 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: overcloud image upload: Drop support for glance https://review.opendev.org/c/openstack/python-tripleoclient/+/864012 | 06:56 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Stop mocking image client https://review.opendev.org/c/openstack/python-tripleoclient/+/866298 | 07:06 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Remove overcloud profiles commands https://review.opendev.org/c/openstack/python-tripleoclient/+/866299 | 07:10 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Remove remaining mocking of compute client https://review.opendev.org/c/openstack/python-tripleoclient/+/866300 | 07:14 |
| tkajinam | https://review.opendev.org/c/openstack/tripleo-ansible/+/857593 | 07:22 |
| tkajinam | https://zuul.opendev.org/t/openstack/build/72a7cdd8c7c34739b160d1f6b4c3a403 | 07:22 |
| tkajinam | building a container image consistently fails with: /bin/sh: error while loading shared libraries: libc.so.6: cannot change memory protections | 07:23 |
| tkajinam | sounds like a recent selinux issue in downstream | 07:24 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Stop mocking image client https://review.opendev.org/c/openstack/python-tripleoclient/+/866298 | 07:25 |
| *** amoralej|off is now known as amoralej | 07:29 | |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Add service options to only undercloud.conf https://review.opendev.org/c/openstack/python-tripleoclient/+/866302 | 07:57 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Add service options to only undercloud.conf https://review.opendev.org/c/openstack/python-tripleoclient/+/866302 | 07:58 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-common stable/train: Fix token cache expires_at check https://review.opendev.org/c/openstack/tripleo-common/+/865248 | 08:05 |
| lmiccini | o/ looking for reviews https://review.opendev.org/c/openstack/puppet-tripleo/+/863041 :) | 08:08 |
| *** ysandeep is now known as ysandeep|lunch | 08:12 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New service role: aodh https://review.opendev.org/c/openstack/tripleo-ansible/+/858674 | 08:17 |
| *** pojadhav|ruck is now known as pojadhav|lunch | 08:26 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Octavia OVN Provider: Fix missing hieradata for NB/SB DB ports https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866303 | 08:28 |
| *** jpena|off is now known as jpena | 08:31 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New service role: aodh https://review.opendev.org/c/openstack/tripleo-ansible/+/858674 | 08:49 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_ironic: Make wsgi processes/threads tunable https://review.opendev.org/c/openstack/tripleo-ansible/+/866290 | 08:49 |
| Tengu | :) | 08:49 |
| Tengu | tkajinam: ah, you want to pass the display_name | 08:50 |
| Tengu | tkajinam: check my last patchset | 08:50 |
| Tengu | else it will show "ironic", not "ironic_wsgi" | 08:50 |
| Tengu | tkajinam: https://review.opendev.org/c/openstack/tripleo-ansible/+/858674/8/tripleo_ansible/roles/tripleo_aodh/tasks/api_httpd.yml#25 | 08:51 |
| Tengu | for instance | 08:51 |
| tkajinam | yeah | 08:52 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_ironic: Make wsgi processes/threads tunable https://review.opendev.org/c/openstack/tripleo-ansible/+/866290 | 08:53 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_ironic: Make wsgi processes/threads tunable https://review.opendev.org/c/openstack/tripleo-ansible/+/866290 | 08:54 |
| tkajinam | Tengu, we need to omit tripleo_aodh_api_httpd_vhost_wsgi_daemon_process by default I believe | 08:55 |
| Tengu | not if we want to fix a sensible default? | 08:57 |
| tkajinam | now it defaults to 1 and overrides tripleo_httpd_vhost_workers | 08:58 |
| tkajinam | while we want to tripleo_httpd_vhost_workers, which is computed based on cores | 08:59 |
| *** yadnesh is now known as yadnesh|afk | 09:00 | |
| *** pojadhav|lunch is now known as pojadhav|ruck | 09:00 | |
| shyamb | Hi, | 09:01 |
| shyamb | Does this call, re-creates cinder volume pacemaker resource with updated image? | 09:01 |
| shyamb | https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/cinder/cinder-volume-pacemaker-puppet.yaml#L213 | 09:01 |
| opendevreview | Juan Larriba proposed openstack/tripleo-common master: Fix tripleo_entrypoint.sh https://review.opendev.org/c/openstack/tripleo-common/+/866305 | 09:01 |
| *** ysandeep|lunch is now known as ysandeep | 09:12 | |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Refactoring default-node-count validation to use openstack-collection modules https://review.opendev.org/c/openstack/tripleo-validations/+/864421 | 09:14 |
| Tengu | tkajinam: sooo. omit? is it something we can mention in the default/main.yml or... ? | 09:30 |
| Tengu | (sorry, was on a call) | 09:30 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New service role: aodh https://review.opendev.org/c/openstack/tripleo-ansible/+/858674 | 09:33 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-ci master: Add a multi-rhel update job. https://review.opendev.org/c/openstack/tripleo-ci/+/865851 | 09:57 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade stable/wallaby: [DNM] Testing a multi-rhel job. https://review.opendev.org/c/openstack/tripleo-upgrade/+/865854 | 09:58 |
| *** yadnesh|afk is now known as yadnesh | 09:58 | |
| opendevreview | Karthik S proposed openstack/os-net-config master: Avoid reconfiguring SR-IOV devices during stack update https://review.opendev.org/c/openstack/os-net-config/+/866312 | 10:07 |
| damani | hi | 10:16 |
| damani | i think this patch can be merge https://review.opendev.org/c/openstack/python-tripleoclient/+/851856 | 10:17 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: Added osp18 option for releases in get hash https://review.opendev.org/c/openstack/tripleo-repos/+/866316 | 10:22 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 10:29 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-ci master: Add rhos-18 settings for container builds https://review.opendev.org/c/openstack/tripleo-ci/+/866319 | 10:35 |
| Tengu | damani: you didn't address marios comment actually. | 10:35 |
| Tengu | care to do? | 10:35 |
| *** dviroel|out is now known as dviroel | 10:43 | |
| opendevreview | Jiri Podivin proposed openstack/tripleo-common master: New constant for THT system path and THT roles directory https://review.opendev.org/c/openstack/tripleo-common/+/866324 | 10:56 |
| *** rlandy|out is now known as rlandy | 11:20 | |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Don't use meta:end_play after growvols check https://review.opendev.org/c/openstack/tripleo-ansible/+/864445 | 11:41 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Configure OVN DB connections by ansible https://review.opendev.org/c/openstack/tripleo-heat-templates/+/853658 | 11:41 |
| opendevreview | Merged openstack/validations-libs stable/1.6: [stable only] Rename linters and cover jobs with validations-libs prefix https://review.opendev.org/c/openstack/validations-libs/+/865806 | 11:41 |
| rlandy | ianw: hi - when you get in - can you review the failures on https://review.opendev.org/c/openstack/diskimage-builder/+/850882 - chatted with stevebaker[m] about it. | 11:52 |
| opendevreview | Merged openstack/validations-common stable/1.6: [stable/1.6 only] ubuntu focal nodeset for the linter and cover jobs https://review.opendev.org/c/openstack/validations-common/+/865657 | 11:53 |
| opendevreview | Fernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes https://review.opendev.org/c/openstack/tripleo-ansible/+/862795 | 12:16 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs stable/1.6: Adding yamllint configuration file and fixing style violations https://review.opendev.org/c/openstack/validations-libs/+/865677 | 12:21 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs stable/1.6: Minor style violations fix https://review.opendev.org/c/openstack/validations-libs/+/865678 | 12:21 |
| shyam2389 | Hi mwhahaha: | 12:22 |
| shyam2389 | Does this code block, updated pacemaker resource if image gets updated? | 12:23 |
| shyam2389 | https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/cinder/cinder-volume-pacemaker-puppet.yaml#L210-L215 | 12:23 |
| shyam2389 | I tried same block for trilio pacemaker resource, but it's not updating the resource in case underlying image changes. | 12:26 |
| shyam2389 | https://github.com/shyam-biradar/triliovault-cfg-scripts/blob/interop-777/redhat-director-scripts/rhosp16/services/triliovault-wlm-cron-pacemaker.yaml#L430-L445 | 12:26 |
| shyam2389 | owalsh: ^^ | 12:26 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Fail if we cannot rename nics on all nodes. https://review.opendev.org/c/openstack/tripleo-upgrade/+/866331 | 12:31 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Do not rename nic interfaces https://review.opendev.org/c/openstack/tripleo-upgrade/+/866332 | 12:39 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Fail if we cannot rename nics on all nodes. https://review.opendev.org/c/openstack/tripleo-upgrade/+/866331 | 12:59 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-common stable/wallaby: Export default location of password file https://review.opendev.org/c/openstack/tripleo-common/+/864201 | 13:03 |
| *** rlandy is now known as rlandy|brb | 13:03 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 13:08 |
| *** ysandeep is now known as ysandeep|afk | 13:12 | |
| *** rlandy|brb is now known as rlandy | 13:18 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 13:18 |
| opendevreview | Fernando Díaz Bravo proposed openstack/python-tripleoclient master: bnr - Deprecation note https://review.opendev.org/c/openstack/python-tripleoclient/+/866334 | 13:23 |
| marios | chem: (just wondering if this rings any bells, didn't file bug yet) any idea about that one The conditional check ''{{ playbook_dir }}/{{ _task_file_path }}' is exists' failed @ https://9c09d6d31505666aa8c8-1c29e72385050ff895c097b41af27e3b.ssl.cf2.rackcdn.com/865052/4/check/tripleo-ci-centos-9-undercloud-upgrade-zed/9e332d6/logs/undercloud/home/zuul/undercloud_upgrade.log | 13:25 |
| marios | chem: context is we are trying to add undercloud-upgrade zed job | 13:26 |
| chem | marios: interesting, that shouldn't happen. Something changed in the way the file are generated maybe. Let me have a look. | 13:26 |
| chem | marios: really strange stuff | 13:27 |
| marios | chem: thx, we'll file a bug with more info today ... thought I'd ask incase you've seen it. weird cos the file _is_ there https://9c09d6d31505666aa8c8-1c29e72385050ff895c097b41af27e3b.ssl.cf2.rackcdn.com/865052/4/check/tripleo-ci-centos-9-undercloud-upgrade-zed/9e332d6/logs/undercloud/home/zuul/tripleo-deploy/undercloud/undercloud-ansible-zrmj8k5s/Undercloud/upgrade_tasks_step4.yaml | 13:28 |
| chem | marios: yeah, the error seems to lead to a syntax error problem | 13:28 |
| marios | chem: but cant see a syntax issue there either https://9c09d6d31505666aa8c8-1c29e72385050ff895c097b41af27e3b.ssl.cf2.rackcdn.com/865052/4/check/tripleo-ci-centos-9-undercloud-upgrade-zed/9e332d6/logs/undercloud/home/zuul/tripleo-deploy/undercloud/undercloud-ansible-zrmj8k5s/upgrade_steps_playbook.yaml | 13:28 |
| marios | chem: ok thanks for checking for now anyway, maybe we have config issue on the job will continue digging for now and will file the bug | 13:29 |
| *** frenzy_friday|rover is now known as frenzy_friday|rover|lunch | 13:33 | |
| chem | marios: just FYI the file in l125 is https://9c09d6d31505666aa8c8-1c29e72385050ff895c097b41af27e3b.ssl.cf2.rackcdn.com/865052/4/check/tripleo-ci-centos-9-undercloud-upgrade-zed/9e332d6/logs/undercloud/home/zuul/tripleo-deploy/undercloud/undercloud-ansible-zrmj8k5s/Undercloud/upgrade_tasks_step4.yaml | 13:34 |
| chem | marios: doesn't change the fact that it indeed exists :) | 13:34 |
| chem | marios: fun fact in that the same code block check was done for step0, ... | 13:35 |
| marios | chem: ack thanks yeah i mentioned it above "weird cos the file _is_ there... " | 13:35 |
| *** dasm|off is now known as dasm | 13:46 | |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 13:50 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 13:51 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 13:52 |
| *** yadnesh is now known as yadnesh|away | 13:55 | |
| opendevreview | Luca Miccini proposed openstack/puppet-tripleo master: Enable fence_watchdog configuration in stonith topology https://review.opendev.org/c/openstack/puppet-tripleo/+/863041 | 13:59 |
| *** frenzy_friday|rover|lunch is now known as frenzy_friday|rover | 14:01 | |
| *** amoralej is now known as amoralej|lunch | 14:03 | |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-quickstart master: Add featureset068 for multi-rhel update. https://review.opendev.org/c/openstack/tripleo-quickstart/+/866131 | 14:05 |
| *** ysandeep|afk is now known as ysandeep | 14:07 | |
| csibbitt | Hi y'all; I'd like to request a final review + merge on a patch to fix CI tests. https://review.opendev.org/c/openstack/tripleo-ansible/+/865465 (master) and https://review.opendev.org/c/openstack/tripleo-ansible/+/865888 (zed) | 14:21 |
| csibbitt | I'm still working on fixing up the wallaby backport, but those are good to go, passing tests, and have already been through a round of review | 14:22 |
| opendevreview | Luca Miccini proposed openstack/puppet-tripleo master: Enable fence_watchdog configuration in stonith topology https://review.opendev.org/c/openstack/puppet-tripleo/+/863041 | 14:32 |
| *** pojadhav|ruck is now known as pojadhav|afk | 14:42 | |
| *** amoralej|lunch is now known as amoralej | 14:42 | |
| opendevreview | John Fulton proposed openstack/tripleo-heat-templates stable/zed: Remove CephHciOsdType and CephHciOsdCount parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866350 | 14:43 |
| opendevreview | Luca Miccini proposed openstack/puppet-tripleo master: Enable fence_watchdog configuration in stonith topology https://review.opendev.org/c/openstack/puppet-tripleo/+/863041 | 14:45 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 14:45 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: Provide a fallback value for httpd wsgi workers https://review.opendev.org/c/openstack/tripleo-ansible/+/866163 | 14:46 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ansible master: New tripleo_nova role https://review.opendev.org/c/openstack/tripleo-ansible/+/866338 | 14:46 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Convert Nova httpd configurations to new role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866177 | 14:48 |
| Tengu | bogdando: I don't understand your patch about the workers... | 14:49 |
| Tengu | what's the point to set it to 0 and fallback onto the equation if it's set to 0? | 14:49 |
| Tengu | oh. ok. when something from t-h-t defaults to 0. ok. | 14:51 |
| Tengu | that's.... ugly, actually. | 14:51 |
| opendevreview | Jiri Podivin proposed openstack/validations-common stable/1.6: Bugfix: Add missing become: true to 2 tasks https://review.opendev.org/c/openstack/validations-common/+/864698 | 14:51 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-ansible stable/wallaby: load existing password file prior to rotation https://review.opendev.org/c/openstack/tripleo-ansible/+/864250 | 14:52 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Convert Nova httpd configurations to new role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866177 | 14:52 |
| bogdando | Tengu: in t-h-t, we used to have conditions, in each service to handle 0 value differently | 14:53 |
| bogdando | as we switch to ansible, it makes sense to provide this fallback for 0 in a single place | 14:53 |
| bogdando | and ditch map_merge complicated logic, for each service to maintain in tht | 14:53 |
| Tengu | yeah, got it while re-reading the ansible_group_vars snippet | 14:53 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-ansible master: tripleo_gnocchi service role https://review.opendev.org/c/openstack/tripleo-ansible/+/858878 | 14:54 |
| bogdando | right, or that map_merge in group vars turns into a monster construction | 14:54 |
| Tengu | yeah | 14:54 |
| Tengu | cool to see nova httpd configs moving to ansible :) | 14:54 |
| bogdando | indeed | 14:58 |
| bogdando | there is also a subtle thinkg about threads ## | 14:58 |
| bogdando | tl;dr it mast be 1 for openstack wsgi | 14:59 |
| bogdando | must | 14:59 |
| bogdando | as I commented here https://review.opendev.org/c/openstack/tripleo-ansible/+/866290/ | 15:00 |
| rlandy | abishop: hello ... following up on https://trello.com/c/qzPntXcs/2802-cixlp1998227tripleociproa-unexpected-api-error-class-novaexceptionglanceconnectionfailed ... "Two solutions have been proposed; one from me and one from Rabi. Reviewers will vote according to their preference." | 15:05 |
| abishop | yes? | 15:05 |
| rlandy | what's required to get a decision on which solution can go forward? | 15:05 |
| rlandy | we have all fs001 check jobs failing atm | 15:05 |
| abishop | I'd say it's up to the reviewers :-/ | 15:05 |
| rlandy | abishop: ramishra looks to be on PTO for the rest of the week | 15:06 |
| abishop | I thought that fixing my typo would be quick and easy, but apparently there's controversy | 15:06 |
| *** blarnath is now known as d34dh0r53 | 15:06 | |
| abishop | I think he and I have stated our cases (plenty of comments in the two patches) | 15:07 |
| rlandy | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865875 has multiple -1's | 15:07 |
| rlandy | CI team are not core reviewers there | 15:07 |
| rlandy | expect for marios | 15:07 |
| abishop | that patch isn't relevant (yet), it's a zed backport | 15:07 |
| rlandy | correct - sorry | 15:08 |
| abishop | marios is encouraged to choose, and fultonj said he'd take a look (he's familiar with $topic) | 15:08 |
| rlandy | ok | 15:09 |
| rlandy | fultonj: when you have time, if you could state your preference, that would be helpful | 15:09 |
| marios | o/ abishop from the comments today i thought you were advocating Rabi patch but with your typo correction included | 15:09 |
| rlandy | abishop: thank you | 15:09 |
| marios | abishop: (comments i *read* today , not comments from today) | 15:09 |
| marios | abishop: ok i'll revisit then | 15:10 |
| fultonj | i will be reading the updates on the discussion today | 15:10 |
| *** ysandeep is now known as ysandeep|out | 15:10 | |
| abishop | marios: I still prefer my patch and Rabi advocates for his, so we're leave it to the reviewers to settle it | 15:11 |
| abishop | my latest comment is that even if we go with Rabi's, I think it needs an update | 15:12 |
| fultonj | marios: rlandy yesterday I looked at THT patches 866099 and 866165. Both fix LP 1998227 . Yesteday I prefered 866165 since I know it was verified in edge and it seemed simpler. I need to re-review both now as there have been updates I'm not yet up to speed on them. | 15:14 |
| rlandy | fultonj: sure - please take your time to see which is preferable | 15:15 |
| * fultonj just needs to respond to an un-related cix first | 15:15 | |
| rlandy | I just didn't want us to leave this hanging out | 15:15 |
| fultonj | i agree we need to merge one of them today | 15:15 |
| rlandy | or at least by the weekend | 15:16 |
| fultonj | we need this for OSSN-0090 too | 15:16 |
| opendevreview | Jiri Podivin proposed openstack/validations-common stable/1.6: DNM testing new doc job overrides for stable branches https://review.opendev.org/c/openstack/validations-common/+/866343 | 15:16 |
| fultonj | tkajinam: have you seen THT patches 866099 and 866165? | 15:18 |
| abishop | btw, for the zed (and wallaby) backport, which is currently being held -w, I plan to update the backport to incorporate whichever fix we choose from master | 15:18 |
| fultonj | (maybe it's too late for him) | 15:18 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 15:18 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 15:21 |
| Tengu | pfrrt. that error raised by ansible-lint is as meaningless as possible. | 15:21 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 15:29 |
| opendevreview | Jiri Podivin proposed openstack/validations-common stable/1.6: DNM testing new doc job overrides for stable branches https://review.opendev.org/c/openstack/validations-common/+/866343 | 15:30 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 15:39 |
| marios | odyssey4me: o/ please can someone check this and give us your thoughts https://bugs.launchpad.net/tripleo/+bug/1998501 (not on cix board yet but planning to add it depending on initial feedback) - tl;dr undercloud-upgrade wallaby->zed (not ffu) does it make sense? should we continue to dig there? any idea about the error we see if so | 15:43 |
| marios | dasm: ^^ fyi | 15:43 |
| marios | chem: the issue we discussed earlier fyi at https://bugs.launchpad.net/tripleo/+bug/1998501 ^^ | 15:43 |
| opendevreview | Jiri Podivin proposed openstack/validations-common stable/1.6: DNM testing new doc job overrides for stable branches https://review.opendev.org/c/openstack/validations-common/+/866343 | 15:43 |
| odyssey4me | jbadiapa ^ | 15:43 |
| opendevreview | Merged openstack/tripleo-common master: Fix tripleo_entrypoint.sh https://review.opendev.org/c/openstack/tripleo-common/+/866305 | 15:45 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: Refactoring node-health validation https://review.opendev.org/c/openstack/tripleo-validations/+/864555 | 15:45 |
| * jbadiapa checking | 15:55 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 16:13 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: DNM - testing selinux things in CI https://review.opendev.org/c/openstack/tripleo-ansible/+/866347 | 16:13 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: DNM - testing selinux things in CI https://review.opendev.org/c/openstack/tripleo-ansible/+/866347 | 16:16 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 16:16 |
| *** amoralej is now known as amoralej|off | 16:27 | |
| jbadiapa | marios, it's very likely that https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/ironic/ironic-api-container-puppet.yaml#L396 is unrelated, but this task is included in every upgrade_tasks_step. | 16:43 |
| *** marios is now known as marios|out | 16:49 | |
| marios|out | jbadiapa: thanks for checking - will revisit that tomorrow | 16:50 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/zed: [DNM] Adding debug https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866349 | 16:55 |
| opendevreview | Merged openstack/tripleo-validations stable/zed: Add validation to check if a reboot of a node is necessary https://review.opendev.org/c/openstack/tripleo-validations/+/865558 | 17:13 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: Add validation to check if a reboot of a node is necessary https://review.opendev.org/c/openstack/tripleo-validations/+/865559 | 17:13 |
| opendevreview | Merged openstack/validations-libs master: Check the presence of the output callback and the roles path https://review.opendev.org/c/openstack/validations-libs/+/864727 | 17:13 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 17:17 |
| *** jpena is now known as jpena|off | 17:27 | |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-heat-templates master: Configure a step when the old ironic-api configuration is cleaned https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866373 | 17:30 |
| opendevreview | Merged openstack/tripleo-upgrade stable/wallaby: [ffwd3] Introduce roles_data adjustment for wallaby https://review.opendev.org/c/openstack/tripleo-upgrade/+/866141 | 17:31 |
| opendevreview | Merged openstack/tripleo-upgrade stable/wallaby: Removed tasks using unsupported NIC config script https://review.opendev.org/c/openstack/tripleo-upgrade/+/866142 | 17:31 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove shebang for the CLI python file https://review.opendev.org/c/openstack/validations-libs/+/866336 | 17:35 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-heat-templates master: DNM test bug/1998501 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866381 | 19:08 |
| fultonj | rlandy: I +2'd https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866165 to fix LP 1998227 | 19:18 |
| fultonj | any tripleo cores able to review ^ ? so we can unblock tempest tests are failing in featureset001 master ? | 19:24 |
| rlandy | fultonj: thanks ... slagle / | 19:36 |
| rlandy | or otherwise stevebaker[m] when he gets in | 19:37 |
| fultonj | sounds good rlandy | 19:37 |
| stevebaker[m] | hey | 19:38 |
| stevebaker[m] | fultonj, rlandy done | 19:47 |
| fultonj | thank you stevebaker[m] | 19:47 |
| fultonj | abishop: ^ fyi | 19:47 |
| rlandy | stevebaker[m]: thank you ... | 19:52 |
| rlandy | I also pinged ianw about the outstanding failures on https://review.opendev.org/c/openstack/diskimage-builder/+/850882 | 19:53 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/zed: Run virtqemud with umask 0027 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865830 | 19:55 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix logs permissions for swtpm https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865141 | 19:55 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Use python to template cell urls https://review.opendev.org/c/openstack/tripleo-heat-templates/+/864475 | 20:22 |
| opendevreview | Merged openstack/tripleo-ansible master: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-ansible/+/865630 | 20:22 |
| *** dviroel is now known as dviroel|out | 21:02 | |
| opendevreview | Fernando Díaz Bravo proposed openstack/python-tripleoclient master: bnr - Deprecation note https://review.opendev.org/c/openstack/python-tripleoclient/+/866334 | 21:11 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/zed: Remove CephHciOsdType and CephHciOsdCount parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866350 | 21:12 |
| *** dasm is now known as dasm|off | 22:07 | |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 23:19 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 23:25 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 23:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!