| *** jhesketh has joined #openstack-oslo | 00:52 | |
| *** threestrands has quit IRC | 04:15 | |
| openstackgerrit | Merged openstack/oslo.utils master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728625 | 04:21 |
|---|---|---|
| openstackgerrit | Merged openstack/oslo.upgradecheck master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728624 | 04:22 |
| openstackgerrit | Merged openstack/oslotest master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728622 | 04:25 |
| *** rpittau|afk is now known as rpittau | 06:51 | |
| openstackgerrit | Masayuki Igawa proposed openstack/os-api-ref master: Remove .testr.conf https://review.opendev.org/729154 | 07:10 |
| *** TheJulia has quit IRC | 07:24 | |
| *** mnasiadka has quit IRC | 07:25 | |
| *** mnasiadka has joined #openstack-oslo | 07:27 | |
| *** TheJulia has joined #openstack-oslo | 07:29 | |
| *** ralonsoh has joined #openstack-oslo | 07:33 | |
| *** tosky has joined #openstack-oslo | 07:56 | |
| *** tkajinam has quit IRC | 08:02 | |
| *** adriant1 has joined #openstack-oslo | 08:39 | |
| *** adriant has quit IRC | 08:41 | |
| *** adriant1 is now known as adriant | 08:41 | |
| AJaeger | stephenfin: did you see my comments above, please? | 08:52 |
| stephenfin | AJaeger: not yet. Looking | 09:01 |
| AJaeger | thanks - I fear we need another release ;( 5 fails out of over 70 so far for that bug | 09:07 |
| AJaeger | and I wonder about the PDF version number... | 09:07 |
| AJaeger | stephenfin: https://opendev.org/openstack/openstackdocstheme/src/branch/master/openstackdocstheme/ext.py#L379-L380 - do we need to move to config-inited? | 09:15 |
| openstackgerrit | Merged openstack/openstackdocstheme master: Fix auto_name logic https://review.opendev.org/729031 | 09:15 |
| stephenfin | AJaeger: I don't think it gives us a whole lot extra so we should, but there's no rush. We don't need any of these config options really early in the build process | 09:15 |
| stephenfin | What's this about PDF version number? | 09:16 |
| AJaeger | stephenfin: I remove the version setting from https://review.opendev.org/729027 - and now the version number is not shown anymore in the PDF | 09:17 |
| AJaeger | but if you set version and release in conf.py, it's shown in the PDF. So, that smells like wrong order of initialization | 09:17 |
| AJaeger | but that's a shot in the dark | 09:19 |
| stephenfin | Hmm, it does sound like it :-\ | 09:19 |
| stephenfin | I'll try get a LaTeX environment up and running later today and see if I can reproduce | 09:19 |
| AJaeger | I was asking whether there's an easy fix to get the release displayed in *all* PDF files, that would indeed be nice | 09:19 |
| AJaeger | If not: We can use depends-On in CI - I can help with that one... | 09:20 |
| AJaeger | (not for debugging) | 09:20 |
| AJaeger | even without LateX environment: The first step (generation of .tex files) might be enough | 09:20 |
| stephenfin | that's a fair point, actually, yeah | 09:21 |
| AJaeger | Ok, then let's wait with the next theme release a day or so, we only broke a few repos this time - and only if the new auto_name is set to False, so nothing is blocked ;) | 09:24 |
| AJaeger | stephenfin: just changing https://opendev.org/openstack/openstackdocstheme/src/branch/master/openstackdocstheme/ext.py#L474 to use config-inited gets a doc/build/pdf/doc-openstackdocstheme.tex with \release{2.2.1.dev1} - while without changes it has \release{} | 09:43 |
| stephenfin | hmm, sounds like that's it so. I assume the switchover it trivial enough? | 09:44 |
| AJaeger | I don't know whether anything else needs changing and whether everything works, I'm not familiar with that code | 09:45 |
| stephenfin | I think the only implication is that another extension, even one in core, could re-override one of the variables we set | 09:46 |
| stephenfin | but I guess that's just a manual testing job | 09:46 |
| stephenfin | really need to figure out how to get unit tests going for that project :-\ | 09:46 |
| AJaeger | yeah ;( | 09:48 |
| AJaeger | Want a cross-test for octavianclient PDF building ? ;) | 09:48 |
| stephenfin | if it's easy to do, why not | 09:49 |
| AJaeger | let's see... | 09:49 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Add octaviaclient cross job https://review.opendev.org/729193 | 09:54 |
| AJaeger | here's a change - depends-on the change that removes the version number. Let's see whether that works nicely ^ | 09:54 |
| *** tkajinam has joined #openstack-oslo | 10:30 | |
| *** rpittau is now known as rpittau|bbl | 10:31 | |
| openstackgerrit | Merged openstack/os-api-ref master: Remove .testr.conf https://review.opendev.org/729154 | 11:05 |
| AJaeger | stephenfin: argh, http://files.ham.ie/openstack/openstackdocstheme-code-issue.png - see https://review.opendev.org/729009 | 11:25 |
| AJaeger | stephenfin: the cross-octavia job seems to work | 11:27 |
| *** sean-k-mooney has quit IRC | 11:53 | |
| *** raildo has joined #openstack-oslo | 11:57 | |
| *** rpittau|bbl is now known as rpittau | 12:04 | |
| *** sean-k-mooney has joined #openstack-oslo | 12:09 | |
| *** kgiusti has joined #openstack-oslo | 12:16 | |
| openstackgerrit | Merged openstack/oslo.serialization master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728619 | 13:14 |
| openstackgerrit | Merged openstack/oslo.middleware master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728614 | 13:16 |
| openstackgerrit | Merged openstack/oslo.db master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728547 | 13:16 |
| openstackgerrit | Merged openstack/oslo.tools master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728623 | 13:17 |
| openstackgerrit | Merged openstack/oslo.context master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728546 | 13:17 |
| openstackgerrit | Merged openstack/oslo.cache master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728267 | 13:22 |
| openstackgerrit | Merged openstack/oslo-specs master: Switch to newer openstackdocstheme version https://review.opendev.org/728621 | 13:28 |
| openstackgerrit | Merged openstack/oslo.config master: Resolve UnboundLocalError https://review.opendev.org/724880 | 13:30 |
| openstackgerrit | Merged openstack/oslo.service master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728620 | 13:34 |
| *** tkajinam has quit IRC | 14:11 | |
| *** tkajinam has joined #openstack-oslo | 14:11 | |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config stable/ussuri: Remove redundant class docstring info https://review.opendev.org/721808 | 14:19 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/719810 | 14:20 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719811 | 14:20 |
| openstackgerrit | Merged openstack/oslo.policy master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728615 | 14:22 |
| openstackgerrit | Merged openstack/oslo.rootwrap master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728618 | 14:25 |
| openstackgerrit | Merged openstack/oslo.vmware master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728627 | 14:26 |
| openstackgerrit | Merged openstack/oslo.reports master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728617 | 14:28 |
| openstackgerrit | Merged openstack/oslo.messaging master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728613 | 14:36 |
| openstackgerrit | Merged openstack/oslo.privsep master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728616 | 14:40 |
| AJaeger | stephenfin: https://review.opendev.org/#/c/724898/1 breaks the designateclient as mugsie reported. Reverting that change locally, the boxes look fine again. Not sure what's the best action is here. | 16:02 |
| *** rpittau is now known as rpittau|afk | 16:22 | |
| stephenfin | AJaeger: I just pushed a tweak to that patch | 16:30 |
| stephenfin | To restore the "native" theme | 16:31 |
| stephenfin | Let's see how that works | 16:31 |
| stephenfin | If it doesn't, I'll have to debug that locally too \o/ | 16:31 |
| mugsie | stephenfin: I was also seeing CORS failures on that patch, let me dig them outj | 16:38 |
| mugsie | oh, its the supported version banner, ignore me | 16:39 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config master: Raise hacking minimum to 3.0.1 https://review.opendev.org/727244 | 16:52 |
| AJaeger | stephenfin: thanks! | 16:54 |
| *** yoctozepto has joined #openstack-oslo | 17:18 | |
| yoctozepto | AJaeger: but does it only apply to victoria+? kolla did not branch ussuri yet (cycle trailing), so we got hit but not necessarily want to apply the common patch either | 17:19 |
| yoctozepto | (continuing pygments discussion from openstack-qa) | 17:19 |
| AJaeger | yoctozepto: if you don't use upper-constraints for ussuri, you will get the newer version | 17:20 |
| AJaeger | yoctozepto: minmal change is just changing the style, will work with older (there's a noop) and newer version | 17:20 |
| yoctozepto | AJaeger: ok, that is probably what I need right now | 17:22 |
| yoctozepto | upper constraints got overridden by zuul afair ;/ | 17:22 |
| yoctozepto | sphinx->native then | 17:22 |
| *** rcernin has quit IRC | 18:34 | |
| *** ralonsoh has quit IRC | 18:58 | |
| *** raildo has quit IRC | 19:04 | |
| *** raildo has joined #openstack-oslo | 19:19 | |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config master: Raise hacking minimum to 3.0.1 https://review.opendev.org/727244 | 20:16 |
| *** moguimar has quit IRC | 20:39 | |
| *** moguimar has joined #openstack-oslo | 20:39 | |
| openstackgerrit | Merged openstack/openstackdocstheme master: Add cross-tests for nova https://review.opendev.org/728377 | 21:19 |
| *** raildo has quit IRC | 21:24 | |
| *** rcernin has joined #openstack-oslo | 22:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
