| *** tetsuro has joined #openstack-release | 00:38 | |
| *** dave-mccowan has quit IRC | 01:01 | |
| *** tetsuro has quit IRC | 03:15 | |
| *** tetsuro has joined #openstack-release | 03:19 | |
| *** tetsuro has quit IRC | 03:20 | |
| *** tetsuro has joined #openstack-release | 03:52 | |
| *** ykarel|away is now known as ykarel | 04:10 | |
| *** evrardjp has quit IRC | 04:36 | |
| *** evrardjp has joined #openstack-release | 04:36 | |
| *** udesale has joined #openstack-release | 05:12 | |
| *** udesale has quit IRC | 05:14 | |
| *** udesale has joined #openstack-release | 05:14 | |
| *** vishalmanchanda has joined #openstack-release | 05:22 | |
| *** irclogbot_1 has quit IRC | 06:49 | |
| *** prometheanfire has quit IRC | 06:51 | |
| *** prometheanfire has joined #openstack-release | 06:51 | |
| *** jhesketh has quit IRC | 06:52 | |
| *** irclogbot_3 has joined #openstack-release | 06:53 | |
| *** irclogbot_3 has quit IRC | 06:53 | |
| *** jhesketh has joined #openstack-release | 06:53 | |
| *** tonyb has quit IRC | 06:54 | |
| *** irclogbot_3 has joined #openstack-release | 06:54 | |
| *** tonyb has joined #openstack-release | 06:59 | |
| *** priteau has joined #openstack-release | 07:00 | |
| *** slaweq has joined #openstack-release | 07:03 | |
| *** jbadiapa has joined #openstack-release | 07:08 | |
| *** tosky has joined #openstack-release | 07:32 | |
| *** e0ne has joined #openstack-release | 07:35 | |
| *** rpittau|afk is now known as rpittau | 07:37 | |
| *** dtantsur|afk is now known as dtantsur | 08:04 | |
| *** ykarel is now known as ykarel|lunch | 08:30 | |
| *** priteau has quit IRC | 09:03 | |
| *** tetsuro has quit IRC | 09:09 | |
| *** jbadiapa is now known as jbadiapa|away | 09:27 | |
| ttx | smcginnis: ok commented | 09:47 |
|---|---|---|
| ttx | smcginnis: TL;DR: it's really a tension between a rule meant to enable multi-vendor consumption of the code, and a reasonable ask for a project effectively consumed by a single vendor | 09:49 |
| ttx | I put it on the agenda for tomorrow's meeting | 09:50 |
| *** ykarel|lunch is now known as ykarel | 09:59 | |
| *** rpittau is now known as rpittau|bbl | 10:16 | |
| *** dhellmann_ has joined #openstack-release | 11:31 | |
| *** dhellmann has quit IRC | 11:32 | |
| *** dhellmann_ is now known as dhellmann | 11:32 | |
| *** ricolin has quit IRC | 11:54 | |
| *** jbadiapa|away is now known as jbadiapa | 11:59 | |
| *** rpittau|bbl is now known as rpittau | 12:07 | |
| *** dmellado has quit IRC | 12:15 | |
| *** dmellado has joined #openstack-release | 12:23 | |
| *** dave-mccowan has joined #openstack-release | 12:49 | |
| smcginnis | ttx: Looks like it worked out in the end. | 12:53 |
| *** dave-mccowan has quit IRC | 12:54 | |
| *** ricolin_ has joined #openstack-release | 13:01 | |
| *** jbadiapa has quit IRC | 13:02 | |
| *** jbadiapa has joined #openstack-release | 13:04 | |
| *** udesale_ has joined #openstack-release | 13:09 | |
| *** udesale has quit IRC | 13:11 | |
| *** ykarel is now known as ykarel|afk | 13:43 | |
| *** ykarel|afk is now known as ykarel | 14:02 | |
| openstackgerrit | Maciej Józefczyk proposed openstack/releases master: Initial release of ovn-octavia-provider https://review.opendev.org/710200 | 14:21 |
| *** ykarel is now known as ykarel|away | 15:22 | |
| *** zxiiro has joined #openstack-release | 15:23 | |
| openstackgerrit | Thierry Carrez proposed openstack/releases master: Update milestone2-to-milestone3 process https://review.opendev.org/716635 | 15:49 |
| smcginnis | stephenfin: Sorry, you're basically our only reno expert now. Any updates on that issue with heat/reno that's blocking their final stable/rocky release? | 15:53 |
| stephenfin | smcginnis: The fix is this one https://review.opendev.org/#/c/715481/ | 15:54 |
| stephenfin | I need to get that merged and backported | 15:55 |
| smcginnis | stephenfin: Oh right, that one is just waiting on other failures. | 15:55 |
| stephenfin | yup | 15:55 |
| smcginnis | stephenfin: There was another one I was thinking of with reno itself. I need to find that. | 15:55 |
| stephenfin | this one | 15:55 |
| stephenfin | https://review.opendev.org/#/c/715479/ | 15:56 |
| smcginnis | Yep, that's the one. | 15:56 |
| stephenfin | that was needed if I wanted to use v1 of the previous patch https://review.opendev.org/#/c/715481/1/releasenotes/config.yaml | 15:56 |
| stephenfin | it's still a bug, but it's not necessary with v2 | 15:56 |
| smcginnis | OK, I was confused. I thought it was to address the other common reno issue we have now with scanning earlier branches pre-reno release notes. | 15:57 |
| stephenfin | Ah, no, not really. The solution for all of those is to set an earliest_version so reno doesn't try scanning old, crufty branches | 15:58 |
| smcginnis | Yeah, we've been doing that when we hit them. I'm just afraid of the chaos that is going to happen at RC time. ;) | 15:59 |
| stephenfin | If it comes to that, we'll just have to disable that functionality in reno by default or something | 16:00 |
| stephenfin | Perhaps it would be worth adding a flag to do that now? Something environment'y so we could set it in our jobs | 16:00 |
| stephenfin | Just in case (TM) | 16:01 |
| smcginnis | That would actually be my preference. We have so many repos that have been around for a long time. These failures keep popping up and no one understands why since the release notes are fine. | 16:01 |
| stephenfin | Okay, can you draft that or will I? Will need to discuss with bnemec to get a FFE | 16:01 |
| smcginnis | If you can, I would appreciate that. I've been in meetings non-stop this morning, sometimes more than one at once. | 16:02 |
| stephenfin | Cool. Can do | 16:03 |
| smcginnis | I don't think me writing any code would be prudent at this point. | 16:03 |
| stephenfin | :) | 16:03 |
| bnemec | If we're fixing a bug I don't think we need an FFE. | 16:03 |
| smcginnis | It's a bit of a gray area, but I think it's a bug. | 16:04 |
| smcginnis | Oh, but actually reno is a Release Management deliverable officially, not an oslo one. | 16:04 |
| smcginnis | So we should be good. | 16:04 |
| stephenfin | TIL | 16:04 |
| smcginnis | I actually wasn't sure and had to look what was "official": https://opendev.org/openstack/governance/src/branch/master/reference/projects.yaml#L2936 | 16:05 |
| openstackgerrit | Abhishek Kekane proposed openstack/releases master: Release glance_store 2.0.0 https://review.opendev.org/716382 | 16:13 |
| *** rpittau is now known as rpittau|afk | 16:15 | |
| stephenfin | smcginnis: Might not be necessary to modify reno - you can override the branch by passing the '--branch' argument | 16:25 |
| stephenfin | so if you pass '--branch stable/rocky', your issues should go away | 16:25 |
| smcginnis | stephenfin: How does that work with how pbr (I think that's where the integration is) builds the release notes as part of creating an sdist? | 16:26 |
| smcginnis | That's why release note builds are fine. Those specify which branch to do with the sphinx integration. | 16:26 |
| stephenfin | oh, it wouldn't /o\ | 16:26 |
| smcginnis | But the failure is when it's jist trying to build everything all in one. | 16:26 |
| smcginnis | I don't like the behavior, and not sure when that was introduced. | 16:27 |
| smcginnis | But the combination of that with the scanning changes causes a lot of headaches. | 16:27 |
| stephenfin | it's one and the same | 16:28 |
| stephenfin | change Ibe92a9fa5f42b5a7160201cd64ee6a93f9a78229 | 16:28 |
| stephenfin | the idea was that we needed to scan all branches to make sure we captured release notes that only appeared on stable branches | 16:29 |
| stephenfin | the issues we're seeing occur because we're scanning branches we never scanned before | 16:29 |
| smcginnis | Yeah, I mean that along with whenever the presence of reno was automatically picked up when creating an sdist and including release notes builds in that process. | 16:29 |
| stephenfin | oh, yeah, that was me too \o/ | 16:30 |
| smcginnis | Haha, so it's ALL your fault. :) | 16:30 |
| stephenfin | change 3a6b96c799de732364d414d63d05b10305c9c5ba | 16:30 |
| stephenfin | whoops | 16:30 |
| stephenfin | I54cb7a894770040165cc918469cb0ed486518789 | 16:30 |
| stephenfin | dhellmann told me to do it :) | 16:31 |
| stephenfin | *asked me nicely to do it | 16:31 |
| smcginnis | ;) | 16:31 |
| * dhellmann denies culpability | 16:31 | |
| smcginnis | So ideally, I would think reno just handles it and if something isn't there, which is the case for any repo that has been around for awhile, it just ignores that and doesn't blow up. | 16:31 |
| dhellmann | is this another of those cases where reno needs to ignore a very old branch? | 16:32 |
| smcginnis | Yeah. | 16:32 |
| stephenfin | yup | 16:33 |
| smcginnis | We've had several of those. | 16:33 |
| smcginnis | And it's usually at the most inopportune time that a project hits it. | 16:33 |
| smcginnis | Like when trying to close out rocky-em. | 16:33 |
| dhellmann | of course | 16:33 |
| stephenfin | though in this case, we're also tangling with the fact reno currently sorted branches alphabetically, so heat's old 'vN' branches appear "newer" than Ussuri | 16:34 |
| dhellmann | ah | 16:34 |
| smcginnis | Or 2014.1 is newer than 13.0. | 16:34 |
| stephenfin | haven't seen that, but yup, probably that too | 16:35 |
| dhellmann | yeah, this is one of the reasons we made it configurable, to deal with projects that had things outside of the "standard" | 16:35 |
| dhellmann | I'm not sure why the order would matter, though? | 16:35 |
| smcginnis | Which is a fair percentage of openstack/* | 16:35 |
| dhellmann | do you have a log with an error? | 16:35 |
| smcginnis | https://review.opendev.org/715282 is a new one. | 16:36 |
| *** evrardjp has quit IRC | 16:36 | |
| smcginnis | Then there were several related to https://review.opendev.org/#/c/707498/ | 16:36 |
| *** evrardjp has joined #openstack-release | 16:36 | |
| dhellmann | my openstack dev environment is very old, give me some time to set up a reproducere | 16:38 |
| smcginnis | Thanks dhellmann! That would be appreciated. | 16:40 |
| dhellmann | which heat repo was failing, on which branch? | 16:41 |
| smcginnis | I think openstack/heat on stable/rocky. Let me double check. | 16:42 |
| dhellmann | looks like openstack/heat on the rocky branch | 16:43 |
| dhellmann | yeah | 16:43 |
| smcginnis | You may have a better way to test, but quickest way for me has been "tox -e venv --notest; .tox/venv/bin/pip install reno; .tox/venv/bin/python setup.py sdist" | 16:44 |
| dhellmann | that looks like what I would do | 16:44 |
| dhellmann | cloning now | 16:44 |
| openstackgerrit | Stephen Finucane proposed openstack/reno master: Allow configuration via environment variables https://review.opendev.org/716666 | 16:49 |
| stephenfin | smcginnis, (dhellmann): I need to go walk the dog before it gets dark, but I'll come back to that later ^ | 16:50 |
| smcginnis | Thanks! Need to step away for a bit too. | 16:50 |
| smcginnis | Thanks for your help with this guys. Much appreciated. | 16:50 |
| openstackgerrit | Merged openstack/releases master: Release glance_store 2.0.0 https://review.opendev.org/716382 | 17:00 |
| openstackgerrit | Doug Hellmann proposed openstack/reno master: scan the whole branch if a version cannot be found https://review.opendev.org/716672 | 17:05 |
| dhellmann | smcginnis , stephenfin : see if ^ helps; it fixed sdist building on heat's stable/rocky branch for me | 17:05 |
| dhellmann | that may have other unintended consequences when building specific branches, but only to include possibly more info than expected | 17:06 |
| dhellmann | and that can be fixed by explicitly configuring the base version for the branch in the sphinx directive | 17:06 |
| *** dtantsur is now known as dtantsur|afk | 17:07 | |
| *** udesale_ has quit IRC | 17:29 | |
| *** jbadiapa has quit IRC | 17:49 | |
| *** jbadiapa has joined #openstack-release | 17:50 | |
| *** gmann is now known as gmann_lunch | 17:56 | |
| smcginnis | dhellmann: That looks great. I've tested with heat and with cinder minus the reno.yaml file that was added to get around the failure. Both work and look fine to me. | 18:04 |
| dhellmann | *whew* | 18:05 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release os-brick 3.0.1 https://review.opendev.org/716720 | 18:25 |
| *** gmann_lunch is now known as gmann | 18:28 | |
| *** diablo_rojo has quit IRC | 18:32 | |
| *** diablo_rojo has joined #openstack-release | 18:33 | |
| *** e0ne has quit IRC | 18:38 | |
| *** e0ne has joined #openstack-release | 18:38 | |
| *** e0ne has quit IRC | 18:39 | |
| openstackgerrit | Merged openstack/reno master: scan the whole branch if a version cannot be found https://review.opendev.org/716672 | 18:43 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release reno 3.0.1 https://review.opendev.org/716724 | 18:46 |
| smcginnis | ttx, hberaud: I think we want that out right away to get around some of these reno related failures ^^ | 18:47 |
| hberaud | smcginnis: ack | 18:47 |
| * hberaud take a look | 18:48 | |
| openstackgerrit | Julia Kreger proposed openstack/releases master: Release sushy 3.2.0 for Ussuri https://review.opendev.org/716728 | 19:06 |
| openstackgerrit | Julia Kreger proposed openstack/releases master: Release ironic-lib 4.2.0 for Ussuri https://review.opendev.org/716732 | 19:16 |
| openstackgerrit | Merged openstack/releases master: Release reno 3.0.1 https://review.opendev.org/716724 | 19:22 |
| openstackgerrit | Michael Johnson proposed openstack/releases master: Release octavia-lib 1.6.0 for Ussuri https://review.opendev.org/716741 | 19:51 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final heat-translator for ussuri https://review.opendev.org/716742 | 19:54 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final tosca-parser for ussuri https://review.opendev.org/716743 | 19:56 |
| slaweq | smcginnis: hi, I have a question about Your comment in https://review.opendev.org/#/c/710200/ | 19:58 |
| slaweq | smcginnis: is there anything we can do to make initial release of this now? | 19:58 |
| slaweq | smcginnis: I know it's late in the cycle, but previously this driver was in networking-ovn repo and that will not be released in Ussuri anymore | 19:59 |
| smcginnis | slaweq: I just want to discuss it with the rest of the team in our meeting tomorrow to see if anyone has any concerns about making an exception. | 19:59 |
| slaweq | smcginnis: ok, thx a lot | 20:00 |
| *** weshay is now known as weshay|ruck | 20:00 | |
| slaweq | if I can do anything to help with that, please let me know | 20:00 |
| smcginnis | slaweq: Thanks, I'll ping you if any questions or concerns come up about it. | 20:01 |
| slaweq | smcginnis: sure, thx a lot | 20:01 |
| slaweq | at what hour is Your meeting tomorrow? | 20:01 |
| openstackgerrit | Michael Johnson proposed openstack/releases master: Release octavia-lib 2.0.0 for Ussuri https://review.opendev.org/716741 | 20:03 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final tosca-parser for ussuri https://review.opendev.org/716748 | 20:04 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final keystonemiddleware for ussuri https://review.opendev.org/716751 | 20:06 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final keystoneauth for ussuri https://review.opendev.org/716748 | 20:07 |
| *** jbadiapa has quit IRC | 20:07 | |
| *** jbadiapa has joined #openstack-release | 20:08 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final metalsmith for ussuri https://review.opendev.org/716753 | 20:08 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final mistral-extra for ussuri https://review.opendev.org/716755 | 20:10 |
| *** e0ne has joined #openstack-release | 20:11 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final monasca-common for ussuri https://review.opendev.org/716756 | 20:11 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final neutron-lib for ussuri https://review.opendev.org/716757 | 20:12 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final octavia-lib for ussuri https://review.opendev.org/716758 | 20:13 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final openstacksdk for ussuri https://review.opendev.org/716759 | 20:14 |
| johnsom | smcginnis FYI, the octavia-lib release has already been posted in https://review.opendev.org/716741. Also, shouldn't your proposals have the stable branch in them? | 20:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final os-client-config for ussuri https://review.opendev.org/716760 | 20:15 |
| smcginnis | johnsom: Ah, thanks. I missed removing that from the list. | 20:15 |
| smcginnis | johnsom: Good point, I probably should have included that with all of these. I'll abandon my octavia-lib one though. | 20:16 |
| johnsom | +1 | 20:16 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final ovsdbapp for ussuri https://review.opendev.org/716761 | 20:17 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final keystonemiddleware for ussuri https://review.opendev.org/716751 | 20:19 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final heat-translator for ussuri https://review.opendev.org/716742 | 20:20 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final metalsmith for ussuri https://review.opendev.org/716753 | 20:21 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final tosca-parser for ussuri https://review.opendev.org/716743 | 20:21 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final monasca-common for ussuri https://review.opendev.org/716756 | 20:22 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final neutron-lib for ussuri https://review.opendev.org/716757 | 20:23 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final mistral-extra for ussuri https://review.opendev.org/716755 | 20:24 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final openstacksdk for ussuri https://review.opendev.org/716759 | 20:24 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final os-client-config for ussuri https://review.opendev.org/716760 | 20:25 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release final keystoneauth for ussuri https://review.opendev.org/716748 | 20:25 |
| *** vishalmanchanda has quit IRC | 20:39 | |
| *** slaweq has quit IRC | 20:54 | |
| *** slaweq has joined #openstack-release | 21:05 | |
| *** slaweq has quit IRC | 21:09 | |
| *** slaweq has joined #openstack-release | 21:12 | |
| *** jbadiapa has quit IRC | 21:14 | |
| *** frickler_ has joined #openstack-release | 22:03 | |
| *** melwitt has quit IRC | 22:08 | |
| *** fnordahl has quit IRC | 22:08 | |
| *** frickler has quit IRC | 22:08 | |
| *** e0ne has quit IRC | 22:20 | |
| *** e0ne has joined #openstack-release | 22:21 | |
| *** slaweq has quit IRC | 22:31 | |
| *** slaweq has joined #openstack-release | 22:42 | |
| *** slaweq has quit IRC | 22:46 | |
| *** melwitt has joined #openstack-release | 22:51 | |
| *** tosky has quit IRC | 23:03 | |
| *** e0ne has quit IRC | 23:13 | |
| *** tetsuro has joined #openstack-release | 23:44 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!