| *** gmann is now known as gmann_afk | 00:07 | |
| *** gmann_afk is now known as gmann | 01:11 | |
| *** ysandeep|out is now known as ysandeep|sick | 05:11 | |
| *** jpena|off is now known as jpena | 06:58 | |
| *** frickler_ is now known as frickler | 07:25 | |
| *** odyssey4me is now known as Guest1483 | 08:12 | |
| *** ykarel is now known as ykarel|lunch | 08:50 | |
| *** ykarel|lunch is now known as ykarel | 10:20 | |
| *** jpodivin is now known as jpodivin|ruck | 10:27 | |
| *** jcapitao is now known as jcapitao_lunch | 10:59 | |
| *** rlandy is now known as rlandy|ruck | 11:03 | |
| *** jpena is now known as jpena|lunch | 11:27 | |
| *** jpena|lunch is now known as jpena | 12:26 | |
| *** jcapitao_lunch is now known as jcapitao | 12:30 | |
| *** ykarel is now known as ykarel|afk | 12:43 | |
| *** ykarel|afk is now known as ykarel | 13:29 | |
| stephenfin | can someone explain to me what populates e.g. https://docs.openstack.org/adjutant/xena/index.html ? | 13:52 |
|---|---|---|
| stephenfin | and why e.g. https://docs.openstack.org/keystone/xena/index.html is a 404 right now? | 13:52 |
| fungi | a change needs to merge to that branch of that project | 13:54 |
| *** ysandeep|sick is now known as ysandeep | 13:55 | |
| fungi | usually the core reviewers would merge the autogenerated .gitreview file update change which is proposed when the stable branch is created | 13:56 |
| stephenfin | Ah, so those are missing. Good to know | 13:56 |
| * stephenfin updates commit message to add that context | 13:56 | |
| fungi | https://review.opendev.org/q/status:open+-is:wip+project:openstack/keystone+branch:stable/xena | 13:57 |
| fungi | looks like they were proposed a couple days ago and have some core reviews but no approvals yet | 13:57 |
| fungi | specifically it's the stable-core reviewers who need to approve those i think? | 13:58 |
| stephenfin | yup, typically anyway | 13:58 |
| stephenfin | though they still need to merge them for victoria so it could take a while :) https://review.opendev.org/q/status:open+-is:wip+project:openstack/python-keystoneclient+branch:stable/wallaby | 13:59 |
| fungi | but really, any change merging to that branch should trigger a docs update, in theory | 13:59 |
| fungi | worth bringing up in the release team meeting, starting now in #openstack-release | 14:00 |
| *** ykarel is now known as ykarel|away | 14:26 | |
| *** ysandeep is now known as ysandeep|out | 15:00 | |
| fungi | rlandy|ruck: https://review.opendev.org/812005 now has parent changes to exercise the tox.requires behavior and fix the previously failing module, if you want to test something with a depends-on to that | 16:08 |
| fungi | would be good to know if you still see the same error with it as before, or possibly a different error this time | 16:08 |
| *** jpena is now known as jpena|off | 16:09 | |
| rlandy|ruck | fungi: thank you - will test that shortly | 16:09 |
| rlandy|ruck | rotating meetings atm :) | 16:10 |
| fungi | yeah, i know the feeling | 16:16 |
| fungi | isn't friday supposed to be meeting free? somebody tell somebody | 16:16 |
| *** ysandeep|out is now known as ysandeep | 17:17 | |
| melwitt | does anyone know if there's a better way than using DNM patch with LIBS_FROM_GIT=oslo.* to run against unmerged oslo changes in the gate? | 17:37 |
| fungi | melwitt: you could add a job which sets that | 17:38 |
| fungi | testing against released dependencies and testing against unreleased dependencies under development are of course two different things, so you would do both, not choose different ones for differend changes | 17:39 |
| melwitt | I thought about that but what I've got is failing unit tests (AttributeError) and I also want to run with tempest and grenade. just wanted to check and make sure I wasn't missing another way to do it :) | 17:41 |
| fungi | how it's usually been done is that the oslo projects would have a "source" job which set that, so they'd block changes which broke nova unit tests | 17:43 |
| melwitt | I'm working on unified limits in nova and I have proposed oslo.limit interface changes to do what I want in nova and want to run it in the gate and make sure I haven't done something wrong before the oslo.limit changes are eventually merged and released | 17:43 |
| fungi | basically change to oslo.config or whatever would run nova's unit tests with the proposed change applied | 17:43 |
| fungi | yeah, maybe you just need to add jobs like that for oslo.limit? look at some of the other popular oslo projects for examples of similar source jobs | 17:44 |
| melwitt | the nova changes that cause the error are unmerged too ... basically want to test two unmerged things together to make sure they work well together | 17:45 |
| melwitt | it's normally easy to do that but with oslo have to do something to make it pull from gerrit change | 17:46 |
| fungi | ahh, yeah, that gets convoluted, but i agree a throwaway change to test combining them is fine, it's how i normally do it | 17:46 |
| melwitt | I was thinking of putting up a DNM devstack patch to hardcode it into LIBS_FROM_GIT and then DNM patch in my series to Depends-On that | 17:47 |
| fungi | yeah, and stick the DNM change at the end of course so you can just abandon it when you're done without needing to rebase anything | 17:49 |
| *** ysandeep is now known as ysandeep|out | 18:17 | |
| melwitt | does anyone know what's the difference between setting a devstack var under devstack_local_conf or devstack_localrc in a zuul config? | 18:18 |
| fungi | melwitt: looking at roles/write-devstack-local-conf/tasks/main.yaml one is a subset of the other | 18:21 |
| melwitt | oh, thanks. sorry I didn't realize it was in devstack 😓 | 18:22 |
| fungi | devstack_local_conf lets you set a number of things about the config, one of which is the localrc array, which defaults to the value of devstack_localrc | 18:22 |
| fungi | no worries, i was curious, but yeah #openstack-qa might yield better answers | 18:22 |
| melwitt | thanks! | 18:23 |
| fungi | any time! | 18:23 |
| melwitt | ah looks like devstack_local_conf is when you want to also specify which phase you want the things to apply in, like post-config or test-config | 18:40 |
| fungi | makes sense, thanks for clarifying! | 18:41 |
| simondodsley | is there an issue with review.opendev.org? | 19:29 |
| hberaud | same thing here... review.opendev.org doesn't load | 19:32 |
| fungi | yes, i'm discussing it in #opendev | 19:37 |
| fungi | seems like the server has become unreachable suddenly | 19:37 |
| -opendevstatus- NOTICE: The review.opendev.org Gerrit server has become unreachable as of approximately 19:10 UTC due to a networking issue in the provider, but should be reachable again shortly | 19:44 | |
| rlandy|ruck | fungi: put in https://review.opendev.org/c/openstack/tripleo-quickstart/+/812225 with tox change (https://review.opendev.org/812005) depends-on | 21:02 |
| rlandy|ruck | it will run openstack-tox-molecule | 21:02 |
| fungi | rlandy|ruck: thanks, i don't think there's much risk of the unrevert getting approved before that reports anyway | 21:17 |
| rlandy|ruck | fungi; vote +1 on https://review.opendev.org/812005 - the tox jobs are passing right now on the test patch | 21:18 |
| fungi | thanks a bunch rlandy|ruck! | 21:24 |
| rlandy|ruck | sure - thanks for the fix | 21:24 |
| fungi | test coverage makes all the difference ;) | 21:24 |
| rlandy|ruck | we need a t-shirt with ^^ saying | 21:36 |
| fungi | it still won't be as popular as the "it worked in devstack" shirts | 22:05 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!