| tonyb | kevko: I've created a hold on 929771 for kolla-ansible-ubuntu-multinode-ipv6 | 08:45 |
|---|---|---|
| tonyb | please recehck the job and I can ass your SSH key (same as yesterday) to the nodes | 08:46 |
| kevko | yep | 08:46 |
| tonyb | *add not ass #sorry all | 08:46 |
| *** jroll01 is now known as jroll0 | 09:00 | |
| kevko | tonyb: you can unhold it ..done ...it looks like i've fixed it already yesterday ..but forgot to change one line in review | 09:42 |
| tonyb | Okay hold removed. | 09:43 |
| tonyb | Hopefully any recheck will just work now | 09:44 |
| kevko | tonyb: I am live watching the online job status ...and from experience, I know that it is already working (that kolla multijnode ipv6 job). If it weren't working, it would have crashed a long time ago - but it's running ..so thanks ! | 10:33 |
| *** ykarel__ is now known as ykarel | 10:33 | |
| tonyb | awesome! | 10:33 |
| kevko | btw, it was also a lesson for me to always remember that IPv6URLs are written with [] brackets and IPs without | 10:37 |
| tonyb | hehe. | 10:43 |
| fungi | yeah, well really [] quoted addresses are to deal with the fact that : is a url separator in addition to being an address segment separator, e.g. for specify an explicit port number after a literal ip address | 12:32 |
| kevko | I know, we in Kolla have the put_address_in_context('url') filter ... the problem was that it wasn't used :) | 12:35 |
| *** ykarel_ is now known as ykarel | 12:41 | |
| opendevreview | Amit Uniyal proposed openstack/project-config master: Add official OpenStack jobs for whitebox-tempest-plugin https://review.opendev.org/c/openstack/project-config/+/930029 | 13:35 |
| opendevreview | Amit Uniyal proposed openstack/project-config master: Add official OpenStack jobs for whitebox-tempest-plugin https://review.opendev.org/c/openstack/project-config/+/930029 | 14:39 |
| opendevreview | Brian Rosmaita proposed opendev/irc-meetings master: Update agenda URL for cinder festival of reviews https://review.opendev.org/c/opendev/irc-meetings/+/930066 | 14:50 |
| opendevreview | Merged openstack/project-config master: Add official OpenStack jobs for whitebox-tempest-plugin https://review.opendev.org/c/openstack/project-config/+/930029 | 15:34 |
| opendevreview | Merged openstack/project-config master: Convert python2 dib element scripts to python3 https://review.opendev.org/c/openstack/project-config/+/929166 | 15:34 |
| opendevreview | Merged openstack/project-config master: Add 'Review-Priority' for tap-as-a-service https://review.opendev.org/c/openstack/project-config/+/929776 | 15:38 |
| opendevreview | Merged openstack/project-config master: charms: add official repo jobs https://review.opendev.org/c/openstack/project-config/+/928326 | 15:39 |
| opendevreview | Merged opendev/irc-meetings master: Update agenda URL for cinder festival of reviews https://review.opendev.org/c/opendev/irc-meetings/+/930066 | 15:43 |
| opendevreview | Merged openstack/project-config master: Run ansible-lint on Ubuntu Noble https://review.opendev.org/c/openstack/project-config/+/926848 | 15:47 |
| opendevreview | Merged openstack/project-config master: Retire kuryr-kubernetes and tempest plugin: end gate and update acl https://review.opendev.org/c/openstack/project-config/+/923072 | 15:47 |
| opendevreview | Merged openstack/project-config master: Add missing DEPRECATED marker to networking-odl https://review.opendev.org/c/openstack/project-config/+/919416 | 16:01 |
| opendevreview | Merged openstack/project-config master: kolla: Rework Grafana dashboards https://review.opendev.org/c/openstack/project-config/+/918732 | 16:22 |
| opendevreview | Merged openstack/project-config master: Remove driverfixes branch acls from manila https://review.opendev.org/c/openstack/project-config/+/918147 | 16:22 |
| *** dhill is now known as Guest4045 | 16:51 | |
| clarkb | I found https://github.com/sphinx-contrib/kroki as a sphinx extension that supports blockdiag too. Problem is it seems to farm out the rendering to an external service rather than doing it locally | 18:17 |
| clarkb | we could run that external service ourselves locally and point the builds to that but that seems overly complicated | 18:18 |
| clarkb | really does feel like maybe we should put a blockdiag file in the repo then find something that can render it and copy the resulting image over | 18:19 |
| clarkb | for example blockdiag itself? | 18:19 |
| clarkb | oh though maybe it too is no longer maintained? | 18:20 |
| clarkb | ya it seems like blockdiag itself is the issue not the sphinxcontrib lib | 18:20 |
| clarkb | maybe we should switch to graphviz which is a built in extension to sphinx | 18:22 |
| clarkb | ya I'm going to try that | 18:23 |
| JayF | for Ironic, we basically just committed the results | 18:32 |
| JayF | and have a not-using-global-requirements tox environment to generate them manually | 18:32 |
| JayF | it's ... not ideal | 18:32 |
| clarkb | JayF: graphviz + sphinx.ext.graphviz seems to be working well for this particular use case but it is very simple | 18:40 |
| clarkb | I'm about to push up the change for it if you'd like to see it | 18:40 |
| opendevreview | Clark Boylan proposed opendev/base-jobs master: Replace blockdiag with graphviz in doc image rendering https://review.opendev.org/c/opendev/base-jobs/+/930082 | 18:44 |
| clarkb | JayF: ^ | 18:44 |
| JayF | cid was going to look into sphinx stuff for Ironic, ^^^ | 18:44 |
| JayF | might be a good thing to look at if you're around too | 18:44 |
| JayF | clarkb: have you met cid? He's been working with me now for several months :D | 18:44 |
| JayF | according to bitergio for this release he's #3 most prolific Ironic committer :D | 18:44 |
| clarkb | I've seen cid hanging around ironic | 18:45 |
| clarkb | nice | 18:45 |
| JayF | it's awesome to get new folks working in/around the ecosystem | 18:45 |
| JayF | and good to have a completely external perspective too | 18:45 |
| JayF | he might be a good resource for opendev generally about feedback for dev processes and onboarding if you ever wanted one | 18:45 |
| clarkb | the one thing with graphviz is that you have to install the graphviz package with bindep since it isn't a python native tool. But it is maintained both itself and in sphinx so probably the best option over time | 18:47 |
| clarkb | hrm local build works without a warning but job in zuul produces a warning which is treated as an error | 18:49 |
| clarkb | oh wait that is from the seqdiag which I guess was depending on pillow too? | 18:50 |
| JayF | yep | 18:50 |
| JayF | as someone who <3 seqdiag style graphs that was a killer too | 18:51 |
| clarkb | JayF: the good news is the warning emits dot code so I think maybe I can just copy the dot code into a graphviz entry | 18:51 |
| clarkb | but more testing necessary | 18:51 |
| clarkb | ok definitely not a straight port seems blockdiag and Pillow are using some dialect of dot that graphviz doesn't immediately grok | 19:01 |
| clarkb | anyway I'm disappointed that seqdiag and blockdiag are basically the same thing and I missed that thinking I could solve these independently. But Still seems doable I may just need to write some dot from scratch | 19:02 |
| clarkb | https://stackoverflow.com/questions/32436856/using-graphviz-to-create-tcp-flow-diagrams | 19:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!