| opendevreview | Takashi Kajinami proposed openstack/cinder master: Validate zoning_mode option in config layer https://review.opendev.org/c/openstack/cinder/+/931119 | 03:59 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/930664 | 04:18 |
| opendevreview | Shogo Takazawa proposed openstack/cinder master: Hitachi: Fix snap pool settings issue during migration https://review.opendev.org/c/openstack/cinder/+/931124 | 04:26 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Fix to retype extra_specs correctly https://review.opendev.org/c/openstack/cinder/+/930984 | 05:22 |
| opendevreview | Shogo Takazawa proposed openstack/cinder master: Hitachi: Fix snap pool settings issue during migration https://review.opendev.org/c/openstack/cinder/+/931124 | 05:22 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: hpe3par: Use integer values for snapshot expiration/retention https://review.opendev.org/c/openstack/cinder/+/931128 | 05:35 |
| opendevreview | Shogo Takazawa proposed openstack/cinder master: Hitachi: Volume Replication for VSP Driver https://review.opendev.org/c/openstack/cinder/+/869991 | 07:44 |
| *** geguileo is now known as Guest5196 | 07:58 | |
| opendevreview | Jan Hartkopf proposed openstack/cinder-tempest-plugin master: Add test for Ceph backup snapshot cleanup option https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/855876 | 09:23 |
| opendevreview | Jan Hartkopf proposed openstack/cinder-tempest-plugin master: Add test for Ceph backup snapshot cleanup option https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/855876 | 09:50 |
| tkajinam | Hi ! I wonder if https://review.opendev.org/c/openstack/cinder/+/929400 can be merged early in 2025.1 cycle. I've been fixing the same issue multiple times and the change prevents similar problems in the future | 13:21 |
| tkajinam | also it'd be nice if https://review.opendev.org/c/openstack/cinder/+/929932 can be merged early, too, to unblock bump of netaddr | 13:22 |
| jbernard | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi: meeting in #openstack-meeting-alt | 14:01 |
| opendevreview | Takashi Natsume proposed openstack/cinder master: Replace deprecated datetime.utcfromtimestamp() https://review.opendev.org/c/openstack/cinder/+/931005 | 14:07 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Pure Storage - bump version numbers for 2025.1 (Epoxy) https://review.opendev.org/c/openstack/cinder/+/931178 | 14:19 |
| tobias-urdin | arggh missed the meeting – requesting review on this bugfix https://review.opendev.org/c/openstack/cinder/+/928781/2 (triaged by eharney) | 14:38 |
| tobias-urdin | rosmaita: were you able to get any feedback on adding query param to existing API here? https://review.opendev.org/c/openstack/cinder/+/926263 | 14:38 |
| tobias-urdin | now that we've entered a new cycle also requesting feedback for this minimal new API https://review.opendev.org/c/openstack/cinder/+/928846 if you want a spec or if review is enough | 14:39 |
| tobias-urdin | and for us living in the land of backports i bow to you for any reviews \o/ - https://review.opendev.org/c/openstack/cinder/+/927414 https://review.opendev.org/c/openstack/cinder/+/927347 https://review.opendev.org/c/openstack/cinder/+/927359 https://review.opendev.org/c/openstack/cinder/+/927346 | 14:41 |
| simondodsley | anyone know why `cinder.tests.unit.test_cmd.TestCinderBackupCmd.test_main_multiprocess` is failing in the `py312` cinder gate? | 15:20 |
| eharney | simondodsley: i think it passes most of the time, looks like a bug we have to chase down :/ | 15:47 |
| simondodsley | eharney - ok i'll recheck | 15:48 |
| eharney | simondodsley: oh i see what the bug is | 15:50 |
| eharney | we have IntOpt backup_workers with max=processutils.get_worker_count() and that unit test job ran on a 1-cpu node, so max=1 and the unit test set it to 2 | 15:51 |
| eharney | need to figure out how to fix up the unit tests so they will still pass in that scenario | 15:51 |
| simondodsley | ooh - interesting | 15:51 |
| eharney | results in "ValueError: Should be less than or equal to 1" from "CONF.set_override('backup_workers', 2)" | 15:52 |
| clarkb | eharney: can you point to a log for that 1 cpu node case? We shouldn't have any test nodes with one cpu | 15:53 |
| eharney | https://zuul.opendev.org/t/openstack/build/66e86bad2ef1422ea1783704590a84c3 | 15:53 |
| clarkb | the new raxflex cloud should have 4vcpus on test nodes and everything else has 8 (raxflex hardware is newer and those jobs still run quicker with fewer vcpus) | 15:53 |
| clarkb | that job ran on rax-ord which is not the new region so should have 8 vcpus. Lets see if we log any more info | 15:54 |
| clarkb | https://zuul.opendev.org/t/openstack/build/66e86bad2ef1422ea1783704590a84c3/log/zuul-info/host-info.ubuntu-noble.yaml#540 that is very unexpected | 15:54 |
| clarkb | is it only python3.12 that hits that? | 15:56 |
| eharney | i haven't spotted this failure before, not sure | 15:56 |
| clarkb | I would expect an issue like this to be more random and affect other python versions too. But maybe ubuntu noble isn't able to see all the CPUs for some reason and it is specific to that platform | 15:57 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Testing without ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931209 | 16:53 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Get cephadm lod for ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931212 | 17:28 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Testing without ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931209 | 17:30 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Testing without ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931209 | 17:31 |
| opendevreview | Merged openstack/devstack-plugin-ceph stable/2023.1: Standalone nfs-ganesha with cephadm deployment https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/930501 | 18:11 |
| opendevreview | Pablo Colson proposed openstack/cinder master: fix: typo in pure driver https://review.opendev.org/c/openstack/cinder/+/931214 | 18:19 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.1: Fix ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/929186 | 18:23 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph master: [DNM] Get cephadm log for ingress https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/931212 | 19:30 |
| clarkb | https://opendev.org/openstack/oslo.concurrency/src/branch/master/oslo_concurrency/processutils.py#L602-L611 is what cinder is calling to get the cpu count which ultimately calls multiprocessing.cpu_count() if that fails we default to 1. I wonder if ansible is doing a similar fallback | 19:36 |
| clarkb | and cpu_count() raises if os.cpu_count() returns None | 19:38 |
| clarkb | I did more debugging with my breadcumb trail in #opendev but long story short in the last week cinder has had 7 success openstack-tox-py312 job runs in rax-ord and one failure. The successes all ran on differeny hypervisors with a reported different xen version than the failure | 20:24 |
| clarkb | a hunch is that this is related to the new kerenel and specific xen versions not getting along properly but that is still a hunch | 20:24 |
| clarkb | some notes here https://etherpad.opendev.org/p/Kk-QduOuLQV6grR444bw | 20:25 |
| pabloclsn | Hello | 20:32 |
| pabloclsn | for this merge request who should I add as a reviewer ? (https://review.opendev.org/c/openstack/cinder/+/931214) | 20:33 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Port to the tempest stable plugin interface (clients) https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/929620 | 21:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!