| *** thorst has quit IRC | 00:02 | |
| *** rajathagasthya has quit IRC | 00:16 | |
| *** yamamoto_ has joined #openstack-release | 00:18 | |
| *** yamamoto_ has quit IRC | 00:24 | |
| *** thorst has joined #openstack-release | 00:25 | |
| *** thorst has quit IRC | 00:26 | |
| *** zhurong has joined #openstack-release | 00:45 | |
| *** yamamoto_ has joined #openstack-release | 01:21 | |
| *** yamamoto_ has quit IRC | 01:26 | |
| *** thorst has joined #openstack-release | 01:26 | |
| *** thorst has quit IRC | 01:31 | |
| *** bobh has quit IRC | 01:32 | |
| *** hongbin has joined #openstack-release | 01:34 | |
| *** gcb has joined #openstack-release | 01:36 | |
| *** huats has quit IRC | 01:49 | |
| *** huats has joined #openstack-release | 01:50 | |
| *** thorst has joined #openstack-release | 01:51 | |
| *** thorst has quit IRC | 01:51 | |
| *** iyamahat has quit IRC | 02:03 | |
| *** liujiong has joined #openstack-release | 02:08 | |
| *** yamahata has quit IRC | 02:10 | |
| *** gcb has quit IRC | 02:12 | |
| *** gcb has joined #openstack-release | 02:15 | |
| *** yamamoto_ has joined #openstack-release | 02:22 | |
| *** yamamoto_ has quit IRC | 02:28 | |
| *** ekcs has quit IRC | 02:30 | |
| *** tellesnobrega has joined #openstack-release | 02:31 | |
| *** rosmaita has quit IRC | 02:39 | |
| *** hongbin has quit IRC | 02:48 | |
| *** thorst has joined #openstack-release | 02:52 | |
| *** hongbin has joined #openstack-release | 02:54 | |
| *** yamamoto has joined #openstack-release | 02:56 | |
| *** thorst has quit IRC | 02:57 | |
| *** yamamoto has quit IRC | 03:02 | |
| *** iyamahat has joined #openstack-release | 03:17 | |
| *** dave-mccowan has quit IRC | 03:40 | |
| *** udesale has joined #openstack-release | 03:42 | |
| ricolin_ | Hi is there any release core awake?:) | 03:45 |
|---|---|---|
| *** thorst has joined #openstack-release | 03:53 | |
| *** hongbin has quit IRC | 03:56 | |
| *** hongbin has joined #openstack-release | 03:58 | |
| *** thorst has quit IRC | 03:59 | |
| *** yamamoto has joined #openstack-release | 04:03 | |
| *** hongbin has quit IRC | 04:04 | |
| *** yamamoto has quit IRC | 04:09 | |
| *** thorst has joined #openstack-release | 04:54 | |
| *** thorst has quit IRC | 04:58 | |
| *** ekcs has joined #openstack-release | 05:04 | |
| *** yamamoto has joined #openstack-release | 05:05 | |
| *** yamamoto has quit IRC | 05:10 | |
| *** ekcs has quit IRC | 05:11 | |
| *** gcb has quit IRC | 05:21 | |
| *** gcb has joined #openstack-release | 05:23 | |
| *** claudiub has joined #openstack-release | 05:27 | |
| *** ricolin_ has quit IRC | 05:31 | |
| *** ricolin has joined #openstack-release | 05:31 | |
| *** edmondsw has joined #openstack-release | 05:42 | |
| *** edmondsw has quit IRC | 05:47 | |
| *** thorst has joined #openstack-release | 05:55 | |
| *** thorst has quit IRC | 05:59 | |
| *** yamamoto has joined #openstack-release | 06:06 | |
| *** yamamoto has quit IRC | 06:12 | |
| *** shardy has joined #openstack-release | 06:27 | |
| *** yamahata has joined #openstack-release | 06:32 | |
| *** armax has quit IRC | 06:39 | |
| *** pcaruana has joined #openstack-release | 06:45 | |
| *** coolsvap has joined #openstack-release | 06:55 | |
| *** thorst has joined #openstack-release | 06:56 | |
| *** thorst has quit IRC | 07:00 | |
| *** csomerville has joined #openstack-release | 07:05 | |
| *** cody-somerville has quit IRC | 07:05 | |
| openstackgerrit | Spyros Trigazis (strigazi) proposed openstack/releases master: Release magnum-ui 3.0.0 https://review.openstack.org/495918 | 07:08 |
| *** tmorin has joined #openstack-release | 07:10 | |
| *** iyamahat has quit IRC | 07:18 | |
| *** jtomasek has joined #openstack-release | 07:18 | |
| *** amoralej|off is now known as amoralej | 07:29 | |
| *** edmondsw has joined #openstack-release | 07:30 | |
| *** yamahata has quit IRC | 07:31 | |
| *** jpich has joined #openstack-release | 07:34 | |
| *** edmondsw has quit IRC | 07:35 | |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to slow performance | 07:37 | |
| -openstackstatus- NOTICE: Gerrit has been restarted successfully | 07:41 | |
| *** iyamahat has joined #openstack-release | 07:54 | |
| *** thorst has joined #openstack-release | 07:57 | |
| *** thorst has quit IRC | 08:01 | |
| *** pbandark has joined #openstack-release | 08:07 | |
| *** alexchadin has joined #openstack-release | 08:09 | |
| *** liujiong_lj has joined #openstack-release | 08:13 | |
| *** liujiong has quit IRC | 08:14 | |
| *** iyamahat has quit IRC | 08:25 | |
| *** shardy has quit IRC | 08:25 | |
| *** liujiong has joined #openstack-release | 08:25 | |
| *** liujiong_lj has quit IRC | 08:26 | |
| *** liujiong_lj has joined #openstack-release | 08:28 | |
| *** liujiong has quit IRC | 08:30 | |
| *** yamamoto has joined #openstack-release | 08:35 | |
| *** dtantsur|afk is now known as dtantsur | 08:36 | |
| *** thorst has joined #openstack-release | 08:57 | |
| *** yamamoto has quit IRC | 08:59 | |
| *** yamamoto has joined #openstack-release | 08:59 | |
| *** yamamoto has quit IRC | 08:59 | |
| *** thorst has quit IRC | 09:02 | |
| *** electrofelix has joined #openstack-release | 09:03 | |
| *** shardy has joined #openstack-release | 09:18 | |
| *** liujiong_lj has quit IRC | 09:27 | |
| *** liujiong has joined #openstack-release | 09:31 | |
| *** csomerville has quit IRC | 09:50 | |
| *** thorst has joined #openstack-release | 09:58 | |
| *** yamamoto has joined #openstack-release | 10:00 | |
| *** thorst has quit IRC | 10:03 | |
| *** yamamoto has quit IRC | 10:05 | |
| *** alexchadin has quit IRC | 10:05 | |
| *** alexchadin has joined #openstack-release | 10:07 | |
| *** alexchadin has quit IRC | 10:10 | |
| *** alexchadin has joined #openstack-release | 10:11 | |
| *** alexchadin has quit IRC | 10:13 | |
| *** liujiong has quit IRC | 10:13 | |
| *** zhurong has quit IRC | 10:17 | |
| dims | ricolin : pong | 10:29 |
| *** yamamoto has joined #openstack-release | 10:30 | |
| ricolin | dims, Hi thanks for pong back, tonyb already provide me some very useful answers:) | 10:32 |
| dims | cool thx | 10:32 |
| *** udesale has quit IRC | 10:36 | |
| *** dtantsur is now known as dtantsur|bbl | 10:40 | |
| *** shardy has quit IRC | 10:42 | |
| *** wolverineav has joined #openstack-release | 10:57 | |
| *** freerunner has quit IRC | 11:00 | |
| *** junbo has quit IRC | 11:01 | |
| *** Dinesh_Bhor has quit IRC | 11:01 | |
| *** stephenfin has quit IRC | 11:01 | |
| *** toabctl has quit IRC | 11:01 | |
| *** toabctl has joined #openstack-release | 11:04 | |
| *** vryzhenkin has joined #openstack-release | 11:04 | |
| *** Dinesh_Bhor has joined #openstack-release | 11:05 | |
| *** junbo has joined #openstack-release | 11:05 | |
| *** stephenfin has joined #openstack-release | 11:05 | |
| *** edmondsw has joined #openstack-release | 11:06 | |
| *** thorst has joined #openstack-release | 11:07 | |
| *** edmondsw has quit IRC | 11:10 | |
| *** thorst has quit IRC | 11:11 | |
| *** mburned has joined #openstack-release | 11:11 | |
| *** alexchadin has joined #openstack-release | 11:13 | |
| *** yamamoto has quit IRC | 11:18 | |
| *** yamamoto has joined #openstack-release | 11:30 | |
| openstackgerrit | Spyros Trigazis (strigazi) proposed openstack/releases master: Release magnum-ui 3.0.0 https://review.openstack.org/495918 | 11:32 |
| openstackgerrit | Merged openstack/releases master: New python-neutronclient stable releases https://review.openstack.org/495986 | 11:35 |
| openstackgerrit | Merged openstack/releases master: ocata: release nova 15.0.7 https://review.openstack.org/495903 | 11:35 |
| *** edmondsw has joined #openstack-release | 11:45 | |
| *** yamamoto has quit IRC | 11:46 | |
| *** yamamoto has joined #openstack-release | 11:48 | |
| *** rosmaita has joined #openstack-release | 11:49 | |
| *** rosmaita has quit IRC | 11:50 | |
| *** rosmaita has joined #openstack-release | 11:51 | |
| *** tellesnobrega has quit IRC | 12:01 | |
| *** yamamoto has quit IRC | 12:03 | |
| *** rosmaita has quit IRC | 12:03 | |
| *** rosmaita has joined #openstack-release | 12:04 | |
| *** tmorin has left #openstack-release | 12:04 | |
| *** yamamoto has joined #openstack-release | 12:04 | |
| *** yamamoto has quit IRC | 12:10 | |
| *** thorst has joined #openstack-release | 12:11 | |
| *** yamamoto has joined #openstack-release | 12:12 | |
| *** amoralej is now known as amoralej|lunch | 12:14 | |
| dhellmann | dims, smcginnis : o/ | 12:20 |
| openstackgerrit | Merged openstack/releases master: Release magnum-ui 3.0.0 https://review.openstack.org/495918 | 12:21 |
| dims | dhellmann : smcginnis : o/ | 12:22 |
| *** dave-mccowan has joined #openstack-release | 12:34 | |
| *** udesale has joined #openstack-release | 12:37 | |
| *** thorst is now known as thorst_afk | 12:49 | |
| smcginnis | dims, dhellmann: o/ | 12:49 |
| *** tellesnobrega has joined #openstack-release | 12:49 | |
| *** alexchadin has quit IRC | 12:51 | |
| *** alexchadin has joined #openstack-release | 12:51 | |
| *** pkovar has joined #openstack-release | 12:53 | |
| *** zhurong has joined #openstack-release | 12:56 | |
| *** yamamoto has quit IRC | 13:12 | |
| *** yamamoto has joined #openstack-release | 13:14 | |
| *** yamamoto has quit IRC | 13:22 | |
| *** bobh has joined #openstack-release | 13:35 | |
| *** amoralej|lunch is now known as amoralej | 13:36 | |
| *** tellesnobrega has quit IRC | 13:40 | |
| *** shardy has joined #openstack-release | 13:49 | |
| *** pkovar has quit IRC | 13:53 | |
| *** tellesnobrega has joined #openstack-release | 13:53 | |
| *** alexchadin has quit IRC | 13:55 | |
| *** alexchadin has joined #openstack-release | 13:56 | |
| *** yamahata has joined #openstack-release | 13:56 | |
| *** hongbin has joined #openstack-release | 13:59 | |
| *** witek_ has joined #openstack-release | 13:59 | |
| *** witek has quit IRC | 13:59 | |
| *** iyamahat has joined #openstack-release | 14:07 | |
| *** armax has joined #openstack-release | 14:14 | |
| *** zhurong has quit IRC | 14:19 | |
| *** yamamoto has joined #openstack-release | 14:23 | |
| *** rosmaita has quit IRC | 14:24 | |
| otherwiseguy | dims, any word on library release timeframe for ovsdbapp? https://review.openstack.org/#/c/493235/ | 14:25 |
| smcginnis | otherwiseguy: We are still trying to wrap up some final pike things. Once queens opens up we can get that through. | 14:26 |
| otherwiseguy | smcginnis, I have someone asking me every day for builds, so I just pass those on to you all. :D Got an updated estimate I can give them? | 14:27 |
| *** yamamoto has quit IRC | 14:28 | |
| smcginnis | otherwiseguy: I'm hoping within the next couple days. | 14:30 |
| smcginnis | dhellmann: We need to get those final trailing releases through first, right? ^T^ | 14:30 |
| dhellmann | we don't usually start doing releases for N+1 until after the final date for N | 14:31 |
| dhellmann | http://git.openstack.org/cgit/openstack/releases/tree/PROCESS.rst#n343 | 14:32 |
| *** rosmaita has joined #openstack-release | 14:32 | |
| otherwiseguy | dhellmann, the problem is libraries that are used by non-OS projects as well. | 14:32 |
| otherwiseguy | The RHEV/oVirt folks are chomping at the bit. | 14:32 |
| otherwiseguy | One reason I was pushing for an "independent" release cycle, but that got shot down. | 14:33 |
| otherwiseguy | which I understand, it just has some trade offs. | 14:33 |
| *** alexchadin has quit IRC | 14:34 | |
| dhellmann | smcginnis : how many stragglers are we waiting for? | 14:34 |
| smcginnis | dhellmann: There's still a few in the list from the R-1 missing: https://etherpad.openstack.org/p/pike-relmgt-tracking | 14:35 |
| dhellmann | http://paste.openstack.org/show/619048/ | 14:35 |
| smcginnis | dhellmann: I hadn't used that one yet - nice! | 14:36 |
| smcginnis | dhellmann: I think at least the majority of them have been contacted. | 14:37 |
| dhellmann | tempest won't branch so we don't need to count that one | 14:37 |
| dhellmann | and none of those things use ovsdbapp | 14:38 |
| otherwiseguy | dhellmann, I was just going to mention that. :D | 14:38 |
| smcginnis | :) | 14:38 |
| dhellmann | http://paste.openstack.org/show/619050/ | 14:38 |
| dhellmann | so the remaining question is whether the release validation and announce scripts work properly for a "future" series | 14:39 |
| smcginnis | dhellmann: Shall we find out? | 14:39 |
| dhellmann | I think the announcement is likely to show up as coming for "pike" when it's not | 14:39 |
| smcginnis | dhellmann: Where is that data pulled from? | 14:39 |
| dhellmann | looking | 14:40 |
| dhellmann | oh, maybe it's using the filename | 14:41 |
| *** udesale has quit IRC | 14:44 | |
| * dims peeks | 14:45 | |
| smcginnis | dhellmann: That would be good. | 14:45 |
| dhellmann | looking at check_branch_sha in gitutils.py in the releases repo, I think we fixed the issue we had the last time something like this came up | 14:46 |
| dhellmann | I also think the "master" argument to that function isn't being used | 14:46 |
| dhellmann | smcginnis , dims : I think it's probably safe to go ahead | 14:47 |
| dhellmann | smcginnis , dims : we need the PTL or liaison to +1 | 14:48 |
| dims | dhellmann : the ovsdbapp? | 14:48 |
| dhellmann | yes | 14:48 |
| dims | otherwiseguy : can you get the PTL or liaison to +1 it please? looks like we can ship quickly | 14:49 |
| otherwiseguy | dims, dhellmann smcginnis cool! thanks! pinging kevinbenton. :) | 14:53 |
| otherwiseguy | I think armax is release liaison? | 14:54 |
| dhellmann | yes | 14:54 |
| * armax looks | 14:55 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: remove unused master argument from check_branch_sha() https://review.openstack.org/496276 | 14:55 |
| otherwiseguy | armax, basically just need to release the first "queens" ovsdbapp for an non-Openstack user since it has OVN stuff in it they need. | 14:56 |
| dhellmann | otherwiseguy : after this release, it would be good to start thinking about settling the API and releasing a 1.0.0 version | 14:56 |
| otherwiseguy | armax, https://review.openstack.org/#/c/493235/ | 14:56 |
| otherwiseguy | dhellmann, absolutely. | 14:56 |
| armax | otherwiseguy: ok | 14:57 |
| otherwiseguy | I *think* 0.5.0 will essentially become 1.0. Just hedging a bit since I now have a whole release cycle and want to convert networking-ovn over to using it. I don't think I'll find anything else that needs to change, though. | 14:57 |
| dhellmann | sounds good | 14:57 |
| dhellmann | some folks get antsy about putting 0.y.z versions into production | 14:58 |
| otherwiseguy | new projects are a pain. | 14:58 |
| armax | otherwiseguy: on a different note, do we test ovsdbapp against neutron? | 14:58 |
| otherwiseguy | dhellmann, yeah, i intended to have 0.5.0 be 1.0...I just didn't seen that the library deadline was earlier than the rest of them. And some of the patches sat for review for like 3 weeks. | 14:58 |
| armax | I can’t see anything of that sort either in the periodic or check queue | 14:59 |
| otherwiseguy | armax, still working on making a gating test against master ovsdbapp. | 14:59 |
| armax | OK | 14:59 |
| openstackgerrit | Eric Harney proposed openstack/releases master: Cinder stable/ocata 10.0.5 release https://review.openstack.org/496282 | 14:59 |
| otherwiseguy | There is a periodic check. | 14:59 |
| smcginnis | armax: While you're here - know anything about the status of neutron-fwaas-dashboard? | 14:59 |
| armax | otherwiseguy: I don’t see it here: http://grafana.openstack.org/dashboard/db/neutron-failure-rate?panelId=4&fullscreen :) | 14:59 |
| armax | smcginnis: not on the top of my head, but I can try and help you nonetheless | 15:00 |
| smcginnis | armax: It's one of the projects on the list that has not done a pike release and requested a pike branch. | 15:00 |
| smcginnis | armax: So just looking for when that might be coming. | 15:00 |
| armax | otherwiseguy: http://logs.openstack.org/periodic/periodic-neutron-py35-with-ovsdbapp-master/dd4a10b/console.html.gz | 15:00 |
| armax | I see it ^ | 15:00 |
| otherwiseguy | armax, search for ovsdbapp :) There's something weird with the py35 check. | 15:00 |
| armax | but it looks busted | 15:00 |
| armax | otherwise: how do we know if the release you’re asking doesn’t break the world :P? | 15:01 |
| otherwiseguy | perhaps tomorrow will be a improve testing day. | 15:01 |
| armax | smcginnis: I thought amotoki had taken care of that | 15:01 |
| armax | smcginnis: let me check | 15:01 |
| amotoki | armax: smcginnis: I am taking care of it. | 15:02 |
| otherwiseguy | armax, well, the release won't break anything. using it might, but I think the requirements change runs neutron tests. | 15:02 |
| armax | otherwiseguy: right, once we bump the requirements, I mean | 15:02 |
| smcginnis | amotoki: Oh right, sorry. I think there was a note about that. | 15:02 |
| otherwiseguy | armax, but yeah, absolutely testing improvements are a top priority right now. | 15:03 |
| otherwiseguy | I'm tired of running tons of tests manually to verify. | 15:03 |
| *** cody-somerville has joined #openstack-release | 15:06 | |
| *** cody-somerville has quit IRC | 15:06 | |
| *** cody-somerville has joined #openstack-release | 15:06 | |
| otherwiseguy | armax, the failure in the periodic is "tox.ConfigError: ConfigError: substitution key '[py35]setenv' not found" | 15:06 |
| armax | right | 15:06 |
| otherwiseguy | why i don't quite understand yet | 15:07 |
| *** tellesnobrega has quit IRC | 15:09 | |
| *** dtantsur|bbl is now known as dtantsur | 15:13 | |
| openstackgerrit | Matt Riedemann proposed openstack/releases master: newton: release nova 14.0.8 https://review.openstack.org/496285 | 15:20 |
| openstackgerrit | Merged openstack/releases master: Release ovsdbapp 0.5.0 https://review.openstack.org/493235 | 15:23 |
| *** tmorin has joined #openstack-release | 15:23 | |
| *** tmorin has left #openstack-release | 15:24 | |
| *** yamamoto has joined #openstack-release | 15:25 | |
| *** yamamoto has quit IRC | 15:30 | |
| otherwiseguy | dims, smcginnis dhellmann armax thanks for the rush on ovsdbapp. the oVirt people are happy. :) | 15:33 |
| armax | np | 15:33 |
| smcginnis | otherwiseguy: No problem, glad we could get it going. | 15:34 |
| otherwiseguy | One group that is bugging me to get something done for them down, several more to go. :) | 15:35 |
| smcginnis | It never ends. ;) | 15:37 |
| *** csomerville has joined #openstack-release | 15:39 | |
| *** cody-somerville has quit IRC | 15:43 | |
| openstackgerrit | Rob Cresswell proposed openstack/releases master: Tag Horizon Pike RC2 https://review.openstack.org/496297 | 15:55 |
| *** thorst_afk has quit IRC | 15:55 | |
| *** thorst_afk has joined #openstack-release | 15:57 | |
| *** thorst_afk has quit IRC | 16:01 | |
| *** thorst_afk has joined #openstack-release | 16:05 | |
| *** shardy has quit IRC | 16:06 | |
| *** rajathagasthya has joined #openstack-release | 16:14 | |
| *** iyamahat has quit IRC | 16:15 | |
| *** yamahata has quit IRC | 16:16 | |
| openstackgerrit | Akihiro Motoki proposed openstack/releases master: Release neutron-fwaas-dashboard 1.0.0 for Pike https://review.openstack.org/496307 | 16:18 |
| lbragstad | hey all - quick question on release notes | 16:21 |
| lbragstad | is this the best channel for that or should i ask somewhere else? | 16:21 |
| smcginnis | lbragstad: This is probably good. What's up? | 16:23 |
| lbragstad | so - reno renders release notes, right | 16:23 |
| lbragstad | it looks like it renders release notes based on when they were touched in the $project/releasenotes/ directory | 16:24 |
| smcginnis | lbragstad: I've seen some oddness there, but never had the time to track that down. | 16:24 |
| smcginnis | dhellmann: Are you aware of any open issues? ^^ | 16:25 |
| lbragstad | a lot of the projects also migrated their docs this release, and one of the action items in that was to update all references to links | 16:25 |
| *** vryzhenkin is now known as freerunner | 16:25 | |
| lbragstad | so - we had a contributor go through and do that for keystone - which included updating the contents of release notes for ocata or newton | 16:25 |
| * smcginnis checks our updates to Cinder... | 16:25 | |
| lbragstad | for example - https://github.com/openstack/keystone/blob/master/releasenotes/notes/bp-shadow-mapping-06fc7c71a401d707.yaml | 16:25 |
| amotoki | lbragstad: if you need to touch release notes for past releases, you need to edit release notes in stable branches. | 16:25 |
| amotoki | lbragstad: if you touch them in the master, it will be considered as part of the current release. | 16:26 |
| lbragstad | i assume that's why things that went into ocata and newton are rendering in the pike release notes? | 16:26 |
| *** yamamoto has joined #openstack-release | 16:26 | |
| smcginnis | Yeah, I think reno looks at where the commit happened relative to the tags. | 16:27 |
| amotoki | lbragstad: you already landed a patch to touch older release notes and reverting it does not change the situaiton. | 16:27 |
| amotoki | lbragstad: you need to use ignore-notes like https://github.com/openstack/horizon/blob/master/releasenotes/source/pike.rst | 16:28 |
| amotoki | lbragstad: horizon did the similar thing and I did a workaround like ^ | 16:28 |
| lbragstad | amotoki: ah - ok, that makes sense | 16:28 |
| amotoki | lbragstad: this is the corresponding commit https://github.com/openstack/horizon/commit/85fe8f3b5fdf526302831107aee0c372ac5a9fec and | 16:29 |
| lbragstad | it sounds like there are two ways to fix it then? one is to write a patch for stable branches that update the doc urls and the other is to black list those release notes from pike | 16:29 |
| amotoki | see L.96-98 of http://paste.openstack.org/show/617310/ (which describes what I did) | 16:29 |
| amotoki | lbragstad: i think you need both. | 16:29 |
| lbragstad | amotoki: aha - ok, that makes sense | 16:30 |
| lbragstad | I suppose this will require a new release candidate | 16:30 |
| lbragstad | for pike | 16:31 |
| lbragstad | so that the release notes will render properly | 16:32 |
| amotoki | perhaps it does not need another RC as the release notes for pike is always generated from the latest commit of stable/pike | 16:32 |
| *** yamamoto has quit IRC | 16:32 | |
| smcginnis | amotoki: Oh nice, I wasn't aware of the ignore-notes. | 16:32 |
| amotoki | smcginnis: I chatted with dhellmann on this topic and it turns out it is the only solution for this. | 16:33 |
| lbragstad | amotoki: ack - that's why I was curious :) | 16:33 |
| dims | robcresswell : would you need a RC3 or all done for pike? :) | 16:33 |
| lbragstad | smcginnis: just to confirm - this change shouldn't require a new release candidate, should it? | 16:33 |
| smcginnis | lbragstad: I don't believe so as the publish job is triggered on tagging, IIRC. | 16:34 |
| smcginnis | dims, dhellmann: I'd feel better if one of you confirmed that though. ^^ :) | 16:34 |
| robcresswell | dims: That should be it | 16:34 |
| lbragstad | and like amotoki said - it renders release notes based on stable branches | 16:34 |
| dims | w00t robcresswell congrats! | 16:34 |
| *** iyamahat has joined #openstack-release | 16:35 | |
| robcresswell | \o/ | 16:35 |
| dhellmann | lbragstad, smcginnis : you will need a new release candidate, because the final release is tagged from an existing candidate. | 16:35 |
| *** yamahata has joined #openstack-release | 16:35 | |
| dhellmann | otherwise things can end up in the final release that were never in a candidate | 16:35 |
| smcginnis | dhellmann: Oh right. Thanks for clearing that up. | 16:36 |
| lbragstad | ok - that makes sense | 16:36 |
| lbragstad | looks like that will be our priority today | 16:36 |
| lbragstad | keystone is going to need an rc3 | 16:36 |
| *** pcaruana has quit IRC | 16:37 | |
| openstackgerrit | Merged openstack/releases master: Tag Horizon Pike RC2 https://review.openstack.org/496297 | 16:38 |
| smcginnis | dhellmann: So updates to past release notes should be done directly in those stable branches, and there's just going to be a mismatch of content in master, correct? | 16:42 |
| dhellmann | smcginnis : yes, that's right | 16:43 |
| dhellmann | the note files are only ever read out of the branch so it doesn't matter if the master file doesn't match | 16:43 |
| smcginnis | dhellmann: Alternatively, update in master for consistency and use the ignore parameter in the new release note page. | 16:43 |
| smcginnis | dhellmann: OK, got it. Thank you. | 16:43 |
| dhellmann | yeah, updating master is just extra work | 16:44 |
| dhellmann | if the file is updated on the branch, reno should correctly figure out that the note applies to the older release and not the latest | 16:44 |
| smcginnis | Just thinking if someone came across stale info looking through the repo, but I suppose that's a pretty poor way to read release notes. :) | 16:44 |
| lbragstad | i have fixes up for keystone here - https://review.openstack.org/#/q/topic:bug/1710572 | 16:56 |
| lbragstad | for the ignore notes patch - it looks like that applies to unreleased notes | 16:57 |
| lbragstad | do we write a patch directly against stable/pike for that? | 16:57 |
| dhellmann | lbragstad : I'm not sure I understand the question | 16:58 |
| lbragstad | dhellmann: looking at this change - https://review.openstack.org/#/c/490250/3 | 16:58 |
| lbragstad | it looks like we can opt specific notes out of being rendered for a release | 16:58 |
| lbragstad | https://review.openstack.org/#/c/490250/3/releasenotes/source/unreleased.rst | 16:58 |
| dhellmann | yes, that's right. that option was added to address this situation :-) | 16:59 |
| lbragstad | do we write that patch directly against the stable branch or do we write it to master and somehow denote that "pike is suppose to ignore these" and backport it? | 16:59 |
| lbragstad | ^ not sure if that makes any more sense :) | 16:59 |
| dhellmann | you would put the patch on the branch where the notes are showing but where you don't want them to show | 16:59 |
| lbragstad | aha - ok | 16:59 |
| *** pbandark has quit IRC | 16:59 | |
| dhellmann | branch(es) | 16:59 |
| lbragstad | so in my case, i need to write it against stable/pike for sure | 17:00 |
| dhellmann | yeah, it sounds like it | 17:00 |
| dhellmann | maybe master, too? depending on if the note was backported after the branch was created | 17:00 |
| dhellmann | lbragstad : I have to run an errand but if you want me to help with reviews ping me here with URLs and I'll do that when I return | 17:01 |
| lbragstad | dhellmann: in that case - wouldn't we want the note to render? | 17:01 |
| *** tellesnobrega has joined #openstack-release | 17:02 | |
| lbragstad | dhellmann: will do, thank you for the help | 17:02 |
| dhellmann | if the note was originally part of newton or something and it got changed in master and backported to pike, you wouldn't want it to show in either pike or master so you'd need to block it in both places | 17:02 |
| lbragstad | got it - ok, that makes sense | 17:04 |
| smcginnis | dhellmann: Thoughts? https://review.openstack.org/496318 | 17:07 |
| *** electrofelix has quit IRC | 17:12 | |
| *** thorst_afk has quit IRC | 17:13 | |
| *** eranrom_ has joined #openstack-release | 17:22 | |
| *** eranrom has quit IRC | 17:25 | |
| *** wolverineav has quit IRC | 17:26 | |
| *** wolverineav has joined #openstack-release | 17:26 | |
| *** jpich has quit IRC | 17:28 | |
| *** yamamoto has joined #openstack-release | 17:28 | |
| *** yamamoto has quit IRC | 17:35 | |
| *** rajathagasthya has quit IRC | 17:45 | |
| *** coolsvap has quit IRC | 17:45 | |
| *** ekcs has joined #openstack-release | 17:47 | |
| dhellmann | smcginnis : looking | 17:49 |
| *** thorst_afk has joined #openstack-release | 17:49 | |
| dhellmann | smcginnis : good content. I wonder if it should go into the reno docs instead? then the ptg could link to that? | 17:50 |
| dhellmann | seems better than the other way around | 17:50 |
| openstackgerrit | Armando Migliaccio proposed openstack/releases master: Neutron projects RC2 https://review.openstack.org/494058 | 17:55 |
| lbragstad | dhellmann: https://review.openstack.org/#/c/496343/ | 17:59 |
| dhellmann | lbragstad : lgtm. We'll see if that trailing comma confuses reno. :-) | 18:00 |
| lbragstad | dhellmann: good catch - updated | 18:01 |
| dhellmann | oh, I really wanted to see if it would confuse the parser :-) | 18:02 |
| dhellmann | I can test that out myself, though, so no worries | 18:02 |
| *** rajathagasthya has joined #openstack-release | 18:03 | |
| *** dtantsur is now known as dtantsur|afk | 18:05 | |
| *** thorst_a_ has joined #openstack-release | 18:18 | |
| *** thorst_afk has quit IRC | 18:20 | |
| *** rmcall has joined #openstack-release | 18:21 | |
| *** amoralej is now known as amoralej|off | 18:27 | |
| *** bobh has quit IRC | 18:31 | |
| *** yamamoto has joined #openstack-release | 18:31 | |
| *** yamamoto has quit IRC | 18:37 | |
| *** amotoki has quit IRC | 18:53 | |
| smcginnis | dhellmann: I was thinking reno docs first, but then thought there was some openstack specific info there that might be better in the ptg. | 19:11 |
| smcginnis | dhellmann: I can switch that over though and link to from the ptg to the reno docs and include that os specific info. | 19:12 |
| dhellmann | what part of that seems openstack-specific? | 19:12 |
| dhellmann | I guess the example has a series name | 19:12 |
| smcginnis | dhellmann: The notes about ignore and such. | 19:12 |
| lbragstad | dhellmann: kmalloc and i have a few other questions regarding reno | 19:12 |
| dhellmann | but the idea of stable branches is pretty well baked into reno | 19:13 |
| dhellmann | lbragstad : ok | 19:13 |
| kmalloc | it looks like changes on master are being reflected in stable/pike releases | 19:13 |
| dhellmann | really? have an example I can look at? | 19:13 |
| lbragstad | dhellmann: even though we have https://review.openstack.org/#/c/492774/1 | 19:13 |
| kmalloc | files were changed in master. do we need an ignore in stable/pike then? or in master? | 19:13 |
| lbragstad | dhellmann: https://docs.openstack.org/releasenotes/keystone/pike.html | 19:13 |
| lbragstad | dhellmann: https://docs.openstack.org/releasenotes/keystone/pike.html#id7 for example | 19:14 |
| lbragstad | we implemented the allow-expired blueprint in ocata | 19:14 |
| lbragstad | but that release note was also touched in https://review.openstack.org/#/c/493479/ (which merged 8 days ago( | 19:15 |
| kmalloc | on master only, the pike change was -2'd today by me before landing | 19:15 |
| lbragstad | does the proposal bot propose the change to master to update the release notes, but require someone to backport it to stable/pike? | 19:15 |
| dhellmann | is the allow_expired note in 493479? | 19:15 |
| lbragstad | its not.... weird! let me find another exmple | 19:16 |
| dhellmann | let me run the report command and see where it's pulling that info | 19:17 |
| lbragstad | ah - here | 19:17 |
| lbragstad | https://docs.openstack.org/releasenotes/keystone/pike.html#id7 | 19:18 |
| lbragstad | the shadow-mapping blueprint was implemented in ocata | 19:18 |
| lbragstad | it was also updated here - https://review.openstack.org/#/c/493479/ | 19:18 |
| lbragstad | (but i'm still missing how allow-expired is included in the pike release notes when we didn't touch it) | 19:18 |
| dhellmann | the shadow-mapping one makes sense, the other one doesn't | 19:18 |
| kmalloc | ^^ that is concerning me. | 19:18 |
| kmalloc | anyway, before we wedged ignores in, i wanted to make sure we're not also broken in some other way | 19:19 |
| lbragstad | it's like musical release notes :) | 19:19 |
| lbragstad | let me pull this history on allow-expired and verify if it was updated in pike | 19:20 |
| lbragstad | aha - it was | 19:20 |
| lbragstad | https://github.com/openstack/keystone/commit/2fadadbb8c29be031396b5f8a47c72c7d2c3c1eb | 19:20 |
| lbragstad | which explains why bp-support-federated-attr is also being rendered, among others that weren't included in https://review.openstack.org/#/c/493479/ | 19:21 |
| dhellmann | this command will show you what file each note comes from and why it is picking up those files: .tox/releasenotes/bin/reno report --branch stable/pike | 19:21 |
| dhellmann | and the content it's going to use | 19:21 |
| dhellmann | in this case it reports: | 19:22 |
| dhellmann | f5d845b9601bc1ef: update to releasenotes/notes/bp-allow-expired-f5d845b9601bc1ef.yaml in commit 2fadadbb8c29be031396b5f8a47c72c7d2c3c1eb | 19:22 |
| lbragstad | yep - that's https://github.com/openstack/keystone/commit/2fadadbb8c29be031396b5f8a47c72c7d2c3c1eb | 19:22 |
| lbragstad | which was a wording clean up | 19:22 |
| dhellmann | I suppose I could try to make reno smart enough to notice that the note also appears on an older stable branch, and ignore it | 19:22 |
| dhellmann | but that involves scanning the entire history of the project, which I was trying to avoid | 19:23 |
| dhellmann | because projects with a lot of history would take forever | 19:23 |
| lbragstad | yeah - or we could just have a better policy around not tinkering with wording in release notes | 19:23 |
| smcginnis | I wonder if we should have a convention of notes/[branch] to help make some of this more obvious. | 19:23 |
| smcginnis | Especially with updating older notes. | 19:23 |
| dhellmann | smcginnis : the problem with that is when you backport a fix, the filename is wrong | 19:23 |
| lbragstad | smcginnis: yeah - that was an action item that came out of today's keystone meeting | 19:24 |
| dhellmann | backporting is why we didn't just use filenames to begin with | 19:24 |
| smcginnis | dhellmann: Well, that's actually interesting. If a fix is backported with a release note, then it's going to show up in both the new and old release notes, right? | 19:24 |
| dhellmann | because if I fix something early in queens and backport to pike, then I have to change the patch as I backport it instead of being able to do a clean backport | 19:24 |
| dhellmann | yes, it will, that's by design | 19:24 |
| smcginnis | So we want queens to have a note that something is fixed if it was already fixed in pike? | 19:25 |
| smcginnis | I suppose that makes sense for more stable releases, but we're in a weird phase for that right now. | 19:25 |
| dhellmann | yes, because it shows exactly what version the fix is in in both branches | 19:25 |
| lbragstad | well - it must be fixed in queens before it can be backported to pike, right? | 19:25 |
| dhellmann | so if I'm on an early pike release without the fix, I may just wait and upgrade to queens | 19:25 |
| smcginnis | lbragstad: Right. | 19:25 |
| dhellmann | but not all fixes are backported | 19:26 |
| dhellmann | so we want the notes to reflect the real content of each release | 19:26 |
| lbragstad | the problem we're running into is that we had some patches to trivial updates to various release notes but they are getting picked up in the pike notes | 19:27 |
| lbragstad | the backport of a fix with a release note makes sense to me | 19:27 |
| dhellmann | I wonder if I make reno not show files that are only updated, but never have the "add" event, if this would go away | 19:27 |
| kmalloc | dhellmann: that probably would do it | 19:27 |
| lbragstad | so - for that | 19:27 |
| dhellmann | I'll have to think about whether I can do that. I'm not sure if I have enough info in the scanner. | 19:27 |
| lbragstad | let's say we backport a patch that updates a broken link | 19:28 |
| lbragstad | the stable release notes will never render properly and you'll still need an .htaccess fix? | 19:28 |
| kmalloc | lbragstad: it would be fine, because the reno would have an "add" event. if he has enough info | 19:28 |
| kmalloc | it would render | 19:28 |
| lbragstad | well - they will render, but it will ignore the backport because it was explicitly an add | 19:28 |
| lbragstad | wasn't* | 19:28 |
| kmalloc | the case where it's an update with no add, it wont be included | 19:28 |
| lbragstad | got it | 19:29 |
| dhellmann | lbragstad : not quite. The scanner stops when it reaches the point on master where the previous branch was created. So when building the pike history, it looks all the way back to the point where stable/ocata was branched. | 19:29 |
| dhellmann | so if the note was indeed added on master before stable/pike but after stable/ocata and then it's updated in master *after* the branch is created and then that change is backported, it should work fine. | 19:30 |
| dhellmann | without any changes to reno, actually | 19:30 |
| dhellmann | the issue is that we've been updating notes that are much older than that | 19:30 |
| dhellmann | and that "crosses the streams" | 19:30 |
| lbragstad | yes | 19:30 |
| lbragstad | so updating notes from stable/newton for example | 19:30 |
| dhellmann | because reno sees that edit and thinks "aha! a new note file!" | 19:30 |
| dhellmann | but it should wait until it sees the file added, not modified | 19:31 |
| dhellmann | it can tell the difference based on the commit content, I think | 19:31 |
| dhellmann | it has been a few months since I've looked at the code, so it may take me a while to figure out the fix | 19:31 |
| dhellmann | but I'll make a note of it and look after the ptg for sure | 19:31 |
| lbragstad | no worries - i think we can get by with the ignore-notes approach, right? | 19:32 |
| dhellmann | yes | 19:32 |
| lbragstad | and an .htaccess file | 19:32 |
| *** yamamoto has joined #openstack-release | 19:33 | |
| kmalloc | lbragstad: reverted -2 on the pike change for ignore | 19:34 |
| lbragstad | kmalloc: awesome - thanks | 19:35 |
| kmalloc | so,now +2/+A | 19:35 |
| lbragstad | i just saw that | 19:35 |
| kmalloc | but ugh. | 19:35 |
| lbragstad | yes - documentation | 19:35 |
| lbragstad | i feel like most of this needs to live in a common place, so i should probably review the patch smcginnis has up | 19:35 |
| lbragstad | because the things we're hitting here should be applicable to all projects using reno, and not just keystone | 19:36 |
| smcginnis | lbragstad: Update coming to put some of that in the reno docs, some of it in the project team guide. | 19:36 |
| lbragstad | right? | 19:36 |
| lbragstad | smcginnis: ack | 19:36 |
| lbragstad | smcginnis: i'll watch for a new version | 19:36 |
| dhellmann | lbragstad : yes, I agree, common docs | 19:36 |
| smcginnis | lbragstad: Yeah, I think we need docs somewhere because I think this will likely affect every project at some point. | 19:36 |
| lbragstad | ok - i'm going to grab some coffee, thanks for all the help here kmalloc, dhellmann, smcginnis | 19:38 |
| *** yamamoto has quit IRC | 19:38 | |
| lbragstad | it's nice to have this untangled | 19:38 |
| dhellmann | ++ | 19:38 |
| smcginnis | dhellmann: Do you think https://github.com/openstack/reno/blob/master/doc/source/user/usage.rst would be the right place for this info? | 19:42 |
| dhellmann | smcginnis : yes, that looks like a good page | 19:43 |
| smcginnis | dhellmann: OK, I'll just have to differentiate from the Edit Release Note for the intially created note and editing an historical note. | 19:43 |
| *** armax has quit IRC | 19:52 | |
| openstackgerrit | Sean McGinnis proposed openstack/reno master: Add user details for editing stable branch notes https://review.openstack.org/496391 | 20:02 |
| smcginnis | lbragstad, dhellmann: ^^ | 20:02 |
| smcginnis | lbragstad: Any suggestions of what to add about avoiding updates? | 20:16 |
| smcginnis | lbragstad: Don't mess up in the first place. :) | 20:17 |
| lbragstad | #waterfallforthewin | 20:17 |
| smcginnis | Hah | 20:17 |
| lbragstad | i'm not sure - i know we're probably going to add a section to the keystone docs (if one isn't added to reno) that just says to be mindful of the format, wording, and follow the existing conventions | 20:19 |
| lbragstad | The existing conventions being - https://docs.openstack.org/keystone/latest/contributor/release-notes.html for keystone | 20:20 |
| smcginnis | Yeah, looks like a good place to add a warning of caution. | 20:24 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: update handling of branches in list-changes https://review.openstack.org/496402 | 20:26 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: show missing changes before included changes https://review.openstack.org/496403 | 20:29 |
| *** claudiub has quit IRC | 20:30 | |
| *** rmcall has quit IRC | 20:30 | |
| *** rmcall has joined #openstack-release | 20:31 | |
| *** yamamoto has joined #openstack-release | 20:35 | |
| *** yamamoto has quit IRC | 20:38 | |
| *** yamamoto has joined #openstack-release | 20:38 | |
| *** yamamoto has quit IRC | 20:39 | |
| * dims peeks | 20:54 | |
| *** dave-mccowan has quit IRC | 20:58 | |
| *** dave-mccowan has joined #openstack-release | 21:02 | |
| *** armax has joined #openstack-release | 21:04 | |
| openstackgerrit | Merged openstack/releases master: update handling of branches in list-changes https://review.openstack.org/496402 | 21:08 |
| *** yamamoto has joined #openstack-release | 21:10 | |
| *** yamamoto has quit IRC | 21:15 | |
| *** thorst_a_ has quit IRC | 21:21 | |
| *** thorst_afk has joined #openstack-release | 21:24 | |
| *** thorst_afk has quit IRC | 21:28 | |
| *** thorst_afk has joined #openstack-release | 22:02 | |
| *** thorst_afk has quit IRC | 22:06 | |
| *** thorst_afk has joined #openstack-release | 22:08 | |
| *** yamamoto_ has joined #openstack-release | 22:11 | |
| *** wolverineav has quit IRC | 22:15 | |
| *** wolverineav has joined #openstack-release | 22:16 | |
| *** wolverineav has quit IRC | 22:16 | |
| *** yamamoto_ has quit IRC | 22:17 | |
| *** wolverineav has joined #openstack-release | 22:17 | |
| *** edmondsw has quit IRC | 22:17 | |
| *** thorst_afk has quit IRC | 22:18 | |
| *** thorst_afk has joined #openstack-release | 22:18 | |
| *** wolverineav has quit IRC | 22:21 | |
| *** thorst_afk has quit IRC | 22:22 | |
| *** thorst_afk has joined #openstack-release | 23:00 | |
| *** hongbin has quit IRC | 23:13 | |
| *** yamamoto_ has joined #openstack-release | 23:13 | |
| *** yamamoto_ has quit IRC | 23:18 | |
| ianw | tonyb: so as background, we run a slightly forked version of bandersnatch to mirror pypi. changes have been integrated into it's 2.0 release, but it depends on python3.5. which means we need to upgrade the pypi mirror host to xenial | 23:25 |
| ianw | it's not urgent, but also not something that will go away, so just want to make sure we don't do something right as someone is releasing something | 23:26 |
| tonyb | ianw: If it's a 'short' outage and you're mostly worried about someone on the release team hitting the +W then Monday or Friday are good days to do the work as we generally don't do releases on those days. That doesn't stop Hosted projects from pushing a tag | 23:34 |
| tonyb | I *think* most of the release tooling is fine for py27 or py35 | 23:35 |
| dhellmann | I think the only release tool not currently running under python 3 is the one that updates launchpad bugs to indicate what release the fix is in. | 23:36 |
| dhellmann | That does not work on 3 because one of the dependencies is not available to be installed on python 3 | 23:36 |
| ianw | tonyb: cool, i think a monday then is a good idea. definitely should just be a cut-over situation | 23:37 |
| tonyb | dhellmann: Right. | 23:37 |
| tonyb | ianw: Cool. Give us a little notice and it shoudl be fine. | 23:38 |
| ianw | dhellmann: shouldn't affect release tools ... it's just we need to replace the host as it explicitly says "python3.5" required | 23:38 |
| tonyb | ianw: Would this be post PTG? | 23:38 |
| tonyb | ianw: We'd want to have a really good backout plan if we weer to do it befoer the pike release date | 23:39 |
| ianw | yeah, the theory is that with AFS we don't release the new volume until it's mirrored correctly | 23:40 |
| ianw | so the risk is we're just a bit slow | 23:40 |
| *** efried has quit IRC | 23:42 | |
| tonyb | ianw: okay cool. | 23:42 |
| *** dave-mccowan has quit IRC | 23:48 | |
| *** edmondsw has joined #openstack-release | 23:55 | |
| *** efried has joined #openstack-release | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!