| *** wolverineav has joined #openstack-release | 00:00 | |
| *** hongbin has quit IRC | 00:04 | |
| *** wolverineav has quit IRC | 00:05 | |
| *** zhurong has joined #openstack-release | 00:38 | |
| *** wolverineav has joined #openstack-release | 00:39 | |
| *** wolverineav has quit IRC | 00:43 | |
| *** gcb has joined #openstack-release | 01:05 | |
| *** Qiming has quit IRC | 01:09 | |
| *** Qiming has joined #openstack-release | 01:13 | |
| *** gcb has quit IRC | 01:18 | |
| *** wolverineav has joined #openstack-release | 01:18 | |
| *** wolverineav has quit IRC | 01:23 | |
| *** wolverineav has joined #openstack-release | 01:33 | |
| *** wolverineav has quit IRC | 01:37 | |
| *** wolverineav has joined #openstack-release | 01:53 | |
| *** zhurong has quit IRC | 01:57 | |
| *** wolverineav has quit IRC | 01:58 | |
| *** d0ugal_ has quit IRC | 02:03 | |
| *** d0ugal_ has joined #openstack-release | 02:14 | |
| *** hongbin has joined #openstack-release | 02:25 | |
| *** wolverineav has joined #openstack-release | 02:26 | |
| *** wolverineav has quit IRC | 02:30 | |
| *** gcb has joined #openstack-release | 02:30 | |
| *** wolverineav has joined #openstack-release | 03:01 | |
| *** d0ugal_ has quit IRC | 03:02 | |
| *** wolverineav has quit IRC | 03:06 | |
| *** edmondsw has joined #openstack-release | 03:07 | |
| *** gcb has quit IRC | 03:07 | |
| *** edmondsw has quit IRC | 03:11 | |
| *** d0ugal_ has joined #openstack-release | 03:13 | |
| *** gcb has joined #openstack-release | 03:20 | |
| *** ricolin has joined #openstack-release | 03:21 | |
| *** kumarmn has joined #openstack-release | 03:21 | |
| *** kumarmn_ has quit IRC | 03:24 | |
| *** yamamoto has joined #openstack-release | 03:54 | |
| *** d0ugal_ has quit IRC | 04:06 | |
| *** dave-mccowan has quit IRC | 04:10 | |
| *** dave-mccowan has joined #openstack-release | 04:11 | |
| *** wolverineav has joined #openstack-release | 04:11 | |
| *** udesale has joined #openstack-release | 04:13 | |
| *** d0ugal_ has joined #openstack-release | 04:15 | |
| *** wolverineav has quit IRC | 04:16 | |
| *** dave-mccowan has quit IRC | 04:18 | |
| *** ykarel|away has joined #openstack-release | 04:25 | |
| *** ykarel|away is now known as ykarel | 04:44 | |
| *** wolverineav has joined #openstack-release | 04:48 | |
| *** wolverineav has quit IRC | 04:53 | |
| *** hongbin has quit IRC | 04:56 | |
| *** wolverineav has joined #openstack-release | 05:20 | |
| *** wolverineav has quit IRC | 05:25 | |
| openstackgerrit | Tony Breeds proposed openstack/releases master: Request for release of Sahara Pike 7.0.1 https://review.openstack.org/531435 | 05:34 |
|---|---|---|
| openstackgerrit | Merged openstack/releases master: heat-translator is used as a library https://review.openstack.org/542857 | 05:55 |
| *** wolverineav has joined #openstack-release | 05:57 | |
| prometheanfire | tonyb: yo | 05:59 |
| *** wolverineav has quit IRC | 06:01 | |
| tonyb | prometheanfire: Just about to quit for the day ... 'sup? | 06:04 |
| *** yamamoto has quit IRC | 06:05 | |
| prometheanfire | tonyb: nothing, just saying hi | 06:07 |
| tonyb | prometheanfire: oh hi :) | 06:07 |
| prometheanfire | tonyb: nn :P | 06:08 |
| openstackgerrit | Emilien Macchi proposed openstack/releases master: TripleO pike release https://review.openstack.org/541725 | 06:08 |
| *** yamamoto has joined #openstack-release | 06:15 | |
| openstackgerrit | Matthew Thode proposed openstack/releases master: branch requirements for stable/queens https://review.openstack.org/540454 | 06:22 |
| *** wolverineav has joined #openstack-release | 06:34 | |
| *** wolverineav has quit IRC | 06:38 | |
| *** gcb has quit IRC | 06:41 | |
| *** gcb has joined #openstack-release | 06:56 | |
| *** gcb has quit IRC | 07:02 | |
| *** yamamoto has quit IRC | 07:20 | |
| *** eumel8 has joined #openstack-release | 07:24 | |
| *** lvdombrkr has joined #openstack-release | 07:37 | |
| *** wolverineav has joined #openstack-release | 07:40 | |
| *** jtomasek has joined #openstack-release | 07:43 | |
| *** slaweq has joined #openstack-release | 07:45 | |
| *** wolverineav has quit IRC | 07:45 | |
| *** pcaruana has joined #openstack-release | 08:07 | |
| *** wolverineav has joined #openstack-release | 08:16 | |
| *** wolverineav has quit IRC | 08:20 | |
| *** yamamoto has joined #openstack-release | 08:21 | |
| *** yamamoto has quit IRC | 08:29 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| *** amoralej|off is now known as amoralej | 08:38 | |
| *** yamahata has quit IRC | 08:39 | |
| *** bauwser is now known as bauzas | 08:43 | |
| *** jpich has joined #openstack-release | 08:54 | |
| *** claudiub has joined #openstack-release | 09:02 | |
| *** d0ugal_ has quit IRC | 09:03 | |
| *** d0ugal has joined #openstack-release | 09:03 | |
| *** d0ugal has joined #openstack-release | 09:03 | |
| *** pbandark has joined #openstack-release | 09:13 | |
| *** ykarel is now known as ykarel|lunch | 09:15 | |
| *** alexchadin has joined #openstack-release | 09:16 | |
| *** shardy has joined #openstack-release | 09:33 | |
| *** dirk has quit IRC | 09:33 | |
| *** dirk has joined #openstack-release | 09:34 | |
| *** yamamoto has joined #openstack-release | 09:43 | |
| *** shardy has quit IRC | 09:43 | |
| *** shardy has joined #openstack-release | 09:43 | |
| openstackgerrit | Merged openstack/releases master: update get_release_type to take a repo name https://review.openstack.org/542387 | 09:43 |
| *** yamamoto has quit IRC | 09:44 | |
| *** ricolin has quit IRC | 09:47 | |
| openstackgerrit | Merged openstack/releases master: add validation step for pypi permissions https://review.openstack.org/542388 | 09:48 |
| amoralej | hi, i've seen that tag 0.4.0 of pankoclient created in https://review.openstack.org/#/c/539618/ is not in stable/queens but in master branch | 09:49 |
| amoralej | smcginnis, ^ i guess there is some issue there? | 09:49 |
| openstackgerrit | Merged openstack/releases master: show which repos use a library https://review.openstack.org/542312 | 09:51 |
| *** alexchadin has quit IRC | 10:03 | |
| openstackgerrit | Dougal Matthews proposed openstack/releases master: Release Mistral client 3.3.0 https://review.openstack.org/543402 | 10:05 |
| *** alexchadin has joined #openstack-release | 10:12 | |
| *** ykarel|lunch is now known as ykarel | 10:19 | |
| *** electrofelix has joined #openstack-release | 10:36 | |
| *** lucas-afk is now known as lucasagomes | 10:44 | |
| *** wolverineav has joined #openstack-release | 10:44 | |
| *** yamamoto has joined #openstack-release | 10:45 | |
| *** yamamoto has quit IRC | 10:51 | |
| *** udesale_ has joined #openstack-release | 10:58 | |
| *** udesale has quit IRC | 11:01 | |
| openstackgerrit | Thomas Bechtold proposed openstack/releases master: pymod2pkg: Release version 0.13.2 https://review.openstack.org/543424 | 11:04 |
| *** udesale_ has quit IRC | 11:07 | |
| *** coreycb has quit IRC | 11:19 | |
| *** coreycb has joined #openstack-release | 11:20 | |
| *** yamamoto has joined #openstack-release | 11:20 | |
| *** yamamoto has quit IRC | 11:32 | |
| *** finucannot is now known as stephenfin | 11:40 | |
| *** alexchadin has quit IRC | 11:45 | |
| *** pkovar has joined #openstack-release | 11:58 | |
| *** yamamoto has joined #openstack-release | 12:02 | |
| *** dave-mccowan has joined #openstack-release | 12:08 | |
| *** pkovar has quit IRC | 12:38 | |
| smcginnis | amoralej: Hmm, looks normal to me. The specified commit is the point where it was tagged and branched. | 13:02 |
| amoralej | smcginnis, correct, sorry, i don't know why i detected an issue there | 13:06 |
| smcginnis | amoralej: No worries. :) | 13:12 |
| *** edmondsw has joined #openstack-release | 13:15 | |
| *** edmondsw has quit IRC | 13:15 | |
| *** edmondsw_ has joined #openstack-release | 13:16 | |
| *** rosmaita has joined #openstack-release | 13:18 | |
| *** kumarmn_ has joined #openstack-release | 13:29 | |
| *** kumarmn has quit IRC | 13:32 | |
| *** edmondsw_ is now known as edmondsw | 13:37 | |
| ttx | smcginnis: o/ | 13:40 |
| smcginnis | o/ | 13:40 |
| ttx | I keep a tracker at line 457 of https://etherpad.openstack.org/p/queens-relmgt-tracking | 13:40 |
| ttx | helps me keep up to date | 13:41 |
| ttx | dave-mccowan: any chance we could get a barbican RC1 ASAP? | 13:41 |
| ttx | Happy to wait a bit if you need something in, but if no news we'll have to force a RC1 tag tomorrow-ish | 13:42 |
| dave-mccowan | ttx yes. i'll do it now. sorry, i missed the date. | 13:42 |
| ttx | dave-mccowan: OK! Thanks | 13:42 |
| ttx | smcginnis: the only significant new thing of this morning is a proposed python-mistralclient re-release | 13:43 |
| smcginnis | ttx: Yep, that's all I've seen. | 13:44 |
| smcginnis | Looks like we also have an issue with the sahara stable release. Somehow passed check, but the gate validation caught some issues. | 13:44 |
| smcginnis | Need to look into that yet, but I need to step away for 15-30 minutes. Be right back. | 13:45 |
| openstackgerrit | Merged openstack/releases master: Release Glance 15.0.1 https://review.openstack.org/542903 | 13:47 |
| openstackgerrit | Merged openstack/releases master: Release Glance 14.0.1 https://review.openstack.org/542897 | 13:47 |
| openstackgerrit | Merged openstack/releases master: Release Heat 9.0.3 https://review.openstack.org/543119 | 13:47 |
| openstackgerrit | Merged openstack/releases master: Release Mistral Pike 5.2.1 https://review.openstack.org/543098 | 13:47 |
| *** eumel8 has quit IRC | 13:48 | |
| EmilienM | release-managers: please review https://review.openstack.org/#/c/541725/ | 13:48 |
| *** dtantsur is now known as dtantsur|brb | 13:52 | |
| ttx | EmilienM: I'm on it | 13:53 |
| ttx | lgtm | 13:53 |
| *** ricolin has joined #openstack-release | 13:55 | |
| EmilienM | thx | 13:55 |
| *** kong has quit IRC | 13:55 | |
| *** pkovar has joined #openstack-release | 13:58 | |
| openstackgerrit | Merged openstack/releases master: TripleO pike release https://review.openstack.org/541725 | 14:00 |
| *** amoralej is now known as amoralej|lunch | 14:08 | |
| *** kumarmn_ has quit IRC | 14:12 | |
| *** udesale has joined #openstack-release | 14:20 | |
| *** tenobreg is now known as tellesnobrega | 14:21 | |
| *** yamamoto has quit IRC | 14:24 | |
| smcginnis | Oh joy, more job failures. | 14:24 |
| *** udesale_ has joined #openstack-release | 14:24 | |
| *** udesale has quit IRC | 14:25 | |
| ttx | beh | 14:25 |
| smcginnis | ttx: Thanks for adding those. | 14:26 |
| smcginnis | I'm going to start with the sahara issue. | 14:26 |
| ttx | sahara? | 14:26 |
| smcginnis | ttx: They passed check, but are hitting gate validation errors: https://review.openstack.org/#/c/531435/4 | 14:27 |
| ttx | oh, ok, not a postjob fail | 14:27 |
| ttx | I'm debunking the two latest | 14:28 |
| smcginnis | Perfect | 14:30 |
| *** kumarmn has joined #openstack-release | 14:30 | |
| *** slaweq has quit IRC | 14:31 | |
| *** slaweq has joined #openstack-release | 14:32 | |
| *** annabelleB has joined #openstack-release | 14:33 | |
| ttx | The glance one is an AFS copy fail, probably transient | 14:36 |
| ttx | The mistral-extra one is weirder | 14:36 |
| *** slaweq has quit IRC | 14:36 | |
| smcginnis | I took a quick look. Do they have mistral-extra in their own requirements.txt? | 14:36 |
| smcginnis | I think we saw that one before where it would fail because the new version was higher than the requirements version. | 14:36 |
| ttx | checking | 14:37 |
| smcginnis | Sahara appears to be a split out of different release types that was done in queens but not in pike. | 14:37 |
| *** yamahata has joined #openstack-release | 14:37 | |
| *** edmondsw has quit IRC | 14:38 | |
| *** lbragstad has joined #openstack-release | 14:39 | |
| *** yamahata has quit IRC | 14:42 | |
| *** yamahata has joined #openstack-release | 14:42 | |
| ttx | doesn't seem to have itself in requirements | 14:45 |
| ttx | (mistral-extra) | 14:45 |
| *** lucasagomes is now known as lucas-hungry | 14:48 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Request for release of Sahara Pike 7.0.1 https://review.openstack.org/531435 | 14:50 |
| *** edmondsw has joined #openstack-release | 14:52 | |
| *** amoralej|lunch is now known as amoralej | 14:57 | |
| *** yamamoto has joined #openstack-release | 14:57 | |
| smcginnis | ttx: So should we bother with the glance release notes job, or just leave until next time to update? | 14:58 |
| smcginnis | ttx: Oh, mistral-extra is due to upper-constraints. | 15:01 |
| smcginnis | Odd though... it's in master u-c, but not pike. This was a pike release, so pike jobs must be looking at master u-c. | 15:03 |
| smcginnis | So mistral-extra was removed from master with this commit: http://git.openstack.org/cgit/openstack/requirements/commit/?id=f9b51b4b7d00bb917b3f48cea9180aa50b3d178d | 15:08 |
| smcginnis | Doesn't say anything about why, but I seem to remember having problems with this or another under a similar situation. | 15:08 |
| smcginnis | Only in rpm-packaging requirements.txt | 15:09 |
| prometheanfire | smcginnis: ? | 15:09 |
| smcginnis | prometheanfire: We got a release job failure with mistral-extra. | 15:10 |
| smcginnis | prometheanfire: I remember seeing this type of failure before, but don't remember the exact details now. | 15:10 |
| smcginnis | prometheanfire: http://logs.openstack.org/13/13fec4048a3c57d77307f4384b26788227179113/release/release-openstack-python/1dc8c7a/job-output.txt.gz#_2018-02-12_13_54_48_807947 | 15:10 |
| smcginnis | prometheanfire: It appears to be because mistral-extra is in upper-constraints, so it can't install itself. | 15:11 |
| *** eumel8 has joined #openstack-release | 15:11 | |
| smcginnis | It was added here actually: https://review.openstack.org/#/c/523908/ | 15:14 |
| *** lucas-hungry is now known as lucasagomes | 15:19 | |
| *** armax has joined #openstack-release | 15:32 | |
| *** lvdombrkr has quit IRC | 15:33 | |
| prometheanfire | smcginnis: shouldn't that be handled the same as any other job? | 15:34 |
| prometheanfire | smcginnis: oslo stuff is in UC as well | 15:34 |
| smcginnis | Yeah, I'm not seeing why it is causing a problem, just that it appears to be from u-c. | 15:35 |
| smcginnis | And more troubling, that it appears to be checking a stable/pike job against master u-c. | 15:35 |
| smcginnis | ttx: Any ideas with mistral-extra? | 15:36 |
| prometheanfire | smcginnis: occurs on recheck? | 15:38 |
| smcginnis | prometheanfire: Well, it's the release job, so not so easy to recheck, unfortunately. | 15:39 |
| ttx | in meetings, will bbl | 15:40 |
| smcginnis | It was on the sdist step, so it has already tagged it. | 15:40 |
| smcginnis | ttx: ack | 15:40 |
| prometheanfire | oh, the job itself failed after approval | 15:40 |
| smcginnis | Yep. :/ | 15:40 |
| *** ykarel is now known as ykarel|away | 15:44 | |
| prometheanfire | smcginnis: was that the ffe you wanted to discuss? | 15:44 |
| prometheanfire | ya, I'm not sure why it's using the git path at all | 15:45 |
| *** udesale has joined #openstack-release | 15:53 | |
| smcginnis | prometheanfire: I just saw the one this morning, and noticed osprofiler finally posted something, so just wanted to hold and make sure we had discussed those. | 15:53 |
| *** udesale_ has quit IRC | 15:53 | |
| openstackgerrit | Dougal Matthews proposed openstack/releases master: Release Mistral client 3.3.0 https://review.openstack.org/543402 | 15:53 |
| smcginnis | prometheanfire: And that too I guess. ^^ | 15:54 |
| *** ykarel|away has quit IRC | 15:54 | |
| smcginnis | fungi: I know we had this with another release in the not so distant past. Do you recall the cause and/or fix? http://logs.openstack.org/13/13fec4048a3c57d77307f4384b26788227179113/release/release-openstack-python/1dc8c7a/job-output.txt.gz#_2018-02-12_13_54_48_807947 | 15:57 |
| fungi | smcginnis: that error generally means there's a source install of a project appearing in the constraints list | 15:58 |
| fungi | i'm not sure what the recommended solution is these days, but in the past using edit-constraints from a pip wrapper script under tox was used to work around it | 15:59 |
| smcginnis | fungi: In that repo's tox.ini? | 15:59 |
| fungi | yeah, i've also seen projects perform a --no-deps install of the project separately from installing dependencies under constraints, or installing the project without constraints after installing constrained deps | 16:00 |
| smcginnis | fungi: --no-deps should be fine for sdist, right? | 16:02 |
| smcginnis | fungi: The other concerning thing here is it appears to be looking at upper-constraints from master. | 16:03 |
| fungi | has requirements branched? | 16:03 |
| smcginnis | Sorry, this was on a stable/pike job. | 16:04 |
| *** pcaruana has quit IRC | 16:04 | |
| fungi | ahh | 16:05 |
| *** ykarel|away has joined #openstack-release | 16:07 | |
| prometheanfire | smcginnis: why a client is being used as a lib I don't know, but ya, needs a release | 16:07 |
| prometheanfire | http://codesearch.openstack.org/?q=mistralclient&i=nope&files=.*requirements.*&repos= | 16:07 |
| prometheanfire | and reqs bump probably | 16:07 |
| prometheanfire | I'd honestly rather just branch now, then get that into the queens / master after | 16:08 |
| fungi | smcginnis: looking at http://logs.openstack.org/13/13fec4048a3c57d77307f4384b26788227179113/release/release-openstack-python/1dc8c7a/zuul-info/inventory.yaml there may be some unspoken assumptions that zuul somehow guesses an appropriate branch to check out for required-projects when the item under test is a non-change ref (in this case a tag) | 16:09 |
| smcginnis | fungi: I thought that might be the case, but does this mean the local cached repo is checked out to the tag? http://logs.openstack.org/13/13fec4048a3c57d77307f4384b26788227179113/release/release-openstack-python/1dc8c7a/job-output.txt.gz#_2018-02-12_13_51_36_668161 | 16:10 |
| smcginnis | prometheanfire: Should we wait for a response on that client? Or should I just process the branching now? | 16:13 |
| prometheanfire | I don't think branching now harms us | 16:14 |
| prometheanfire | as long as the reqs update is submitted to queens we'll be fine | 16:15 |
| smcginnis | prometheanfire: OK, on its way. | 16:15 |
| prometheanfire | not sure if it'll need to be submitted to master as well (if not the bot will pick it up anyway) | 16:15 |
| prometheanfire | k, I'll send a notice to the list letting them know reqs is unfrozen | 16:15 |
| smcginnis | ++ | 16:15 |
| *** ykarel|away has quit IRC | 16:16 | |
| *** ekcs_ has quit IRC | 16:21 | |
| *** ekcs_ has joined #openstack-release | 16:21 | |
| *** dtantsur|brb is now known as dtantsur | 16:24 | |
| prometheanfire | smcginnis: http://logs.openstack.org/54/540454/2/gate/openstack-tox-validate/d29e258/job-output.txt.gz#_2018-02-12_16_23_23_431578 | 16:27 |
| smcginnis | Bah! | 16:27 |
| *** bobh has joined #openstack-release | 16:30 | |
| jroll | prometheanfire: most clients are usable as a library (read: python sdk) for communicating between services, that's half the point of the clients | 16:32 |
| jroll | more than half now that osc is the thing | 16:32 |
| fungi | smcginnis: i'll finish digging into the mistral-extra pike tag situation after my current meeting ends | 16:33 |
| dtantsur | prometheanfire: hey, does your ML post mean that requirements are branched? right now I don't see a stable/queens on openstack/requirements | 16:34 |
| prometheanfire | dtantsur: well, http://logs.openstack.org/54/540454/2/gate/openstack-tox-validate/d29e258/job-output.txt.gz#_2018-02-12_16_23_23_431578 | 16:34 |
| smcginnis | dtantsur: We had an issue with the post-release job. Working on that now. | 16:34 |
| fungi | as a separate heads-up, we just performed an emergency restart of the zuul scheduler so may have lost any jobs queued in the tag/release/pre-release/release-post/post pipelines | 16:34 |
| fungi | as well as possibly any unprocessed gerrit events qhich were still queued | 16:35 |
| dtantsur | "cannot marshal None" oh that Marshal None, seems a problematic dude :D | 16:35 |
| prometheanfire | fungi: ok, restarted the restart | 16:38 |
| smcginnis | prometheanfire: We have a publish-to-pypi job on openstack/requirements. What are we publishing to pypi? | 16:45 |
| prometheanfire | smcginnis: we are a library as well | 16:46 |
| prometheanfire | was tox updated? | 16:46 |
| ttx | back | 16:46 |
| prometheanfire | smcginnis: do we need a tag as well? | 16:46 |
| smcginnis | prometheanfire: Maybe that's why it's failing. Had to take care of something else here, so just looking at the failure now. | 16:47 |
| prometheanfire | k | 16:48 |
| smcginnis | It looks like it failed on getting the pypi roles for the project. | 16:50 |
| smcginnis | Doug just added this last week. I'll just remove it for now so we can troubleshoot at a better time. | 16:51 |
| *** hongbin has joined #openstack-release | 16:51 | |
| mriedem | prometheanfire: so i should merge https://review.openstack.org/#/c/542657/ to queens now? | 16:52 |
| openstackgerrit | Matthew Thode proposed openstack/releases master: branch requirements for stable/queens https://review.openstack.org/540454 | 16:52 |
| smcginnis | mriedem: Not yet. | 16:52 |
| mriedem | oh i see ^ | 16:52 |
| prometheanfire | smcginnis: lets see if it's happier with a tag | 16:52 |
| *** ricolin has quit IRC | 16:58 | |
| smcginnis | prometheanfire: That passes local validation, but I don't see any output for the pypi check. | 16:58 |
| smcginnis | prometheanfire: Had to rebase it: openstackci does not have permission to upload packages for openstack_requirements. Current owners include: | 17:00 |
| smcginnis | prometheanfire: It does show openstackci as a Package Index Owner, so I think we will have to wait for my disable of that check for now. | 17:00 |
| prometheanfire | smcginnis: which one we talking about now? | 17:02 |
| smcginnis | prometheanfire: The validation failure with branching requirements. | 17:02 |
| prometheanfire | ah | 17:02 |
| *** jpich has quit IRC | 17:07 | |
| fungi | i'm not immediately seeing where we have any logic to attempt to infer a branch corresponding to a tag | 17:07 |
| fungi | (back to looking into the mistral-extra release failure) | 17:08 |
| smcginnis | fungi: I guess that makes sense then. It checks out the tag, but the tag exists on both master and stable/queens. Unless we pass that in, how would it know which to use. | 17:09 |
| smcginnis | Unless we look at branches and just say whatever is the last and not master? | 17:09 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: branch requirements for stable/queens https://review.openstack.org/540454 | 17:11 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Disable pypi permission verification https://review.openstack.org/543551 | 17:11 |
| fungi | smcginnis: i seem to remember it being suggested that projects with stable branches should at least merge a stable branch prep commit soon after branching. so in theory the only tag which appears on two branches will be the initial release for that series (or an rc tag during reqs freeze) so which one it picks in that case shouldn't matter | 17:12 |
| smcginnis | fungi: Oh, so having the tag and branch together is an issue, but if they merge something else to stable and release from there it wouldn't be a problem? | 17:14 |
| fungi | yeah, i guess if they're a cycle-trailing project using python upper-constraints from the requirements repo then it could get hairy | 17:15 |
| smcginnis | I'm still a little confused though. Because with that u-c on master, the next master release would be for a version newer than the constraint. | 17:15 |
| smcginnis | So I guess I don't understand how that process works (though it obviously must). | 17:16 |
| fungi | the risk is tagging and branching when the requirements repo has divergent constraints lists for the new branch and master | 17:16 |
| prometheanfire | fungi: basically we are very careful | 17:16 |
| *** udesale has quit IRC | 17:17 | |
| fungi | yeah, so anyway the reason i'm asking is that we're discussing in #openstack-infra what tag-triggered jobs can do to guess the right branch of the requirements repo to checkout so they get the right upper-constraints.txt | 17:17 |
| smcginnis | I'm not sure what we do here. It appears to have tagged, but no pypi packages nor tarballs. | 17:17 |
| prometheanfire | manualy tarball/pypi, remove tag and rerun, try to tag a patch release and rerun | 17:18 |
| prometheanfire | one of those three options I think | 17:18 |
| smcginnis | I don't believe we can remove a tag. | 17:19 |
| prometheanfire | nor I, just stating all options :D | 17:19 |
| smcginnis | :) | 17:19 |
| prometheanfire | infra would have to do that if it's even available | 17:20 |
| smcginnis | Assuming 2 is not an option, 1 would need infra help, 3 we could probably ask mistral to do, but sounds like they would need to merge something to the repo first before requesting the .1 release. | 17:21 |
| smcginnis | ttx: Any thoughts or ideas? ^^ | 17:21 |
| fungi | smcginnis: well, i'm looking at how we can update the job to guess the correct branch, but i don't think that's the reason it failed. looks like it failed because of other constraints handling unrelated to the branch it selected | 17:23 |
| smcginnis | fungi: Oh? | 17:24 |
| smcginnis | fungi: Can you point me at what you're looking at? | 17:24 |
| fungi | the specific error there comes when you attempt to apply constraints to an install from source | 17:24 |
| smcginnis | fungi: So sdist shouldn't even be using constraints? | 17:24 |
| fungi | http://logs.openstack.org/13/13fec4048a3c57d77307f4384b26788227179113/release/release-openstack-python/1dc8c7a/job-output.txt.gz#_2018-02-12_13_54_48_807947 | 17:24 |
| fungi | i think other projects do something in their tox configuration to work around that | 17:25 |
| smcginnis | Oh, so I read that error as it being a problem because the new one is above the constraint, but the issue is that it is trying to apply constraints in the first place? | 17:25 |
| fungi | yeah | 17:26 |
| corvus | smcginnis: i think the medium-term goal is to stop using constraints in sdist, but some libraries still require it i think. | 17:26 |
| fungi | pip refuses to install from source with a matching constraint for the package name | 17:26 |
| corvus | this comment from dhellmann seems to imply that at least: https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/jobs.yaml#n196 | 17:26 |
| fungi | the invocation (some several lines up from that error) is: pip install -c/home/zuul/src/git.openstack.org/openstack/requirements/upper-constraints.txt --no-deps -e /home/zuul/src/git.openstack.org/openstack/mistral-extra | 17:26 |
| fungi | so i think their tox/wrapper is probably slightly wrong | 17:27 |
| smcginnis | I don't see tox being used for this though. | 17:28 |
| smcginnis | Shouldn't this call python setup.py sdist directly? Running tox: tox -evenv -vv python setup.py sdist | 17:29 |
| fungi | smcginnis: http://logs.openstack.org/13/13fec4048a3c57d77307f4384b26788227179113/release/release-openstack-python/1dc8c7a/job-output.txt.gz#_2018-02-12_13_54_44_949871 says it's running under tox | 17:29 |
| smcginnis | fungi: Yes, my point was they don't have anything in their tox.ini for this. So why the command I noted above? | 17:30 |
| fungi | well, -e venv will inherit http://git.openstack.org/cgit/openstack/mistral-extra/tree/tox.ini?h=stable/pike#n8 | 17:31 |
| smcginnis | Yes, but why do we call tox -e venv? Why do we not just call "python setup.py sdist" was the question. | 17:32 |
| clarkb | pbr is one reason | 17:34 |
| fungi | smcginnis: note that, e.g., oslo.config uses a tools/tox_install.sh wrapper around pip to solve this conundrum: http://git.openstack.org/cgit/openstack/oslo.config/tree/tox.ini?h=stable/pike#n11 | 17:37 |
| fungi | the magic happens at http://git.openstack.org/cgit/openstack/oslo.config/tree/tools/tox_install.sh?h=stable/pike#n27 | 17:39 |
| fungi | basically, before using upper-constraints.txt the entry for the package itself gets filtered out of it | 17:40 |
| smcginnis | fungi: So we need to have mistral add a similar script to their repo, then re-release? | 17:41 |
| fungi | smcginnis: i've seen several solutions to this same problem, so at this point i'd defer to tonyb and/or #openstack-stable on what the current recommended solution is | 17:42 |
| fungi | my money's on oslo.config probably using the recommended mechanism, but i'm hesitant to go cargo-culting if there's new advice to be had | 17:43 |
| smcginnis | I don't see anyting in the os-brick repo, but would have assumed I would have run into the same thing there. | 17:43 |
| * fungi looks | 17:43 | |
| smcginnis | fungi: I guess for now I will notify mistral and wait for input. | 17:43 |
| fungi | smcginnis: yeah, os-brick uses a different tox.ini-only solution: http://git.openstack.org/cgit/openstack/os-brick/tree/tox.ini?h=stable/pike#n16 | 17:44 |
| fungi | basically that says install dependencies using constraints but doesn't override the install_command to apply constraints | 17:45 |
| fungi | that might be the "new" recommendation, i don't really know | 17:45 |
| fungi | so as long as one of your dependencies doesn't depend on you... | 17:46 |
| fungi | if it does, then you probably need to go the edit-constraints in a wrapper route | 17:46 |
| smcginnis | fungi: Oh, I see why I'm confused. I was looking at their master tox.ini, which does match os-brick. | 17:46 |
| smcginnis | So looks like that needs to be backported and a new release done. | 17:46 |
| fungi | that sounds expedient | 17:46 |
| smcginnis | fungi: And it was already recognized as causing release failures: https://github.com/openstack/mistral-extra/commit/566dc85cca11512537b786a8a8ffc35a2331871c | 17:47 |
| fungi | got iy | 17:47 |
| fungi | it | 17:47 |
| fungi | smcginnis: unfortunately, we're still missing a solution to apply the right branch of constraints for stable point release tags, but i think i know a way we can do that fairly cleanly by depending on some basic assumptions about our release workflow | 17:47 |
| smcginnis | fungi: I would imagine it's safe to make those assumptions 99+% of the time/ | 17:48 |
| fungi | and odds are applying master constraints when generating the tarball/wheel isn't going to cause a problem for most projects in the near term (it hasn't even been noticed until now, apparently?) | 17:48 |
| *** yamamoto has quit IRC | 17:49 | |
| fungi | i think as long as we can safely assume that we'll only ever run jobs for tags which appear on multiple branches is at the creation of a new series, which will be prior to requirements branching for that same series where the fallback to checking out master is what we want anyway | 17:51 |
| fungi | and even if we move requirements branching earlier into the requirements freeze, in that case we don't care whether we pick the new stable branch or master because they should have ~identical constraints | 17:52 |
| *** ykarel|away has joined #openstack-release | 17:53 | |
| fungi | and for cycle-trailing releases after requirements is unfrozen, cross our fingers and hope that they haven't diverged enough to cause an issue or we add some special logic to prefer a matching stable branch over master | 17:53 |
| *** yamamoto has joined #openstack-release | 17:54 | |
| fungi | as long as these hold true, we can `git branch --contains <tag>` to guess the right branch | 17:55 |
| *** armax has quit IRC | 17:55 | |
| smcginnis | OK, ML posted for mistral solution. | 17:57 |
| smcginnis | fungi: While I'm distracting you from other things - do you think we should try to rerun this? http://logs.openstack.org/31/31181cba50df52a1a723f7f34af2eef6d0e3e673/tag/publish-openstack-releasenotes/dfe7a5a/ | 17:59 |
| smcginnis | fungi: Specifically - http://logs.openstack.org/31/31181cba50df52a1a723f7f34af2eef6d0e3e673/tag/publish-openstack-releasenotes/dfe7a5a/ara/result/08f952e4-67dc-47bc-9742-f6579324b5d7/ | 17:59 |
| *** yamamoto has quit IRC | 18:00 | |
| fungi | smcginnis: i'll take a look | 18:01 |
| fungi | smcginnis: also, i'm encouraged to find that apparently before zuul v3 we did basically what i describe above to get the right branch for a given tag: https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml?id=d4cc58d06#n200 | 18:02 |
| smcginnis | fungi: Oh, that's a good sign. ;) | 18:03 |
| fungi | introduced by https://review.openstack.org/349627 a couple years ago | 18:05 |
| *** armax has joined #openstack-release | 18:05 | |
| *** ekcs has quit IRC | 18:06 | |
| *** annabelleB has quit IRC | 18:07 | |
| fungi | smcginnis: i've not before seen the error linked in that releasenotes job... i wouldn't be surprised if it's something transient with the rsync subprocess, but regardless it should be safe to reenqueue that ref in the tag pipeline | 18:07 |
| fungi | since i believe that's the only job we run in tag for that project | 18:08 |
| smcginnis | fungi: I think you are right. | 18:08 |
| smcginnis | fungi: Might be worth reenqueueing. I think the main thing they wanted out there was release note related. | 18:09 |
| fungi | i'm digging into whether this was related to the zookeeper disconnect we had a few hours ago | 18:10 |
| *** yamamoto has joined #openstack-release | 18:11 | |
| *** annabelleB has joined #openstack-release | 18:11 | |
| *** yamamoto has quit IRC | 18:11 | |
| * ttx catching up | 18:14 | |
| *** harlowja has joined #openstack-release | 18:14 | |
| ttx | smcginnis: anything you still need me for ? | 18:15 |
| prometheanfire | ok, back | 18:15 |
| smcginnis | ttx: I think we are OK. | 18:15 |
| smcginnis | ttx: mistral-extra got mostly sorted. | 18:16 |
| ttx | ok | 18:16 |
| smcginnis | ttx: fungi is looking at rerunning the failed glance release-notes job. | 18:16 |
| smcginnis | ttx: We have a change up to help with requirements branching if you have time and want to ack. | 18:16 |
| smcginnis | ttx: https://review.openstack.org/#/c/543551/ | 18:16 |
| smcginnis | ttx: Figured I would just disable that new test for now and leave it for Doug. Unless I find the time and inclination to debug it before then. | 18:17 |
| ttx | looking | 18:18 |
| fungi | smcginnis: prevailing wisdom suggests that release notes job raced another job also updating glance docs at roughly the same time | 18:19 |
| fungi | so anyway, transient and can be reenqueued | 18:19 |
| ttx | Looks good, +2a since we may miss enough core for speedy reaction | 18:19 |
| smcginnis | fungi: Shoot, yeah, I think I approved pike and ocata releases not far apart. I should have thought of that by this point. | 18:20 |
| smcginnis | ttx: Thanks | 18:20 |
| fungi | and i've now reenqueued refs/tags/14.0.1 for openstack/glance into the "tag" pipeline | 18:20 |
| smcginnis | Thanks fungi | 18:21 |
| smcginnis | ttx: Might be good to have you look at https://review.openstack.org/#/c/531435/ too. | 18:21 |
| smcginnis | ttx: But I can approve later if you don't have the time. | 18:21 |
| ttx | +2a since it was reviewed by tony previously | 18:27 |
| ttx | need to run. cheers! | 18:28 |
| lbragstad | o/ couple quick questions before we cut rc2 | 18:29 |
| lbragstad | we were originally waiting on a couple translation patches - https://review.openstack.org/#/c/542886/ and https://review.openstack.org/#/c/541567/ | 18:30 |
| lbragstad | just double checking, but both of those ^ will need to be backported to stable/queens since that branch exists now before cutting rc2 | 18:30 |
| lbragstad | right? | 18:31 |
| *** electrofelix has quit IRC | 18:34 | |
| *** electrofelix has joined #openstack-release | 18:37 | |
| *** armstrong has joined #openstack-release | 18:39 | |
| armstrong | @smcginnis: Hello, which days are allocated for the release team during the PTG at Dublin? | 18:40 |
| openstackgerrit | Dave McCowan proposed openstack/releases master: Release Barbican Queens RC1 https://review.openstack.org/543579 | 18:48 |
| *** ekcs has joined #openstack-release | 18:49 | |
| *** electrofelix has quit IRC | 18:51 | |
| openstackgerrit | Merged openstack/releases master: Request for release of Sahara Pike 7.0.1 https://review.openstack.org/531435 | 18:55 |
| lbragstad | also - does https://review.openstack.org/#/c/542886/ need to be backported as well? | 18:56 |
| *** ykarel|away has quit IRC | 18:57 | |
| smcginnis | lbragstad: No, that's fine to leave in master. I do believe you will need to backport the translations though. | 19:00 |
| lbragstad | smcginnis ok - cool, i have one proposed already, but i'll need a stable core to review | 19:01 |
| smcginnis | armstrong: Most of us are involved in multiple things, but I think we will be trying to sync up at some point on Wednesday and Thursday. | 19:01 |
| *** lucasagomes is now known as lucas-pto | 19:03 | |
| lbragstad | smcginnis here is one of them- https://review.openstack.org/#/c/543573/1 | 19:03 |
| lbragstad | smcginnis proposing https://review.openstack.org/#/c/543087/1 results in a conflict, which doesn't seem normal for a translation backport | 19:04 |
| lbragstad | i checked with the i18n team, but no response yet | 19:05 |
| smcginnis | lbragstad: That's a little odd. Must be a chance in a string causing problems. | 19:05 |
| smcginnis | lbragstad: I would bet you can just replace the conflict with the new value(s). | 19:05 |
| lbragstad | smcginnis proposed - https://review.openstack.org/#/c/543584/ | 19:09 |
| smcginnis | lbragstad: They're translating things under release notes? | 19:11 |
| lbragstad | apparently | 19:11 |
| lbragstad | https://review.openstack.org/#/c/543573/ was too | 19:12 |
| *** yamamoto has joined #openstack-release | 19:12 | |
| smcginnis | Weird. Some of the things in those translations are a bit odd, but ok. | 19:12 |
| *** pkovar has quit IRC | 19:13 | |
| lbragstad | yeah - i'm not very familiar with the translation process, but i didn't question it | 19:13 |
| *** jtomasek has quit IRC | 19:14 | |
| smcginnis | I normally don't either, but just happened to notice and take a look. ¯\_(ツ)_/¯ | 19:14 |
| *** yamamoto has quit IRC | 19:19 | |
| *** amoralej is now known as amoralej|off | 19:27 | |
| *** bobh has quit IRC | 19:27 | |
| *** yamahata has quit IRC | 19:28 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: pymod2pkg: Release version 0.13.2 https://review.openstack.org/543424 | 19:28 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: cyborg 0.1.0 queens independant release request https://review.openstack.org/542280 | 19:30 |
| *** lbragstad_ has joined #openstack-release | 19:41 | |
| *** lbragstad_ has quit IRC | 19:43 | |
| *** dtantsur is now known as dtantsur|afk | 19:44 | |
| lbragstad | smcginnis ok - i have some more information from the translation team | 19:47 |
| lbragstad | http://eavesdrop.openstack.org/irclogs/%23openstack-i18n/%23openstack-i18n.2018-02-12.log.html#t2018-02-12T18:48:46 | 19:47 |
| lbragstad | the tl;dr is to wait until 0600 UTC tomorrow and see if the translation job proposes anything to keystone's stable/queens branch | 19:48 |
| lbragstad | if so - then we need to merge whatever that is | 19:48 |
| lbragstad | if not, then we should be good to use HEAD of stable/queens as RC2 (pending the translations you and morgan reviewed are merged) | 19:49 |
| smcginnis | lbragstad: OK, thanks! | 19:50 |
| lbragstad | yep! | 19:50 |
| smcginnis | lbragstad: Any potential for further backports to stable/queens possible? | 19:50 |
| smcginnis | lbragstad: Reason I ask is we want to limit the number of RCs, so it might be best to wait until late this week to propose an RC2 just in case anything more significant is needed. | 19:51 |
| lbragstad | smcginnis not yet - we dug into one issue, but it wasn't introduced in queens and it isn't a release stopper (just something we wanted to get in if we could) | 19:51 |
| lbragstad | sounds good | 19:51 |
| lbragstad | we can wait to do RC2 | 19:51 |
| lbragstad | but at this point, i don't expect any other backports | 19:51 |
| lbragstad | that would result in an RC | 19:51 |
| smcginnis | lbragstad: OK, sounds pretty safe then. Whenever you think you're ready, just go ahead with the request. | 19:51 |
| prometheanfire | wish that test disablment would test faster is all | 19:52 |
| smcginnis | prometheanfire: Things do seem unusually slow right now, don't they. | 19:52 |
| lbragstad | i'll keep an eye on the current translation backports and see what the translation job says | 19:52 |
| prometheanfire | smcginnis: :D | 19:52 |
| prometheanfire | smcginnis: give me time to bug tonyb about what reqs needs to do when we unfreeze (stuff we merge to master causes cycle-trailing to test against rocky) | 19:56 |
| prometheanfire | so not sure how much we are 'business as usual still' | 19:56 |
| *** armax has quit IRC | 19:56 | |
| smcginnis | Ah, OK. That sounds important. ;) | 19:56 |
| prometheanfire | ya, I'm staying semi-frozen til then | 19:56 |
| prometheanfire | stuff can go into queens of course | 19:56 |
| prometheanfire | but master(rocky) may stay frozen | 19:56 |
| prometheanfire | not sure | 19:57 |
| *** bobh has joined #openstack-release | 19:57 | |
| smcginnis | I haven't heard of any pressing rocky needs yet, so should be safe enough to wait a little longer and make sure it's all clear. | 19:58 |
| prometheanfire | yarp | 19:58 |
| prometheanfire | that was my thinking as well | 19:58 |
| *** armax has joined #openstack-release | 19:59 | |
| prometheanfire | smcginnis: the reqs rebase needs +2+W btw | 20:00 |
| *** annabelleB has quit IRC | 20:01 | |
| *** yamahata has joined #openstack-release | 20:05 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Add missing stable/queens branches https://review.openstack.org/543601 | 20:06 |
| *** annabelleB has joined #openstack-release | 20:07 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Add missing stable/queens branches https://review.openstack.org/543601 | 20:07 |
| prometheanfire | smcginnis: ^???? | 20:18 |
| prometheanfire | smcginnis: also, mind hitting https://review.openstack.org/540454 | 20:18 |
| smcginnis | prometheanfire: Got too many in there, but just cleaning up on the release requests where we didn't want to hold things up for a branch included. | 20:18 |
| prometheanfire | ok | 20:18 |
| prometheanfire | I thought tripleo was cycle trailing | 20:18 |
| smcginnis | Oh good, that finally made it through. | 20:19 |
| prometheanfire | and puppet | 20:19 |
| smcginnis | It is. I'm removing those now. | 20:19 |
| prometheanfire | ah, k | 20:19 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Add missing stable/queens branches https://review.openstack.org/543601 | 20:23 |
| *** alexchadin has joined #openstack-release | 20:23 | |
| *** kong has joined #openstack-release | 20:28 | |
| prometheanfire | tonyb: when yo have time | 20:33 |
| *** alexchadin has quit IRC | 20:34 | |
| tonyb | prometheanfire: I'm on a call right now but 'sup? | 20:34 |
| prometheanfire | 13:58 < prometheanfire > smcginnis: give me time to bug tonyb about what reqs needs to do when we unfreeze (stuff we merge to master causes cycle-trailing to test against rocky) | 20:36 |
| prometheanfire | 13:58 < prometheanfire > so not sure how much we are 'business as usual still' | 20:36 |
| tonyb | Hmmm yeah the g-r changes are okay ish as we have agrement that they'll check with us | 20:39 |
| tonyb | u-c is harder as it starts to invalidate testing for cycle-trailing projects | 20:39 |
| tonyb | so yeah we need to go a little slow for a while | 20:40 |
| prometheanfire | ya | 20:40 |
| prometheanfire | I'll update the notes to make it more explicit | 20:40 |
| tonyb | longer term we should check if we get the branch opened earlier ? | 20:42 |
| prometheanfire | ya, also, the publish to specific contraints urls (publish to tarballs should help) | 20:42 |
| tonyb | prometheanfire: Yeah but that's messy | 20:43 |
| prometheanfire | didn't get it this cycle | 20:43 |
| tonyb | prometheanfire: Yeah. | 20:44 |
| openstackgerrit | Merged openstack/releases master: Disable pypi permission verification https://review.openstack.org/543551 | 20:45 |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Create stable/queens for devstack https://review.openstack.org/543611 | 20:45 |
| smcginnis | tonyb, prometheanfire: Do I need to remove +W from the requirements branching patch, or are we still good there? | 20:45 |
| prometheanfire | smcginnis: we are still good | 20:45 |
| smcginnis | OK, thanks. | 20:46 |
| tonyb | smcginnis: It's all good | 20:46 |
| prometheanfire | smcginnis: we were just talking about requirements changes after the branch, how we should handle it | 20:46 |
| tonyb | smcginnis: it's a requiremenst team thing at this point, we don't want to hold up the rest of the community | 20:46 |
| smcginnis | OK, that's what I thought you were saying, but figured I better make sure before it made it through gate. ;) | 20:46 |
| prometheanfire | yarp | 20:47 |
| *** eumel8 has quit IRC | 20:48 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Create stable/queens branch for grenade https://review.openstack.org/543614 | 20:53 |
| tonyb | smcginnis: re openstack_requirements not havign openstackci as an owner it certainly used to | 21:00 |
| *** bobh has quit IRC | 21:00 | |
| openstackgerrit | Merged openstack/releases master: branch requirements for stable/queens https://review.openstack.org/540454 | 21:01 |
| tonyb | Hmmm I can't log into pypi to check/fix it ruhroh :( | 21:01 |
| prometheanfire | tonyb: you've been reassigned | 21:01 |
| *** ChanServ has quit IRC | 21:02 | |
| smcginnis | tonyb: It did show openstackci as an owner. The problem was the script to verify that was getting back something unexpected, so for now I've just removed that check since we never had it until last week. | 21:03 |
| tonyb | smcginnis: Oh okay | 21:04 |
| smcginnis | tonyb: If you are having trouble logging in, maybe that's part of the reason we weren't able to get the info with the script. Might just be external issues with pypi out of our control. | 21:05 |
| tonyb | smcginnis, fungi: To be frank I know how the constratints stuff *used* to work but then mordred and I messed around with it and I don't really know where it landed. I'll make soem time to refresh where we're at but I a quick look at the mistral-extra tox.ini look right to me | 21:07 |
| tonyb | smcginnis: yeah possibly I think I'll need a coffee befoer I dig into that ;P | 21:07 |
| fungi | tonyb: it seems like the constraints application is mostly worked out | 21:08 |
| fungi | and i've pushed https://review.openstack.org/543607 to hopefully solve the branch guessing problem | 21:08 |
| tonyb | fungi: yeah but that "can't match constraints and sdist I *think* measn it's doign the wrong thing | 21:09 |
| fungi | tonyb: right, mistral-extra was broken, they needed a tox.ini fix backported from master to make things work | 21:09 |
| tonyb | fungi: Oh which one? | 21:10 |
| *** ChanServ has joined #openstack-release | 21:10 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 21:10 | |
| fungi | it looks like projects either (new-style?) install their deps using constraints but then install themselves in an independent step without, or (slightly older-stype?) use a wrapper script as their install_command and make it call edit-constraints to filter the project out of the constraints list before applying it | 21:11 |
| fungi | tonyb: backported fix for mistral-extra is https://review.openstack.org/543563 | 21:12 |
| tonyb | fungi: That sounds right, the new-style is the recomened way to combine tox and constraints | 21:12 |
| tonyb | fungi: oooh I see yeah that's broken and seem to be the right fix | 21:13 |
| fungi | it may not have been broken for them in pike because they might not have been in the global requirements list at the time | 21:14 |
| tonyb | fungi: Correct they added themselves in Decemeber and we didn't check that they'd done this step befoer we +W'd the chnage :( | 21:15 |
| fungi | well, the reason it broke for them on stable/pike is because of what i'm hoping to fix with 543607 | 21:16 |
| tonyb | smcginnis: So I got into pypi it was a pebkac error to do with me havign more accounts then expected :( | 21:16 |
| tonyb | fungi: No I think even with your chnage the job would still fail in the same way | 21:17 |
| fungi | no, because they're not actually _in_ pike's global-requirements.txt or upper-constraints.txt | 21:17 |
| tonyb | fungi: as even *if* they have the pike constraints file the version therein != $(git describe) | 21:17 |
| tonyb | Oh of course | 21:17 |
| fungi | 543607 would in theory cause their build to happen with the stable/pike constraints which they don't appear in | 21:18 |
| tonyb | Carry on ;P | 21:18 |
| fungi | but because we don't currently guess the right branch, they got build with the constraints file from master which did include them | 21:18 |
| tonyb | fungi: Yup I get it now, did I mention -ENOCOFFEE | 21:19 |
| tonyb | ? | 21:19 |
| fungi | that sounds like a critical priority bug you should be fixing | 21:20 |
| tonyb | :) | 21:21 |
| tonyb | I'm in a (voice) meeting so alas I'm stuck for a bit :( | 21:22 |
| * tonyb can see a bunch of machine driven constraints chnages in the future (perhaps PTG+2weeks?) | 21:23 | |
| *** wolverineav has quit IRC | 21:39 | |
| *** edmondsw has quit IRC | 21:39 | |
| *** onovy has quit IRC | 21:54 | |
| *** bobh has joined #openstack-release | 22:00 | |
| *** bobh has quit IRC | 22:04 | |
| *** bobh has joined #openstack-release | 22:06 | |
| *** bobh has quit IRC | 22:07 | |
| *** bobh has joined #openstack-release | 22:08 | |
| *** wolverineav has joined #openstack-release | 22:12 | |
| andreykurilin | hi folks! | 22:13 |
| andreykurilin | There is a change to python-novaclient which updates u-c link in tox.ini for stable/queens branch | 22:14 |
| andreykurilin | It looks like this change(from openstack release bot) proposes a wrong/broken link - https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=stable/queens | 22:15 |
| andreykurilin | Should `h=stable/queens` be replaced with `b=stable/queens` ? | 22:16 |
| *** wolverineav has quit IRC | 22:18 | |
| *** wolverineav has joined #openstack-release | 22:18 | |
| openstackgerrit | Merged openstack/releases master: Release Barbican Queens RC1 https://review.openstack.org/543579 | 22:19 |
| andreykurilin | hm...it looks like everything works now | 22:20 |
| andreykurilin | sorry | 22:20 |
| smcginnis | andreykurilin: Those are normal. | 22:21 |
| smcginnis | andreykurilin: The patch gets proposed right away at branching, but it will not be valid until requirements is unfrozen and branched too. | 22:21 |
| andreykurilin | smcginnis: yes. I saw "unfreezed" message and thought that everything should work now.. but gates delayed this a bit | 22:22 |
| andreykurilin | thanks for reply, will recheck the patch to novaclient | 22:22 |
| *** wolverineav has quit IRC | 22:23 | |
| *** annabelleB has quit IRC | 22:30 | |
| *** wolverineav has joined #openstack-release | 22:34 | |
| openstackgerrit | Sean McGinnis proposed openstack/releases master: Add queens release note links https://review.openstack.org/543645 | 22:39 |
| *** wolverin_ has joined #openstack-release | 22:41 | |
| *** wolverin_ has quit IRC | 22:42 | |
| *** wolverin_ has joined #openstack-release | 22:42 | |
| *** wolverineav has quit IRC | 22:45 | |
| *** wolverin_ has quit IRC | 22:55 | |
| *** armstrong has quit IRC | 22:58 | |
| *** rosmaita has quit IRC | 23:07 | |
| *** wolverineav has joined #openstack-release | 23:12 | |
| *** wolverineav has quit IRC | 23:15 | |
| *** wolverineav has joined #openstack-release | 23:15 | |
| *** kumarmn has quit IRC | 23:15 | |
| *** wolverin_ has joined #openstack-release | 23:20 | |
| *** wolverineav has quit IRC | 23:21 | |
| *** wolverin_ has quit IRC | 23:25 | |
| *** wolverineav has joined #openstack-release | 23:26 | |
| *** wolverineav has quit IRC | 23:26 | |
| *** wolverineav has joined #openstack-release | 23:27 | |
| *** wolverin_ has joined #openstack-release | 23:29 | |
| *** wolverin_ has quit IRC | 23:29 | |
| *** wolverin_ has joined #openstack-release | 23:30 | |
| *** bobh has quit IRC | 23:32 | |
| *** wolverineav has quit IRC | 23:33 | |
| *** claudiub has quit IRC | 23:34 | |
| *** kumarmn has joined #openstack-release | 23:38 | |
| *** wolverin_ has quit IRC | 23:42 | |
| *** wolverineav has joined #openstack-release | 23:42 | |
| *** wolverineav has quit IRC | 23:43 | |
| *** wolverineav has joined #openstack-release | 23:43 | |
| *** kumarmn has quit IRC | 23:44 | |
| *** pbandark has quit IRC | 23:46 | |
| *** wolverineav has quit IRC | 23:48 | |
| *** wolverineav has joined #openstack-release | 23:54 | |
| *** abelur has quit IRC | 23:54 | |
| *** kumarmn has joined #openstack-release | 23:55 | |
| *** kumarmn has quit IRC | 23:57 | |
| *** wolverineav has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!