| songwenping | -----------------++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ | 06:34 |
|---|---|---|
| *** efried1 is now known as efried | 08:43 | |
| opendevreview | Sylvain Bauza proposed openstack/nova master: document how to ask for more a single vGPU https://review.opendev.org/c/openstack/nova/+/906151 | 09:57 |
| songwenping_ | hi, team, any body can help to look at the problem about the tempest failed:https://zuul.opendev.org/t/openstack/build/5ea2c69c3b0c4a4d882944be580ba27f | 10:14 |
| bauzas | songwenping_: something is wrong with the cyborg tempest plugin | 10:25 |
| bauzas | 2024-01-22 09:51:08.246168 | controller | ++ /opt/stack/cyborg/devstack/plugin.sh:source:8 : source /opt/stack/cyborg/devstack/lib/cyborg 2024-01-22 09:51:08.248710 | controller | /opt/stack/cyborg/devstack/lib/cyborg: line 293: syntax error: unexpected end of file | 10:25 |
| bauzas | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5ea/906246/1/check/cyborg-tempest/5ea2c69/job-output.txt | 10:25 |
| bauzas | that's seen again in the devstack install logs : | 10:26 |
| bauzas | 2024-01-22 09:51:08.231 | + functions-common:run_plugins:1820 : local dir=/opt/stack/cyborg 2024-01-22 09:51:08.233 | + functions-common:run_plugins:1821 : [[ -f /opt/stack/cyborg/devstack/plugin.sh ]] 2024-01-22 09:51:08.236 | + functions-common:run_plugins:1822 : source /opt/stack/cyborg/devstack/plugin.sh stack pre-install 2024-01-22 09:51:08.239 | ++ /opt/stack/cyborg/devstack/plugin.sh:source:7 : | 10:26 |
| bauzas | echo_summary 'cyborg devstack plugin.sh called: stack/pre-install' 2024-01-22 09:51:08.241 | ++ ./stack.sh:echo_summary:468 : [[ -t 3 ]] 2024-01-22 09:51:08.243 | ++ ./stack.sh:echo_summary:474 : echo -e cyborg devstack plugin.sh called: stack/pre-install 2024-01-22 09:51:08.246 | ++ /opt/stack/cyborg/devstack/plugin.sh:source:8 : source /opt/stack/cyborg/devstack/lib/cyborg 2024-01-22 09:51:08.2 | 10:26 |
| bauzas | 48 | /opt/stack/cyborg/devstack/lib/cyborg: line 293: syntax error: unexpected end of file 2024-01-22 09:51:08.250 | ++++ /opt/stack/cyborg/devstack/lib/cyborg:source:1 : exit_trap | 10:26 |
| bauzas | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5ea/906246/1/check/cyborg-tempest/5ea2c69/controller/logs/devstacklog.txt | 10:26 |
| songwenping_ | thanks bauzas, i will check my commit. | 10:33 |
| tkajinam | looks like Pirrow was bumped to 10.x.x and this killed doc job | 12:11 |
| tkajinam | 'ImageDraw' object has no attribute 'textsize' | 12:11 |
| tkajinam | https://pillow.readthedocs.io/en/stable/releasenotes/10.0.0.html#font-size-and-offset-methods | 12:16 |
| tkajinam | https://github.com/blockdiag/blockdiag/issues/176 | 12:29 |
| tkajinam | https://github.com/blockdiag/blockdiag/issues/177 | 12:30 |
| frickler | tkajinam: oh, that. I remember people started to replace their blockdiags with other types of drawings. I guess we'll have to revert that bump once again for now | 12:53 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Checking doc job https://review.opendev.org/c/openstack/nova/+/906265 | 12:54 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Checking doc job with fixed u-c https://review.opendev.org/c/openstack/nova/+/906266 | 12:54 |
| tkajinam | frickler, yeah | 12:54 |
| tkajinam | frickler, I hope https://review.opendev.org/c/openstack/requirements/+/906262 is enough. testing in the above patches | 12:54 |
| tkajinam | probably I can try it in my local. one sec | 12:54 |
| opendevreview | sean mooney proposed openstack/nova master: add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015 | 12:55 |
| frickler | see also https://review.opendev.org/c/openstack/requirements/+/897537 , seems we need to go on with that for some cycles. also I though we did have some doc cross jobs in reqs that tests that, hmm | 12:56 |
| tkajinam | ah, ok. so this is not the first time we hit this | 12:58 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | 12:59 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck tracker to nova context https://review.opendev.org/c/openstack/nova/+/829468 | 12:59 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck utils and constants https://review.opendev.org/c/openstack/nova/+/829469 | 12:59 |
| opendevreview | sean mooney proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | 12:59 |
| tkajinam | frickler, I tested that requirement change and confirmed doc build now succeeds | 13:00 |
| tkajinam | just fyi | 13:00 |
| tkajinam | (I was concerned about potential conflicts with the other bumps but seems a simple and single downgrade works. nice | 13:00 |
| frickler | tkajinam: yes, once I saw your comment I remembered that we had that thing before. thx for confirming it works, already approved 906262 | 13:02 |
| tkajinam | frickler, thank you, too | 13:02 |
| frickler | nova ppl please also check https://bugs.launchpad.net/ironic/+bug/2026345 related to the above | 13:49 |
| bauzas | frickler: lookign | 13:55 |
| bauzas | frickler: fwiw, now I've upgraded to F39 which defaults to py312, I need to target my docs and relnotes tox runs by capping them with py39 or py310 | 13:55 |
| bauzas | and I saw some problem with Pillow due to some dependency | 13:56 |
| frickler | sounds like we should add some py312 testing into requirements and elsewhere soon. I'm just not sure yet which distro to use for that. maybe I should start looking at some noble stuff, that would likely make sense for devstack, too | 13:59 |
| bauzas | frickler: lemme know when we need to merge the new reqs patch in Nova (but I think we don't need it ) | 14:12 |
| bauzas | yeah, Pillow is neither in reqs.txt or in test-reqs.txt | 14:13 |
| opendevreview | Marius Leustean proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/c/openstack/nova/+/699176 | 14:13 |
| bauzas | sean-k-mooney: fwiw, I reworded entirely the doc patch for the multi-vgpu-per_instance bugfix https://review.opendev.org/c/openstack/nova/+/906151/2 | 14:34 |
| frickler | bauzas: that's not the point, stopping to use blockdiag seems to be what is needed | 14:34 |
| bauzas | sean-k-mooney: long story short, I just say that SRIOV GPUs are fine, but you would need to do something else for non-SRIOV GPUs | 14:35 |
| bauzas | frickler: well, if blockdiag is unmaintained, we may need to do this indeed | 14:35 |
| bauzas | frickler: but nothing magically happens even if we wish it, we need some people to do the dirty work :) | 14:36 |
| bauzas | so I can try to file a bug report but no promises we could change that shortly | 14:36 |
| bauzas | (ideally, if many projects share the same usage of blockdiag, I'd certainly be happy if the TC could take a stance on that situation like they did with eventlet) | 14:37 |
| bauzas | gmann: JayF: just sayin' ^ | 14:37 |
| tkajinam | https://bugs.launchpad.net/ironic/+bug/2026345 | 14:44 |
| tkajinam | bauzas, there is already a bug for it so you can use it | 14:44 |
| bauzas | tkajinam: well, that one is for the pillow error | 14:45 |
| bauzas | I'd like to track ourselves the fact that we're incented to move from blockdiag | 14:45 |
| tkajinam | ok | 14:45 |
| bauzas | there was a thread https://lists.openstack.org/pipermail/openstack-discuss/2023-July/034561.html | 14:46 |
| bauzas | tkajinam: frickler: ^ | 14:46 |
| tkajinam | afaik designate already replaced blockdiag by graphviz. however graphviz does not cover all of the use cases needed by ironic and now they are looking into plantuml. | 14:47 |
| tkajinam | yeah I wasn't aware of that thread. thanks for sharing it. | 14:48 |
| bauzas | well, we need to evaluate the required changes, but sean-k-mooney's suggestion to statically create the SVGs sounds to me the shortest and easiest path | 14:49 |
| bauzas | hum, interesting | 14:51 |
| bauzas | https://github.com/search?q=repo%3Aopenstack%2Fnova+blockdiag&type=code | 14:52 |
| bauzas | the only place where we pull blockdiag is from conf | 14:52 |
| bauzas | I'm not a doc expert, maybe stephenfin may help :) | 14:53 |
| bauzas | stephenfin: tl;dr: blockdiag seems unmaintained and is broken due to a new Pillow release, do you know what's left in nova because we can remove it from sphinx's conf ? | 14:54 |
| bauzas | before* | 14:54 |
| JayF | We use it in ironic still. We've got folks working on replacements, but it's just on a long list of things with a short people to do them. | 14:55 |
| JayF | Our weekly meeting is in 5 minutes I can see if anyone is actively committed to completing our retirement of blockdiag with any kind of timeline | 14:56 |
| bauzas | JayF: nova's approach was to statically create the SVGs | 14:58 |
| bauzas | JayF: apparently now we no longer *need* blockdiag | 14:58 |
| bauzas | oh waiy | 14:58 |
| bauzas | we need to statically create for seqdiag | 14:59 |
| bauzas | https://github.com/search?q=repo%3Aopenstack%2Fnova+seqdiag&type=code | 14:59 |
| bauzas | I can take that action | 14:59 |
| bauzas | JayF: that may help you guys https://bugs.launchpad.net/tacker/+bug/2026345/comments/11 | 15:15 |
| JayF | Ah, that's a good idea actually | 15:15 |
| JayF | Hadn't even considered that | 15:15 |
| JayF | Thanks! | 15:15 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Explicitly disable signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | 16:04 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Omit signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | 16:05 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Omit signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | 16:06 |
| opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | 17:10 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | 17:15 |
| opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | 17:16 |
| opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | 17:16 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: doc: drop blockdiag usage https://review.opendev.org/c/openstack/nova/+/906309 | 17:28 |
| bauzas | sean-k-mooney: gibi: let's drop blockdiag for now ^ | 17:30 |
| bauzas | stephenfin: you may also be interested in https://review.opendev.org/c/openstack/nova/+/906309 | 17:30 |
| *** efried1 is now known as efried | 21:24 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!