| *** syed__ has quit IRC | 01:56 | |
| *** syed__ has joined #craton | 02:17 | |
| *** Mudpuppy_ has quit IRC | 03:13 | |
| *** valw has joined #craton | 03:29 | |
| *** VW has joined #craton | 03:41 | |
| *** valw has quit IRC | 03:49 | |
| *** Mudpuppy has joined #craton | 03:56 | |
| *** valw has joined #craton | 04:26 | |
| *** VW has quit IRC | 04:35 | |
| *** mhayden has quit IRC | 04:38 | |
| *** valw has quit IRC | 04:49 | |
| *** mhayden has joined #craton | 04:50 | |
| *** VW has joined #craton | 05:14 | |
| *** VW has quit IRC | 05:18 | |
| *** valw has joined #craton | 05:20 | |
| *** valw has quit IRC | 05:25 | |
| *** tojuvone has quit IRC | 06:34 | |
| *** valw has joined #craton | 06:38 | |
| *** valw has quit IRC | 06:42 | |
| *** Mudpuppy has quit IRC | 06:56 | |
| *** Mudpuppy has joined #craton | 07:27 | |
| *** Mudpuppy has quit IRC | 07:32 | |
| *** syed__ has quit IRC | 08:06 | |
| *** valw has joined #craton | 08:39 | |
| *** valw has quit IRC | 08:43 | |
| *** tojuvone has joined #craton | 10:30 | |
| *** Mudpuppy has joined #craton | 12:26 | |
| *** Mudpuppy has quit IRC | 12:28 | |
| *** valw has joined #craton | 12:41 | |
| *** valw has quit IRC | 12:46 | |
| *** valw has joined #craton | 13:30 | |
| *** VW has joined #craton | 13:36 | |
| *** VW has quit IRC | 13:40 | |
| *** valw has quit IRC | 13:49 | |
| *** VW has joined #craton | 14:27 | |
| sigmavirus | sulo: https://gist.github.com/sigmavirus24/33bcf97f378690f93b274b1481e32732 is what I get when I try out your functional test suite patch | 14:39 |
|---|---|---|
| *** valw has joined #craton | 14:48 | |
| *** valw has quit IRC | 14:54 | |
| sulo | sigmavirus: :( | 14:59 |
| sigmavirus | sulo: it's very weird, but I'm also running into the problems jimbaker mentioned yesterday | 14:59 |
| sulo | hmm thats interesting though .. | 14:59 |
| sigmavirus | And my ubuntu images was created 5/6 days ago | 14:59 |
| sigmavirus | I'm running this all on a cloud server | 14:59 |
| * sigmavirus tries running docker build today | 14:59 | |
| sulo | but that error sounds like it failed to get the image built ? | 15:00 |
| sigmavirus | Still failing with http://archive.ubuntu.com/ubuntu/pool/main/m/mysql-5.7/mysql-common_5.7.15-0ubuntu0.16.04.1_all.deb | 15:00 |
| sulo | i dont know why i dont get this problem | 15:00 |
| sigmavirus | Or rather, it can't fetch mysql-common and libmysqlclient stuff | 15:00 |
| sigmavirus | Might be using a different mirror | 15:00 |
| sigmavirus | I'm hitting IP: 91.189.88.162 80 | 15:00 |
| sulo | 91.189.88.149 | 15:01 |
| sulo | well that package doest exist | 15:02 |
| *** valw has joined #craton | 15:03 | |
| sigmavirus | huh | 15:03 |
| sigmavirus | Idk what's specifying that version (if anything) | 15:03 |
| sulo | sigmavirus: your image id for ubuntu e4415b714b62 ? | 15:05 |
| sigmavirus | Yep! | 15:05 |
| sigmavirus | "Created 6 days ago" | 15:05 |
| sulo | yes | 15:05 |
| sigmavirus | me is going to clear teh image cache | 15:05 |
| sigmavirus | ^ should have been /me | 15:07 |
| sigmavirus | SO MANY FREE GIGS NOW | 15:07 |
| sigmavirus | :P | 15:07 |
| * sigmavirus only hates docker a little | 15:07 | |
| sigmavirus | wow | 15:15 |
| sigmavirus | there were a lot of dangling images :/ | 15:15 |
| sigmavirus | STALE IMAGE CACHE | 15:15 |
| * sigmavirus shakes fist | 15:15 | |
| sigmavirus | jimbaker: you might need to do `docker rmi -f $(docker images -qf dangling=true)` to get past the build errors | 15:15 |
| sulo | sigmavirus: Get:1 http://archive.ubuntu.com/ubuntu xenial-updates/main amd64 mysql-common all 5.7.16-0ubuntu0.16.04.1 [15.0 kB] | 15:16 |
| sulo | is what my image is doing | 15:16 |
| sigmavirus | sulo: yeah, I've no trouble building the image now | 15:17 |
| sigmavirus | I had to clear out the old cached bits | 15:17 |
| sulo | cool | 15:18 |
| sulo | sigmavirus: does that help the test maybe ? | 15:18 |
| sigmavirus | I think this is why docker's docs recommend deleting the apt cache in /var/lists/apt after an apt-get update && apt-get install | 15:18 |
| sigmavirus | sulo: I would expect it will | 15:18 |
| sigmavirus | waiting for the build to finish | 15:18 |
| sulo | ok | 15:18 |
| sigmavirus | figure building right before might make the tests a bit faster? | 15:18 |
| sulo | yeah | 15:19 |
| sulo | it does | 15:19 |
| sigmavirus | also building ensures it won't error :D | 15:19 |
| sigmavirus | (or lessens the likelihood probably) | 15:19 |
| * sigmavirus thinks it's building the image again lol | 15:20 | |
| sulo | it build the craton image | 15:20 |
| sigmavirus | woo | 15:21 |
| sigmavirus | looks to have been successful | 15:21 |
| sigmavirus | Yep! | 15:21 |
| sulo | it should be quicker though (not having to download the base) | 15:21 |
| sulo | also https://review.openstack.org/#/c/401312/ | 15:22 |
| *** Mudpuppy has joined #craton | 15:29 | |
| *** syed__ has joined #craton | 15:54 | |
| *** valw has quit IRC | 15:56 | |
| *** valw has joined #craton | 15:57 | |
| jimbaker | sigmavirus, thanks for the guidance. i have only 3 dangling images around. good thing? ;) ~3GB to be freed... | 16:17 |
| sigmavirus | I had ... a lot | 16:18 |
| jimbaker | i did successively run docker rmi with docker images yesterday to get past ref count issues (or so i assume) | 16:18 |
| jimbaker | actually i had no idea what i was doing, just felt good running it multiple times. kill! kill! :) | 16:19 |
| jimbaker | but now a cleaner system | 16:19 |
| sulo | docker rm $(docker ps -a -q); docker rmi $(docker images | grep "^<none>" | awk "{print $3}") | 16:20 |
| jimbaker | sulo, so https://hub.docker.com/_/ubuntu/ - does this mean we should use FROM ubuntu:latest in Dockerfile ? | 16:21 |
| jimbaker | sulo, generally commands like that are just golf versions of removing everything one ever cared about on a laptop ;) | 16:21 |
| sulo | jimbaker: haha | 16:23 |
| jimbaker | i was not aware of docker ps -a - so that explains those zombies holding onto images | 16:25 |
| jimbaker | but hey, -f | 16:26 |
| jimbaker | sulo, back to my other question re ubuntu:latest | 16:26 |
| jimbaker | i assume this is the right tag for us | 16:26 |
| sulo | jimbaker: re: update, i personally like images update without letting me know .. | 16:27 |
| sulo | *i dont like* | 16:27 |
| sulo | if we want to just have latest .. we should just pull latest on the image rather than using lastest image | 16:28 |
| sulo | latest image will change without letting us know | 16:28 |
| jimbaker | sulo, maybe then just a quick doc update then on the build steps | 16:28 |
| jimbaker | in Dockerfile | 16:28 |
| jimbaker | and corresponding docs | 16:28 |
| sulo | yeah +1 | 16:29 |
| sulo | let me file an issue and fix that too | 16:30 |
| jimbaker | sulo, sounds good | 16:31 |
| sulo | https://review.openstack.org/#/c/401343/ | 16:38 |
| sulo | jimbaker: also did tests work for you with these changes ? | 16:39 |
| sulo | it worked for sigmavirus after the images cleanup | 16:40 |
| jimbaker | sulo, i was re-running, and just realized i forgot to pull latest. let my try again... | 16:40 |
| sulo | jimbaker: also curious if you saw any errors yesterday, when it was hanging | 16:41 |
| sulo | it can hang only if we didnt catch the excption .. i didnt think there was a place where that could happen | 16:41 |
| sulo | but maybe i am wrong | 16:41 |
| jimbaker | yeah, this hang is almost certainly because of the use of threading | 16:42 |
| jimbaker | and nontermination thereof | 16:43 |
| jimbaker | sulo, so here's what i saw yesterday - https://gist.github.com/jimbaker/9fe0907a402df96ddd8b5ad21cf853c2 | 16:44 |
| jimbaker | i'm going to rerun now | 16:44 |
| jimbaker | also sounds good about removing py34 support. wanted to see that happen :) | 16:44 |
| jimbaker | asap | 16:45 |
| jimbaker | i assume 3.6 is still on its release schedule for later this year | 16:45 |
| sigmavirus | jimbaker: yes, but we'll probably support 3.5 and 3.6 so long as xenial is the base image for the infra gate | 16:53 |
| sigmavirus | also one snag with moving to py.test is the fact that it's not already in global-requirements | 16:54 |
| jimbaker | 3.5 and 3.6 is good; just wanted to drop 3.4 | 16:57 |
| sigmavirus | 3.4 is so great though! | 16:57 |
| sigmavirus | :P | 16:57 |
| jimbaker | strictly speaking, i don't believe 3.4 gets in the way of type hints. but using type hints + mypy is the next thing i would like to see us trying | 16:57 |
| jimbaker | right there with coverage | 16:58 |
| jimbaker | having said that: 3.6 seems like an awesome codebase from my modest following to-date. i should spend a little time today going through the what's new | 16:59 |
| jimbaker | wordcode... ordered dicts by default... | 16:59 |
| sulo | jimbaker: ok i see what needs to fix for your error yesterday .. i want properly catching it | 17:00 |
| sulo | wasnt | 17:00 |
| sigmavirus | oh yeah that's not what I was running into | 17:04 |
| jimbaker | sulo, functional testing still stalling on my laptop, even with the latest change about 20 min ago | 17:20 |
| jimbaker | sulo, assuming you meant patch set 4 was supposed to fix | 17:21 |
| sigmavirus | jimbaker: sulo (regarding our functional tests) http://lists.openstack.org/pipermail/openstack-dev/2016-November/107784.html is relevant to our interest | 17:29 |
| sigmavirus | *interests | 17:29 |
| jimbaker | sigmavirus, very nice | 18:09 |
| jimbaker | so we should add tools/test-setup.sh for this purpose to https://review.openstack.org/#/c/397872/ | 18:10 |
| jimbaker | sigmavirus, does this impact the discussion in https://review.openstack.org/#/c/397872/4/craton/tests/functional/test_cell_calls.py ? | 18:11 |
| jimbaker | actually it sounds like this proposed support is still WIP. so a separate branch then | 18:13 |
| jimbaker | commented accordingly | 18:21 |
| jimbaker | re 3.6 - nice to see pep 526 was adopted | 18:25 |
| *** valw has quit IRC | 18:29 | |
| *** jovon has joined #craton | 19:08 | |
| *** valw has joined #craton | 19:10 | |
| *** Mudpuppy_ has joined #craton | 19:15 | |
| *** Mudpuppy has quit IRC | 19:19 | |
| *** Mudpuppy_ has quit IRC | 19:19 | |
| *** VW_ has joined #craton | 19:36 | |
| *** VW_ has quit IRC | 19:39 | |
| *** VW_ has joined #craton | 19:39 | |
| sigmavirus | jimbaker: which one is 526 again? | 19:39 |
| *** VW has quit IRC | 19:39 | |
| *** VW_ has quit IRC | 19:43 | |
| *** VW has joined #craton | 20:09 | |
| syed__ | guys, a general question, for https://bugs.launchpad.net/craton/+bug/1644270 writing tests for endpoints in routes.py | 20:12 |
| openstack | Launchpad bug 1644270 in craton "Add test to validate routes/schema" [Medium,Triaged] | 20:12 |
| syed__ | should the tests be for api endpoints | 20:12 |
| *** VW has quit IRC | 20:13 | |
| *** VW has joined #craton | 20:43 | |
| sigmavirus | syed__: what? | 20:43 |
| syed__ | I am not sure to what we means for endpoint tests ? sigmavirus | 20:45 |
| syed__ | https://github.com/openstack/craton/blob/master/craton/api/v1/routes.py#L16, should it be checking urls | 20:45 |
| sigmavirus | I think it actually wants you to verify that for every route in that list there's an entry in https://github.com/openstack/craton/blob/master/craton/api/v1/schemas.py#L1055 that has a valid schema associated with it | 20:49 |
| sigmavirus | It might be a bit ... icky though | 20:49 |
| -openstackstatus- NOTICE: Due to a problem with the cinder volume backing the log server, jobs are failing with POST_FAILURE. Please avoid issuing 'recheck' commands until the issue is resolved. | 21:05 | |
| jimbaker | sigmavirus, pep 526 is new syntax for being able to declare the type of variables, and without requiring the variable to be initializaed. mypy only supported this in comments prior to 3.6 | 21:10 |
| jimbaker | so it complements being able to do such declarations in the function signature | 21:11 |
| jimbaker | often one can infer the types without declarations, but not always | 21:12 |
| jimbaker | without *such* declarations | 21:12 |
| jimbaker | esp from the function signature | 21:12 |
| *** Mudpuppy has joined #craton | 21:19 | |
| *** jovon has quit IRC | 21:28 | |
| *** VW has quit IRC | 21:35 | |
| *** valw has quit IRC | 22:08 | |
| *** jovon has joined #craton | 22:16 | |
| *** VW has joined #craton | 22:18 | |
| -openstackstatus- NOTICE: The affected filesystems on the log server are repaired. Please leave 'recheck' comments on any changes which failed with POST_FAILURE. | 22:50 | |
| *** VW has quit IRC | 23:29 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!