| *** sree has joined #openstack-release | 00:00 | |
| *** kumarmn has joined #openstack-release | 00:01 | |
| *** wolverineav has quit IRC | 00:04 | |
| *** kumarmn has quit IRC | 00:05 | |
| *** sree has quit IRC | 00:06 | |
| *** mriedem has joined #openstack-release | 00:10 | |
| openstackgerrit | Michael Johnson proposed openstack/releases master: Release Octavia projects for Queens RC1 https://review.openstack.org/541092 | 00:24 |
|---|---|---|
| *** ekcs has joined #openstack-release | 00:35 | |
| *** claudiub|2 has quit IRC | 00:45 | |
| *** zhurong has joined #openstack-release | 00:51 | |
| *** annabelleB has joined #openstack-release | 00:52 | |
| *** edmondsw has joined #openstack-release | 00:53 | |
| *** edmondsw has quit IRC | 00:58 | |
| *** annabelleB has quit IRC | 01:03 | |
| *** annabelleB has joined #openstack-release | 01:14 | |
| *** jtomasek has quit IRC | 01:17 | |
| *** ricolin has joined #openstack-release | 01:20 | |
| *** kumarmn has joined #openstack-release | 01:30 | |
| *** Qiming_ has joined #openstack-release | 01:40 | |
| *** Qiming has quit IRC | 01:41 | |
| *** Qiming has joined #openstack-release | 01:43 | |
| openstackgerrit | baiwenteng proposed openstack/releases master: Update http with https https://review.openstack.org/541976 | 01:43 |
| *** hongbin has joined #openstack-release | 01:45 | |
| *** Qiming_ has quit IRC | 01:45 | |
| *** kumarmn has quit IRC | 01:49 | |
| mnaser | dhellmann: cool, left a comment on that review | 01:56 |
| *** kumarmn has joined #openstack-release | 02:12 | |
| *** kumarmn has quit IRC | 02:16 | |
| *** sree has joined #openstack-release | 02:16 | |
| *** annabelleB has quit IRC | 02:16 | |
| *** iyamahat has quit IRC | 02:18 | |
| *** yamahata has quit IRC | 02:20 | |
| *** sree has quit IRC | 02:20 | |
| *** annabelleB has joined #openstack-release | 02:23 | |
| openstackgerrit | Merged openstack/releases master: Release ceilometer 10.0.0 for Queens https://review.openstack.org/540697 | 02:24 |
| openstackgerrit | Tovin Seven proposed openstack/releases master: Add osprofiler release for stable/queens https://review.openstack.org/541645 | 02:27 |
| *** kumarmn has joined #openstack-release | 02:32 | |
| *** kumarmn has quit IRC | 02:36 | |
| *** edmondsw has joined #openstack-release | 02:41 | |
| *** edmondsw has quit IRC | 02:46 | |
| *** annabelleB has quit IRC | 02:49 | |
| EmilienM | tonyb: I know why | 02:51 |
| EmilienM | tonyb: the Depends-On | 02:51 |
| EmilienM | no worries I'll work on it | 02:51 |
| EmilienM | it was more WIP patches | 02:51 |
| EmilienM | thanks for caring though, I appreciate | 02:51 |
| *** sree has joined #openstack-release | 02:51 | |
| openstackgerrit | zhurong proposed openstack/releases master: Release solum and cut stable/queens https://review.openstack.org/541994 | 02:52 |
| tonyb | EmilienM: cool. Let us know when they're less WiP and more merge now :) | 02:54 |
| EmilienM | tonyb: let me update, it should be fine now | 02:54 |
| EmilienM | tonyb: give me one sec | 02:54 |
| *** armax has quit IRC | 02:54 | |
| tonyb | EmilienM: oooh Yeah that's help ;P | 02:55 |
| EmilienM | tonyb: I did recheck | 02:55 |
| EmilienM | they should pass, let's see | 02:55 |
| EmilienM | oh actually no, the pike one needs update | 02:56 |
| *** sree has quit IRC | 02:56 | |
| openstackgerrit | Emilien Macchi proposed openstack/releases master: TripleO pike release https://review.openstack.org/541725 | 02:56 |
| openstackgerrit | Emilien Macchi proposed openstack/releases master: TripleO pike release https://review.openstack.org/541725 | 02:56 |
| tonyb | EmilienM: Should I recheck https://review.openstack.org/#/c/541448/ which looks like it'll block the pike releases | 02:58 |
| EmilienM | ah shit | 02:59 |
| EmilienM | oops sorry | 02:59 |
| tonyb | LOL | 02:59 |
| EmilienM | this one yeah I need it | 02:59 |
| EmilienM | I need to WIP | 02:59 |
| tonyb | Just -W it will be fine. | 03:00 |
| EmilienM | yeah done | 03:01 |
| tonyb | cool | 03:01 |
| *** ekcs has quit IRC | 03:10 | |
| *** udesale has joined #openstack-release | 03:15 | |
| *** gongysh has joined #openstack-release | 03:16 | |
| *** mriedem has quit IRC | 03:16 | |
| *** armax has joined #openstack-release | 03:28 | |
| *** wolverineav has joined #openstack-release | 03:29 | |
| *** wolverineav has quit IRC | 03:33 | |
| *** gongysh has quit IRC | 03:33 | |
| *** wolverineav has joined #openstack-release | 03:35 | |
| *** yamamoto has joined #openstack-release | 03:38 | |
| *** dave-mccowan has quit IRC | 03:39 | |
| *** wolverineav has quit IRC | 03:39 | |
| openstackgerrit | Akihiro Motoki proposed openstack/releases master: Release and cut queens branch for neutron-vpnaas and dashboard https://review.openstack.org/542006 | 03:42 |
| *** ykarel has joined #openstack-release | 03:57 | |
| *** wolverineav has joined #openstack-release | 03:58 | |
| *** wolverin_ has joined #openstack-release | 04:00 | |
| *** wolverineav has quit IRC | 04:00 | |
| *** gongysh has joined #openstack-release | 04:03 | |
| *** kumarmn has joined #openstack-release | 04:04 | |
| *** wolverin_ has quit IRC | 04:04 | |
| openstackgerrit | Julia Kreger proposed openstack/releases master: Release Bifrost 5.0.0 for Queens https://review.openstack.org/542012 | 04:05 |
| *** edmondsw has joined #openstack-release | 04:30 | |
| *** edmondsw has quit IRC | 04:34 | |
| *** armax has quit IRC | 04:48 | |
| openstackgerrit | Qiming Teng proposed openstack/releases master: Release Senlin Queens RC1 https://review.openstack.org/542023 | 04:49 |
| *** zhurong has quit IRC | 04:56 | |
| *** hongbin has quit IRC | 04:58 | |
| *** kumarmn has quit IRC | 04:58 | |
| *** kumarmn has joined #openstack-release | 04:58 | |
| *** kumarmn has quit IRC | 05:02 | |
| *** claudiub|2 has joined #openstack-release | 05:25 | |
| *** lbragstad has quit IRC | 05:37 | |
| *** ekcs has joined #openstack-release | 05:47 | |
| *** zhurong has joined #openstack-release | 05:49 | |
| *** yamahata has joined #openstack-release | 05:50 | |
| *** rosmaita has quit IRC | 05:52 | |
| openstackgerrit | Ifat Afek proposed openstack/releases master: Release vitrage 2.0.1 https://review.openstack.org/541928 | 05:58 |
| *** ekcs has quit IRC | 06:10 | |
| *** ekcs has joined #openstack-release | 06:14 | |
| *** edmondsw has joined #openstack-release | 06:18 | |
| *** ekcs has quit IRC | 06:22 | |
| *** edmondsw has quit IRC | 06:22 | |
| *** yamamoto_ has joined #openstack-release | 06:26 | |
| *** yamamoto has quit IRC | 06:30 | |
| *** coolsvap has joined #openstack-release | 07:00 | |
| *** jtomasek has joined #openstack-release | 07:23 | |
| *** jtomasek has quit IRC | 07:24 | |
| *** jtomasek has joined #openstack-release | 07:25 | |
| *** pcaruana has joined #openstack-release | 07:31 | |
| *** lvdombrkr has joined #openstack-release | 07:32 | |
| openstackgerrit | Michał Dulko proposed openstack/releases master: Release kuryr-kubernetes 0.4.0 and cut queens RC1 https://review.openstack.org/542070 | 07:54 |
| *** claudiub has joined #openstack-release | 08:02 | |
| *** claudiub|2 has quit IRC | 08:04 | |
| *** edmondsw has joined #openstack-release | 08:06 | |
| openstackgerrit | Michał Dulko proposed openstack/releases master: Release kuryr-kubernetes 0.4.0 and cut queens RC1 https://review.openstack.org/542070 | 08:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic 9.1.3 for Pike https://review.openstack.org/542077 | 08:09 |
| *** dtantsur|afk is now known as dtantsur | 08:09 | |
| dtantsur | tonyb: hey, when you're around ^^^ contains a security hardening fix, we'd appreciate a stable review asap | 08:10 |
| *** edmondsw has quit IRC | 08:11 | |
| *** slaweq has joined #openstack-release | 08:13 | |
| *** iyamahat has joined #openstack-release | 08:16 | |
| *** witek has quit IRC | 08:16 | |
| *** witek_ has joined #openstack-release | 08:16 | |
| *** mnaser has quit IRC | 08:17 | |
| *** kmalloc has quit IRC | 08:17 | |
| *** mnaser has joined #openstack-release | 08:17 | |
| *** kmalloc has joined #openstack-release | 08:18 | |
| *** robcresswell has quit IRC | 08:18 | |
| *** iyamahat_ has joined #openstack-release | 08:19 | |
| *** iyamahat__ has joined #openstack-release | 08:20 | |
| *** iyamahat has quit IRC | 08:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic 7.0.4 for Ocata https://review.openstack.org/542082 | 08:22 |
| dtantsur | tonyb: ditto ^^^ | 08:23 |
| *** iyamahat_ has quit IRC | 08:23 | |
| *** alexchadin has joined #openstack-release | 08:25 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-inspector 6.0.1 for Pike https://review.openstack.org/542084 | 08:25 |
| *** eumel8 has joined #openstack-release | 08:25 | |
| openstackgerrit | Ifat Afek proposed openstack/releases master: Release vitrage 2.0.1 https://review.openstack.org/541928 | 08:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-inspector 5.0.2 for Ocata https://review.openstack.org/542085 | 08:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-ui 3.0.2 for Pike https://review.openstack.org/542086 | 08:30 |
| *** witek_ is now known as witek | 08:30 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-ui 2.2.2 for Ocata https://review.openstack.org/542087 | 08:31 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release bifrost 4.0.1 for Pike https://review.openstack.org/542088 | 08:34 |
| *** alexchadin has quit IRC | 08:34 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release bifrost 3.1.1 for Ocata https://review.openstack.org/542089 | 08:36 |
| dtantsur | TheJulia: releasing everything ^^^ from stable branches in preparation for PTL take over :) | 08:36 |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-lib 2.10.1 for Pike https://review.openstack.org/542091 | 08:38 |
| *** alexchadin has joined #openstack-release | 08:40 | |
| openstackgerrit | Ifat Afek proposed openstack/releases master: Release vitrage 2.1.0 https://review.openstack.org/541928 | 08:48 |
| *** robcresswell has joined #openstack-release | 08:49 | |
| *** jpich has joined #openstack-release | 08:54 | |
| *** yamamoto has joined #openstack-release | 08:57 | |
| *** yamamoto_ has quit IRC | 09:00 | |
| lvdombrkr | MariaDB [keystone]> select count(*) from token; | 09:16 |
| lvdombrkr | +----------+ | 09:16 |
| lvdombrkr | | count(*) | | 09:16 |
| lvdombrkr | +----------+ | 09:16 |
| lvdombrkr | | 2019757 | | 09:16 |
| lvdombrkr | +----------+ | 09:16 |
| lvdombrkr | 1 row in set (1.84 sec) | 09:16 |
| lvdombrkr | o sorry folks | 09:16 |
| *** claudiub has quit IRC | 09:18 | |
| *** iyamahat__ has quit IRC | 09:25 | |
| *** amoralej|off is now known as amoralej | 09:26 | |
| *** gongysh has quit IRC | 09:36 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-python-agent 3.2.0 and create stable/queens https://review.openstack.org/542112 | 09:41 |
| *** alexchadin has quit IRC | 09:48 | |
| *** edmondsw has joined #openstack-release | 09:55 | |
| *** kumarmn has joined #openstack-release | 09:59 | |
| *** edmondsw has quit IRC | 09:59 | |
| *** kumarmn has quit IRC | 10:03 | |
| *** zhurong has quit IRC | 10:04 | |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release monasca-kibana-plugin, cut stable/queens https://review.openstack.org/541776 | 10:15 |
| smcginnis | ttx: Are you around? | 10:15 |
| *** iyamahat has joined #openstack-release | 10:23 | |
| openstackgerrit | Merged openstack/releases master: add a warning for repo used in release but not in repository-settings https://review.openstack.org/540521 | 10:32 |
| openstackgerrit | Merged openstack/releases master: only check release jobs if there are new releases to tag https://review.openstack.org/540975 | 10:34 |
| openstackgerrit | Merged openstack/releases master: make validate report more verbose https://review.openstack.org/541011 | 10:35 |
| openstackgerrit | Merged openstack/releases master: only validate releases that have not yet been tagged https://review.openstack.org/541012 | 10:35 |
| openstackgerrit | Merged openstack/releases master: by default skip projects that have had tags applied https://review.openstack.org/541492 | 10:35 |
| openstackgerrit | Merged openstack/releases master: skip list-changes for retired repositories https://review.openstack.org/541820 | 10:35 |
| *** alexchadin has joined #openstack-release | 10:36 | |
| ttx | smcginnis: yes | 10:44 |
| smcginnis | ttx: Oh, was going to ask if you wanted to take another look through Doug's changes, but I thought maybe you were out and didn't want to wait too long before more release requests came in so I approved things. | 10:45 |
| ttx | ok! | 10:45 |
| * ttx looks up queue | 10:45 | |
| smcginnis | ttx: Any concerns there? | 10:45 |
| ttx | no no | 10:46 |
| smcginnis | I am going to rebase the one point where there is a merge conflict so we can get the whole set through. | 10:46 |
| ttx | ok I can W+1 after the rebase | 10:46 |
| openstackgerrit | Merged openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 10:48 |
| *** alexchadin has quit IRC | 10:49 | |
| *** alexchadin has joined #openstack-release | 10:50 | |
| ttx | I'll hold on release approving until the set is in | 10:50 |
| *** iyamahat has quit IRC | 10:50 | |
| smcginnis | Looks like that would be safer. | 10:51 |
| *** alexchadin has quit IRC | 10:55 | |
| *** alexchadin has joined #openstack-release | 10:56 | |
| *** claudiub has joined #openstack-release | 10:57 | |
| openstackgerrit | Merged openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 11:00 |
| *** armax has joined #openstack-release | 11:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic 10.1.0 and create stable/queens https://review.openstack.org/542132 | 11:03 |
| *** lucas-afk is now known as lucasagomes | 11:05 | |
| *** dtantsur is now known as dtantsur|bbl | 11:06 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 11:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 11:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 11:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 11:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 11:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 11:15 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: try to validate the starting points for branches https://review.openstack.org/541906 | 11:15 |
| smcginnis | Whew, doesn't look like I screwed any of that up. | 11:16 |
| *** yamahata has quit IRC | 11:27 | |
| * ttx looks | 11:28 | |
| ttx | oh, you approved it already. All set then :) | 11:29 |
| * ttx goes to lunch | 11:29 | |
| *** zhurong has joined #openstack-release | 11:30 | |
| smcginnis | Not the root one of the stack, but I can just push that through if you are out. | 11:31 |
| smcginnis | Validate takes some time when you modify every deliverable. :) | 11:32 |
| *** alexchadin has quit IRC | 11:33 | |
| *** alexchadin has joined #openstack-release | 11:33 | |
| smcginnis | Looks like we can safely ignore those release job failures. They were just for the modifications of all the deliverables for pike and ocata. | 11:35 |
| smcginnis | OK, validation all checked out. Pushing that through. | 11:39 |
| *** armax has quit IRC | 11:40 | |
| openstackgerrit | Merged openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 11:56 |
| openstackgerrit | Merged openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 11:57 |
| openstackgerrit | Merged openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 11:57 |
| openstackgerrit | Merged openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 11:57 |
| openstackgerrit | Merged openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 11:57 |
| openstackgerrit | Merged openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 11:57 |
| openstackgerrit | Merged openstack/releases master: try to validate the starting points for branches https://review.openstack.org/541906 | 11:57 |
| smcginnis | johnsom: Please take a look at my comment on https://review.openstack.org/#/c/541092/ when you have a chance. | 11:59 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release solum and cut stable/queens https://review.openstack.org/541994 | 12:01 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release and cut queens branch for neutron-vpnaas and dashboard https://review.openstack.org/542006 | 12:02 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release Senlin Queens RC1 https://review.openstack.org/542023 | 12:04 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release vitrage 2.1.0 https://review.openstack.org/541928 | 12:06 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release kuryr-kubernetes 0.4.0 and cut queens RC1 https://review.openstack.org/542070 | 12:07 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release ironic-python-agent 3.2.0 and create stable/queens https://review.openstack.org/542112 | 12:08 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release and cut queens branch for neutron-vpnaas and dashboard https://review.openstack.org/542006 | 12:09 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release monasca-kibana-plugin, cut stable/queens https://review.openstack.org/541776 | 12:10 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release ironic 10.1.0 and create stable/queens https://review.openstack.org/542132 | 12:12 |
| *** dave-mccowan has joined #openstack-release | 12:12 | |
| *** alexchadin has quit IRC | 12:14 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release Octavia projects for Queens RC1 https://review.openstack.org/541092 | 12:16 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release kuryr-kubernetes 0.4.0 and cut queens RC1 https://review.openstack.org/542070 | 12:18 |
| *** alexchadin has joined #openstack-release | 12:18 | |
| *** armax has joined #openstack-release | 12:21 | |
| *** alexchadin has quit IRC | 12:22 | |
| *** ykarel_ has joined #openstack-release | 12:32 | |
| *** ykarel has quit IRC | 12:34 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release Octavia projects for Queens RC1 https://review.openstack.org/541092 | 12:36 |
| *** ykarel__ has joined #openstack-release | 12:36 | |
| *** ykarel_ has quit IRC | 12:39 | |
| *** ykarel__ is now known as ykarel | 12:44 | |
| *** alexchadin has joined #openstack-release | 12:45 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release Bifrost 5.0.0 for Queens https://review.openstack.org/542012 | 12:46 |
| *** alexchadin has quit IRC | 12:49 | |
| openstackgerrit | Merged openstack/releases master: Release ironic-python-agent 3.2.0 and create stable/queens https://review.openstack.org/542112 | 12:50 |
| openstackgerrit | Merged openstack/releases master: Release monasca-kibana-plugin, cut stable/queens https://review.openstack.org/541776 | 12:50 |
| *** alexchadin has joined #openstack-release | 12:54 | |
| *** pkovar has joined #openstack-release | 12:55 | |
| *** armax has quit IRC | 12:58 | |
| *** coolsvap has quit IRC | 12:58 | |
| *** armax has joined #openstack-release | 12:58 | |
| openstackgerrit | Masahito Muroi proposed openstack/releases master: Release blazar 1.0.0.0rc1 and blazar-nova 1.0.1 https://review.openstack.org/542161 | 13:00 |
| *** edmondsw has joined #openstack-release | 13:11 | |
| smcginnis | Latest release job failure was just the update to the queens deliverables files. Safe to ignore. | 13:12 |
| *** rosmaita has joined #openstack-release | 13:17 | |
| *** coolsvap has joined #openstack-release | 13:18 | |
| openstackgerrit | Merged openstack/releases master: Release Bifrost 5.0.0 for Queens https://review.openstack.org/542012 | 13:19 |
| openstackgerrit | Merged openstack/releases master: Release Senlin Queens RC1 https://review.openstack.org/542023 | 13:19 |
| *** wolverineav has joined #openstack-release | 13:22 | |
| *** zhurong has quit IRC | 13:27 | |
| dhellmann | o/ | 13:27 |
| dhellmann | smcginnis : you've been busy | 13:27 |
| smcginnis | I got a few things out of the way this morning. | 13:29 |
| dhellmann | I'm just catching up on email, so let me know if there's anything I can help with | 13:30 |
| smcginnis | Nothing urgent right now. | 13:30 |
| smcginnis | We had release failures on your patches that updated all of the deliverable files, so I don't think we need to worry about that. | 13:30 |
| dhellmann | yeah, those shouldn't have triggered any actual operations | 13:31 |
| smcginnis | I've rebased the outstanding release requests with those new changes and fixed up a couple that had highlights. | 13:31 |
| dhellmann | we may need to think about extending the timeout on that job though if we run into problems with patches that don't touch all of the deliverables in a series | 13:31 |
| smcginnis | Other than that, I think I've made a pass through all requests and flagged a couple. | 13:31 |
| smcginnis | I honestly didn't look too closely at those failures, but I think it was just due to the sheer volume due to every deliverable file being touched. | 13:32 |
| dhellmann | yeah, I assume so, too | 13:32 |
| dhellmann | I'll take a closer look at one just to be sure | 13:32 |
| smcginnis | Good, probably best to check on that. | 13:32 |
| smcginnis | I had seen timeouts due to git review -s hanging. | 13:32 |
| smcginnis | In which case, extending the timeout of the job is just going to make it hang that much longer. | 13:33 |
| smcginnis | I haven't looked at making a similar change like you did for the git fetch hang. | 13:33 |
| smcginnis | I assume it's the same kind of thing. | 13:33 |
| smcginnis | dhellmann: Could you take a quick scan through https://etherpad.openstack.org/p/queens-email-r2 before I send that out? | 13:33 |
| dhellmann | does git review -s talk to the network? | 13:34 |
| dhellmann | sure, looking now | 13:34 |
| smcginnis | I didn't think it would need to, but based on the timeout, I suppose it must. | 13:34 |
| dhellmann | wow, I've been failing at calendaring -- I thought rc1 was next week | 13:34 |
| smcginnis | Hah! That's why I was a little concerned about all your changes. ;) | 13:35 |
| edmondsw | dims TheJulia... lost your votes on https://review.openstack.org/#/c/540165/ on a respin | 13:35 |
| smcginnis | The mess up with q-3 taking an extra week kind of threw off the cadence for me too. | 13:35 |
| *** d0ugal has quit IRC | 13:36 | |
| dhellmann | yeah, I think that's what confused me | 13:36 |
| dhellmann | smcginnis : we tend to get a lot of questions about branching around this time in the cycle. Do you want to explain the sequence so folks know what to expect? | 13:39 |
| dhellmann | like, "after all of the cycle-with-milestone projects have branched we will branch devstack, etc. and the requirements repository..." | 13:39 |
| smcginnis | Sure, I'll add a note for that. | 13:40 |
| smcginnis | Hows that under General Information? | 13:42 |
| smcginnis | Ah, grenade. | 13:42 |
| *** d0ugal has joined #openstack-release | 13:43 | |
| smcginnis | dhellmann: Good to send? Or still reviewing? | 13:45 |
| dhellmann | do you think we want to mention the requirements freeze lifting, too? that's the other question we get a lot | 13:46 |
| smcginnis | I kind of did with the last part about branching requirements, but figured I would leave it for prometheanfire to actual state something about the freeze being lifted. | 13:47 |
| dhellmann | makes sense | 13:47 |
| dhellmann | I think it's good this way then | 13:47 |
| smcginnis | s/I kind of did/I kind of eluded to/ | 13:47 |
| smcginnis | OK, thanks! | 13:47 |
| dhellmann | I've suggested to pkovar that he contact you next week to find out when all of the repos are branched so the docs team can run their end-of-cycle process to link to the cycle-specific docs | 13:48 |
| smcginnis | OK, great. I'll watch out for that. | 13:49 |
| * smcginnis grabs some breakfast | 13:49 | |
| pkovar | thanks, dhellmann smcginnis | 13:51 |
| dhellmann | np | 13:51 |
| * ttx does another pass at reviewing | 13:52 | |
| *** mriedem has joined #openstack-release | 13:54 | |
| openstackgerrit | Merged openstack/releases master: Release kuryr-kubernetes 0.4.0 and cut queens RC1 https://review.openstack.org/542070 | 13:56 |
| *** dtantsur|bbl is now known as dtantsur | 13:59 | |
| openstackgerrit | Alexander Chadin proposed openstack/releases master: Queens release candidate for watcher https://review.openstack.org/542197 | 14:00 |
| dhellmann | prometheanfire : we have a release request for a bug fix in osprofiler in https://review.openstack.org/#/c/541645/ | 14:02 |
| ttx | smcginnis: reviewed the ironic release contents ad +2a-ed it | 14:04 |
| dtantsur | folks, do you know why https://review.openstack.org/#/c/542089/ shows a merge conflict? Oo I can hardly imagine someone changing this file | 14:04 |
| dtantsur | (and a few other releases as well) | 14:05 |
| dhellmann | dtantsur : we just landed a bunch of small settings changes to all of the deliverable files | 14:05 |
| dtantsur | dhellmann: ah! so should I rebase myself? | 14:05 |
| dhellmann | dtantsur : I'm doing that now, stand by | 14:05 |
| *** pkovar has quit IRC | 14:05 | |
| *** kumarmn has joined #openstack-release | 14:06 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Release bifrost 3.1.1 for Ocata https://review.openstack.org/542089 | 14:06 |
| ttx | We have a bunch of stuff blocked on release-highlights presence (or content nitpicks) -- I'm not sure we should block them on that... those can always be fixed afterwards | 14:06 |
| ttx | I would rather issue RCs and branch -- maybe be more picky for further RCs if highlights are still missing? | 14:07 |
| dhellmann | yes, I would prefer not to block on the highlights stuff | 14:07 |
| dtantsur | dhellmann: thanks! | 14:08 |
| dhellmann | I thought we already said that we'd not block? | 14:08 |
| ttx | some reviews did not get approved so I assumed the rule changed | 14:08 |
| dhellmann | ah, maybe smcginnis was trying to nudge and give them a little time? | 14:09 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Release ironic 9.1.3 for Pike https://review.openstack.org/542077 | 14:09 |
| * ttx will unblock a few | 14:10 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Release ironic 7.0.4 for Ocata https://review.openstack.org/542082 | 14:10 |
| openstackgerrit | Armando Migliaccio proposed openstack/releases master: Neutron project RC1 https://review.openstack.org/540476 | 14:11 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Release ironic-ui 2.2.2 for Ocata https://review.openstack.org/542087 | 14:11 |
| *** ykarel_ has joined #openstack-release | 14:11 | |
| *** ykarel has quit IRC | 14:11 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Release ironic-inspector 5.0.2 for Ocata https://review.openstack.org/542085 | 14:11 |
| dhellmann | dtantsur : that should be all of yours, rebased | 14:12 |
| dtantsur | thanks again | 14:12 |
| openstackgerrit | Merged openstack/releases master: Release ironic 10.1.0 and create stable/queens https://review.openstack.org/542132 | 14:13 |
| ttx | OK approved all I could | 14:13 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: expand on openstacksdk cycle highlights https://review.openstack.org/541732 | 14:15 |
| openstackgerrit | Alexander Chadin proposed openstack/releases master: Queens release candidate for watcher-dashboard https://review.openstack.org/542211 | 14:18 |
| *** alexchadin has quit IRC | 14:18 | |
| openstackgerrit | Merged openstack/releases master: Release vitrage 2.1.0 https://review.openstack.org/541928 | 14:18 |
| *** pkovar has joined #openstack-release | 14:21 | |
| *** lucasagomes is now known as lucas-hungry | 14:26 | |
| openstackgerrit | Merged openstack/releases master: Release Octavia projects for Queens RC1 https://review.openstack.org/541092 | 14:26 |
| openstackgerrit | Jay Bryant proposed openstack/releases master: Release Cinder Queens RC1 (12.0.0.0rc1) https://review.openstack.org/542213 | 14:27 |
| *** yamamoto has quit IRC | 14:34 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-tempest-plugin 1.1.0 https://review.openstack.org/542235 | 14:37 |
| *** lbragstad has joined #openstack-release | 14:37 | |
| dtantsur | I'm going to spam you today :) | 14:37 |
| openstackgerrit | Jay Bryant proposed openstack/releases master: Release Cinder 11.0.3 https://review.openstack.org/542236 | 14:40 |
| dims | edmondsw : done! | 14:40 |
| edmondsw | dims tx! | 14:41 |
| *** yamamoto has joined #openstack-release | 14:46 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release networking-generic-switch 1.0.0 and create stable/queens https://review.openstack.org/542241 | 14:47 |
| openstackgerrit | Manoj Kumar proposed openstack/releases master: Tagging trove for RC1 https://review.openstack.org/542242 | 14:47 |
| *** yamamoto has quit IRC | 14:51 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release networking-baremetal 1.0.0 and create stable/queens https://review.openstack.org/542245 | 14:54 |
| *** dtantsur is now known as dtantsur|brb | 14:55 | |
| *** ykarel_ is now known as ykarel | 14:58 | |
| smcginnis | dhellmann, ttx: Correct about nudging and giving a little time. If there wasn't a response quickly I was going to still go ahead with those. | 14:59 |
| *** iyamahat has joined #openstack-release | 15:00 | |
| *** yamahata has joined #openstack-release | 15:02 | |
| openstackgerrit | Merged openstack/releases master: Release and cut queens branch for neutron-vpnaas and dashboard https://review.openstack.org/542006 | 15:03 |
| *** lucas-hungry is now known as lucasagomes | 15:26 | |
| *** yamamoto has joined #openstack-release | 15:32 | |
| openstackgerrit | Spyros Trigazis (strigazi) proposed openstack/releases master: Release magnum 6.0.0 https://review.openstack.org/542265 | 15:32 |
| *** ykarel is now known as ykarel|away | 15:32 | |
| strigazi | smcginnis: apologies for the delay ^^ | 15:34 |
| *** dtantsur|brb is now known as dtantsur | 15:34 | |
| prometheanfire | dhellmann: ok, add it to the list :P | 15:34 |
| prometheanfire | as far as the freeze lifting I'll need to run the validation (or have tony do it since I can't 'git review -s' (our infra leaks tcp connections)...) | 15:35 |
| prometheanfire | but as soon as we are validated I'll send a branch request and then once that hits I'll send an email | 15:36 |
| smcginnis | strigazi: No problem, perfect timing. ;) | 15:36 |
| smcginnis | prometheanfire: ++ | 15:36 |
| prometheanfire | smcginnis: we gonna get more before branching? | 15:36 |
| *** yamamoto has quit IRC | 15:37 | |
| smcginnis | prometheanfire: More requirements FFEs? | 15:38 |
| *** coolsvap has quit IRC | 15:38 | |
| smcginnis | I hope not. | 15:38 |
| prometheanfire | ya | 15:38 |
| prometheanfire | ok | 15:38 |
| prometheanfire | as of right now we don't have any | 15:38 |
| prometheanfire | maybe osprofiler (as doug mentioned earlier) | 15:38 |
| *** bobh has joined #openstack-release | 15:39 | |
| smcginnis | I hope we've gotten everything through by now that's needed, but we'll see if we get any last minute surprises. ;) | 15:39 |
| prometheanfire | oh, you had monasca-common as wel | 15:39 |
| prometheanfire | oh, you had monasca-common as well | 15:39 |
| prometheanfire | I'd like to know more about this one | 15:39 |
| prometheanfire | 08:04 < dhellmann > prometheanfire : we have a release request for a bug fix in osprofiler in https://review.openstack.org/#/c/541645/ | 15:39 |
| smcginnis | Oh yeah, that's still out there. (monasca-common) | 15:40 |
| *** hongbin has joined #openstack-release | 15:41 | |
| prometheanfire | I just approved it | 15:42 |
| smcginnis | They had responded on that right? Getting fuzzy now, but I think that was the one that they said was only impacting their own projects? | 15:43 |
| smcginnis | Or wait... | 15:43 |
| *** yamamoto has joined #openstack-release | 15:44 | |
| *** yamamoto has quit IRC | 15:44 | |
| smcginnis | prometheanfire: There's this one, though they keep asking on the patch and not responding on the ML where they were asked: https://review.openstack.org/541638 | 15:44 |
| *** ykarel|away has quit IRC | 15:45 | |
| openstackgerrit | Merged openstack/releases master: expand on openstacksdk cycle highlights https://review.openstack.org/541732 | 15:51 |
| openstackgerrit | Thomas Bechtold proposed openstack/releases master: pymod2pkg: Release version 0.13.1 https://review.openstack.org/542272 | 15:55 |
| *** slaweq has quit IRC | 15:56 | |
| *** annabelleB has joined #openstack-release | 15:56 | |
| prometheanfire | smcginnis: added myself to that, so I'll get a ping if he responds via the review | 15:56 |
| *** slaweq has joined #openstack-release | 15:56 | |
| *** gongysh has joined #openstack-release | 16:00 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Update stable branch link to direct to repo root https://review.openstack.org/542276 | 16:00 |
| *** slaweq has quit IRC | 16:01 | |
| *** pcaruana has quit IRC | 16:02 | |
| openstackgerrit | gongysh proposed openstack/releases master: Request tacker server project queens release https://review.openstack.org/541544 | 16:07 |
| openstackgerrit | Zhipeng Huang proposed openstack/releases master: cyborg 0.1.0 queens independant release request https://review.openstack.org/542280 | 16:07 |
| smcginnis | ttx, dhellmann: Thoughts on https://review.openstack.org/#/c/541994/ | 16:08 |
| smcginnis | Should we just update it for them and remove the new release? | 16:08 |
| *** udesale has quit IRC | 16:10 | |
| EmilienM | release-managers: please look at https://review.openstack.org/#/c/541724/ | 16:12 |
| openstackgerrit | Emilien Macchi proposed openstack/releases master: TripleO pike release https://review.openstack.org/541725 | 16:12 |
| *** gongysh has quit IRC | 16:13 | |
| ttx | smcginnis: we could yes. Maybe do it tomorrow if they haven't picked it up | 16:14 |
| smcginnis | OK, that sounds like a reasonable plan. | 16:14 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Adjust grammar in Blazar release note. https://review.openstack.org/542285 | 16:17 |
| *** iyamahat has quit IRC | 16:22 | |
| *** armax has quit IRC | 16:22 | |
| *** armax has joined #openstack-release | 16:23 | |
| openstackgerrit | Merged openstack/releases master: Queens release candidate for watcher https://review.openstack.org/542197 | 16:28 |
| *** masayukig has quit IRC | 16:29 | |
| openstackgerrit | Eric Berglund proposed openstack/releases master: Release ceilometer-powervm Queens RC1 (6.0.0.0rc1) https://review.openstack.org/542293 | 16:30 |
| *** masayukig has joined #openstack-release | 16:30 | |
| openstackgerrit | Merged openstack/releases master: Release blazar 1.0.0.0rc1 and blazar-nova 1.0.1 https://review.openstack.org/542161 | 16:31 |
| openstackgerrit | Merged openstack/releases master: TripleO ocata release https://review.openstack.org/541724 | 16:31 |
| *** iyamahat has joined #openstack-release | 16:31 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release magnum 6.0.0 https://review.openstack.org/542265 | 16:32 |
| *** pkovar has quit IRC | 16:37 | |
| *** pkovar has joined #openstack-release | 16:38 | |
| *** bobh has quit IRC | 16:38 | |
| openstackgerrit | Graham Hayes proposed openstack/releases master: Release Designate 6.0.0.0rc1 https://review.openstack.org/542303 | 16:38 |
| *** ykarel|away has joined #openstack-release | 16:40 | |
| openstackgerrit | Graham Hayes proposed openstack/releases master: Release Designate Dashboard 6.0.0.0rc1 https://review.openstack.org/542305 | 16:43 |
| openstackgerrit | Manoj Kumar proposed openstack/releases master: Tagging trove for RC1 https://review.openstack.org/542242 | 16:43 |
| *** eumel8 has quit IRC | 16:44 | |
| *** yamamoto has joined #openstack-release | 16:44 | |
| corvus | dhellmann: can you take another look at https://review.openstack.org/540148 please? | 16:47 |
| *** openstackgerrit has quit IRC | 16:48 | |
| dhellmann | corvus : approved | 16:48 |
| *** hongbin has quit IRC | 16:48 | |
| corvus | thx | 16:49 |
| dhellmann | corvus : the release jobs are mostly useless without the log files, so that's why I wanted to keep the flag in place, but if the flag itself is not going to be useful then *shrug* | 16:49 |
| *** hongbin has joined #openstack-release | 16:50 | |
| *** ykarel|away has quit IRC | 16:51 | |
| *** openstackgerrit has joined #openstack-release | 16:51 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Request tacker server project queens release https://review.openstack.org/541544 | 16:51 |
| corvus | yeah, setting as a site variable, which is the intention, won't allow it to be overridden. hopefully we'll have a better log system in place before the next time that happens. | 16:51 |
| dhellmann | ok | 16:51 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: show which repos use a library https://review.openstack.org/542312 | 16:56 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release Glance 16.0.0.0rc1 https://review.openstack.org/542313 | 16:56 |
| *** lvdombrkr has quit IRC | 16:56 | |
| dhellmann | smcginnis , prometheanfire : https://review.openstack.org/542312 will be of interest to both of you | 16:56 |
| smcginnis | dhellmann: Nice! I was just thinking earlier today that it would be nice to have that automatically pulled into our validate job or something. | 16:57 |
| * ttx rerereviews | 16:57 | |
| ttx | Just approved a bunch, let's see how that goes | 17:01 |
| *** yamamoto has quit IRC | 17:02 | |
| openstackgerrit | Merged openstack/releases master: Remove log handling override https://review.openstack.org/540148 | 17:02 |
| *** slaweq has joined #openstack-release | 17:04 | |
| openstackgerrit | Steve McLellan proposed openstack/releases master: Release Searchlight queens RC1 and stable branch https://review.openstack.org/542319 | 17:05 |
| *** slaweq has quit IRC | 17:09 | |
| openstackgerrit | Merged openstack/releases master: Adjust grammar in Blazar release note. https://review.openstack.org/542285 | 17:09 |
| openstackgerrit | Merged openstack/releases master: Queens release candidate for watcher-dashboard https://review.openstack.org/542211 | 17:09 |
| openstackgerrit | Merged openstack/releases master: Release Cinder Queens RC1 (12.0.0.0rc1) https://review.openstack.org/542213 | 17:09 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: show which repos use a library https://review.openstack.org/542312 | 17:12 |
| openstackgerrit | Merged openstack/releases master: Release networking-generic-switch 1.0.0 and create stable/queens https://review.openstack.org/542241 | 17:15 |
| openstackgerrit | Merged openstack/releases master: Release networking-baremetal 1.0.0 and create stable/queens https://review.openstack.org/542245 | 17:15 |
| dhellmann | another timeout on tag-releases | 17:16 |
| dhellmann | http://logs.openstack.org/11/1160e02315eaef3a8380af3d6dd9f707eccc214e/release-post/tag-releases/ff8305f/ | 17:16 |
| smcginnis | TripleO Ocata release. | 17:16 |
| dhellmann | that doesn't look like git hung | 17:18 |
| dhellmann | the retry logic is in place there | 17:18 |
| dhellmann | although maybe it's not working | 17:20 |
| dhellmann | because that git pull did block for almost 30 minutes | 17:20 |
| smcginnis | Last thing was the git pull. | 17:20 |
| dhellmann | maybe we have that set to high | 17:20 |
| smcginnis | Is that in the same script that we set to retry? | 17:20 |
| smcginnis | Oh, yep. Sorry, was finishing an email. Now I'm actually looking. | 17:21 |
| dhellmann | time limit is set to 300 but I'm not sure what units those are. I expected seconds? | 17:21 |
| smcginnis | That was my assumption. | 17:21 |
| openstackgerrit | Merged openstack/releases master: Release magnum 6.0.0 https://review.openstack.org/542265 | 17:21 |
| dhellmann | http://blogs.collab.net/git/tips-on-git says seconds | 17:22 |
| dhellmann | I guess that means its not working? | 17:22 |
| smcginnis | Or hanging in some other odd way. | 17:23 |
| dhellmann | hmm, it's not there in the output | 17:23 |
| dhellmann | oh, we set -x after processing the arguments | 17:23 |
| dhellmann | fungi , corvus : do either of you have any ideas about why the git timeout variable doesn't seem to be having any effect? ^^ | 17:25 |
| fungi | which script was it in again? | 17:26 |
| dhellmann | project-config/jenkins/scripts/release-tools/clone_repo.sh | 17:27 |
| *** bobh has joined #openstack-release | 17:28 | |
| fungi | i'll have a look after the security meeting ends | 17:29 |
| dhellmann | thanks | 17:29 |
| fungi | skimming scrollback, i guess there's also some question as to whether the updated version of the script ran in that job? | 17:29 |
| dhellmann | fungi : there was, but I'm convinced the updated version ran because I see it using the "retry" function | 17:30 |
| fungi | okay, that helps rule out one possibility at least | 17:30 |
| *** armax has quit IRC | 17:31 | |
| *** armax has joined #openstack-release | 17:32 | |
| openstackgerrit | Saad Zaher proposed openstack/releases master: Release Freezer 6.0.0.0rc1 https://review.openstack.org/542342 | 17:35 |
| dhellmann | stephenfin : regarding those reno bugs: some of them may be challenging to solve because reno is only scanning one branch at a time. so doing things like recognizing what has already been seen, or is older and belongs somewhere else, will require some fundamental changes to the scanner | 17:37 |
| *** jpich has quit IRC | 17:37 | |
| openstackgerrit | Saad Zaher proposed openstack/releases master: Release freezer-web-ui 6.0.0.0rc1 https://review.openstack.org/542346 | 17:37 |
| stephenfin | dhellmann: I figured it wouldn't be easy. Out of curiosity, what's the current behavior for backported fixes? Do we consider them fixed on master or stable/xyz? | 17:38 |
| corvus | dhellmann: i know in zuul we set the same variables and found that it still wasn't sufficient in some cases, so we wrapped the whole thing in a timeout that kills the process (though if we hit that, we have to delete the whole git tree because it can be inconsistent) | 17:38 |
| stephenfin | Sorry - expected behavior, in your opinion. I can verify the actual behavior | 17:39 |
| dhellmann | corvus : yeah, the issue is we've hit that process timeout here instead of the expected (shorter) git timeout :-( | 17:40 |
| dhellmann | stephenfin : I'm not sure what you mean? are you asking where the note should show up? | 17:40 |
| stephenfin | Yup | 17:40 |
| dhellmann | if a note appears on a branch, it should show up in the release history for that branch. | 17:41 |
| dhellmann | even if that means it shows up in multiple places | 17:41 |
| dhellmann | if I fix a bug in 10.0.1 and backport that to 9.5.2 then users who installed 10.0.0 have just as much right to know the bug was fixed when they go to upgrade as users of 9.5.1 | 17:41 |
| dhellmann | reno is *branch based* not version based. Versions are just points in time along a branch. | 17:42 |
| stephenfin | Riiight. So I've been working on a different set of assumptions | 17:43 |
| dhellmann | we have some logic to try to figure out a point on a branch where continuing to report history is likely to not be interesting (or is likely to be confusing) but we're still scanning the branch | 17:44 |
| stephenfin | That would explain why branching from the release tag (as opposed to making the release tag on the branch, as we normally do) doesn't work | 17:44 |
| dhellmann | yeah, the "where does this branch stop" logic looks at tags | 17:44 |
| dhellmann | we could probably change that to look at commits without breaking anything we're doing for openstack | 17:45 |
| dtantsur | smcginnis: left a comment on review, but I guess I'll ask here: which branch are you referring to in your comment on https://review.openstack.org/#/c/542235/ ? | 17:45 |
| stephenfin | dhellmann: I'm looking at this for Sphinx and I think they tag after branching. Not too big an issue, in that case | 17:46 |
| stephenfin | Though yeah, we can't probably short circuit it by looking at the base of the two branches or something | 17:47 |
| dhellmann | they probably have the version number in their setup.py file | 17:47 |
| dhellmann | yeah, I think the logic for finding the base of the branch can be adapted to work for that case, too | 17:47 |
| stephenfin | dhellmann: Agreed. The merging back one is the bigger issue (i.e. https://bugs.launchpad.net/reno/+bug/1588309) | 17:51 |
| openstack | Launchpad bug 1588309 in reno "Reno is confused when a stable tag is merged with the master branch" [Medium,Confirmed] | 17:51 |
| stephenfin | I think that might be solvable by way of the '--first-parent' argument (or dulwich equivalent) | 17:51 |
| stephenfin | but I haven't fully fleshed it out | 17:52 |
| dhellmann | so they patch in stable branches and then merge forward? | 17:52 |
| stephenfin | Yup. It's really irritating | 17:52 |
| stephenfin | *rather | 17:52 |
| dhellmann | the topo traversal logic is already a bit hairy, so I don't know if we want to solve it at that level | 17:52 |
| *** amoralej is now known as amoralej|off | 17:53 | |
| dhellmann | especially since that might break the handling for the merge commits zuul creates | 17:53 |
| dhellmann | as you said, we might actually need to do something like scan the other branches first and then filter things out when they appear to have been merged into the "current" branch | 17:54 |
| dhellmann | that's a whole different traversal implementation though | 17:54 |
| *** pbandark has joined #openstack-release | 17:55 | |
| stephenfin | That would break the behaviour you referred to above though | 17:55 |
| stephenfin | "if I fix a bug in 10.0.1 and backport that to 9.5.2 then users who installed 10.0.0 have just as much right to know the bug was fixed when they go to upgrade as users of 9.5.1" | 17:55 |
| dhellmann | right | 17:55 |
| stephenfin | Hmm, let me think on it. I'll see how eager the Sphinx folks are to have this. If they're actually going to use it, then I'll go ahead and work on a solution | 17:56 |
| dhellmann | ok | 17:56 |
| stephenfin | If not, I guess we just want to call it out in the docs in an Assumptions section | 17:56 |
| dhellmann | there are a couple of other tools that work sort of like reno but may not have some of the same assumptions baked in | 17:56 |
| dhellmann | towncrier and the thing python-dev created (I can't remember the name) | 17:57 |
| stephenfin | Yup, I spotted towncrier (tox use it, I think) | 17:57 |
| dhellmann | which is not to say I wouldn't like them to use reno, just that there may be another path of least resistance | 17:57 |
| stephenfin | It just didn't seem as well developed | 17:57 |
| dhellmann | yeah, it probably does as much as the twisted folks need | 17:57 |
| stephenfin | If it works and doesn't mean rewriting chunks of reno for one user though... | 17:58 |
| dhellmann | right | 17:58 |
| *** dtantsur is now known as dtantsur|afk | 17:58 | |
| stephenfin | Alrighty then, for now I'll just do a doc fix stating these assumptions (trunk-based development, no backmerges from stable) and have a chat with them | 17:59 |
| stephenfin | Thanks, dhellmann :) | 17:59 |
| dhellmann | sounds good, thank stephenfin | 18:01 |
| *** slaweq_ has joined #openstack-release | 18:13 | |
| *** slaweq_ has quit IRC | 18:19 | |
| *** ekcs has joined #openstack-release | 18:30 | |
| *** pkovar has quit IRC | 18:47 | |
| *** ekcs has quit IRC | 18:51 | |
| *** ekcs has joined #openstack-release | 18:53 | |
| openstackgerrit | Thomas Bechtold proposed openstack/releases master: pymod2pkg: Release version 0.13.1 https://review.openstack.org/542272 | 18:54 |
| *** slaweq has joined #openstack-release | 18:56 | |
| *** armax has quit IRC | 19:00 | |
| *** slaweq has quit IRC | 19:01 | |
| *** armax has joined #openstack-release | 19:01 | |
| openstackgerrit | Ben Swartzlander proposed openstack/releases master: Release Queens RC1 for Manila https://review.openstack.org/542376 | 19:04 |
| *** lucasagomes is now known as lucas-afk | 19:09 | |
| fungi | dhellmann: so looking at the value we've chosen for GIT_HTTP_LOW_SPEED_LIMIT in clone_repo.sh and the size of the tripleo-image-elements repo, it could easily sustain a higher average transfer rate than that while needing several times longer than the 30 minutes that job timed out in | 19:14 |
| smcginnis | Over 30 minutes on a git fetch? | 19:15 |
| fungi | oh, good point, i was looking at time to clone | 19:15 |
| fungi | the call which timed out was a git pull on master, and just before that when the master branch was checked out it claimed "Your branch is up-to-date with 'origin/master'." so there really should have been nothing to pull | 19:16 |
| fungi | this is pretty strange | 19:16 |
| dhellmann | I thought the low speed limit was the value to watch for as meaning we're getting a bad connection, not a throttle on the connection. Did I interpret that wrong? | 19:16 |
| dhellmann | doesn't git pull also go upstream again? | 19:17 |
| dhellmann | yeah, pull is like fetch + merge | 19:17 |
| dhellmann | so maybe we want to change that to a merge operation? | 19:17 |
| fungi | not a throttle, no, i was merely pointing out that with tie being some 12mb in size you could sustain a slightly higher data rate than 1000 bytes a second (so not trip the check) and still need hours to complete the transfer | 19:18 |
| dhellmann | oh, I see | 19:18 |
| dhellmann | I don't really know what a reasonable value for that field is | 19:19 |
| fungi | yeah, though this mental exercise may be an opportunity to agree on a value | 19:19 |
| *** kong has quit IRC | 19:26 | |
| *** kong has joined #openstack-release | 19:27 | |
| *** armax has quit IRC | 19:31 | |
| *** lbragstad has quit IRC | 19:31 | |
| *** lbragstad has joined #openstack-release | 19:32 | |
| *** armax has joined #openstack-release | 19:32 | |
| *** pkovar has joined #openstack-release | 19:37 | |
| *** pkovar has quit IRC | 19:38 | |
| fungi | so thinking through this, i expect we can set an arbitrarily high value for GIT_HTTP_LOW_SPEED_LIMIT because if a git operation takes longer than the 5 minutes we have for GIT_HTTP_LOW_SPEED_TIME then we should assume it's broken anyway, right? | 19:43 |
| smcginnis | That seems reasonable to me. | 19:44 |
| dhellmann | sure | 19:44 |
| openstackgerrit | Eric Kao proposed openstack/releases master: Congress Q-rc1 and stable branch https://review.openstack.org/542382 | 19:44 |
| smcginnis | dhellmann: What has been the past convention - should I wait for a second +2 on the RC, or are these fine with single review? | 19:45 |
| dhellmann | smcginnis : regarding the magnum release failure, that's a service. did we change services to upload to pypi now? | 19:45 |
| dhellmann | smcginnis : a single review is ok | 19:45 |
| smcginnis | dhellmann: No, we left it open to the project, and I think they chose to go with publishing. | 19:46 |
| smcginnis | Let me double check to make sure there. | 19:46 |
| smcginnis | Hmm, deliverable type is service. Let me check project-config. | 19:46 |
| smcginnis | publish-to-pypi | 19:46 |
| smcginnis | Do we do two +2's for RC2 and on? I just seem to remember one point where you and Thierry were waiting on that. Or maybe it was just situation specific. | 19:48 |
| openstackgerrit | Eric Kao proposed openstack/releases master: Congress dashboard Q-rc1 and stable branch https://review.openstack.org/542384 | 19:50 |
| openstackgerrit | Miguel Lavalle proposed openstack/releases master: Neutron projects RC1 https://review.openstack.org/540476 | 19:51 |
| openstackgerrit | Lance Bragstad proposed openstack/releases master: Tag keystone rc1 https://review.openstack.org/542385 | 19:52 |
| openstackgerrit | Miguel Lavalle proposed openstack/releases master: Neutron projects RC1 https://review.openstack.org/540476 | 19:54 |
| dhellmann | I think we hold later release candidates a bit to try to avoid having a bunch of them | 19:56 |
| dhellmann | but I don't remember specific rules for doing 2 reviewers | 19:56 |
| smcginnis | OK, that works for me. Just wanted to make sure I was following whatever conventions you guys had in place before. | 19:57 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: update get_release_type to take a repo name https://review.openstack.org/542387 | 19:58 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add validation step for pypi permissions https://review.openstack.org/542388 | 19:58 |
| dhellmann | fungi : do we have a place in project-config where I could add logic like ^^ to prevent a repo from having the pypi publish job without granting permission first? | 19:58 |
| dhellmann | smcginnis : yeah, that's good. we should make notes about the review process changes in our process doc, too | 19:59 |
| smcginnis | Ooh, nice. | 19:59 |
| dhellmann | there's still a little bit of this stuff that's in our heads | 19:59 |
| smcginnis | Nothing that conflicts with info in the process doc, but some areas are a little grey inbetween what's listed in there. | 19:59 |
| *** armax has quit IRC | 20:02 | |
| prometheanfire | dhellmann: smcginnis I've recieved no FFEs today so far, so looking good? | 20:02 |
| smcginnis | prometheanfire: Yep, I believe so. | 20:02 |
| smcginnis | Nothing contentious that I've seen. | 20:03 |
| dhellmann | prometheanfire : what happened with osprofiler? https://review.openstack.org/541645 | 20:03 |
| *** armax has joined #openstack-release | 20:03 | |
| prometheanfire | dhellmann: they still haven't replied | 20:03 |
| dhellmann | ok | 20:04 |
| fungi | dhellmann: not sure off the top of my head whether we have a good place to prevent projects from adding a job without meeting some arbitrary set of conditions | 20:04 |
| fungi | we normally rely on human review for that | 20:04 |
| dhellmann | fungi : there's zuul-projects-checks.py and it does look at release jobs | 20:04 |
| fungi | oh, nice | 20:04 |
| dhellmann | I could add it there? | 20:04 |
| dhellmann | it's looking for dupes, but it would be an easy extension | 20:04 |
| fungi | looking | 20:04 |
| dhellmann | oh, although figuring out the sdist name is actually hard without source code | 20:05 |
| openstackgerrit | Merged openstack/releases master: Release Designate 6.0.0.0rc1 https://review.openstack.org/542303 | 20:05 |
| *** armax has quit IRC | 20:05 | |
| dhellmann | and that job isn't checking out all of the git repos, and probably shouldn't | 20:05 |
| dhellmann | so maybe the releases repo is the best place to check this | 20:05 |
| openstackgerrit | Merged openstack/releases master: Release Designate Dashboard 6.0.0.0rc1 https://review.openstack.org/542305 | 20:05 |
| *** armax has joined #openstack-release | 20:06 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add validation step for pypi permissions https://review.openstack.org/542388 | 20:06 |
| openstackgerrit | Merged openstack/releases master: Tagging trove for RC1 https://review.openstack.org/542242 | 20:06 |
| fungi | dhellmann: yeah, and also we normally expect projects to add the jobs they want up front, even if they may not yet have any content for the repository | 20:06 |
| dhellmann | right | 20:06 |
| dhellmann | ok, so we'll just leave it in the releases repo for now | 20:06 |
| fungi | that seems like a reasonable place for it | 20:07 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release Glance 16.0.0.0rc1 https://review.openstack.org/542313 | 20:07 |
| openstackgerrit | Merged openstack/releases master: Release Searchlight queens RC1 and stable branch https://review.openstack.org/542319 | 20:07 |
| fungi | if a team under release management requests a release and it would get pushed to pypi but they don't have registration/permission right there, then that's the time to catch it | 20:07 |
| fungi | if a project isn't under release management, it's up to their authors to care about (and address) that | 20:08 |
| smcginnis | Only problem I see is like this last one. Type was service, so we would have to take a peek at project-config to see that they had a publish-to-pypi job set up. | 20:08 |
| dhellmann | yep, I figured if we could extend that help to other teams we would but it doesn't seem like a good fit, so... | 20:08 |
| fungi | how much longer is that going to be the case though? | 20:09 |
| dhellmann | smcginnis : yeah, it's looking at the configured jobs, not the release-type | 20:09 |
| fungi | we're just at the final phase of logistical challenges before we start publishing services to pypi anyway, afaik | 20:09 |
| dhellmann | right now that patch requires any project with any job template with 'pypi' in the name to give openstackci permission to upload | 20:10 |
| dhellmann | if we change the server publishing job we can add that one to the list of things for it to check | 20:10 |
| fungi | mordred: looks like we've reached the one-month mark on https://github.com/dcrosta/keystone/issues/2 with no reply too | 20:11 |
| dhellmann | smcginnis : for that tripleo release that failed part way through, how do you want to handle recovery? try re-enqueueing the whole thing? running it by hand? | 20:12 |
| openstackgerrit | Merged openstack/releases master: Congress Q-rc1 and stable branch https://review.openstack.org/542382 | 20:13 |
| smcginnis | dhellmann: Oh right. I'm scattered today, did that tag some but not others? | 20:14 |
| dhellmann | I think so | 20:14 |
| dhellmann | let me run missing-releases on it | 20:14 |
| smcginnis | dhellmann: If we get it rerun, then it should skip over the ones that are already set, right? | 20:14 |
| dhellmann | yes | 20:15 |
| dhellmann | I'm getting an error in the tool: http://paste.openstack.org/show/666815/ | 20:16 |
| dhellmann | oh, maybe because it's asking pypi about a puppet module | 20:16 |
| smcginnis | That looks vaguely familiar. | 20:17 |
| smcginnis | I think I've hit that, but wasn't sure what was being returned that it that the GET succeeded but the json result was bad. | 20:18 |
| dhellmann | yeah, I think I have a work around. patch incoming | 20:22 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: update missing-releases to handle non-python projects better https://review.openstack.org/542412 | 20:22 |
| dhellmann | http://paste.openstack.org/show/666817/ | 20:24 |
| dhellmann | it looks like we are just missing 2 tags | 20:24 |
| smcginnis | So just the two. | 20:25 |
| dhellmann | so we could re-enqueue that job and try again and it should do what we need | 20:25 |
| smcginnis | No tags, so should be safe enough to just rerun... yeah. | 20:25 |
| dhellmann | assuming we don't get another timeout, of course | 20:25 |
| smcginnis | Let's hope seconds times a charm. | 20:25 |
| smcginnis | You want to ask? | 20:25 |
| dhellmann | I actually need to drop offline for a bit. I should be back in about 90 minutes. | 20:26 |
| prometheanfire | heres the currently untagged/branched queens stuff https://gist.github.com/prometheanfire/9449355352d97207aa85172cd9ef4b9f | 20:26 |
| smcginnis | dhellmann: ack | 20:26 |
| dhellmann | prometheanfire : that looks like it includes some things that aren't official projects | 20:27 |
| prometheanfire | dhellmann: ya, it's a bit rought now | 20:28 |
| dhellmann | prometheanfire : have a look at the list-deliverables command in the releases repo (tox -e venv -- list-deliverables --no-stable-branch) | 20:28 |
| openstackgerrit | Miguel Lavalle proposed openstack/releases master: Neutron projects RC1 https://review.openstack.org/540476 | 20:28 |
| prometheanfire | os-service-types, the heat-agents stuff and tricircle are the ones I'm paying the most attention to | 20:28 |
| smcginnis | here's the official project list: http://paste.openstack.org/show/666818/ | 20:28 |
| smcginnis | Several of those are out there right now. | 20:29 |
| prometheanfire | dhellmann: k, using this atm https://review.openstack.org/492382 | 20:29 |
| dhellmann | oh, wow, yeah, please look at list-deliverables | 20:30 |
| dhellmann | it does all sorts of queries like this | 20:30 |
| prometheanfire | k | 20:31 |
| openstackgerrit | Miguel Lavalle proposed openstack/releases master: Neutron projects RC1 https://review.openstack.org/540476 | 20:31 |
| dhellmann | it needs more documentation, like most of our tools | 20:32 |
| dhellmann | ok, really dropping off for a bit now | 20:32 |
| prometheanfire | k | 20:32 |
| openstackgerrit | Brian Rosmaita proposed openstack/releases master: Release Glance 16.0.0.0rc1 https://review.openstack.org/542313 | 20:39 |
| prometheanfire | dhellmann: that tool is so much cleaner | 20:40 |
| prometheanfire | list-deliverables --no-stable-branch --series queens --unreleased | 20:40 |
| prometheanfire | prety sure that's the command and output I should care about | 20:40 |
| prometheanfire | right smcginnis? | 20:40 |
| smcginnis | prometheanfire: Yep | 20:41 |
| smcginnis | My output I pasted was without the unreleased, just in case there were any that did do a release but have not done a branch. | 20:41 |
| prometheanfire | tripleo-quickstart should probably be cycle trailing with the rest of tripleo right? | 20:41 |
| prometheanfire | ah, ya, we do care about the branching | 20:42 |
| smcginnis | Probably. I don't really know the nature of -quickstart though, so maybe it doesn't need to be? | 20:43 |
| smcginnis | Oh, it is cycle-trailing. At least in the queens deliverable file. | 20:43 |
| prometheanfire | --missing-rc might be what I want | 20:45 |
| openstackgerrit | Merged openstack/releases master: update missing-releases to handle non-python projects better https://review.openstack.org/542412 | 20:45 |
| prometheanfire | rcs have to have branches right? or no? | 20:45 |
| smcginnis | prometheanfire: Potentially no. They _should_ include branches. But we've said if they are missing we will go back after the RC crunch and create them if the requestor did not include branching in their patch. | 20:46 |
| prometheanfire | ok, fairly sure the reqs unfreeze requires the branch | 20:47 |
| smcginnis | prometheanfire: OK, let's do a final check before that then. | 20:47 |
| prometheanfire | yarp | 20:47 |
| smcginnis | I think we've gotten all but maybe one with a branch. | 20:47 |
| prometheanfire | you happen to know if list-deliverables can output projects missing a particular branch> | 20:48 |
| smcginnis | Like --series pike --no-stable-branch? Or something different? | 20:49 |
| prometheanfire | oh ya, but for queens :D | 20:49 |
| smcginnis | Yep, that's what I ran for queens. ;) | 20:50 |
| prometheanfire | I guess I also want --model cycle-with-milestones | 20:54 |
| prometheanfire | any other models I should run? | 20:54 |
| smcginnis | Maybe cycle-with-intermediary? | 20:55 |
| prometheanfire | tonyb: ping? | 20:55 |
| prometheanfire | ya | 20:55 |
| tonyb | prometheanfire: I'm in and out as the kids are still breaking fast etc ... 'sup? | 20:56 |
| prometheanfire | tonyb: I want to make sure the output from list-deliverables is usable for requirements to decide when to unfreeze | 20:56 |
| prometheanfire | list-deliverables --no-stable-branch --series queens --model cycle-with-milestones | 20:56 |
| prometheanfire | list-deliverables --no-stable-branch --series queens --model cycle-with-intermediary | 20:57 |
| *** ChanServ has quit IRC | 20:57 | |
| prometheanfire | vs your script's output it also lists things like nova | 20:57 |
| smcginnis | tonyb: Example output: http://logs.openstack.org/32/32af7e52ebe56a47b62e2ab44a22b5c63fa521e0/pre-release/release-openstack-python/1ae06b6/job-output.txt.gz#_2018-02-08_20_29_08_979129 | 20:57 |
| smcginnis | Oh, fail. Ignore that. | 20:58 |
| tonyb | okay | 20:58 |
| smcginnis | tonyb: http://paste.openstack.org/show/666818/ | 20:58 |
| prometheanfire | smcginnis: well, my commands didn't have stuff like tripleo as they are cycle trailing | 20:58 |
| openstackgerrit | Merged openstack/releases master: Congress dashboard Q-rc1 and stable branch https://review.openstack.org/542384 | 20:59 |
| openstackgerrit | Merged openstack/releases master: Release Glance 16.0.0.0rc1 https://review.openstack.org/542313 | 20:59 |
| prometheanfire | tonyb: all that the tool would need to do is match the projects to their managing project and then we'd be good, prety sure we don't care about python-moganclient | 21:00 |
| tonyb | prometheanfire: So the main problem is thew data source IIRC list-deliverables only looks at (whatever we're calling big tent) projects where as requirements needs to look at everythign in projects.txt | 21:02 |
| smcginnis | Yeah, we only look at official projects. | 21:03 |
| tonyb | Also he hack I wrote is predictive whereas list-deliverables is "currect state" | 21:03 |
| smcginnis | So you do care about things like python-moganclient? | 21:03 |
| tonyb | so sure we could massage things but then we start to get way from what list-deliverables is designed for | 21:03 |
| prometheanfire | smcginnis: if they have a requiremnts sync job then we need to care | 21:04 |
| prometheanfire | I suppose the only thing we don't 'care' about are cycle trailing | 21:04 |
| tonyb | smcginnis: if it's in prpojects.txt we care about it *but* as time goes on we need to weigh the "were frozen 'till everybody is good" vs "OpenStack is stalled waiting for $project" | 21:04 |
| tonyb | prometheanfire: No we care about cycle-trainlin too just we have to handle them differently | 21:05 |
| prometheanfire | tonyb: ya, we just warn them | 21:05 |
| smcginnis | Especially if $project has two less than part time contributors. | 21:05 |
| prometheanfire | that's why the quotes | 21:05 |
| openstackgerrit | Merged openstack/releases master: Release Freezer 6.0.0.0rc1 https://review.openstack.org/542342 | 21:07 |
| openstackgerrit | Merged openstack/releases master: Release freezer-web-ui 6.0.0.0rc1 https://review.openstack.org/542346 | 21:07 |
| tonyb | smcginnis: Right it's a trade off so the tool I wrote was supposed to inform the decision rather than make it | 21:08 |
| prometheanfire | tonyb: the ungoverned ones are the ones hard to get to do anything | 21:09 |
| tonyb | prometheanfire: Sure | 21:09 |
| prometheanfire | tonyb: however, the tool doesn't show some things that list-deliverables does show | 21:09 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Release solum and cut stable/queens https://review.openstack.org/541994 | 21:09 |
| prometheanfire | like nova | 21:09 |
| smcginnis | Do we care about them? | 21:10 |
| smcginnis | :) | 21:10 |
| prometheanfire | I imagine we care about nova/keystone/etc | 21:10 |
| tonyb | prometheanfire: When we looed at it a coupld of days ago my sense was that if we took care of swift and ironci the overall view would be fine ... assuming we had ACKs from the cycyle-trailing projects | 21:10 |
| tonyb | prometheanfire: Right that's by design | 21:11 |
| prometheanfire | hmm, checking on an example (glance), it DOES have a queens branch, so list-deliverables doesn't check for pure branching | 21:11 |
| tonyb | prometheanfire: the tools *knows* that requiremenst wont branch until *all* the cycle-with-mailestones have also branched so they *can't* be a problem (that's the predictive part) | 21:11 |
| tonyb | I'm clearly confused about the aim here. | 21:12 |
| prometheanfire | tonyb: I'm trying to see if I can get any extra info from list-deliverables vs your tool, doesn't look like it atm | 21:12 |
| prometheanfire | the tool gives false positives for thigns that have a branch but no release it looks like | 21:13 |
| prometheanfire | and we only care about branching... so | 21:13 |
| prometheanfire | tonyb: anyway, atm, I'd like to see tricircle and heat-agents done https://paste.pound-python.org/show/QJW6TgeEHkiFljO5yUaz/ | 21:14 |
| *** iyamahat has quit IRC | 21:16 | |
| *** yamahata has quit IRC | 21:16 | |
| tonyb | prometheanfire: okay. That's pobably fair | 21:16 |
| openstackgerrit | Ben Swartzlander proposed openstack/releases master: Release Queens RC1 for Manila https://review.openstack.org/542376 | 21:17 |
| prometheanfire | even then, if no response by tomorrow afternoon my time I might move on to unfreeze (send out warning email to cycle-trailing) | 21:17 |
| prometheanfire | because those are minor things | 21:17 |
| * prometheanfire wants to be a bit more agressive about moving things forward | 21:19 | |
| prometheanfire | ok, sent email | 21:21 |
| prometheanfire | with the wrong subject, fml | 21:21 |
| tonyb | :( | 21:21 |
| smcginnis | That's kind of how my afternoon has been going. | 21:21 |
| tonyb | prometheanfire: have you had ACKs from cycle-trailing projects? | 21:21 |
| prometheanfire | no, that email is next | 21:22 |
| tonyb | okay | 21:22 |
| prometheanfire | puppet, tripleo, osa, kola, that's all of them | 21:23 |
| prometheanfire | easier at least | 21:23 |
| smcginnis | Have to drop off for awhile. Will be back online later. | 21:28 |
| prometheanfire | k | 21:29 |
| prometheanfire | ok, cycle-trailing email sent now too, (asking for acks) | 21:30 |
| *** dave-mccowan has quit IRC | 21:30 | |
| *** ChanServ has joined #openstack-release | 21:31 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 21:31 | |
| *** yamamoto has joined #openstack-release | 21:32 | |
| *** edmondsw has quit IRC | 21:38 | |
| *** dave-mccowan has joined #openstack-release | 21:47 | |
| *** ricolin has quit IRC | 21:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/releases master: queens: release nova 17.0.0.0rc1 https://review.openstack.org/542459 | 22:00 |
| mriedem | ^ no idea if the cycle highlights rst formatting will work in there | 22:00 |
| dhellmann | mriedem : that should be ok, but let's see what the build looks like | 22:03 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: update get_release_type to take a repo name https://review.openstack.org/542387 | 22:08 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add validation step for pypi permissions https://review.openstack.org/542388 | 22:08 |
| *** kumarmn has quit IRC | 22:16 | |
| *** Marx314 has joined #openstack-release | 22:44 | |
| *** claudiub|2 has joined #openstack-release | 22:48 | |
| *** claudiub has quit IRC | 22:50 | |
| *** wolverineav has quit IRC | 22:56 | |
| *** wolverineav has joined #openstack-release | 22:57 | |
| dhellmann | ttx, smcginnis : I've made notes about the release failures in the tracking etherpad under R-2 | 23:00 |
| smcginnis | Thanks dhellmann | 23:00 |
| *** wolverineav has quit IRC | 23:01 | |
| *** bobh has quit IRC | 23:06 | |
| smcginnis | dhellmann: Any thoughts on https://review.openstack.org/541544 | 23:06 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: searchlight-ui rc2 https://review.openstack.org/542476 | 23:07 |
| dhellmann | smcginnis : looking | 23:07 |
| dhellmann | oh, tacker | 23:07 |
| dhellmann | I have no idea what they're trying to do | 23:07 |
| dhellmann | until gongysh responds I guess we can't really do much | 23:08 |
| smcginnis | Yeah, I'm lost on that. I'm afraid to approve it until we get some kind of response on there. | 23:08 |
| dhellmann | maybe leave a -1 so it's clear we're not going to do anything with it | 23:08 |
| dhellmann | not everyone reads the emails from gerrit | 23:09 |
| smcginnis | Oh, good point. | 23:09 |
| dhellmann | also please have a look at that searchlight-ui rc2 in https://review.openstack.org/542476 if you have a minute | 23:09 |
| dhellmann | if you don't I'll self-approve it | 23:09 |
| dhellmann | don't have time that is :-) | 23:09 |
| smcginnis | Yep, I added myself to there to make sure I see the zuul results come in. | 23:09 |
| smcginnis | Not that I expect anything unusual there. I suppose I could just +w right away. | 23:09 |
| *** dave-mccowan has quit IRC | 23:12 | |
| *** kektrain_ has joined #openstack-release | 23:17 | |
| *** kektrain_ has left #openstack-release | 23:18 | |
| openstackgerrit | Miguel Lavalle proposed openstack/releases master: Neutron projects RC1 https://review.openstack.org/540476 | 23:19 |
| *** pbandark has quit IRC | 23:21 | |
| smcginnis | armax: Since Miguel isn't in here I'll check with you. Is that RC1 good to go? Or if I approve it is someone going to swoop in a -W it? :) | 23:21 |
| armax | We had agreed to wait until tomorrow morning ET | 23:22 |
| armax | if that’s ok with you | 23:22 |
| smcginnis | OK, that should be fine. Did someone -W that patch to make sure one of us doesn't come along and approve it? | 23:25 |
| *** hongbin has quit IRC | 23:26 | |
| openstackgerrit | Merged openstack/releases master: Release Queens RC1 for Manila https://review.openstack.org/542376 | 23:27 |
| smcginnis | dhellmann: I seem to remember an override for sphinx titles, but maybe I'm only thinking of anchor names. | 23:29 |
| *** annabelleB has quit IRC | 23:29 | |
| smcginnis | Looked a little in the toctree source and didn't see anything obvious, | 23:29 |
| dhellmann | yeah, there's a syntax to specify an alternate title when including something in the toctree | 23:30 |
| smcginnis | Would love to have the TOC have just the project name, but the content have title and description here https://releases.openstack.org/queens/highlights.html | 23:30 |
| *** annabelleB has joined #openstack-release | 23:30 | |
| smcginnis | Oh great. Is that in the title defintion or the toctree itself. | 23:30 |
| dhellmann | http://www.sphinx-doc.org/en/stable/markup/toctree.html?highlight=toctree#directive-toctree | 23:30 |
| dhellmann | under "Entries" | 23:30 |
| smcginnis | I know there's the "Fancy title <blah>" but I don't think that will work in this case. | 23:30 |
| dhellmann | oh | 23:31 |
| smcginnis | Ah, yeah. | 23:31 |
| dhellmann | oh, I see | 23:31 |
| dhellmann | no | 23:31 |
| dhellmann | that's not a toctree | 23:31 |
| smcginnis | I kinda need the inverse in a way. | 23:31 |
| dhellmann | sorry, I misunderstood | 23:31 |
| smcginnis | No, that's exactly where my queries led me. | 23:31 |
| smcginnis | I just think what I want doesn't exist unfortunately. | 23:31 |
| dhellmann | I'm not aware of a way to do what you want | 23:32 |
| smcginnis | Oh well. I think the highlights page looks good enough. Just one of those minor niggles. | 23:32 |
| *** kumarmn has joined #openstack-release | 23:32 | |
| smcginnis | Now looking at that page, how the heck does openstacksdk get sorted to the top? | 23:32 |
| dhellmann | is the name actually lower-case? | 23:33 |
| smcginnis | Oh well, gotta remember it's good enough for it's purpose. | 23:33 |
| smcginnis | Oh! | 23:33 |
| smcginnis | I think you're on to something there. | 23:33 |
| smcginnis | That might be one of the few that is not. | 23:33 |
| smcginnis | So sorting gets it on the top, then calling .title makes them all look that way. | 23:33 |
| *** Jeffrey4l has quit IRC | 23:34 | |
| dhellmann | yeah | 23:34 |
| dhellmann | let me see if I can fix that | 23:34 |
| smcginnis | Line 428 in sphinxext.py it appears. | 23:35 |
| dhellmann | we have released a lot of stuff, this build takes a while | 23:36 |
| smcginnis | Might have to look at some optimization in a couple cycles. | 23:36 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use case-insensitive sort for teams on highlights page https://review.openstack.org/542490 | 23:36 |
| *** kumarmn has quit IRC | 23:37 | |
| *** claudiub|2 has quit IRC | 23:37 | |
| smcginnis | Easy peasy | 23:37 |
| openstackgerrit | Merged openstack/releases master: Release solum and cut stable/queens https://review.openstack.org/541994 | 23:37 |
| smcginnis | OK, neutron has asked to release tomorrow, nova has a couple last patches going through. The rest are waiting for some input. | 23:38 |
| smcginnis | Stable patches all need stable review. | 23:38 |
| smcginnis | I think we're sitting pretty good for now. | 23:39 |
| dhellmann | yep | 23:39 |
| dhellmann | quick, let's drop offline before something else comes in! | 23:39 |
| smcginnis | Haha, yep! | 23:39 |
| smcginnis | dhellmann: So are you out starting tomorrow through next week? | 23:39 |
| * dhellmann puts down the tea mug and picks up the wine glass | 23:39 | |
| * smcginnis raises his glass to dhellmann | 23:40 | |
| dhellmann | yeah, I'll be around for the meeting tomorrow but then I'm on a plane for a weekend conference | 23:40 |
| dhellmann | I'm back home sunday night, and then PTO for a few days | 23:40 |
| dhellmann | I'll be online over the weekend except for sunday afternoon when I'm actually speaking | 23:40 |
| smcginnis | OK, sounds good. I hope you have fun and can relax a bit. | 23:40 |
| dhellmann | thanks! | 23:40 |
| dhellmann | I plan to try to make a dent in my reading pile | 23:41 |
| smcginnis | Good plan, that sounds like a nice break to me. | 23:41 |
| dhellmann | reading and fancy lunches. I live a wild life. | 23:42 |
| smcginnis | Haha | 23:43 |
| openstackgerrit | Merged openstack/releases master: searchlight-ui rc2 https://review.openstack.org/542476 | 23:44 |
| *** yamahata has joined #openstack-release | 23:45 | |
| openstackgerrit | Merged openstack/releases master: use case-insensitive sort for teams on highlights page https://review.openstack.org/542490 | 23:56 |
| openstackgerrit | Lance Bragstad proposed openstack/releases master: Tag keystone rc1 https://review.openstack.org/542385 | 23:57 |
| lbragstad | smcginnis: ^ that should be ready | 23:58 |
| lbragstad | we just landed our last couple patches that we needed to get for rc1 | 23:58 |
| dhellmann | smcginnis : the searchlight-ui rc2 is published | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!