*** Syed__ has quit IRC | 01:47 | |
*** jcannava has quit IRC | 02:36 | |
*** klindgren_ has joined #craton | 03:26 | |
*** klindgren has quit IRC | 03:29 | |
*** VW has joined #craton | 03:56 | |
*** harlowja has joined #craton | 04:37 | |
*** valw has quit IRC | 04:52 | |
*** valw has joined #craton | 04:53 | |
*** valw has quit IRC | 04:57 | |
*** harlowja has quit IRC | 05:46 | |
*** VW has quit IRC | 06:15 | |
sulo | morning o/ | 09:01 |
---|---|---|
*** klindgren_ has quit IRC | 09:04 | |
tojuvone | Noon. Pity that I normally work till 2pm and I am UTC+2. Anyhow catching the irc if my socks proxy hasn't kicked me out. | 09:57 |
sulo | tojuvone: work only till 2pm sounds like a good deas ;) | 10:13 |
sulo | *deal | 10:13 |
tojuvone | yes, well my "work mates" are more or less behind irc :D | 10:36 |
tojuvone | having these hours as skipping the rush hour in traffic and home when kids comes from school | 10:37 |
tojuvone | but really flexible hours and can easily do remote from home | 10:38 |
*** tojuvone has quit IRC | 11:15 | |
*** klindgren has joined #craton | 12:05 | |
*** tojuvone has joined #craton | 12:45 | |
*** VW has joined #craton | 14:07 | |
*** VW has quit IRC | 14:08 | |
*** VW has joined #craton | 14:08 | |
*** VW has quit IRC | 14:54 | |
*** VW has joined #craton | 15:00 | |
*** Mudpuppy has joined #craton | 15:03 | |
*** valw has joined #craton | 15:06 | |
*** Syed__ has joined #craton | 15:40 | |
*** VW has quit IRC | 15:46 | |
*** VW has joined #craton | 15:46 | |
*** valw has quit IRC | 16:01 | |
*** valw has joined #craton | 16:12 | |
*** klindgren_ has joined #craton | 16:15 | |
*** klindgren has quit IRC | 16:17 | |
*** valw has quit IRC | 16:17 | |
*** VW has quit IRC | 16:19 | |
*** valw has joined #craton | 16:25 | |
*** VW has joined #craton | 16:27 | |
Syed__ | Hi guys | 16:29 |
Syed__ | a question | 16:29 |
Syed__ | https://www.irccloud.com/pastebin/XEAMaxoF/ | 16:29 |
Syed__ | when i try to run tox -e functional, why does it gives me this | 16:29 |
*** VW has quit IRC | 16:31 | |
*** VW has joined #craton | 16:33 | |
tojuvone | I get the same | 16:36 |
*** valw has quit IRC | 16:39 | |
sigmavirus | Syed__: I see why | 16:41 |
sigmavirus | looks like there was a container set-up error | 16:41 |
*** valw has joined #craton | 16:43 | |
*** valw has quit IRC | 16:43 | |
*** valw has joined #craton | 16:43 | |
sigmavirus | Syed__: try out ttps://review.openstack.org/408713 ? | 16:45 |
sigmavirus | tojuvone: you too ^ ;) | 16:45 |
tojuvone | sigmavirus: Yes | 16:47 |
sigmavirus | Yes that helps or Yes you'll try that out? | 16:48 |
git-harry | I'm not a fan of skipping them. If Docker isn't set up and you want to run the tests, set it up. If it's not set up and you don't want to run the tests, don't run them. If the container setup fails for some reason not related to the code, rerun the tests. | 16:48 |
tojuvone | at least I had some problem with docker earlier. | 16:50 |
tojuvone | not related to this functest. | 16:50 |
sigmavirus | git-harry: we could raise an error in the setUp method which would cause those tests to be marked as "Error" | 16:51 |
Syed__ | Trying sigmavirus , thanks | 16:51 |
sigmavirus | but having a missing attribute in a test because we didn't stop the test from running is noisy and a bad experience for other developers | 16:52 |
sigmavirus | So whatever the solution we decide upon is, we need a solution | 16:52 |
git-harry | sigmavirus: that sounds reasonable to me | 16:52 |
sigmavirus | The solution could also to not use something flakey like docker | 16:52 |
jimbaker | tojuvone, Syed__, functional tests work for me | 17:00 |
jimbaker | better solutions welcome of course :) | 17:00 |
tojuvone | jimbaker: ok, I am reinstalling docker. Let's see. | 17:02 |
jimbaker | tojuvone, sigmavirus, sulo, Syed__ - core meeting on vidyo | 17:02 |
Syed__ | https://www.irccloud.com/pastebin/SEC0tPCf/ | 17:04 |
Syed__ | hmm ... | 17:04 |
sigmavirus | hah | 17:04 |
sigmavirus | well that's slightly better :P | 17:05 |
*** jovon has joined #craton | 17:05 | |
tojuvone | docker seems to work, but not in my functional test. | 17:12 |
Syed__ | same haah | 17:12 |
Syed__ | docker works | 17:12 |
jimbaker | git-harry, see https://gist.github.com/jimbaker/9cfc95a1a683bb22562b2c0d87811cd0 | 17:27 |
sigmavirus | Syed__: tojuvone what version of docker are you testing against? | 17:30 |
tojuvone | sigmavirus: docker-engine.x86_64 0:1.12.3-1.el7.centos | 17:37 |
sigmavirus | yeah that's what I'm using too | 17:37 |
sigmavirus | huh | 17:38 |
tojuvone | otherwise have not run craton under docker | 17:43 |
*** valw has quit IRC | 17:44 | |
tojuvone | just verified docker works like with: docker run -it ubuntu bash | 17:44 |
sigmavirus | tojuvone: try "docker build -t craton-api:latest ." | 17:46 |
*** valw has joined #craton | 17:46 | |
tojuvone | sigmavirus: It has s network problem. I need to define proxy somewhere (else than env var) | 17:58 |
sigmavirus | Syed__: ah, bet your'e having a proxy issue too :) | 17:59 |
tojuvone | I think I need to leave that for tomorrow. To me it seems a proxy problem: Cannot initiate the connection to archive.ubuntu.com:80 | 18:01 |
sigmavirus | tojuvone: okay | 18:01 |
sigmavirus | jimbaker: https://wiki.openstack.org/wiki/Keystone/Trusts | 18:01 |
Syed__ | i hope not, let me dig into it more | 18:01 |
tojuvone | I'll be at office tomorrow after 6am (UTC+2). Lullaby time :D | 18:02 |
sulo | so vidyo is acting up for me again | 18:05 |
sigmavirus | sulo: that's a feature | 18:12 |
*** harlowja has joined #craton | 18:17 | |
sulo | well .. it is useful sometims | 18:17 |
*** harlowja_ has joined #craton | 18:20 | |
*** harlowja has quit IRC | 18:22 | |
*** valw has quit IRC | 19:00 | |
*** valw has joined #craton | 19:03 | |
sigmavirus | jimbaker: sulo https://review.openstack.org/408767 | 19:04 |
*** valw has quit IRC | 19:07 | |
*** VW has quit IRC | 19:08 | |
*** VW has joined #craton | 19:08 | |
*** valw has joined #craton | 19:12 | |
*** VW has quit IRC | 19:13 | |
*** valw has quit IRC | 19:13 | |
*** valw has joined #craton | 19:13 | |
*** VW has joined #craton | 19:32 | |
*** VW has quit IRC | 19:36 | |
jimbaker | sigmavirus, it would be interesting to combine the capabilities of keystone with myproxy. over time, i'm sure we will see increased use of delegation | 19:51 |
jimbaker | and connectivity of such delegation systems for last mile aspects | 19:51 |
sigmavirus | what's myproxy? | 19:51 |
sigmavirus | also, https://review.openstack.org/408778 | 19:52 |
*** valw has quit IRC | 19:52 | |
*** valw has joined #craton | 19:55 | |
*** VW has joined #craton | 19:56 | |
*** valw has quit IRC | 19:58 | |
*** valw has joined #craton | 19:58 | |
*** VW has quit IRC | 20:00 | |
*** VW has joined #craton | 20:01 | |
*** tojuvone has quit IRC | 20:04 | |
jimbaker | sigmavirus, myproxy is a foundational component in opengrid; see http://grid.ncsa.illinois.edu/myproxy/protocol/ | 20:09 |
jimbaker | it's pretty cool stuff. unfortunately i don't see how we could actually use myproxy here with craton | 20:09 |
jimbaker | cloud is doomed to reinvent grid (basically the same thing) | 20:10 |
sulo | sigmavirus: Syed__: if y'all are trying to skip test when setup fails see my original patch .. https://review.openstack.org/#/c/397872/4/craton/tests/functional/test_cell_calls.py | 20:10 |
sulo | i dont remember who made the case that we dont want that | 20:11 |
sulo | ah it was git-harry | 20:11 |
jimbaker | git-harry, just so you know, tojuvone's change i just proposed caused a bunch of merge conflicts with your proposed changes | 20:12 |
jimbaker | Syed__, same ^^^ | 20:12 |
*** VW_ has joined #craton | 20:12 | |
jimbaker | all easy, just there to be resolved | 20:12 |
jimbaker | that was garbled | 20:13 |
jimbaker | tojuvone's change i just reviewed - and workflowed! - ... | 20:13 |
jimbaker | doing the malaprop day | 20:14 |
git-harry | sulo: yup, I think skipping is a bad idea. | 20:14 |
sulo | git-harry: yeah i agree | 20:14 |
git-harry | jimbaker: yup, I guess I lost that race and will have to rebase. | 20:15 |
jimbaker | git-harry, yep, it's the usual | 20:15 |
git-harry | aye | 20:15 |
*** VW has quit IRC | 20:16 | |
*** VW_ has quit IRC | 20:17 | |
jimbaker | Syed__, are you still working on https://review.openstack.org/#/c/396750/ - seems to be in the same domain as what git-harry is working on | 20:18 |
jimbaker | presumably we should abandon | 20:19 |
jimbaker | or perhaps not... anyway, want to dispose of old reviews, either by update or by abandoning | 20:24 |
*** valw has quit IRC | 20:25 | |
git-harry | jimbaker: Syed__ that's in the client, I've not done anything there yet. | 20:27 |
jimbaker | git-harry, yeah, i over generalized on your work | 20:27 |
jimbaker | presumably some impact on schema cleanup of course | 20:27 |
jimbaker | including i assume recent work by sulo | 20:28 |
sulo | ?? | 20:28 |
sulo | i really need to change my laptop | 20:29 |
jimbaker | sulo, maybe such things as returning variables | 20:29 |
jimbaker | sulo, you should definitely replace that old laptop. certainly a legacy! | 20:29 |
sulo | heh yeah | 20:29 |
jimbaker | too bad the mac laptops are not really for devs any more | 20:30 |
*** valw has joined #craton | 20:30 | |
*** valw_ has joined #craton | 20:32 | |
*** valw_ has quit IRC | 20:35 | |
*** valw has quit IRC | 20:35 | |
sigmavirus | sulo: we at least shouldn't leave the failure until the test tries to run and doesn't have an attribute to use | 20:38 |
sulo | sigmavirus: yeah i think sane hadling of a few thing might make it easy to see whats happening | 20:40 |
jimbaker | sulo, i wonder how well something like https://system76.com/laptops/oryx works, especially long term. (i had a thinkpad i loved with ubuntu, but support degraded over time.) also what rackspace supports. but dual SSD and 64GB sounds attractive. prices out at about the same as a MBP fwiw | 20:41 |
sulo | sigmavirus: you are refering to url i assume | 20:41 |
*** VW has joined #craton | 20:41 | |
sigmavirus | sulo: well that's the first one | 20:41 |
sigmavirus | i'm sure there will be others that pop up later as we move along | 20:41 |
jimbaker | and 8 hyperthreads! truly all-in-one system :) | 20:41 |
* sigmavirus doesn't like ubuntu very much honestly | 20:42 | |
sulo | it should only be url now i think ... will have to recheck | 20:42 |
sigmavirus | I don't dislike it | 20:42 |
sigmavirus | but I don't like it | 20:42 |
sulo | but that purely becuse we rely on container ip to make the call | 20:42 |
sulo | actually what we can do is | 20:42 |
sulo | map it to 127.0.0.1 to some port | 20:42 |
jimbaker | sigmavirus, i liked it for laptops. just didn't like the support aspect. frustrating given i was working at canonical... ;) | 20:43 |
sulo | but it doesnt make it any easy to see what you getting the error though | 20:43 |
*** VW has quit IRC | 20:43 | |
jimbaker | and supposedly thinkpad was the gold standard then | 20:43 |
*** VW_ has joined #craton | 20:43 | |
sigmavirus | sulo: I just think we should error out on those tests | 20:43 |
sigmavirus | Yeah I like thinkpads for various reasons | 20:43 |
sulo | error out ? | 20:43 |
sulo | raise error ? | 20:43 |
sigmavirus | sulo: so tests can error during setup | 20:43 |
sigmavirus | If we do that when container setup fails, that feels most honest | 20:43 |
sulo | well you saying technically its better because it will say "error" ? | 20:44 |
sulo | yeah i guess we can do that too | 20:44 |
sigmavirus | It'll still say error, but the reasoning will be less cryptic | 20:44 |
sulo | maybe error with some reason | 20:44 |
sigmavirus | Right | 20:45 |
* sigmavirus doesn't much like laptop keyboards with numpads | 20:45 | |
sigmavirus | that laptop is pretty though | 20:45 |
sulo | ok lets try that .. we have error_reason and _container_error or something that can be caught during setup | 20:45 |
sulo | so it might make more sense that way | 20:46 |
sigmavirus | We can also add our own exception handling https://github.com/testing-cabal/testtools/blob/master/testtools/testcase.py#L270 | 20:48 |
sulo | yeah, i was going to say we can simply fail the test right now on setup instead of skipping | 20:59 |
sulo | it will be a very simple change and it will fail with failureException with real reason to failure | 21:00 |
*** valw has joined #craton | 21:02 | |
sulo | something like this: https://gist.github.com/sulochan/ff9fe77ec2195a38bbc1d023a2fd488d | 21:04 |
jimbaker | sulo, fyi, resolved variables are not supported by /v1/cells/<cell-id> - but that's separate from the change you just proposed | 21:10 |
jimbaker | (and which i just workflowed) | 21:11 |
sulo | jimbaker: yeap, catching all that in functionl test right now for cells | 21:12 |
jimbaker | cool | 21:12 |
jimbaker | yeah, i'm reporting a bug about having common logic here | 21:12 |
sulo | jimbaker: ok | 21:12 |
jimbaker | we should also make project a root for variables | 21:12 |
sulo | jimbaker: create it separately for cells and host networks .. its easier to pair with tests | 21:13 |
jimbaker | that would let it be project -> region -> cell? -> device | 21:13 |
sulo | like i am doing cells now .. so i can depend on some of these fixes for func tests | 21:13 |
jimbaker | project -> workflow_def -> workflow | 21:13 |
jimbaker | project -> user | 21:13 |
sulo | jimbaker: you mean in the url ? | 21:14 |
jimbaker | no i mean in variable resolution | 21:14 |
sulo | ah | 21:14 |
sulo | yes +1 | 21:14 |
jimbaker | so one can define project-level variables | 21:14 |
jimbaker | sulo, yeah, i will keep the bug separate from networks, since they have been treated separately from a dev perspective | 21:15 |
jimbaker | could use same logic if we put in base of course | 21:15 |
jimbaker | ok, 3 bugs to be reported :) | 21:16 |
jimbaker | sulo, https://bugs.launchpad.net/craton/+bug/1648627 | 21:25 |
openstack | Launchpad bug 1648627 in craton "Use common logic for resolved variables for regions/cells/hosts REST API" [Undecided,New] | 21:25 |
jimbaker | and https://bugs.launchpad.net/craton/+bug/1648626 | 21:25 |
openstack | Launchpad bug 1648626 in craton "Projects should support variables, which other entities use in resolution" [Undecided,New] | 21:25 |
jimbaker | git-harry, thanks for updating the changes in gerrit | 21:29 |
git-harry | np | 21:30 |
jimbaker | one last one: https://bugs.launchpad.net/craton/+bug/1648630 | 21:32 |
openstack | Launchpad bug 1648630 in craton "Use common logic for resolved variables for network support REST API" [Undecided,New] | 21:32 |
*** valw_ has joined #craton | 21:34 | |
*** valw has quit IRC | 21:35 | |
*** valw_ has quit IRC | 21:41 | |
*** valw has joined #craton | 22:01 | |
*** Mudpuppy has quit IRC | 22:06 | |
*** Mudpuppy has joined #craton | 22:07 | |
*** valw has quit IRC | 22:07 | |
*** Mudpuppy has quit IRC | 22:11 | |
*** david-lyle_ has joined #craton | 23:05 | |
*** david-lyle has quit IRC | 23:07 | |
*** valw has joined #craton | 23:13 | |
*** david-lyle_ is now known as david-lyle | 23:35 | |
*** jovon has quit IRC | 23:38 | |
*** VW_ has quit IRC | 23:41 | |
*** VW has joined #craton | 23:43 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!