openstackgerrit | zhangyc proposed openstack/rpm-packaging master: update oslo.messaging.spec.j2 to 12.6.0 https://review.opendev.org/759330 | 00:21 |
---|---|---|
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: update openstackdocstheme.spec.j2 to 2.2.6 https://review.opendev.org/759331 | 00:24 |
*** gyee has quit IRC | 00:44 | |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: fix url https://review.opendev.org/759341 | 02:00 |
openstackgerrit | chenming proposed openstack/rpm-packaging master: Adjust the URLs to point to docs.openstack.org https://review.opendev.org/759342 | 02:16 |
*** evrardjp has quit IRC | 04:34 | |
*** evrardjp has joined #openstack-rpm-packaging | 04:34 | |
*** rpittau|afk is now known as rpittau | 06:21 | |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: update openstackdocstheme to 2.2.6 https://review.opendev.org/759331 | 06:37 |
*** amoralej|off is now known as amoralej | 06:53 | |
*** sboyron has joined #openstack-rpm-packaging | 07:20 | |
*** sboyron has quit IRC | 07:26 | |
*** sboyron has joined #openstack-rpm-packaging | 07:49 | |
*** sboyron has quit IRC | 07:51 | |
*** sboyron has joined #openstack-rpm-packaging | 07:52 | |
*** jpena|off is now known as jpena | 07:56 | |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: Remove setuotppls Requirement from karbor https://review.opendev.org/758310 | 08:08 |
*** number80 has quit IRC | 08:50 | |
*** sboyron has quit IRC | 09:04 | |
*** sboyron has joined #openstack-rpm-packaging | 10:18 | |
*** sboyron has quit IRC | 10:23 | |
openstackgerrit | Javier Peña proposed openstack/rpm-packaging master: fix url https://review.opendev.org/759341 | 11:13 |
*** jpena is now known as jpena|lunch | 11:31 | |
openstackgerrit | Merged openstack/rpm-packaging master: update ovsdbapp to 1.6.0 https://review.opendev.org/758511 | 11:41 |
openstackgerrit | Merged openstack/rpm-packaging master: update oslo.messaging to 12.5.1 https://review.opendev.org/758885 | 11:41 |
openstackgerrit | Merged openstack/rpm-packaging master: update murano-dashboard to 10.0.0 https://review.opendev.org/758087 | 11:42 |
openstackgerrit | Merged openstack/rpm-packaging master: fix url https://review.opendev.org/759341 | 11:45 |
*** sboyron has joined #openstack-rpm-packaging | 11:51 | |
*** jpena|lunch is now known as jpena | 12:33 | |
sboyron | hello jpena, I've got some questions regarding https://review.opendev.org/#/c/759342/ | 12:35 |
jpena | sboyron: oh, I didn't realize the URLs were 404 | 12:36 |
sboyron | you +2 it, while I disagree with almost all improvements of this review, I would like to understand if I'm wrong or not | 12:36 |
jpena | sboyron: you're totally right, I made a wrong assumption | 12:36 |
sboyron | jpena: ok ^^ great | 12:36 |
sboyron | jpena: That was what I thought but I would liked to understand if I were wrong | 12:37 |
jpena | no no, I just checked that the URLs followed the same syntax, but didn't check that they pointed to the proper doc | 12:37 |
sboyron | jpena, do you know why python-tempestconf url not redirect to /latest ? | 12:37 |
sboyron | https://docs.openstack.org/python-tempestconf/ | 12:38 |
sboyron | this one | 12:38 |
jpena | no, no idea | 12:38 |
jpena | it might be some misconfiguration | 12:38 |
sboyron | yep | 12:39 |
sboyron | but don't know where | 12:39 |
sboyron | I was thinking about where to notify or correct this | 12:39 |
sboyron | From a syntaxt point of view what gives me a doubt is the x/ folder in doc.openstack.org URL... | 12:43 |
sboyron | And on top of that, what I do not undertand, or what we should maybe improve, is that the CI didn't failed when the URL was in error | 12:44 |
jpena | in fact, the URL field in the spec is not checked anywhere, it's just meant to be information | 12:44 |
sboyron | should it be tested ? (I'm jumping on a meeting, will check it later) | 12:44 |
sboyron | ok | 12:44 |
jpena | I wouldn't care too much about it, tbh | 12:44 |
sboyron | yeap, neer used it indeed ;p | 12:45 |
sboyron | s/neer/never | 12:45 |
sboyron | but I'll maybe propose a check if I find some time to look closer on CI tests ^^ | 12:45 |
sboyron | just to save review time on this kind of stuff | 12:47 |
*** amoralej is now known as amoralej|lunch | 12:50 | |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: update oslo.messaging to 12.6.0 https://review.opendev.org/759417 | 12:56 |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: url fix https://review.opendev.org/759423 | 13:32 |
*** amoralej|lunch is now known as amoralej | 13:36 | |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: url fix https://review.opendev.org/759423 | 13:47 |
openstackgerrit | zhangyc proposed openstack/rpm-packaging master: update python-tempestconf Uniform Resource Locator https://review.opendev.org/757709 | 14:03 |
*** rpittau is now known as rpittau|afk | 14:06 | |
*** sboyron has quit IRC | 14:22 | |
*** sboyron has joined #openstack-rpm-packaging | 14:37 | |
*** sboyron has quit IRC | 14:41 | |
*** amoralej is now known as amoralej|off | 15:45 | |
*** gyee has joined #openstack-rpm-packaging | 15:48 | |
*** jpena is now known as jpena|off | 16:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!