Thursday, 2016-12-08

*** Syed__ has quit IRC01:47
*** jcannava has quit IRC02:36
*** klindgren_ has joined #craton03:26
*** klindgren has quit IRC03:29
*** VW has joined #craton03:56
*** harlowja has joined #craton04:37
*** valw has quit IRC04:52
*** valw has joined #craton04:53
*** valw has quit IRC04:57
*** harlowja has quit IRC05:46
*** VW has quit IRC06:15
sulomorning o/09:01
*** klindgren_ has quit IRC09:04
tojuvoneNoon. 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
sulotojuvone: work only till 2pm sounds like a good deas ;)10:13
sulo*deal10:13
tojuvoneyes, well my "work mates" are more or less behind irc :D10:36
tojuvonehaving these hours as skipping the rush hour in traffic and home when kids comes from school10:37
tojuvonebut really flexible hours and can easily do remote from home10:38
*** tojuvone has quit IRC11:15
*** klindgren has joined #craton12:05
*** tojuvone has joined #craton12:45
*** VW has joined #craton14:07
*** VW has quit IRC14:08
*** VW has joined #craton14:08
*** VW has quit IRC14:54
*** VW has joined #craton15:00
*** Mudpuppy has joined #craton15:03
*** valw has joined #craton15:06
*** Syed__ has joined #craton15:40
*** VW has quit IRC15:46
*** VW has joined #craton15:46
*** valw has quit IRC16:01
*** valw has joined #craton16:12
*** klindgren_ has joined #craton16:15
*** klindgren has quit IRC16:17
*** valw has quit IRC16:17
*** VW has quit IRC16:19
*** valw has joined #craton16:25
*** VW has joined #craton16:27
Syed__Hi guys16:29
Syed__a question16:29
Syed__https://www.irccloud.com/pastebin/XEAMaxoF/16:29
Syed__when i try to run tox -e functional, why does it gives me this16:29
*** VW has quit IRC16:31
*** VW has joined #craton16:33
tojuvoneI get the same16:36
*** valw has quit IRC16:39
sigmavirusSyed__: I see why16:41
sigmaviruslooks like there was a container set-up error16:41
*** valw has joined #craton16:43
*** valw has quit IRC16:43
*** valw has joined #craton16:43
sigmavirusSyed__: try out ttps://review.openstack.org/408713 ?16:45
sigmavirustojuvone: you too ^ ;)16:45
tojuvonesigmavirus: Yes16:47
sigmavirusYes that helps or Yes you'll try that out?16:48
git-harryI'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
tojuvoneat least I had some problem with docker earlier.16:50
tojuvonenot related to this functest.16:50
sigmavirusgit-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 , thanks16:51
sigmavirusbut having a missing attribute in a test because we didn't stop the test from running is noisy and a bad experience for other developers16:52
sigmavirusSo whatever the solution we decide upon is, we need a solution16:52
git-harrysigmavirus: that sounds reasonable to me16:52
sigmavirusThe solution could also to not use something flakey like docker16:52
jimbakertojuvone, Syed__, functional tests work for me17:00
jimbakerbetter solutions welcome of course :)17:00
tojuvonejimbaker: ok, I am reinstalling docker. Let's see.17:02
jimbakertojuvone, sigmavirus, sulo, Syed__ - core meeting on vidyo17:02
Syed__https://www.irccloud.com/pastebin/SEC0tPCf/17:04
Syed__hmm ...17:04
sigmavirushah17:04
sigmaviruswell that's slightly better :P17:05
*** jovon has joined #craton17:05
tojuvonedocker seems to work, but not in my functional test.17:12
Syed__same haah17:12
Syed__docker works17:12
jimbakergit-harry, see https://gist.github.com/jimbaker/9cfc95a1a683bb22562b2c0d87811cd017:27
sigmavirusSyed__: tojuvone what version of docker are you testing against?17:30
tojuvonesigmavirus: docker-engine.x86_64 0:1.12.3-1.el7.centos17:37
sigmavirusyeah that's what I'm using too17:37
sigmavirushuh17:38
tojuvoneotherwise have not run craton under docker17:43
*** valw has quit IRC17:44
tojuvonejust verified docker works like with: docker run -it ubuntu bash17:44
sigmavirustojuvone: try "docker build -t craton-api:latest ."17:46
*** valw has joined #craton17:46
tojuvonesigmavirus: It has s network problem. I need to define proxy somewhere (else than env var)17:58
sigmavirusSyed__: ah, bet your'e having a proxy issue too :)17:59
tojuvoneI think I need to leave that for tomorrow. To me it seems a proxy problem: Cannot initiate the connection to archive.ubuntu.com:8018:01
sigmavirustojuvone: okay18:01
sigmavirusjimbaker: https://wiki.openstack.org/wiki/Keystone/Trusts18:01
Syed__i hope not, let me dig into it more18:01
tojuvoneI'll be at office tomorrow after 6am (UTC+2). Lullaby time :D18:02
suloso vidyo is acting up for me again18:05
sigmavirussulo: that's a feature18:12
*** harlowja has joined #craton18:17
sulowell .. it is useful sometims18:17
*** harlowja_ has joined #craton18:20
*** harlowja has quit IRC18:22
*** valw has quit IRC19:00
*** valw has joined #craton19:03
sigmavirusjimbaker: sulo https://review.openstack.org/40876719:04
*** valw has quit IRC19:07
*** VW has quit IRC19:08
*** VW has joined #craton19:08
*** valw has joined #craton19:12
*** VW has quit IRC19:13
*** valw has quit IRC19:13
*** valw has joined #craton19:13
*** VW has joined #craton19:32
*** VW has quit IRC19:36
jimbakersigmavirus, it would be interesting to combine the capabilities of keystone with myproxy. over time, i'm sure we will see increased use of delegation19:51
jimbakerand connectivity of such delegation systems for last mile aspects19:51
sigmaviruswhat's myproxy?19:51
sigmavirusalso, https://review.openstack.org/40877819:52
*** valw has quit IRC19:52
*** valw has joined #craton19:55
*** VW has joined #craton19:56
*** valw has quit IRC19:58
*** valw has joined #craton19:58
*** VW has quit IRC20:00
*** VW has joined #craton20:01
*** tojuvone has quit IRC20:04
jimbakersigmavirus, myproxy is a foundational component in opengrid; see http://grid.ncsa.illinois.edu/myproxy/protocol/20:09
jimbakerit's pretty cool stuff. unfortunately i don't see how we could actually use myproxy here with craton20:09
jimbakercloud is doomed to reinvent grid (basically the same thing)20:10
sulosigmavirus: 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.py20:10
suloi dont remember who made the case that we dont want that20:11
suloah it was git-harry20:11
jimbakergit-harry, just so you know, tojuvone's change i just proposed caused a bunch of merge conflicts with your proposed changes20:12
jimbakerSyed__, same ^^^20:12
*** VW_ has joined #craton20:12
jimbakerall easy, just there to be resolved20:12
jimbakerthat was garbled20:13
jimbakertojuvone's change i just reviewed - and workflowed! - ...20:13
jimbakerdoing the malaprop day20:14
git-harrysulo: yup, I think skipping is a bad idea.20:14
sulogit-harry: yeah i agree20:14
git-harryjimbaker: yup, I guess I lost that race and will have to rebase.20:15
jimbakergit-harry, yep, it's the usual20:15
git-harryaye20:15
*** VW has quit IRC20:16
*** VW_ has quit IRC20:17
jimbakerSyed__, are you still working on https://review.openstack.org/#/c/396750/  - seems to be in the same domain as what git-harry is working on20:18
jimbakerpresumably we should abandon20:19
jimbakeror perhaps not... anyway, want to dispose of old reviews, either by update or by abandoning20:24
*** valw has quit IRC20:25
git-harryjimbaker: Syed__ that's in the client, I've not done anything there yet.20:27
jimbakergit-harry, yeah, i over generalized on your work20:27
jimbakerpresumably some impact on schema cleanup of course20:27
jimbakerincluding i assume recent work by sulo20:28
sulo??20:28
suloi really need to change my laptop20:29
jimbakersulo, maybe such things as returning variables20:29
jimbakersulo, you should definitely replace that old laptop. certainly a legacy!20:29
suloheh yeah20:29
jimbakertoo bad the mac laptops are not really for devs any more20:30
*** valw has joined #craton20:30
*** valw_ has joined #craton20:32
*** valw_ has quit IRC20:35
*** valw has quit IRC20:35
sigmavirussulo: we at least shouldn't leave the failure until the test tries to run and doesn't have an attribute to use20:38
sulosigmavirus: yeah i think sane hadling of a few thing might make it easy to see whats happening20:40
jimbakersulo, 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 fwiw20:41
sulosigmavirus: you are refering to url i assume20:41
*** VW has joined #craton20:41
sigmavirussulo: well that's the first one20:41
sigmavirusi'm sure there will be others that pop up later as we move along20:41
jimbakerand 8 hyperthreads! truly all-in-one system :)20:41
* sigmavirus doesn't like ubuntu very much honestly20:42
suloit should only be url now i think ... will have to recheck20:42
sigmavirusI don't dislike it20:42
sigmavirusbut I don't like it20:42
sulobut that purely becuse we rely on container ip to make the call20:42
suloactually what we can do is20:42
sulomap it to 127.0.0.1 to some port20:42
jimbakersigmavirus, i liked it for laptops. just didn't like the support aspect. frustrating given i was working at canonical... ;)20:43
sulobut it doesnt make it any easy to see what you getting the error though20:43
*** VW has quit IRC20:43
jimbakerand supposedly thinkpad was the gold standard then20:43
*** VW_ has joined #craton20:43
sigmavirussulo: I just think we should error out on those tests20:43
sigmavirusYeah I like thinkpads for various reasons20:43
suloerror out ?20:43
suloraise error ?20:43
sigmavirussulo: so tests can error during setup20:43
sigmavirusIf we do that when container setup fails, that feels most honest20:43
sulowell you saying technically its better because it will say "error" ?20:44
suloyeah i guess we can do that too20:44
sigmavirusIt'll still say error, but the reasoning will be less cryptic20:44
sulomaybe error with some reason20:44
sigmavirusRight20:45
* sigmavirus doesn't much like laptop keyboards with numpads20:45
sigmavirusthat laptop is pretty though20:45
sulook lets try that .. we have error_reason and _container_error or something that can be caught during setup20:45
suloso it might make more sense that way20:46
sigmavirusWe can also add our own exception handling https://github.com/testing-cabal/testtools/blob/master/testtools/testcase.py#L27020:48
suloyeah, i was going to say we can simply fail the test right now on setup instead of skipping20:59
suloit will be a very simple change and it will fail with failureException with real reason to failure21:00
*** valw has joined #craton21:02
sulosomething like this: https://gist.github.com/sulochan/ff9fe77ec2195a38bbc1d023a2fd488d21:04
jimbakersulo, fyi, resolved variables are not supported by /v1/cells/<cell-id> - but that's separate from the change you just proposed21:10
jimbaker(and which i just workflowed)21:11
sulojimbaker: yeap, catching all that in functionl test right now for cells21:12
jimbakercool21:12
jimbakeryeah, i'm reporting a bug about having common logic here21:12
sulojimbaker: ok21:12
jimbakerwe should also make project a root for variables21:12
sulojimbaker: create it separately for cells and host networks .. its easier to pair with tests21:13
jimbakerthat would let it be project -> region -> cell? -> device21:13
sulolike i am doing cells now .. so i can depend on some of these fixes for func tests21:13
jimbakerproject -> workflow_def -> workflow21:13
jimbakerproject -> user21:13
sulojimbaker: you mean in the url ?21:14
jimbakerno i mean in variable resolution21:14
suloah21:14
suloyes +121:14
jimbakerso one can define project-level variables21:14
jimbakersulo, yeah, i will keep the bug separate from networks, since they have been treated separately from a dev perspective21:15
jimbakercould use same logic if we put in base of course21:15
jimbakerok, 3 bugs to be reported :)21:16
jimbakersulo, https://bugs.launchpad.net/craton/+bug/164862721:25
openstackLaunchpad bug 1648627 in craton "Use common logic for resolved variables for regions/cells/hosts REST API" [Undecided,New]21:25
jimbakerand https://bugs.launchpad.net/craton/+bug/164862621:25
openstackLaunchpad bug 1648626 in craton "Projects should support variables, which other entities use in resolution" [Undecided,New]21:25
jimbakergit-harry, thanks for updating the changes in gerrit21:29
git-harrynp21:30
jimbakerone last one: https://bugs.launchpad.net/craton/+bug/164863021:32
openstackLaunchpad bug 1648630 in craton "Use common logic for resolved variables for network support REST API" [Undecided,New]21:32
*** valw_ has joined #craton21:34
*** valw has quit IRC21:35
*** valw_ has quit IRC21:41
*** valw has joined #craton22:01
*** Mudpuppy has quit IRC22:06
*** Mudpuppy has joined #craton22:07
*** valw has quit IRC22:07
*** Mudpuppy has quit IRC22:11
*** david-lyle_ has joined #craton23:05
*** david-lyle has quit IRC23:07
*** valw has joined #craton23:13
*** david-lyle_ is now known as david-lyle23:35
*** jovon has quit IRC23:38
*** VW_ has quit IRC23:41
*** VW has joined #craton23:43

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!