| clarkb | smcginnis: that number is down from 500 something when we got networking worked around | 00:02 |
|---|---|---|
| clarkb | so progress I guess :( | 00:02 |
| *** claudiub|2 has quit IRC | 00:03 | |
| *** kumarmn has joined #openstack-release | 00:23 | |
| mriedem | dhellmann: is this right - i need to fix a release note added in pike, so the fix should go in the stable/pike branch, right? https://review.openstack.org/#/c/541498/ | 00:24 |
| jroll | mriedem: directly to stable/pike, yes: https://docs.openstack.org/reno/latest/user/usage.html#updating-stable-branch-release-notes | 00:25 |
| mriedem | thanks | 00:26 |
| smcginnis | yep | 00:27 |
| *** kumarmn has quit IRC | 00:29 | |
| *** hongbin has quit IRC | 00:35 | |
| *** zhurong has joined #openstack-release | 00:42 | |
| *** armax has quit IRC | 00:44 | |
| *** armax has joined #openstack-release | 00:47 | |
| *** iyamahat_ has joined #openstack-release | 00:56 | |
| openstackgerrit | Merged openstack/releases master: Release monasca-agent 2.2.2 https://review.openstack.org/539862 | 00:57 |
| openstackgerrit | Michael Johnson proposed openstack/releases master: Release Octavia projects for Queens RC1 https://review.openstack.org/541092 | 00:58 |
| *** iyamahat has quit IRC | 00:59 | |
| *** iyamahat has joined #openstack-release | 01:07 | |
| *** iyamahat_ has quit IRC | 01:07 | |
| *** mlavalle has quit IRC | 01:24 | |
| *** kumarmn has joined #openstack-release | 01:27 | |
| *** kumarmn has quit IRC | 02:04 | |
| *** armax has quit IRC | 02:09 | |
| *** hongbin has joined #openstack-release | 02:18 | |
| *** mriedem has quit IRC | 02:24 | |
| *** kumarmn has joined #openstack-release | 02:30 | |
| *** kumarmn has quit IRC | 02:45 | |
| *** kumarmn has joined #openstack-release | 02:47 | |
| *** yamamoto has joined #openstack-release | 02:52 | |
| *** yamamoto has quit IRC | 02:55 | |
| *** yamamoto has joined #openstack-release | 02:56 | |
| *** zhenguo has joined #openstack-release | 03:01 | |
| *** kumarmn has quit IRC | 03:07 | |
| *** kumarmn has joined #openstack-release | 03:12 | |
| *** armax has joined #openstack-release | 03:16 | |
| prometheanfire | smcginnis: mind looking at https://review.openstack.org/541305 | 03:19 |
| *** annabelleB has quit IRC | 03:22 | |
| *** ricolin has joined #openstack-release | 03:24 | |
| *** kumarmn has quit IRC | 03:24 | |
| prometheanfire | dhellmann: thanks | 03:26 |
| *** sree has joined #openstack-release | 03:30 | |
| *** coolsvap has joined #openstack-release | 03:36 | |
| *** kumarmn has joined #openstack-release | 03:37 | |
| openstackgerrit | Merged openstack/releases master: release oslo.serialization and oslo.privsep with updated msgpack dependency https://review.openstack.org/541305 | 03:47 |
| *** ianychoi_ has quit IRC | 03:49 | |
| *** ianychoi_ has joined #openstack-release | 03:50 | |
| prometheanfire | smcginnis: comment 14 is correct? https://bugs.launchpad.net/keystone/+bug/1743445 | 03:50 |
| openstack | Launchpad bug 1743445 in oslo.serialization "Switch from msgpack-python to msgpack" [Undecided,In progress] - Assigned to Matthew Thode (prometheanfire) | 03:50 |
| *** armax has quit IRC | 03:51 | |
| *** udesale has joined #openstack-release | 03:56 | |
| *** kumarmn has quit IRC | 03:57 | |
| *** kumarmn has joined #openstack-release | 03:57 | |
| *** abelur_ has quit IRC | 03:58 | |
| *** abelur has joined #openstack-release | 03:58 | |
| *** openstackgerrit has quit IRC | 04:04 | |
| *** kumarmn has quit IRC | 04:04 | |
| *** kumarmn has joined #openstack-release | 04:05 | |
| *** kumarmn has quit IRC | 04:09 | |
| *** rosmaita has quit IRC | 04:14 | |
| *** ykarel|away has joined #openstack-release | 04:30 | |
| *** harlowja has quit IRC | 04:32 | |
| *** iyamahat has quit IRC | 04:38 | |
| *** hongbin has quit IRC | 05:02 | |
| *** openstackgerrit has joined #openstack-release | 05:10 | |
| openstackgerrit | Rico Lin proposed openstack/releases master: Remove branch heat-tempest-plugin https://review.openstack.org/540269 | 05:10 |
| *** zhenguo has quit IRC | 05:10 | |
| *** claudiub|2 has joined #openstack-release | 05:22 | |
| openstackgerrit | gongysh proposed openstack/releases master: Add tacker server project in queens release https://review.openstack.org/541544 | 05:24 |
| *** armstrong has joined #openstack-release | 05:36 | |
| *** harlowja has joined #openstack-release | 05:37 | |
| *** zhurong has quit IRC | 05:39 | |
| *** zhurong has joined #openstack-release | 05:53 | |
| *** harlowja has quit IRC | 06:52 | |
| *** udesale has quit IRC | 07:02 | |
| *** udesale has joined #openstack-release | 07:05 | |
| *** udesale has quit IRC | 07:06 | |
| *** udesale has joined #openstack-release | 07:06 | |
| *** ykarel|away is now known as ykarel | 07:07 | |
| *** alexchadin has joined #openstack-release | 07:10 | |
| *** sree has quit IRC | 07:28 | |
| *** jtomasek has joined #openstack-release | 07:40 | |
| *** slaweq has joined #openstack-release | 07:43 | |
| *** slaweq has quit IRC | 07:44 | |
| *** slaweq has joined #openstack-release | 07:44 | |
| *** armstrong has quit IRC | 07:45 | |
| *** udesale_ has joined #openstack-release | 07:48 | |
| *** pabelanger has quit IRC | 07:50 | |
| *** udesale has quit IRC | 07:51 | |
| *** otherwiseguy_ has quit IRC | 07:51 | |
| *** sree has joined #openstack-release | 08:01 | |
| *** otherwiseguy has joined #openstack-release | 08:02 | |
| *** pabelanger has joined #openstack-release | 08:02 | |
| *** alexchadin has quit IRC | 08:04 | |
| *** slaweq has quit IRC | 08:06 | |
| *** slaweq has joined #openstack-release | 08:07 | |
| *** slaweq has quit IRC | 08:11 | |
| *** amoralej|off is now known as amoralej | 08:13 | |
| *** ianychoi_ has quit IRC | 08:14 | |
| *** alexchadin has joined #openstack-release | 08:16 | |
| *** ianychoi_ has joined #openstack-release | 08:19 | |
| *** zhurong has quit IRC | 08:25 | |
| *** zhurong has joined #openstack-release | 08:26 | |
| *** ianychoi_ has quit IRC | 08:28 | |
| *** ianychoi_ has joined #openstack-release | 08:30 | |
| *** pcaruana has joined #openstack-release | 08:38 | |
| *** alexchadin has quit IRC | 08:39 | |
| *** ianychoi_ has quit IRC | 08:45 | |
| *** ianychoi_ has joined #openstack-release | 08:46 | |
| *** jpich has joined #openstack-release | 08:59 | |
| *** kumarmn has joined #openstack-release | 09:05 | |
| *** kumarmn has quit IRC | 09:10 | |
| *** slaweq has joined #openstack-release | 09:31 | |
| *** slaweq has quit IRC | 09:38 | |
| *** slaweq has joined #openstack-release | 09:38 | |
| openstackgerrit | gongysh proposed openstack/releases master: bump client version to meet tacker server needs https://review.openstack.org/541638 | 09:40 |
| *** slaweq has quit IRC | 09:43 | |
| *** ykarel is now known as ykarel|lunch | 09:54 | |
| *** pcaruana has quit IRC | 09:57 | |
| *** alexchadin has joined #openstack-release | 10:00 | |
| openstackgerrit | Tovin Seven proposed openstack/releases master: Add osprofiler release for stable/queens https://review.openstack.org/541645 | 10:01 |
| openstackgerrit | Merged openstack/releases master: don't allow branching for tempest plugins https://review.openstack.org/541409 | 10:02 |
| openstackgerrit | Tovin Seven proposed openstack/releases master: Add osprofiler release for stable/queens https://review.openstack.org/541645 | 10:02 |
| openstackgerrit | Merged openstack/releases master: implicitly set type to tempest-plugin based on the name https://review.openstack.org/541410 | 10:03 |
| *** eumel8 has joined #openstack-release | 10:11 | |
| *** pcaruana has joined #openstack-release | 10:13 | |
| *** strigazi_ has joined #openstack-release | 10:16 | |
| *** strigazi has quit IRC | 10:19 | |
| *** strigazi_ is now known as strigazi | 10:19 | |
| *** ykarel|lunch is now known as ykarel | 10:20 | |
| *** ianychoi_ has quit IRC | 10:21 | |
| *** ianychoi_ has joined #openstack-release | 10:24 | |
| *** dtantsur|afk is now known as dtantsur | 10:26 | |
| openstackgerrit | Thomas Bechtold proposed openstack/releases master: pymod2pkg: Release 0.13.0 https://review.openstack.org/541657 | 10:26 |
| openstackgerrit | Merged openstack/releases master: use repository-settings to get a default list of repo names https://review.openstack.org/540520 | 10:26 |
| *** gcb has quit IRC | 10:29 | |
| *** yamahata has quit IRC | 10:34 | |
| *** zhurong has quit IRC | 10:35 | |
| openstackgerrit | Merged openstack/releases master: Add or update pike release notes links https://review.openstack.org/540515 | 10:40 |
| openstackgerrit | Merged openstack/releases master: Add release notes link for branched projects https://review.openstack.org/540512 | 10:40 |
| *** udesale_ is now known as udesale | 10:41 | |
| openstackgerrit | Merged openstack/releases master: Release panko 4.0.0 for Queens https://review.openstack.org/540699 | 10:45 |
| openstackgerrit | Merged openstack/releases master: Release aodh 6.0.0 for Queens https://review.openstack.org/540698 | 10:45 |
| *** lucas-afk is now known as lucasagomes | 10:45 | |
| *** sree has quit IRC | 10:52 | |
| *** sree has joined #openstack-release | 10:53 | |
| openstackgerrit | Thierry Carrez proposed openstack/releases master: Sort team names in highlight output https://review.openstack.org/540470 | 10:54 |
| *** slaweq has joined #openstack-release | 10:55 | |
| *** slaweq has quit IRC | 10:56 | |
| *** stevebaker has quit IRC | 10:56 | |
| *** slaweq has joined #openstack-release | 10:57 | |
| *** sree has quit IRC | 10:57 | |
| *** slaweq has quit IRC | 10:59 | |
| *** slaweq__ has joined #openstack-release | 10:59 | |
| *** slaweq__ has quit IRC | 11:04 | |
| *** stevebaker has joined #openstack-release | 11:04 | |
| *** ianychoi_ has quit IRC | 11:05 | |
| *** ianychoi_ has joined #openstack-release | 11:07 | |
| openstackgerrit | Merged openstack/releases master: Sort team names in highlight output https://review.openstack.org/540470 | 11:09 |
| *** pcaruana has quit IRC | 11:09 | |
| *** udesale has quit IRC | 11:30 | |
| *** slaweq has joined #openstack-release | 11:51 | |
| *** slaweq has quit IRC | 11:56 | |
| *** ykarel has quit IRC | 11:59 | |
| *** ykarel has joined #openstack-release | 12:00 | |
| *** ianychoi_ has quit IRC | 12:00 | |
| *** edmondsw has joined #openstack-release | 12:02 | |
| *** pkovar has joined #openstack-release | 12:10 | |
| *** wolverineav has joined #openstack-release | 12:10 | |
| *** pcaruana has joined #openstack-release | 12:13 | |
| *** dave-mcc_ has joined #openstack-release | 12:16 | |
| *** alexchadin has quit IRC | 12:16 | |
| *** alexchadin has joined #openstack-release | 12:16 | |
| *** yamamoto has quit IRC | 12:17 | |
| *** yamamoto has joined #openstack-release | 12:17 | |
| *** alexchadin has quit IRC | 12:21 | |
| *** alexchadin has joined #openstack-release | 12:21 | |
| *** pkovar has quit IRC | 12:28 | |
| smcginnis | prometheanfire: Regarding comment - yes, I believe that is correct. | 12:33 |
| smcginnis | We got another git command hang and timeout: | 12:47 |
| smcginnis | http://logs.openstack.org/75/751450b7b55719fd0b37c02c651097ba75917b38/release-post/tag-releases/a2678e5/job-output.txt.gz#_2018-02-07_11_14_42_697495 | 12:47 |
| *** slaweq_ has joined #openstack-release | 12:50 | |
| *** slaweq_ has quit IRC | 12:50 | |
| *** slaweq has joined #openstack-release | 12:51 | |
| *** alexchadin has quit IRC | 12:53 | |
| *** alexchadin has joined #openstack-release | 12:54 | |
| *** armax has joined #openstack-release | 12:54 | |
| *** sree has joined #openstack-release | 12:58 | |
| *** udesale has joined #openstack-release | 12:58 | |
| *** alexchadin has quit IRC | 12:58 | |
| *** slaweq has quit IRC | 13:02 | |
| *** slaweq has joined #openstack-release | 13:03 | |
| *** rosmaita has joined #openstack-release | 13:06 | |
| smcginnis | ttx: Looks like you got a lot taken care of this morning. Anything we need to be aware of other than this failed release job? | 13:07 |
| *** slaweq has quit IRC | 13:08 | |
| ttx | That tag fail was a timeout processing http://git.openstack.org/cgit/openstack/releases/commit/?id=751450b7b55719fd0b37c02c651097ba75917b38 | 13:08 |
| ttx | which should result in no tags. | 13:08 |
| ttx | so probably safe to ignore | 13:09 |
| smcginnis | ttx: I think the branching part was skipped. But I suppose it will pick it up on the next one if anything was missed. | 13:09 |
| ttx | it made it to tripleo-ui for the record :) | 13:09 |
| ttx | it should not triffer any branch | 13:10 |
| ttx | gg | 13:10 |
| smcginnis | Oh, it was that patch. I finally see. Yeah, safe to ignore. | 13:10 |
| ttx | failed after 66 files | 13:10 |
| ttx | smcginnis: only new thing is a late request re: tacker | 13:11 |
| ttx | python-tackerclient bunp | 13:11 |
| smcginnis | Any sign of a FFE from them for that? | 13:11 |
| *** pkovar has joined #openstack-release | 13:11 | |
| ttx | I told gongysh earlier in privmsg to expect a need for FFE | 13:12 |
| ttx | I was unsure they would need a requirements FFE | 13:12 |
| *** pkovar has quit IRC | 13:12 | |
| ttx | see https://review.openstack.org/#/c/541638/ | 13:12 |
| *** pkovar has joined #openstack-release | 13:13 | |
| smcginnis | OK, watching that. Hopefully they respond soon. | 13:13 |
| *** sree has quit IRC | 13:14 | |
| *** sree has joined #openstack-release | 13:14 | |
| *** sree has quit IRC | 13:14 | |
| *** sree has joined #openstack-release | 13:15 | |
| *** sree has quit IRC | 13:16 | |
| *** sree has joined #openstack-release | 13:17 | |
| ttx | gongysh is back on IRC if you want to ask him | 13:19 |
| *** sree has quit IRC | 13:21 | |
| *** kumarmn has joined #openstack-release | 13:22 | |
| *** kumarmn has quit IRC | 13:27 | |
| *** amoralej is now known as amoralej|lunch | 13:35 | |
| *** sree has joined #openstack-release | 13:38 | |
| *** lvdombrkr has joined #openstack-release | 13:39 | |
| *** coolsvap has quit IRC | 13:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/releases master: Release ironic-inspector 7.1.0 (final) for Queens https://review.openstack.org/541718 | 13:42 |
| *** sree has quit IRC | 13:47 | |
| *** alexchadin has joined #openstack-release | 13:54 | |
| *** alexchadin has quit IRC | 13:55 | |
| *** alexchadin has joined #openstack-release | 13:56 | |
| *** alexchadin has quit IRC | 13:56 | |
| *** alexchadin has joined #openstack-release | 13:56 | |
| *** alexchadin has quit IRC | 13:57 | |
| *** alexchadin has joined #openstack-release | 13:57 | |
| *** alexchadin has quit IRC | 13:57 | |
| *** alexchadin has joined #openstack-release | 13:58 | |
| *** yamahata has joined #openstack-release | 13:58 | |
| *** alexchadin has quit IRC | 13:58 | |
| *** alexchadin has joined #openstack-release | 13:59 | |
| *** alexchadin has quit IRC | 13:59 | |
| *** alexchadin has joined #openstack-release | 14:02 | |
| dhellmann | o/ | 14:05 |
| *** iyamahat has joined #openstack-release | 14:06 | |
| ttx | dhellmann: I may have triggered yet another merge conflict on your long series | 14:09 |
| openstackgerrit | gongysh proposed openstack/releases master: bump client version to meet tacker server needs https://review.openstack.org/541638 | 14:11 |
| dhellmann | ttx: yeah, I'm working on that now | 14:16 |
| dhellmann | it's fine, some of those patches still aren't passing the validation jobs | 14:16 |
| *** ianychoi has joined #openstack-release | 14:17 | |
| openstackgerrit | gongysh proposed openstack/releases master: bump client version to meet tacker server needs https://review.openstack.org/541638 | 14:20 |
| *** gongysh has joined #openstack-release | 14:22 | |
| *** kumarmn has joined #openstack-release | 14:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/releases master: TripleO ocata release https://review.openstack.org/541724 | 14:22 |
| gongysh | dhellmann, ttx https://review.openstack.org/#/c/541638/ | 14:23 |
| gongysh | please have a look at this patch which request to release a new version of client. it add reno notes during the queens and a new client command for a new tacker feature. | 14:24 |
| openstackgerrit | Emilien Macchi proposed openstack/releases master: TripleO pike release https://review.openstack.org/541725 | 14:25 |
| dhellmann | gongysh : the deadline for client releases for the queens cycle was 2 weeks ago. https://releases.openstack.org/queens/schedule.html | 14:25 |
| gongysh | dhellmann, can we see a new version release of queens? | 14:25 |
| gongysh | can we see it as a newer version release? | 14:26 |
| dhellmann | there are a couple of ways to handle it | 14:26 |
| dhellmann | you could ask the requirements team for a FFE to update the constraints list for the new release and if they approve we can talk about releasing it for queens | 14:27 |
| dhellmann | or you could wait for rocky to open after the PTG, and release it then | 14:27 |
| dhellmann | are the changes you need on the stable/queens branch? | 14:27 |
| dhellmann | yes, they look like it | 14:27 |
| gongysh | dhellmann, we hope so. | 14:28 |
| dhellmann | based on http://logs.openstack.org/38/541638/3/check/releases-tox-list-changes/8c2f2cf/tox/list-changes-results.log | 14:28 |
| dhellmann | since that deliverable doesn't have the tag indicating that it follows the stable policy, that is probably ok, even though it isn't ideal | 14:28 |
| dhellmann | it looks like mistral, tacker, and tacker-horizon are the projects that use the client | 14:29 |
| dhellmann | gongysh: is the client being tested against mistral while it is being developed? | 14:30 |
| dhellmann | how critical is it to release this right now? can it wait? the reason we have a deadline and release freeze period is because we want to stabilize the testing environment for the service projects, so we have to consider each release carefully. | 14:31 |
| gongysh | dhellmann, this version upgrade does not hurt mistral, it is a new feature addition. no other world than tacker team knows how to use it. | 14:31 |
| dhellmann | if we wait to release it, would that have any impact on the other tacker deliverables? does the horizon plugin use it, for example? | 14:31 |
| dhellmann | I guess it can't, since it isn't released. | 14:31 |
| *** slaweq has joined #openstack-release | 14:32 | |
| *** slaweq has quit IRC | 14:32 | |
| *** slaweq has joined #openstack-release | 14:33 | |
| dhellmann | gongysh : I see that ttx and smcginnis have replied on the original patch suggesting a course of action for you to take. | 14:33 |
| *** lucasagomes is now known as lucas-hungry | 14:35 | |
| gongysh | dhellmann, tacker team can exclude this feature from queens release if release team have some concern. I am sorry for the this. | 14:36 |
| *** slaweq has quit IRC | 14:37 | |
| dhellmann | gongysh : I do not think you need to remove the feature from tacker. We only need to make sure that when we release the client it does not cause disruption. smcginnis has suggested that you start a mailing list thread (please see the review comments on https://review.openstack.org/#/c/541638/) so I think that is your next step. | 14:37 |
| gongysh | dhellmann, I will try. thanks | 14:38 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add a warning for repo used in release but not in repository-settings https://review.openstack.org/540521 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only check release jobs if there are new releases to tag https://review.openstack.org/540975 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: make validate report more verbose https://review.openstack.org/541011 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only validate releases that have not yet been tagged https://review.openstack.org/541012 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: by default skip projects that have had tags applied https://review.openstack.org/541492 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 14:41 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 14:41 |
| gongysh | dhellmann, sorry, the maillist is openstack or openstack-dev list? | 14:42 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: Add cycle-highlights for openstacksdk for queens https://review.openstack.org/540173 | 14:43 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: expand on openstacksdk cycle highlights https://review.openstack.org/541732 | 14:43 |
| *** alexchadin has quit IRC | 14:44 | |
| rosmaita | smcginnis i think i need to do point releases for glance stable/pike and stable/ocata so the release includes the eventlet fix ... two questions (1) do i need a release note, and (2) do you want me to hold off until next week? | 14:45 |
| *** amoralej|lunch is now known as amoralej | 14:46 | |
| dhellmann | smcginnis , ttx: prometheanfire, toabctl , and I discussed https://review.openstack.org/#/c/541657/1 yesterday and agreed that we could go ahead with the release while toabctl works on removing pymod2pkg and renderspec from the requirements lists | 14:47 |
| dhellmann | gongysh : the openstack-dev list is where we discuss all contributor issues like this | 14:47 |
| gongysh | dhellmann, got it. | 14:49 |
| openstackgerrit | Merged openstack/releases master: release-test 0.12.0 https://review.openstack.org/541415 | 14:51 |
| *** armax has quit IRC | 15:00 | |
| *** armax has joined #openstack-release | 15:00 | |
| smcginnis | rosmaita: It doesn't sound like you need a release note, but you can decide if you think it's worth raising to the attention of deployers. Either works for me. | 15:05 |
| rosmaita | smcginnis ty | 15:05 |
| smcginnis | rosmaita: You could propose stable releases now. As long as there are not gate issues and we are not overloaded with queens releases, I think we can get that out. | 15:06 |
| smcginnis | Worse case scenario, they sit there ready until next week. | 15:06 |
| smcginnis | rosmaita: Oh, actually... | 15:06 |
| smcginnis | rosmaita: Since glance is stable:follows-policy, we will need to wait to get stable team OK on it, if possible. | 15:06 |
| smcginnis | rosmaita: So the release would like wait until early next week unless they have a chance to take a look earlier. | 15:07 |
| smcginnis | But Monday's are the targetted stable release review days. | 15:07 |
| rosmaita | smcginnis thanks, i'll wait to do it until after RC but before monday | 15:07 |
| *** shardy has joined #openstack-release | 15:08 | |
| smcginnis | rosmaita: Oh, to be clear, you can propose it at any time. I just want to be clear about when it might actually get processed. So any time you are ready is good. | 15:09 |
| rosmaita | smcginnis got it, thanks | 15:09 |
| *** yamamoto has quit IRC | 15:10 | |
| *** sree has joined #openstack-release | 15:14 | |
| *** coolsvap has joined #openstack-release | 15:17 | |
| *** gongysh has quit IRC | 15:23 | |
| *** mriedem has joined #openstack-release | 15:26 | |
| *** yamamoto has joined #openstack-release | 15:26 | |
| *** sree_ has joined #openstack-release | 15:27 | |
| *** sree_ is now known as Guest53871 | 15:27 | |
| *** yamamoto has quit IRC | 15:30 | |
| *** sree has quit IRC | 15:30 | |
| *** lucas-hungry is now known as lucasagomes | 15:32 | |
| openstackgerrit | Merged openstack/releases master: Release ironic-inspector 7.1.0 (final) for Queens https://review.openstack.org/541718 | 15:33 |
| *** Guest53871 has quit IRC | 15:41 | |
| *** sree has joined #openstack-release | 15:42 | |
| ttx | dhellmann: +2a | 15:44 |
| *** sree has quit IRC | 15:48 | |
| *** slaweq_ has joined #openstack-release | 15:58 | |
| openstackgerrit | Merged openstack/releases master: pymod2pkg: Release 0.13.0 https://review.openstack.org/541657 | 16:01 |
| *** slaweq_ has quit IRC | 16:03 | |
| *** pcaruana has quit IRC | 16:03 | |
| *** hongbin has joined #openstack-release | 16:07 | |
| *** annabelleB has joined #openstack-release | 16:10 | |
| *** sree has joined #openstack-release | 16:12 | |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release Monasca and cut stable/queens https://review.openstack.org/541767 | 16:13 |
| *** ykarel has quit IRC | 16:14 | |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release Monasca and cut stable/queens https://review.openstack.org/541767 | 16:14 |
| *** sree has quit IRC | 16:17 | |
| *** sree has joined #openstack-release | 16:18 | |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release monasca-kibana-plugin, cut stable/queens https://review.openstack.org/541776 | 16:20 |
| *** sree has quit IRC | 16:25 | |
| *** ricolin has quit IRC | 16:30 | |
| openstackgerrit | John Dickinson proposed openstack/releases master: swift stable/queens branch https://review.openstack.org/541784 | 16:35 |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release Monasca and cut stable/queens https://review.openstack.org/541767 | 16:42 |
| *** eumel8 has quit IRC | 16:46 | |
| openstackgerrit | Ifat Afek proposed openstack/releases master: Release vitrage-dashboard 1.4.1 https://review.openstack.org/541793 | 16:47 |
| *** yamamoto has joined #openstack-release | 16:50 | |
| *** yamamoto has quit IRC | 16:57 | |
| *** shardy has quit IRC | 17:00 | |
| *** lvdombrkr has quit IRC | 17:02 | |
| *** harlowja has joined #openstack-release | 17:11 | |
| *** sree has joined #openstack-release | 17:13 | |
| *** annabelleB has quit IRC | 17:13 | |
| *** udesale has quit IRC | 17:13 | |
| *** mriedem has quit IRC | 17:14 | |
| *** mriedem has joined #openstack-release | 17:15 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add a warning for repo used in release but not in repository-settings https://review.openstack.org/540521 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only check release jobs if there are new releases to tag https://review.openstack.org/540975 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: make validate report more verbose https://review.openstack.org/541011 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only validate releases that have not yet been tagged https://review.openstack.org/541012 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: by default skip projects that have had tags applied https://review.openstack.org/541492 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 17:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 17:16 |
| *** harlowja has quit IRC | 17:17 | |
| *** annabelleB has joined #openstack-release | 17:18 | |
| *** armstrong has joined #openstack-release | 17:18 | |
| openstackgerrit | Witold Bedyk proposed openstack/releases master: Release monasca-kibana-plugin, cut stable/queens https://review.openstack.org/541776 | 17:19 |
| *** sree has quit IRC | 17:20 | |
| *** lvdombrkr has joined #openstack-release | 17:27 | |
| *** dtantsur is now known as dtantsur|afk | 17:33 | |
| *** jpich has quit IRC | 17:37 | |
| dhellmann | smcginnis , ttx, tonyb : maybe we can tackle that series starting at https://review.openstack.org/#/c/540521/ this week, before all of the RC1 tags and branches start coming in? | 17:41 |
| openstackgerrit | Merged openstack/releases master: swift stable/queens branch https://review.openstack.org/541784 | 17:42 |
| smcginnis | dhellmann: OK, will do. | 17:43 |
| dhellmann | smcginnis : it looks like there's a failure I still need to work on | 17:43 |
| dhellmann | but you could start at the bottom anyway | 17:43 |
| smcginnis | It's a good place to start. ;) | 17:43 |
| *** yamahata has quit IRC | 17:47 | |
| *** iyamahat has quit IRC | 17:47 | |
| * ttx rereviews dhellmann-s list | 17:50 | |
| *** ricolin has joined #openstack-release | 17:52 | |
| ttx | oops -1ed the first one :) | 17:52 |
| smcginnis | Read that comment and just saw warning and warning. Oops. | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 17:54 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: skip list-changes for retires repositories https://review.openstack.org/541820 | 17:54 |
| ttx | smcginnis: need to call it a day, any urgent review ? | 17:55 |
| dhellmann | I want to rewrite all of this validation stuff to use the deliverable module instead of having to remember the nested data structure | 17:55 |
| smcginnis | ttx: No, I think we're in good enough shape right now. Thanks. | 17:55 |
| smcginnis | dhellmann: ++ | 17:56 |
| armstrong | @scmginnis hello | 17:56 |
| smcginnis | Hey armstrong. o/ | 17:56 |
| armstrong | Is there any way I can help to contribute to the release team ? | 17:56 |
| armstrong | I will like to get started | 17:56 |
| smcginnis | armstrong: Great! There's a bit of a learning curve to some of the automation, but dhellmann has a series of patches up right now that could use some extra eyes looking for any issues. | 17:57 |
| smcginnis | armstrong: That starts with this patch: https://review.openstack.org/#/c/540521/ | 17:57 |
| armstrong | Ok great! | 17:57 |
| smcginnis | armstrong: Other than that, there's a review dashboard link at the top of https://etherpad.openstack.org/p/queens-relmgt-tracking | 17:58 |
| smcginnis | armstrong: You can take a look at any of the release requests. The interesting job output is the list-changes one usually. | 17:58 |
| armstrong | Ok | 17:59 |
| smcginnis | armstrong: And usually the main thing there is just checking that they are following semver rules for what they are using for the requested version. | 17:59 |
| smcginnis | armstrong: A good reference for that is here: https://semver.org/ | 17:59 |
| smcginnis | armstrong: But dhellmann's patches may be a better place to start for now. ;) | 18:00 |
| armstrong | @smcginnis: Thanks, I now have a good starting point. | 18:00 |
| *** iyamahat has joined #openstack-release | 18:01 | |
| ttx | "a bit of a learning curve" is like best understatement of year 2018 so far | 18:01 |
| smcginnis | armstrong: Great, feel free to ask if you come up with any questions. | 18:01 |
| smcginnis | ttx: ;) | 18:01 |
| armstrong | @ttx: hahaha what’s your opinion about the “learning curve?” | 18:02 |
| smcginnis | There's just a lot there and it's all just magic until you start to dig into it and understand what's all going on. | 18:04 |
| armstrong | Ok! I will dig deep | 18:05 |
| dhellmann | each individual step is relatively simple. understanding how the steps chain together that can be complex. | 18:05 |
| armstrong | Is there any kind of house training or something like that? | 18:06 |
| dhellmann | smcginnis : what did we do for you? I don't remember if I wrote something or if we just went through it at the ptg | 18:07 |
| dhellmann | it's time for lunch here, bbiab | 18:08 |
| smcginnis | dhellmann: I think you pointed me at PROCESS.rst and said good luck. :P | 18:08 |
| smcginnis | dhellmann: Maybe a little more than that. A little. | 18:08 |
| dhellmann | wow that wasn't very nice of me :-) | 18:08 |
| smcginnis | dhellmann: Nah, you were very good answering the many questions I came up with on the way. ;) | 18:09 |
| *** ricolin has quit IRC | 18:09 | |
| smcginnis | Really, for the most part I think reviewing patches and looking into failures helps to start to absorb how things all tie together. | 18:10 |
| *** annabelleB has quit IRC | 18:10 | |
| *** harlowja has joined #openstack-release | 18:11 | |
| *** annabelleB has joined #openstack-release | 18:13 | |
| *** mriedem has quit IRC | 18:14 | |
| *** harlowja has quit IRC | 18:18 | |
| *** sree has joined #openstack-release | 18:23 | |
| *** sree has quit IRC | 18:27 | |
| *** annabelleB has quit IRC | 18:57 | |
| *** annabelleB has joined #openstack-release | 18:59 | |
| *** amoralej is now known as amoralej|off | 19:03 | |
| *** iyamahat_ has joined #openstack-release | 19:05 | |
| *** iyamahat has quit IRC | 19:05 | |
| *** sree has joined #openstack-release | 19:08 | |
| *** sree has quit IRC | 19:12 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 19:13 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: skip list-changes for retired repositories https://review.openstack.org/541820 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 19:15 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 19:15 |
| openstackgerrit | gordon chung proposed openstack/releases master: ceilometer 10.0.0 https://review.openstack.org/541845 | 19:23 |
| *** pkovar has quit IRC | 19:26 | |
| *** iyamahat_ has quit IRC | 19:29 | |
| *** iyamahat has joined #openstack-release | 19:30 | |
| *** yamahata has joined #openstack-release | 19:32 | |
| *** lucasagomes is now known as lucas-afk | 19:33 | |
| *** bobh has joined #openstack-release | 19:35 | |
| *** mriedem has joined #openstack-release | 19:35 | |
| *** sree has joined #openstack-release | 19:39 | |
| *** coolsvap has quit IRC | 19:41 | |
| *** sree has quit IRC | 19:44 | |
| smcginnis | dhellmann: As far as getting those changes in before the RC1 deluge, should I just push https://review.openstack.org/#/c/540521/ and we follow up as you commented? | 19:51 |
| smcginnis | dhellmann: I assume Thierry will not be back until early tomorrow our morning. | 19:51 |
| dhellmann | the follow up is there now at the top of the stack | 19:51 |
| dhellmann | oh, that's fine | 19:51 |
| dhellmann | if you want to +2 and wait for him I don't mind | 19:51 |
| dhellmann | I can fix anything else up by monday | 19:52 |
| *** pcaruana has joined #openstack-release | 19:52 | |
| dhellmann | or on monday since friday is a travel day | 19:52 |
| dhellmann | there are 14 patches in that series now, so we're finally beating storyboard! :-) | 19:53 |
| smcginnis | OK, if you don't think it's time critical, since there are quite a few changes three and I'm slightly concerned about making too many waves at RC time, I think I'll wait for his review. | 19:54 |
| smcginnis | #winning | 19:54 |
| dhellmann | sure, more eyes are better | 19:54 |
| dhellmann | smcginnis : I'm reviewing the monasca releases right now. They look OK so far. Are things working well enough to approve? | 19:56 |
| smcginnis | dhellmann: Yes, as far as I know we are good. | 19:57 |
| dhellmann | here we go then... | 19:57 |
| *** jpich has joined #openstack-release | 20:04 | |
| *** lvdombrkr has quit IRC | 20:05 | |
| *** pcaruana has quit IRC | 20:07 | |
| *** annabelleB has quit IRC | 20:11 | |
| openstackgerrit | Merged openstack/releases master: Release Monasca and cut stable/queens https://review.openstack.org/541767 | 20:12 |
| *** annabelleB has joined #openstack-release | 20:16 | |
| openstackgerrit | Merged openstack/releases master: Release vitrage-dashboard 1.4.1 https://review.openstack.org/541793 | 20:25 |
| *** annabelleB has quit IRC | 20:26 | |
| *** annabelleB has joined #openstack-release | 20:29 | |
| smcginnis | I really don't like having so many things in the release dashboard that we can't do anything with. | 20:29 |
| *** pkovar has joined #openstack-release | 20:33 | |
| *** kumarmn_ has joined #openstack-release | 20:37 | |
| *** kumarmn has quit IRC | 20:41 | |
| *** dave-mcc_ has quit IRC | 20:49 | |
| *** dave-mccowan has joined #openstack-release | 20:51 | |
| tonyb | dhellmann: Sure I'll look over them today. | 20:52 |
| amotoki | hi, about http://lists.openstack.org/pipermail/openstack-dev/2018-February/127070.html, do we need to cut a stable branch now? | 20:53 |
| smcginnis | amotoki: Hey! Which project? | 20:55 |
| amotoki | neutron-vpnaas-dashboard | 20:55 |
| amotoki | it is cycle-with-intermediary | 20:55 |
| amotoki | I plan to release it around R-1 week to import translation and cut a stable branch at the same time | 20:55 |
| smcginnis | amotoki: Yes, that needs a final queens release (if needed) and a stable/queens branch created. | 20:56 |
| smcginnis | amotoki: Are translations being worked on for that? | 20:56 |
| amotoki | smcginnis: yes. there are some translations on vpnaas dashboard | 20:56 |
| amotoki | it is same as for other horizon plugins like fwaas dashboard | 20:56 |
| smcginnis | amotoki: Ah, OK. Maybe that one should be switched to cycle-trailing then. | 20:57 |
| dhellmann | cycle-trailing is really only for packaging projects | 20:57 |
| dhellmann | I thought the translation stuff was set up to understand branches? | 20:57 |
| dhellmann | that's why we anticipate rc2s that include translations | 20:57 |
| smcginnis | Oh, I thought we did that for some horizon stuff too. Nevermind then. | 20:58 |
| openstackgerrit | Merged openstack/releases master: Add cycle-highlights for openstacksdk for queens https://review.openstack.org/540173 | 20:58 |
| amotoki | dhellmann: good point. I forgot that when I talk about it. | 20:58 |
| amotoki | from translators perspecitve, we need a stable branch to avoid confusions | 20:58 |
| smcginnis | amotoki: Yeah, so best would be a "final" release and branch now, then an update off of stable/queens if needed to pick up translations. | 20:59 |
| amotoki | smcginnis: agree | 20:59 |
| amotoki | I will prepare a release today. there is no issue blocking a release | 21:00 |
| openstackgerrit | gordon chung proposed openstack/releases master: Release ceilometer 10.0.0 for Queens https://review.openstack.org/540697 | 21:01 |
| smcginnis | amotoki: OK, great. Thanks! | 21:01 |
| amotoki | thanks for clarification | 21:01 |
| smcginnis | Thanks for asking. ;) | 21:01 |
| *** jpich has quit IRC | 21:04 | |
| kota_ | hello | 21:14 |
| kota_ | one question, can i delete/reword a tag that already is pushed over gerrit and mirrored to github? | 21:15 |
| kota_ | i heared, it may cause a problem in #openstack-infra channel tho. | 21:16 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add a warning for repo used in release but not in repository-settings https://review.openstack.org/540521 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only check release jobs if there are new releases to tag https://review.openstack.org/540975 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: make validate report more verbose https://review.openstack.org/541011 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only validate releases that have not yet been tagged https://review.openstack.org/541012 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: by default skip projects that have had tags applied https://review.openstack.org/541492 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: skip list-changes for retired repositories https://review.openstack.org/541820 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 21:18 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: try to validate the starting points for branches https://review.openstack.org/541906 | 21:18 |
| dhellmann | kota_ : we do not usually move or delete tags because they trigger building artifacts that won't match the new location | 21:18 |
| dhellmann | kota_ : what repo and what tag are you trying to "fix"? | 21:19 |
| dhellmann | if you give me a little more detail I can try to help you find a solution | 21:19 |
| kota_ | dhellmann: https://github.com/openstack/storlets/releases/tag/1.0.0, i got mistaken at the project name even it's the 1.0.0 release... | 21:20 |
| kota_ | not a significant problem tho, just a typo... | 21:20 |
| dhellmann | kota_ : isn't storlets an official project? | 21:20 |
| dhellmann | yeah, it is. | 21:21 |
| kota_ | it's official | 21:21 |
| dhellmann | you shouldn't even have permission to push tags any more | 21:21 |
| dhellmann | so we can fix that | 21:21 |
| dhellmann | unfortunately, we can't revoke the tag itself | 21:21 |
| dhellmann | is it on the wrong repository? or on the wrong place within the repository? | 21:21 |
| kota_ | wrong tag message, not "Storelts", "Storlets" instead. | 21:22 |
| smcginnis | Just the text on that, right? | 21:22 |
| kota_ | yes | 21:22 |
| dhellmann | https://review.openstack.org/541908 | 21:23 |
| dhellmann | that removes the acls | 21:23 |
| dhellmann | I don't think there's anything we can do about the typo | 21:23 |
| dhellmann | kota_ : please have a look at the README.rst file in openstack/releases. You should be using that for managing releases now that Storlets is an official project. | 21:25 |
| kota_ | dhellmann: after the acl removal, how do i push the tag instead? i referred the docs at https://docs.openstack.org/infra/manual/drivers.html | 21:25 |
| kota_ | README.rst, got it | 21:25 |
| dhellmann | and let us know here or on the mailing list if you have any questions | 21:26 |
| kota_ | ok, thanks dhellmann! | 21:26 |
| dhellmann | smcginnis : didn't we have a tool to find gerrit tagging permission mis-configurations? | 21:27 |
| dhellmann | kota_ : any time! | 21:27 |
| smcginnis | dhellmann: Wasn't that was mnaser that was trying to add a tool? | 21:28 |
| mnaser | yes it was me, but i didn't pick up the work in moving it to another repo | 21:28 |
| * smcginnis cries over his diminishing typing skills | 21:28 | |
| mnaser | which is over here https://review.openstack.org/#/c/533780/ | 21:28 |
| dhellmann | he was working on something in the governance repo, but I thought we had something else already that would at least let us discover the issues | 21:28 |
| dhellmann | maybe I'm thinking of aclmanager.py but that's different | 21:28 |
| smcginnis | Not that I'm aware of. Thanks mnaser/ | 21:29 |
| mnaser | smcginnis: if someone wants to pick up the work, it shouldn't be too hard | 21:29 |
| dhellmann | oh, nice, you're using the gerrit api | 21:29 |
| dhellmann | I thought we'd have to parse the input files | 21:29 |
| mnaser | dhellmann: i thought it was a lot more accurate to ask gerrit in case there was something not sync'd up | 21:30 |
| dhellmann | definitely | 21:30 |
| mnaser | which repo did we say this should go to? i think it wouldn't be too hard to add a job with required-projects: including governance and then just running the job there | 21:30 |
| dhellmann | the problem with automating it is we have to solve the chicken-and-egg problem | 21:31 |
| dhellmann | if we require this step before we add something to governance, then the release team is blocking governance changes | 21:31 |
| dhellmann | if we allow the governance change, then project owners can just keep tagging as they were before (see the scrollback) | 21:32 |
| mnaser | dhellmann: what about using zuul's ignored-files ? | 21:32 |
| dhellmann | we also need to account for things like independent projects, which are not necessarily tagged by the release team | 21:32 |
| mnaser | and have it ignore anything that is not a projects.yaml change | 21:32 |
| dhellmann | and the governance repo doesn't know which those are | 21:32 |
| dhellmann | how would ignoring some files help here? | 21:33 |
| mnaser | dhellmann: ignoring all files except reference/projects.yaml for the `governance-validate-access` job .. and then for the independent releases, i can add required-projects: openstack/releases and parse the yaml files | 21:34 |
| dhellmann | the release team doesn't allow data to be added until projects are listed in the governance repo | 21:34 |
| dhellmann | and we don't want technical rules in place that block governance changes | 21:35 |
| dhellmann | we don't want 3 people who may not be on the TC to have a veto over adding project teams | 21:35 |
| dhellmann | that's just bad policy | 21:36 |
| mnaser | dhellmann: fair enough, maybe this should be enforced in infra? | 21:37 |
| dhellmann | maybe. how would we do that? what event would we block? | 21:39 |
| mnaser | so project-config changes that add/remove a group that can push tags triggers that same job | 21:39 |
| mnaser | so pretty much any changes in project-config/gerrit/acls would run that through (and i guess in this case, check the ini files) | 21:39 |
| mnaser | which should be a lot less impactful on gerrit | 21:39 |
| dhellmann | these groups usually already exist, don't they? | 21:41 |
| dhellmann | a repo is created for an unofficial project; work is done; releases are made; time passes; it is made official; it is now misconfigured | 21:41 |
| mnaser | dhellmann: hmm, you're right | 21:42 |
| mnaser | i guess a periodic job but someone would have to pick up responsibility of tracking it | 21:43 |
| dhellmann | right | 21:43 |
| dhellmann | that's why this is a tough one to validate | 21:43 |
| dhellmann | oh, weird, this API returns a big sha for the rule identifier | 21:45 |
| dhellmann | instead of the team name | 21:45 |
| *** mriedem has quit IRC | 21:53 | |
| dhellmann | and the api to access the group details requires authentication | 21:57 |
| *** claudiub|2 has quit IRC | 22:22 | |
| *** claudiub|2 has joined #openstack-release | 22:22 | |
| openstackgerrit | Ifat Afek proposed openstack/releases master: Release vitrage 2.0.1 https://review.openstack.org/541928 | 22:34 |
| *** bobh has quit IRC | 22:40 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add a script to examine the gerrit acls for issues https://review.openstack.org/541931 | 22:55 |
| dhellmann | smcginnis , mnaser : ^^ | 22:55 |
| dhellmann | and http://paste.openstack.org/show/665371/ for the output | 22:56 |
| *** iyamahat_ has joined #openstack-release | 22:56 | |
| dhellmann | oh, ignore those first 2 lines of json stuff, that's a debug run | 22:56 |
| dhellmann | the script doesn't actually print those any more | 22:56 |
| *** sree has joined #openstack-release | 22:57 | |
| *** iyamahat has quit IRC | 22:57 | |
| *** annabelleB has quit IRC | 22:58 | |
| *** sree has quit IRC | 23:01 | |
| tonyb | dhellmann: I've looked over them all nothing major but perhaps a couple of fix-ups late in the series. The last 1 is clearly a WiP so I didn't really look at it | 23:05 |
| dhellmann | tonyb : yeah, all of those formatting things will go away when we remove the highlights from those files as part of deprecating that "feature" | 23:06 |
| dhellmann | and the other things like true->yes are caused by rewriting the files with our yamlutils module | 23:06 |
| *** annabelleB has joined #openstack-release | 23:06 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add a script to examine the gerrit acls for issues https://review.openstack.org/541931 | 23:06 |
| *** sree has joined #openstack-release | 23:07 | |
| tonyb | dhellmann: That was kinda my impression but for some reason I was committed to pointing them out ... I have no idea why :( | 23:07 |
| dhellmann | :-) | 23:07 |
| *** annabelleB has quit IRC | 23:08 | |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 23:11 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: try to validate the starting points for branches https://review.openstack.org/541906 | 23:11 |
| dhellmann | tonyb : ^^ should fix those test errors and un-wip that final patch | 23:11 |
| *** sree has quit IRC | 23:11 | |
| tonyb | dhellmann: cool, I'll look over them | 23:12 |
| *** armstrong has quit IRC | 23:16 | |
| dhellmann | tonyb : your comment about the filename argument in those tests early on has uncovered a bug, I think, so thank you | 23:17 |
| tonyb | dhellmann: cool. I can't say I knew that at the time ;P | 23:17 |
| dhellmann | the test is poorly constructed | 23:21 |
| dhellmann | unfortunately, since that's at the bottom of the stack, I have to resubmit them all | 23:22 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add a warning for repo used in release but not in repository-settings https://review.openstack.org/540521 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only check release jobs if there are new releases to tag https://review.openstack.org/540975 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: make validate report more verbose https://review.openstack.org/541011 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: only validate releases that have not yet been tagged https://review.openstack.org/541012 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: by default skip projects that have had tags applied https://review.openstack.org/541492 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: skip list-changes for retired repositories https://review.openstack.org/541820 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for pike https://review.openstack.org/540864 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for ocata https://review.openstack.org/540865 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: add repo names to repository-settings for queens https://review.openstack.org/540522 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: deprecate the use of highlights attached to a release https://review.openstack.org/540866 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: split validation of tarball-base into its own function https://review.openstack.org/541019 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: wrap subprocess to capture output in tests https://review.openstack.org/541805 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: use consistent log formatting for list-changes and validate https://review.openstack.org/541806 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: treat extra repos as errors https://review.openstack.org/541842 | 23:23 |
| openstackgerrit | Doug Hellmann proposed openstack/releases master: try to validate the starting points for branches https://review.openstack.org/541906 | 23:23 |
| dhellmann | smcginnis : don't look at me, tonyb made me do it | 23:23 |
| tonyb | LOL | 23:23 |
| tonyb | dhellmann: sorry you've used your timeslice for the week ;P | 23:24 |
| * dhellmann hears wine glasses clinking in the other room | 23:24 | |
| dhellmann | heh | 23:24 |
| tonyb | that's good timing then :) | 23:24 |
| dhellmann | definitely | 23:25 |
| dhellmann | thanks for the reviews, ttyl | 23:25 |
| tonyb | EmilienM: I'n not sure why your pike and ocata releases are so sad :( | 23:27 |
| openstackgerrit | Merged openstack/releases master: Update doc theme to use current logo https://review.openstack.org/533245 | 23:29 |
| *** edmondsw has quit IRC | 23:37 | |
| *** iyamahat_ has quit IRC | 23:37 | |
| *** kumarmn_ has quit IRC | 23:38 | |
| *** iyamahat has joined #openstack-release | 23:38 | |
| *** kumarmn has joined #openstack-release | 23:39 | |
| *** kumarmn has quit IRC | 23:43 | |
| *** dave-mccowan has quit IRC | 23:45 | |
| *** dave-mccowan has joined #openstack-release | 23:47 | |
| *** hongbin has quit IRC | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!