| *** edmondsw has quit IRC | 00:00 | |
| openstackgerrit | melissaml proposed openstack/requirements master: Trivial: Update pypi url to new url https://review.openstack.org/563344 | 00:06 |
|---|---|---|
| openstackgerrit | Isaku Yamahata proposed openstack/requirements master: do not merge: test neutron-lib master https://review.openstack.org/563359 | 01:23 |
| prometheanfire | tonyb: not sure if around, but mind going through the queue? | 01:32 |
| *** edmondsw has joined #openstack-requirements | 01:44 | |
| *** edmondsw has quit IRC | 01:48 | |
| openstackgerrit | melissaml proposed openstack/requirements master: Trivial: Update pypi url to new url https://review.openstack.org/563344 | 02:22 |
| openstackgerrit | melissaml proposed openstack/requirements master: Trivial: Update pypi url to new url https://review.openstack.org/563344 | 02:24 |
| prometheanfire | thanks | 02:29 |
| openstackgerrit | Tony Breeds proposed openstack/requirements master: Trivial: Update pypi url to new url https://review.openstack.org/563344 | 02:32 |
| prometheanfire | tonyb: same with the 3.6 for pypi? | 02:33 |
| prometheanfire | tonyb: same with the 3.6 for mypy | 02:33 |
| tonyb | prometheanfire: Sure I didn't notice loosing them | 02:33 |
| openstackgerrit | melissaml proposed openstack/requirements master: Trivial: Update pypi url to new url https://review.openstack.org/563362 | 02:34 |
| tonyb | prometheanfire: anyway if you wait for all the +W'd u-c updates and then rerun the tool with the right --version-map it'll take care of itself | 02:34 |
| prometheanfire | the current code is broken without the reverts | 02:34 |
| prometheanfire | tonyb: see https://review.openstack.org/562946 | 02:35 |
| tonyb | prometheanfire: Sure but that's a tangent | 02:35 |
| prometheanfire | ? | 02:35 |
| tonyb | melissaml isn't 563362 a dupe of the fixed 563344 | 02:37 |
| openstackgerrit | Matthew Thode proposed openstack/requirements master: update upper-constraints.txt https://review.openstack.org/562351 | 02:37 |
| tonyb | prometheanfire: We seem to be talking passed eachother again | 02:37 |
| prometheanfire | probably | 02:38 |
| prometheanfire | oh irc, how you are both awesome and shitty | 02:38 |
| tonyb | prometheanfire: I'm saying that I wont +2+W 562351 because of the 3.6 issue. so if you re-run the tool uyou used to generate that review, which I assume is our generate tool [which I also gather based on what you've said is locally patched]) with the --version-map args I suggested will take care of *all* the ==="3.6" things | 02:40 |
| tonyb | prometheanfire: I think you're saying it has to be run manually because the bot is broken | 02:40 |
| tonyb | prometheanfire: which to me feel like a tangent to the =="3.6" thing | 02:41 |
| prometheanfire | oh sure | 02:41 |
| prometheanfire | that I agree with | 02:41 |
| tonyb | prometheanfire: I assume the 'revert's are the ones I asked for deatils on? | 02:43 |
| prometheanfire | yep | 02:44 |
| tonyb | prometheanfire: so the bot is double broken right now? | 02:45 |
| tonyb | prometheanfire: how | 02:45 |
| tonyb | 'd the chnages break the bot? | 02:45 |
| prometheanfire | the two changes that are being reverted caused the bot to output to look like this https://review.openstack.org/562946 | 02:46 |
| tonyb | prometheanfire: okay It'd be cool if that was in a commit message somewhere but Meh I've added it to the review and +2+Wd them | 02:48 |
| * tonyb goes for a walk with the kids\ | 02:48 | |
| prometheanfire | ya, I realized that a bit late | 02:48 |
| prometheanfire | cya | 02:48 |
| prometheanfire | the command isn't quite perfect, doesn't want to output three versions | 03:01 |
| prometheanfire | --version-map 3.4:3.5 --version-map 3.5:3.4 --version-map 3.5:3.6 --version-map 3.4:3.6 | 03:01 |
| prometheanfire | only outputs py35/36 | 03:02 |
| *** edmondsw has joined #openstack-requirements | 03:32 | |
| *** edmondsw has quit IRC | 03:37 | |
| openstackgerrit | Merged openstack/requirements master: update constraint for tripleo-common to new release 9.0.1 https://review.openstack.org/563258 | 03:53 |
| openstackgerrit | Merged openstack/requirements master: Trivial: Remove redundant space https://review.openstack.org/561132 | 03:53 |
| openstackgerrit | Merged openstack/requirements stable/queens: Mark trollius only be used under py2 https://review.openstack.org/551111 | 04:04 |
| openstackgerrit | Merged openstack/requirements master: Update skydive version to 0.4.5 https://review.openstack.org/560305 | 04:04 |
| openstackgerrit | Merged openstack/requirements master: Update skydive-client lc version to 0.4.5 https://review.openstack.org/561785 | 04:04 |
| openstackgerrit | Merged openstack/requirements master: Add jira client library to g-r https://review.openstack.org/562151 | 04:04 |
| openstackgerrit | Merged openstack/requirements master: update constraint for mistral-lib to new release 0.5.0 https://review.openstack.org/563182 | 04:10 |
| openstackgerrit | Merged openstack/requirements master: update constraint for python-mistralclient to new release 3.4.0 https://review.openstack.org/563185 | 04:10 |
| openstackgerrit | Merged openstack/requirements master: update constraint for monasca-common to new release 2.9.0 https://review.openstack.org/563121 | 04:10 |
| openstackgerrit | Merged openstack/requirements master: update constraint for monasca-statsd to new release 1.10.0 https://review.openstack.org/563118 | 04:10 |
| openstackgerrit | Merged openstack/requirements master: update constraint for python-monascaclient to new release 1.11.0 https://review.openstack.org/563117 | 04:22 |
| openstackgerrit | Merged openstack/requirements master: Revert "show commands and output together" https://review.openstack.org/563161 | 04:22 |
| openstackgerrit | Merged openstack/requirements master: Revert "simplify error handling in generate command" https://review.openstack.org/563162 | 05:10 |
| openstackgerrit | Merged openstack/requirements master: fix unicode errors https://review.openstack.org/561761 | 05:10 |
| openstackgerrit | Merged openstack/requirements master: update constraint for instack-undercloud to new release 9.0.0 https://review.openstack.org/562586 | 05:46 |
| openstackgerrit | Merged openstack/requirements master: Trivial: Update pypi url to new url https://review.openstack.org/563344 | 05:46 |
| rm_work | anyone know what is going on with http://logs.openstack.org/63/562063/8/check/requirements-check/95e28e2/job-output.txt.gz ? | 06:17 |
| rm_work | these are my requirements changes: https://review.openstack.org/#/c/562063/8/requirements.txt | 06:17 |
| rm_work | and lower-constraints: https://review.openstack.org/#/c/562063/8/lower-constraints.txt | 06:17 |
| rm_work | i copied them directly from what's in g-r: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L87-L88 | 06:17 |
| rm_work | https://github.com/openstack/requirements/blob/master/lower-constraints.txt#L126 | 06:18 |
| rm_work | but, it's complaining about a ton of stuff I didn't touch... T_T | 06:19 |
| prometheanfire | rm_work: first one that wants to touch it gets to update lower-constraints to something working | 06:28 |
| rm_work | erg | 06:29 |
| rm_work | so actually ANYTHING touching requirements would fail now? | 06:29 |
| rm_work | and there's no automated job to fix these? | 06:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.openstack.org/563393 | 06:29 |
| prometheanfire | this section http://logs.openstack.org/63/562063/8/check/requirements-check/95e28e2/job-output.txt.gz#_2018-04-21_00_01_01_145661 needs to be used as source for the change | 06:29 |
| rm_work | ok sooooo | 06:29 |
| prometheanfire | nope, not for the initial change | 06:29 |
| rm_work | I have to manually go through and update this whole list? and ... we need to do this every time something in the g-r repo changes? T_T | 06:29 |
| rm_work | is this real life | 06:30 |
| rm_work | is there a script somewhere to do this? | 06:31 |
| prometheanfire | you should only have to do it once iirc | 06:31 |
| prometheanfire | it's checking your requirements | 06:31 |
| prometheanfire | not gr | 06:31 |
| rm_work | errr | 06:31 |
| prometheanfire | gr doesn't track mins anymore | 06:31 |
| rm_work | hmmmm | 06:31 |
| rm_work | yeah but l-c does | 06:31 |
| rm_work | this is our l-c file in our repo | 06:31 |
| prometheanfire | the lc needs to match your req min | 06:31 |
| rm_work | erg | 06:31 |
| rm_work | so it's our own test-requirements.txt minimum | 06:31 |
| prometheanfire | it's not looking at our repo at all for that | 06:31 |
| rm_work | but, somehow this merged to begin with? | 06:32 |
| rm_work | were these checks not running at that point I guess? | 06:32 |
| rm_work | when i run `tox -e lower-constraints` locally it passes | 06:32 |
| prometheanfire | probably bad ordering, it was more haphazard than I'd have liked | 06:32 |
| rm_work | how do i test this locally | 06:32 |
| prometheanfire | the job for reqs changes is probably in project-config | 06:33 |
| rm_work | so this is a lower-constraints issue but it doesn't get tested by our lower-constraints job? doesn't that mean our job is broken? | 06:33 |
| prometheanfire | the content of lower-constraints gets checked by a reqs job | 06:34 |
| rm_work | hmmm | 06:34 |
| rm_work | i wonder what our `lower-constraints` tox env does | 06:34 |
| prometheanfire | lower-constraints itself is tested (consumed) by the lower-constraints job | 06:35 |
| rm_work | looks like installs our reqs at l-c values and runs unit tests in py2 and py3 | 06:35 |
| prometheanfire | yep | 06:35 |
| rm_work | but it doesn't actually validate that l-c itself is valid | 06:35 |
| prometheanfire | your lc must match your (test)reqs | 06:35 |
| prometheanfire | correct | 06:35 |
| rm_work | just that the package definitions there will work for our unit tests, k | 06:35 |
| prometheanfire | one is a test job, one is a rule check | 06:35 |
| rm_work | ok | 06:35 |
| rm_work | so IS THERE a script to parse this output and handle things? or no | 06:37 |
| rm_work | I guess I will write one if there isn't | 06:37 |
| rm_work | simple awk/sed ought to do it | 06:38 |
| prometheanfire | no | 06:38 |
| rm_work | k | 06:38 |
| openstackgerrit | Dirk Mueller proposed openstack/requirements master: WIP: Add cross testing for py36 https://review.openstack.org/554824 | 06:41 |
| rm_work | cat lcdiff.txt | awk '/is constrained/{ print $7, $11 }'| awk -F\' '{ print $2, $3 }' | 06:45 |
| rm_work | ^^ seems to do the first part | 06:45 |
| openstackgerrit | Matthew Thode proposed openstack/requirements master: update upper-constraints.txt https://review.openstack.org/562351 | 06:46 |
| openstackgerrit | Linda Wang proposed openstack/requirements stable/pike: Update constraint for tempest to 17.1.0 https://review.openstack.org/563394 | 06:50 |
| *** edmondsw has joined #openstack-requirements | 07:09 | |
| *** edmondsw has quit IRC | 07:14 | |
| rm_work | cat lcdiff.txt | awk '/is constrained/{ print $7 $11 }' | awk -F\' '{ printf "s/%s===.*/%s===%s/\n", $2, $2, $3 }' | xargs -I REP sed -i ".bak" 'REP' lower-constraints.txt | 07:16 |
| rm_work | prometheanfire: ^^ seems to do it ... except... there's not actually any changes. I started verifying by hand too, because i thought maybe just my script was broken | 07:17 |
| rm_work | oh, hmm.... maybe I just didn't verify enough | 07:17 |
| rm_work | because yeah my = are off | 07:17 |
| rm_work | lol | 07:17 |
| rm_work | ok well fixed my script... both were true | 07:19 |
| rm_work | my script was wrong but now it's working and there are definitely no changes | 07:19 |
| rm_work | cat lcdiff.txt | awk '/is constrained/{ print $7 $11 }' | awk -F\' '{ printf "s/%s==.*/%s==%s/\n", $2, $2, $3 }' | xargs -I REP sed -i ".bak" 'REP' lower-constraints.txt | 07:19 |
| rm_work | (where lcdiff.txt is just all those lines saying what was "wrong") | 07:20 |
| rm_work | ohhhhh there we go, i used the wrong version, no wonder | 07:20 |
| rm_work | thanks rubber ducky :) | 07:21 |
| rm_work | cool | 07:21 |
| rm_work | cat lcdiff.txt | awk '/is constrained/{ print $7 $20 }' | awk -F\' '{ printf "s/%s==.*/%s==%s/\n", $2, $2, $3 }' | xargs -I REP sed -i ".bak" 'REP' lower-constraints.txt | 07:21 |
| rm_work | works! thanks | 07:22 |
| rm_work | actually can replace the `cat lcdiff.txt |` with `wget http://logs.openstack.org/63/562063/8/check/requirements-check/95e28e2/job-output.txt.gz -O - |` | 07:30 |
| rm_work | ah, doesn't handle if the thing isn't *IN* l-c | 07:58 |
| rm_work | oh, no, case issue | 08:06 |
| rm_work | wget http://logs.openstack.org/96/563396/1/check/requirements-check/0b1fad0/job-output.txt.gz -O - | awk '/is constrained/{ print $7 $20 }' | awk -F\' '{ printf "s/\\\\(%s\\\\)==.*/\\\\1==%s/I\n", $2, $3 }' | xargs -I REP gsed -i.bak 'REP' lower-constraints.txt | 08:07 |
| rm_work | coolbeans | 08:07 |
| openstackgerrit | Merged openstack/requirements master: update constraint for python-novaclient to new release 10.2.0 https://review.openstack.org/563107 | 08:23 |
| *** edmondsw has joined #openstack-requirements | 08:57 | |
| *** edmondsw has quit IRC | 09:01 | |
| *** edmondsw has joined #openstack-requirements | 10:46 | |
| *** edmondsw has quit IRC | 10:50 | |
| *** edmondsw has joined #openstack-requirements | 12:34 | |
| *** edmondsw has quit IRC | 12:38 | |
| dhellmann | prometheanfire , rm_work : the script in https://review.openstack.org/#/c/558610/ will fix your settings | 13:32 |
| dhellmann | that's approved now, so it's in the requirements repo | 13:32 |
| dhellmann | prometheanfire , rm_work : http://lists.openstack.org/pipermail/openstack-dev/2018-April/129056.html | 13:34 |
| *** edmondsw has joined #openstack-requirements | 16:11 | |
| *** edmondsw has quit IRC | 16:15 | |
| *** edmondsw has joined #openstack-requirements | 17:59 | |
| *** edmondsw has quit IRC | 18:04 | |
| *** edmondsw has joined #openstack-requirements | 19:47 | |
| *** edmondsw has quit IRC | 19:52 | |
| *** edmondsw has joined #openstack-requirements | 21:36 | |
| *** edmondsw has quit IRC | 21:40 | |
| *** hongbin_ has joined #openstack-requirements | 22:12 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!