| opendevreview | Merged openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/884564 | 01:27 |
|---|---|---|
| gthiemonge | Hi Folks, it seems that the bump to Pillow 10.0.1 (https://review.opendev.org/c/openstack/requirements/+/884564) breaks a few openstack-tox-docs jobs (I see issues in octavia/nova/ironic in opensearch) | 06:47 |
| gthiemonge | https://zuul.openstack.org/build/caf04259e3ca438eaebe9bcbf9e33a68 | 06:48 |
| gthiemonge | interesting: https://bugs.launchpad.net/tacker/+bug/2026345 | 06:49 |
| frickler | gthiemonge: unless there is a bug in that specific version of the library, issues due to upgrades are expected and to be handled on the side of the projects consuming these libraries. we've been sticking on outdated versions for much too long already | 07:01 |
| gthiemonge | frickler: yes, NP I understand it, I'm raising it in case some other folks face the same issue | 07:16 |
| frickler | gthiemonge: do you happen to know if the failure in octavia is also due to using blockdiag? | 07:20 |
| gthiemonge | frickler: we use seqdiag, I guess this is the same pbl, the repo hasn't been updated since Dec 2021 https://github.com/blockdiag/seqdiag | 07:29 |
| gthiemonge | frickler: I'm removing seqdiag from octavia, it's only used for some diagrams in the specs, I think it's fine to not generate them | 07:31 |
| frickler | gthiemonge: ack, nova is already tagged in the above bug, do you want to use it for octavia, too? | 07:35 |
| gthiemonge | frickler: oh yes, I have to click on "Also affects project", right? | 07:39 |
| frickler | gthiemonge: yes | 07:51 |
| gthiemonge | frickler: thanks! | 07:52 |
| rpittau | in ironic we use seqdiag for some diagrams, using graphviz for the same is not super easy | 10:22 |
| rpittau | wondering if it's worth and possible to introduce an alternative like plantuml | 10:22 |
| rpittau | PlantUML uses graphviz btw | 10:25 |
| frickler | the main question I guess is how difficult the migration of existing documents will be. but replacing dependencies that became unmaintained surely should be possible | 10:26 |
| frickler | maybe worth a cross-team discussion at the ptg. I think we could also agree on reverting that part of the recent version bump until then to unblock projects | 10:27 |
| rpittau | frickler: in ironic we have I think 6-7 sequence diagrams that need to be converted and with pure graphviz is not easy to do the conversion, also the syntax is horrible :) | 10:27 |
| rpittau | if we could revert the bump until we find a solution would be great | 10:28 |
| rpittau | I proposed PlantUML since it's basically graphviz but with an easier syntax | 10:28 |
| frickler | rpittau: can you check whether only the Pillow bump needs to be reverted and propose a patch? I guess I would fast approve it then | 10:28 |
| rpittau | frickler: sure thing | 10:29 |
| rpittau | and thanks! | 10:29 |
| opendevreview | Riccardo Pittau proposed openstack/requirements master: Rvert Pillow version pin to 9.5.0 https://review.opendev.org/c/openstack/requirements/+/897537 | 10:37 |
| rpittau | frickler: I tested with a modified upper constraints, moving Pillow back to 9.5.0 fixes the issue | 10:37 |
| opendevreview | Riccardo Pittau proposed openstack/requirements master: Revert Pillow version pin to 9.5.0 https://review.opendev.org/c/openstack/requirements/+/897537 | 10:37 |
| opendevreview | Riccardo Pittau proposed openstack/requirements master: Revert Pillow version pin to 9.5.0 https://review.opendev.org/c/openstack/requirements/+/897537 | 10:39 |
| rpittau | this ^ should now be typo-free :P | 10:39 |
| frickler | rpittau: ack, thx, +3. I'll create a revert of this revert once it is merged so that you and others have something to test against and such that it doesn't get forgotten again | 12:04 |
| rpittau | thanks frickler :) | 12:06 |
| opendevreview | Merged openstack/requirements master: Revert Pillow version pin to 9.5.0 https://review.opendev.org/c/openstack/requirements/+/897537 | 13:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!