| openstackgerrit | punkrokk proposed openstack/tooz master: Add support for Consul ACL token parameter. https://review.opendev.org/718488 | 01:49 |
|---|---|---|
| openstackgerrit | melissaml proposed openstack/oslo.reports master: Remove translation sections from setup.cfg https://review.opendev.org/728337 | 03:05 |
| openstackgerrit | melissaml proposed openstack/oslo.policy master: Remove translation sections from setup.cfg https://review.opendev.org/728338 | 03:09 |
| openstackgerrit | melissaml proposed openstack/oslo.reports master: Remove translation sections from setup.cfg https://review.opendev.org/728340 | 03:17 |
| openstackgerrit | melissaml proposed openstack/oslo.i18n master: Remove translation sections from setup.cfg https://review.opendev.org/728341 | 03:17 |
| openstackgerrit | melissaml proposed openstack/oslo.middleware master: Remove translation sections from setup.cfg https://review.opendev.org/728342 | 03:17 |
| openstackgerrit | melissaml proposed openstack/oslo.service master: Remove translation sections from setup.cfg https://review.opendev.org/728343 | 03:17 |
| openstackgerrit | melissaml proposed openstack/oslo.log master: Remove translation sections from setup.cfg https://review.opendev.org/728344 | 03:17 |
| openstackgerrit | melissaml proposed openstack/oslo.utils master: Remove translation sections from setup.cfg https://review.opendev.org/728346 | 03:30 |
| openstackgerrit | melissaml proposed openstack/oslo.serialization master: Remove translation sections from setup.cfg https://review.opendev.org/728347 | 03:30 |
| openstackgerrit | melissaml proposed openstack/oslo.versionedobjects master: Remove translation sections from setup.cfg https://review.opendev.org/728348 | 03:30 |
| openstackgerrit | melissaml proposed openstack/oslo.privsep master: Remove translation sections from setup.cfg https://review.opendev.org/728349 | 03:30 |
| openstackgerrit | melissaml proposed openstack/oslo.db master: Remove translation sections from setup.cfg https://review.opendev.org/728352 | 03:42 |
| openstackgerrit | melissaml proposed openstack/oslo.vmware master: Remove translation sections from setup.cfg https://review.opendev.org/728353 | 03:58 |
| openstackgerrit | melissaml proposed openstack/oslo.versionedobjects master: Remove translation sections from setup.cfg https://review.opendev.org/728354 | 04:01 |
| openstackgerrit | melissaml proposed openstack/oslo.context master: Remove translation sections from setup.cfg https://review.opendev.org/728355 | 04:08 |
| *** zzzeek has quit IRC | 04:23 | |
| *** zzzeek has joined #openstack-oslo | 04:24 | |
| *** jaosorior has joined #openstack-oslo | 06:43 | |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Use xelatex as latex_engine https://review.opendev.org/728370 | 06:46 |
| AJaeger | stephenfin, smcginnis, hberaud, bnemec, melwitt, I propose to set xelatex unconditionally again ^ | 06:48 |
| AJaeger | I could not find a way to override the default. stephenfin, if you have a better solution: Happy to go with that one! | 06:49 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Update docstheme https://review.opendev.org/728371 | 06:52 |
| AJaeger | testing now in https://review.opendev.org/728372 | 06:57 |
| *** ralonsoh has joined #openstack-oslo | 07:12 | |
| *** rpittau|afk is now known as rpittau | 07:12 | |
| openstackgerrit | Merged openstack/oslo.cache master: Switch to Victoria tests https://review.opendev.org/726519 | 07:23 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Add cross-tests for nova https://review.opendev.org/728377 | 07:28 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Add cross-tests for nova https://review.opendev.org/728377 | 07:28 |
| AJaeger | melwitt, stephenfin, let's add cross-tests. Those should fail with current state of repo and pass before the next release ;) | 07:30 |
| *** tosky has joined #openstack-oslo | 07:35 | |
| *** tkajinam has quit IRC | 08:14 | |
| hberaud | AJaeger: o/ | 08:14 |
| hberaud | AJaeger: IIRC yesterday we reintroduced the conditional engine, that's right? | 08:15 |
| hberaud | AJaeger: ah no... we even added the engine itself if not defined (was undefined at all) | 08:16 |
| *** dtantsur|afk is now known as dtantsur | 08:17 | |
| AJaeger | hberaud: the engine is always defined, so the conditional was a nop ;( | 08:21 |
| AJaeger | and my cross-tests don't work | 08:21 |
| hberaud | ok | 08:22 |
| AJaeger | they work partially - only on the docs, not on pdf | 08:23 |
| hberaud | ok | 08:23 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Add cross-tests for nova https://review.opendev.org/728377 | 08:30 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: Add cross-tests for nova https://review.opendev.org/728377 | 08:44 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: DNM: Add cross-tests for nova https://review.opendev.org/728377 | 08:56 |
| AJaeger | found the culprit - nova sets envdir and then siblings install does not work ;( | 08:57 |
| hberaud | AJaeger: for the latex engine? | 09:05 |
| hberaud | stephenfin: o/ if you've some spare time please can you take a look to https://review.opendev.org/#/c/712479/ & https://review.opendev.org/#/c/718223/ I addressed your comments and they work as expected. Please ignore the follow up patches I still have some issues with the memcached backend... | 09:09 |
| stephenfin | sure | 09:09 |
| AJaeger | hberaud: testing openstackdocstheme from master does not work with nova directly, the envdir breaks the logic we use for installing from master (sibling install). So, I have testing up now and can discuss further with melwitt and mordred (on #openstack-infra) on how to address this in general | 09:16 |
| hberaud | AJaeger: ack | 09:16 |
| openstackgerrit | Merged openstack/openstackdocstheme master: Use xelatex as latex_engine https://review.opendev.org/728370 | 09:19 |
| openstackgerrit | Stefan Nica proposed openstack/oslo.messaging stable/pike: Use default exchange for direct messaging https://review.opendev.org/728396 | 09:24 |
| AJaeger | stephenfin, hberaud , want to merge https://review.opendev.org/728371 as well? | 09:40 |
| stephenfin | AJaeger: good call, done | 09:41 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: DNM: Add cross-tests for nova https://review.opendev.org/728377 | 09:44 |
| AJaeger | I'm still struggling with a working test for nova with master ;( | 09:44 |
| AJaeger | stephenfin: are you confident this fixes the problem and should we release a new version? I am but didn't test it yet :( | 09:45 |
| stephenfin | I'm reasonably confident, but I can't test locally since Fedora is missing a texlive package that Sphinx needs to build PDFs :( | 09:46 |
| AJaeger | mmh, but we should be able to test that pdflatex is not called - let me try.. | 09:47 |
| openstackgerrit | Merged openstack/openstackdocstheme master: Update docstheme https://review.opendev.org/728371 | 09:53 |
| AJaeger | So, with openstackdocstheme master, file doc/build/pdf/latexmkrc has $pdflatex = 'xelatex' - and with 2.1.0 it's $pdflatex = 'pdflatex ' | 09:54 |
| AJaeger | So, looks correct | 09:54 |
| AJaeger | hberaud, stephenfin, pushed next release - https://review.opendev.org/728404 | 09:56 |
| openstackgerrit | Merged openstack/tooz master: Add support for Consul ACL token parameter. https://review.opendev.org/718488 | 10:01 |
| openstackgerrit | melissaml proposed openstack/oslo.versionedobjects master: Remove translation sections from setup.cfg https://review.opendev.org/728354 | 10:02 |
| AJaeger | stephenfin: so, 728377 build with new openstackdocstheme master correctly if I interpret the log files | 10:04 |
| AJaeger | stephenfin: what do you think of loggging the openstackdocstheme version? that would have helped a bit... | 10:05 |
| stephenfin | AJaeger: I was pretty sure Sphinx logged extension version already | 10:06 |
| AJaeger | stephenfin: I don't see it in output | 10:06 |
| stephenfin | ah, I think we need to provide a version in the return value from 'setup()' https://www.sphinx-doc.org/en/master/extdev/index.html#extension-metadata | 10:06 |
| stephenfin | but yeah, logging the version seems fine by me | 10:07 |
| AJaeger | and cross-jobs is not working completely -but I'm convinced it uses xelatex now | 10:08 |
| AJaeger | so, hberaud, stephenfin , let's release https://review.opendev.org/728404 - third time is a charm ;) | 10:09 |
| openstackgerrit | melissaml proposed openstack/oslo.limit master: Remove translation sections from setup.cfg https://review.opendev.org/728406 | 10:09 |
| *** rpittau is now known as rpittau|bbl | 10:09 | |
| openstackgerrit | Merged openstack/tooz master: Switch to Victoria tests https://review.opendev.org/726520 | 11:28 |
| *** threestrands has quit IRC | 11:46 | |
| AJaeger | bnemec: could you review the docstheme release, please: https://review.opendev.org/728404 | 11:49 |
| *** rpittau|bbl is now known as rpittau | 12:06 | |
| *** raildo has joined #openstack-oslo | 12:15 | |
| AJaeger | bnemec: it's released, nothing to do anymore ;) | 12:31 |
| AJaeger | stephenfin, smcginnis, I'm testing the new release with depends-on the upper-constraints update, see https://review.opendev.org/#/c/728432/1 where I listed the 3 changes I test (nova, cinder, python-cinderclient). | 12:41 |
| hberaud | moguimar: o/ if you are around and if you've some spare time please can you take a look to https://review.opendev.org/#/c/712479/ & https://review.opendev.org/#/c/718223/ I addressed your comments and they work as expected. Please ignore the follow up patches I still have some issues with the memcached backend... but for now we can move forward these 2 patches | 12:58 |
| smcginnis | AJaeger: Looking at the cinder one in Zuul status, pdf job passed. Lower-constraints failed, but I think it at least shows it is better. | 12:58 |
| smcginnis | Thanks for checking that. | 12:58 |
| smcginnis | Confirmed it pulled in 2.1.2. | 12:59 |
| smcginnis | And pdf-docs job completed successfully. | 12:59 |
| smcginnis | s/job/step/ | 12:59 |
| smcginnis | Hmm, nova job failed though. | 13:01 |
| smcginnis | Looks like it could be a legitimate issue with the nova docs. Not sure why the pdf build had an issue with it but not the regular build. | 13:03 |
| AJaeger | smcginnis: let's check that closely. stephenfin, do you have time to help checking? | 13:32 |
| AJaeger | smcginnis: the novaclient might be related to my change... | 13:35 |
| AJaeger | smcginnis: novaclient: This is using openstackclient 2.0.2 for PDF build, let's first merge https://review.opendev.org/728430 | 13:37 |
| moguimar | hberaud, ack | 13:58 |
| moguimar | I have upstream friday today =D | 13:59 |
| hberaud | :) | 14:00 |
| *** bnemec is now known as beekneemech | 14:27 | |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.concurrency master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728269 | 15:07 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.concurrency master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728269 | 15:08 |
| melwitt | AJaeger: do you still need help with a nova cross job? I'm not an expert but I can try to help figure out | 15:08 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.cache master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728267 | 15:10 |
| AJaeger | melwitt: I'm fine now, thanks. | 15:12 |
| melwitt | ok cool | 15:12 |
| AJaeger | melwitt: https://review.opendev.org/728270 is green - novaclient | 15:12 |
| AJaeger | for the cross-job, we need zuul-jobs about your envdir setting in tox.ini; so, I cannot do that directly. | 15:13 |
| melwitt | ah great! (green job) | 15:14 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.cache master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728267 | 15:14 |
| melwitt | AJaeger: sorry what do you mean about needing envdir setting? | 15:14 |
| AJaeger | melwitt, nova is setting in tox.ini envdir and thus sibling install fails since we expect that envdir contains the envlist (pdf-docs) and not another one, see https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/tox/library/tox_install_sibling_packages.py#L190-L194 . | 15:16 |
| AJaeger | melwitt: we expect .tox/pdf-docs but https://opendev.org/openstack/nova/src/branch/master/tox.ini#L182 uses .tox/docs | 15:16 |
| melwitt | ohhh I see | 15:17 |
| melwitt | I guess we must be trying to optimize for local builds, so that all docs related test runs share same envdir | 15:17 |
| AJaeger | yep | 15:18 |
| AJaeger | and nobody checked that CI handles this nicely. avass said in #zuul that he will look into it. | 15:18 |
| AJaeger | But so long as we have this, installation of siblings with nova will not work | 15:18 |
| AJaeger | so, be aware of it | 15:19 |
| melwitt | I see. ok, will do, I'll follow along what avass finds and go from there. see if it's something that CI can handle or if we just need to go ahead and change our envdir for pdf-docs only | 15:20 |
| melwitt | or other, if there's some other option I don't know about | 15:20 |
| AJaeger | let's see that these pass: https://review.opendev.org/#/q/status:open++topic:reno-openstacksdocstheme and then we can merge the upper-constraints of openstackdocstheme 2.1.2 and use it | 15:23 |
| melwitt | ack | 15:23 |
| AJaeger | melwitt: I think the envdir is the only one - and we're good to go forward. I would have loved a cross test directly but that has to wait until avass is done. Parsing the tox.ini is a bit more complex | 15:23 |
| melwitt | ok | 15:24 |
| AJaeger | melwitt: sorry, for all the mess that I overlooked in reviewing. Hope we're good now without too many grey hairs for either of us ;) | 15:28 |
| melwitt | AJaeger: haha np, grey hairs indeed xD | 15:30 |
| stephenfin | AJaeger: Sorry, back to back meetings. Did you get sorted or do you still need me for something? | 15:38 |
| AJaeger | stephenfin: all fine! | 15:40 |
| AJaeger | stephenfin: thanks for asking. But all loooks green - just waiting for nova and cinder builds to finish | 15:40 |
| AJaeger | https://review.opendev.org/#/q/status:open++topic:reno-openstacksdocstheme | 15:40 |
| stephenfin | Ah, good to hear :) | 15:40 |
| AJaeger | stephenfin: you might want to patch the theme to return the version number | 15:40 |
| AJaeger | And I need to wait for some fixes to jobs before we can do an integration test. | 15:41 |
| *** jaosorior has quit IRC | 16:03 | |
| AJaeger | melwitt, stephenfin, nova passes: https://review.opendev.org/728263 | 16:10 |
| stephenfin | AJaeger++ | 16:11 |
| melwitt | cool thanks AJaeger | 16:13 |
| openstackgerrit | Ghanshyam Mann proposed openstack/pycadf master: Fix hacking min version to 3.0.1 https://review.opendev.org/727530 | 16:14 |
| AJaeger | stephenfin: updated 728263 as suggested | 16:15 |
| stephenfin | thanks, +2 | 16:15 |
| AJaeger | stephenfin: let's call it a weekend and celebrate ;) | 16:16 |
| AJaeger | thanks | 16:16 |
| stephenfin | Hear, hear | 16:16 |
| openstackgerrit | Hervé Beraud proposed openstack/tooz master: Use unittest.mock instead of third party mock https://review.opendev.org/728496 | 16:29 |
| openstackgerrit | Hervé Beraud proposed openstack/tooz master: Use unittest.mock instead of third party mock https://review.opendev.org/728496 | 16:30 |
| openstackgerrit | Stephen Finucane proposed openstack/openstackdocstheme master: Provide extension version https://review.opendev.org/728497 | 16:42 |
| openstackgerrit | Stephen Finucane proposed openstack/openstackdocstheme master: trivial: Remove leftover Python 2 code https://review.opendev.org/728498 | 16:42 |
| openstackgerrit | Stephen Finucane proposed openstack/openstackdocstheme master: Use consistent logging https://review.opendev.org/728499 | 16:42 |
| *** dtantsur is now known as dtantsur|afk | 17:08 | |
| openstackgerrit | jacky06 proposed openstack/oslo.serialization master: Remove the unused coding style modules https://review.opendev.org/728509 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.rootwrap master: Remove the unused coding style modules https://review.opendev.org/728510 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.reports master: Remove the unused coding style modules https://review.opendev.org/728511 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.i18n master: Remove the unused coding style modules https://review.opendev.org/728512 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.privsep master: Remove the unused coding style modules https://review.opendev.org/728513 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.limit master: Remove the unused coding style modules https://review.opendev.org/728514 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.policy master: Remove the unused coding style modules https://review.opendev.org/728515 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.cache master: Remove the unused coding style modules https://review.opendev.org/728516 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.middleware master: Remove the unused coding style modules https://review.opendev.org/728517 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.versionedobjects master: Remove the unused coding style modules https://review.opendev.org/728518 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.context master: Remove the unused coding style modules https://review.opendev.org/728519 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.utils master: Remove the unused coding style modules https://review.opendev.org/728520 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.log master: Remove the unused coding style modules https://review.opendev.org/728521 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.service master: Remove the unused coding style modules https://review.opendev.org/728522 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.concurrency master: Remove the unused coding style modules https://review.opendev.org/728523 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.vmware master: Remove the unused coding style modules https://review.opendev.org/728524 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.db master: Remove the unused coding style modules https://review.opendev.org/728525 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.messaging master: Remove the unused coding style modules https://review.opendev.org/728526 | 17:10 |
| openstackgerrit | jacky06 proposed openstack/oslo.config master: Remove the unused coding style modules https://review.opendev.org/728527 | 17:10 |
| *** rpittau is now known as rpittau|afk | 17:11 | |
| openstackgerrit | jacky06 proposed openstack/oslo.config master: Remove the unused coding style modules https://review.opendev.org/728527 | 17:15 |
| openstackgerrit | jacky06 proposed openstack/oslo.messaging master: Remove the unused coding style modules https://review.opendev.org/728526 | 17:16 |
| openstackgerrit | jacky06 proposed openstack/oslo.db master: Remove the unused coding style modules https://review.opendev.org/728525 | 17:16 |
| openstackgerrit | jacky06 proposed openstack/oslo.concurrency master: Remove the unused coding style modules https://review.opendev.org/728523 | 17:17 |
| openstackgerrit | jacky06 proposed openstack/oslo.service master: Remove the unused coding style modules https://review.opendev.org/728522 | 17:17 |
| openstackgerrit | jacky06 proposed openstack/oslo.log master: Remove the unused coding style modules https://review.opendev.org/728521 | 17:17 |
| openstackgerrit | jacky06 proposed openstack/oslo.utils master: Remove the unused coding style modules https://review.opendev.org/728520 | 17:17 |
| openstackgerrit | jacky06 proposed openstack/oslo.context master: Remove the unused coding style modules https://review.opendev.org/728519 | 17:17 |
| openstackgerrit | jacky06 proposed openstack/oslo.versionedobjects master: Remove the unused coding style modules https://review.opendev.org/728518 | 17:18 |
| openstackgerrit | jacky06 proposed openstack/oslo.middleware master: Remove the unused coding style modules https://review.opendev.org/728517 | 17:18 |
| openstackgerrit | jacky06 proposed openstack/oslo.policy master: Remove the unused coding style modules https://review.opendev.org/728515 | 17:18 |
| openstackgerrit | jacky06 proposed openstack/oslo.vmware master: Remove the unused coding style modules https://review.opendev.org/728524 | 17:18 |
| openstackgerrit | jacky06 proposed openstack/oslo.limit master: Remove the unused coding style modules https://review.opendev.org/728514 | 17:19 |
| openstackgerrit | jacky06 proposed openstack/oslo.privsep master: Remove the unused coding style modules https://review.opendev.org/728513 | 17:19 |
| openstackgerrit | jacky06 proposed openstack/oslo.i18n master: Remove the unused coding style modules https://review.opendev.org/728512 | 17:19 |
| openstackgerrit | jacky06 proposed openstack/oslo.reports master: Remove the unused coding style modules https://review.opendev.org/728511 | 17:19 |
| openstackgerrit | jacky06 proposed openstack/oslo.cache master: Remove the unused coding style modules https://review.opendev.org/728516 | 17:20 |
| *** ralonsoh has quit IRC | 17:55 | |
| smcginnis | Oh my. | 17:56 |
| beekneemech | +1 | 18:24 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.context master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728546 | 18:24 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.context master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728546 | 18:24 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.db master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728547 | 18:25 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.i18n master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728548 | 18:26 |
| *** iurygregory has quit IRC | 18:35 | |
| *** iurygregory has joined #openstack-oslo | 18:38 | |
| *** iurygregory has quit IRC | 18:45 | |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.context master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728546 | 18:46 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.limit master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728567 | 19:54 |
| openstackgerrit | Andreas Jaeger proposed openstack/oslo.log master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728568 | 19:56 |
| openstackgerrit | Andreas Jaeger proposed openstack/openstackdocstheme master: WIP: Add cross-tests for nova https://review.opendev.org/728377 | 20:24 |
| *** iurygregory has joined #openstack-oslo | 20:42 | |
| *** raildo_ has joined #openstack-oslo | 21:06 | |
| *** raildo has quit IRC | 21:08 | |
| *** raildo_ has quit IRC | 21:57 | |
| *** tosky has quit IRC | 23:39 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!