| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-01-22 | |||
| 14:44:15 | tkajinam | bauzas, there is already a bug for it so you can use it | |
| 14:45:04 | bauzas | tkajinam: well, that one is for the pillow error | |
| 14:45:23 | bauzas | I'd like to track ourselves the fact that we're incented to move from blockdiag | |
| 14:45:43 | tkajinam | ok | |
| 14:46:39 | bauzas | there was a thread https://lists.openstack.org/pipermail/openstack-discuss/2023-July/034561.html | |
| 14:46:48 | bauzas | tkajinam: frickler: ^ | |
| 14:47:51 | 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:48:00 | tkajinam | yeah I wasn't aware of that thread. thanks for sharing it. | |
| 14:49:19 | 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:51:28 | bauzas | hum, interesting | |
| 14:52:44 | bauzas | https://github.com/search?q=repo%3Aopenstack%2Fnova+blockdiag&type=code | |
| 14:52:54 | bauzas | the only place where we pull blockdiag is from conf | |
| 14:53:15 | bauzas | I'm not a doc expert, maybe stephenfin may help :) | |
| 14:54:02 | 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:08 | bauzas | before* | |
| 14:55:42 | 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:56:07 | 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:58:21 | bauzas | JayF: nova's approach was to statically create the SVGs | |
| 14:58:44 | bauzas | JayF: apparently now we no longer *need* blockdiag | |
| 14:58:59 | bauzas | oh waiy | |
| 14:59:12 | bauzas | we need to statically create for seqdiag | |
| 14:59:39 | bauzas | https://github.com/search?q=repo%3Aopenstack%2Fnova+seqdiag&type=code | |
| 14:59:55 | bauzas | I can take that action | |
| 15:15:20 | bauzas | JayF: that may help you guys https://bugs.launchpad.net/tacker/+bug/2026345/comments/11 | |
| 15:15:42 | JayF | Ah, that's a good idea actually | |
| 15:15:47 | JayF | Hadn't even considered that | |
| 15:15:53 | JayF | Thanks! | |
| 16:04:08 | opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Explicitly disable signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | |
| 16:05:34 | opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Omit signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | |
| 16:06:08 | opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Omit signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | |
| 17:10:18 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 17:15:52 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 17:15:53 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 17:15:53 | 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:54 | 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:54 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 17:15:55 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 17:15:55 | 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:56 | 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:57 | 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:57 | 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:59 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 17:15:59 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 17:16:01 | 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:01 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 17:16:03 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 17:16:03 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 17:16:05 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 17:16:05 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 17:16:07 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | |
| 17:16:07 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 17:16:09 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 17:16:09 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 17:16:11 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 17:16:11 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 17:16:13 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 17:16:13 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 17:16:15 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 17:16:15 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 17:28:48 | opendevreview | Sylvain Bauza proposed openstack/nova master: doc: drop blockdiag usage https://review.opendev.org/c/openstack/nova/+/906309 | |
| 17:30:21 | bauzas | sean-k-mooney: gibi: let's drop blockdiag for now ^ | |
| 17:30:36 | bauzas | stephenfin: you may also be interested in https://review.opendev.org/c/openstack/nova/+/906309 | |
| #openstack-nova - 2024-01-23 | |||
| 01:11:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: GMR: Omit signal handleris in wsgi app https://review.opendev.org/c/openstack/nova/+/906279 | |
| 06:28:50 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 09:56:51 | bauzas | sean-k-mooney: gibi: stephenfin: could you please look at https://review.opendev.org/c/openstack/nova/+/906309 for removing our usage of blockdiag ? | |
| 10:58:55 | sean-k-mooney | we already did this for nova-specs so i dont see a reason not to do it for nova propper as well | |
| 10:59:49 | sean-k-mooney | there is one downside with using the image:: macro | |
| 11:00:05 | sean-k-mooney | whic is tha tthe svg wont be rendered in teh browser as a svg | |
| 11:00:13 | sean-k-mooney | it will use the html image tag | |
| 11:00:29 | sean-k-mooney | so extentions like dark reader wont be able ot recoler them properly | |
| 11:01:07 | sean-k-mooney | thats a minor annoyance that iu can live with but it is an accsiblity change | |
| 11:53:56 | opendevreview | Merged openstack/nova master: doc: drop blockdiag usage https://review.opendev.org/c/openstack/nova/+/906309 | |
| 12:14:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: [libvirt]log XML if nova fails to parse it https://review.opendev.org/c/openstack/nova/+/906364 | |
| 15:14:20 | opendevreview | Pierre Riteau proposed openstack/nova master: doc: Fix markup syntax https://review.opendev.org/c/openstack/nova/+/906379 | |
| 15:15:01 | opendevreview | Pierre Riteau proposed openstack/nova master: doc: Fix markup syntax https://review.opendev.org/c/openstack/nova/+/906379 | |
| 15:16:23 | opendevreview | Pierre Riteau proposed openstack/nova master: doc: Fix markup syntax and typo https://review.opendev.org/c/openstack/nova/+/906379 | |
| 15:26:16 | opendevreview | Simon Hensel proposed openstack/nova master: Always delete NVRAM files when deleting instances https://review.opendev.org/c/openstack/nova/+/906380 | |
| 15:48:19 | opendevreview | Vasyl Saienko proposed openstack/nova master: Fix returning empty availability zone https://review.opendev.org/c/openstack/nova/+/902875 | |
| 15:55:05 | bauzas | reminder: nova meeting in 5 mins here | |
| 16:01:36 | bauzas | #startmeeting nova | |
| 16:01:36 | opendevmeet | Meeting started Tue Jan 23 16:01:36 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:36 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:36 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:44 | fwiesel | \o | |
| 16:01:44 | bauzas | sorry I was looking at some change | |
| 16:01:47 | grandchild | o/ | |
| 16:02:06 | auniyal | o/ | |
| 16:02:17 | elodilles | o/ | |
| 16:02:25 | bauzas | I'll need to move in 45 mins so let's start now | |
| 16:02:34 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:39 | bauzas | #info No Critical bug | |
| 16:02:43 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 46 new untriaged bugs (+0 since the last meeting) | |
| 16:02:46 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:03:13 | bauzas | auniyal: could you be using the bug baton for this next week ? :) | |
| 16:03:24 | auniyal | sure :) | |
| 16:03:37 | bauzas | cool | |
| 16:03:42 | bauzas | #info bug baton is auniyal | |
| 16:03:45 | bauzas | #topic Gate status | |
| 16:03:49 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:03:55 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2026345 Nova docs jobs were broken but now this is fixed | |