/srv/irclogs.ubuntu.com/2014/10/10/#juju.txt

=== CyberJacob is now known as CyberJacob|Away
=== CyberJacob|Away is now known as CyberJacob
lazyPower|Sprintstub: ping07:24
stublazyPower|Sprint: pong07:26
lazyPower|SprintHey stub, i'm looking @ https://code.launchpad.net/~james-w/charms/precise/postgresql/metrics/+merge/23132207:26
lazyPower|Sprintsorry incomplete thought and moving sessions, 1 moment07:27
stublazyPower|Sprint: Dammit... I thought my 2 months of PG charm patches was getting a review ;)07:28
lazyPower|Sprintstub: they aren't in the queue that i see07:30
=== CyberJacob is now known as CyberJacob|Away
lazyPower|Sprintstub: http://review.juju.solutions07:30
stublazyPower|Sprint: http://review.juju.solutions/review/1086, 6 days since last touch07:31
stub(but this is the  rollup branch requested, containing MPs dating from over 2 months ago)07:32
lazyPower|Sprintooooo07:32
lazyPower|Sprinttests07:32
lazyPower|Sprintargh, rollup branches :(07:32
stubHey, it used to be in 7 nicely separated patches07:33
lazyPower|Sprintwait, you mean this is 1 branch to review then/07:33
lazyPower|Sprint<3 you just made my day, i thought i had to go back through the 7 pending branches07:33
stublazyPower|Sprint: Yeah, all those old branches got mushed together into this big MP on request07:34
lazyPower|Sprinthrm, this is a rather large MP07:34
lazyPower|Sprinti blame tvansteenburgh07:34
stubAlthough I'm not sure why you guys like that - for the last 10 years everyone around here has been focused on ways of keeping the MPs of small and manageable size ;)07:34
lazyPower|Sprintwell, seeing the sheer size of this, i agree w/ that sentiment07:35
stublazyPower|Sprint: I've got another 1500 lines of diff nearly ready to go ;)07:35
lazyPower|Sprinti'm just used to the GH workflow. I'm a LP noob07:35
stubI'd consider separating review and landing07:36
stubReview is just 'does this code look good'. 'Is this code an improvement'.07:36
stubAnd you want a small, self contained diff for that.07:36
stubLanding is about lint, tests etc. and ideally handled by a bot.07:36
lazyPower|Sprintwe're getting closer to that07:37
lazyPower|Sprintour CI is just about there for this, which means we're literally wrapping the step before introducing bot landing07:37
lazyPower|Sprintty for fixing the tests, i'm about to pull and run this07:38
stubYup. I'm looking forward to that. I'm hoping to get the tests stable enough too, although that might make then annoyingly slow due to 'juju wait' needing a longer pause.07:38
stubThe earlier reviews had picked up some genuine races, which I think I've fixed.07:39
stubBut this branch still works better than trunk atm :)07:39
stublazyPower|Sprint: I was planning on landing https://code.launchpad.net/~james-w/charms/precise/postgresql/metrics/+merge/231322 once I've got my queue cleared (getting the charm in shape to deploy Launchpad with it atm). I think there were things james wanted to fix on it, but I don't recall what now and it should be in a landable state as it is.07:55
stubMaybe just make the minor changes, stick run-one in the crontab stuff.07:55
stubIIRC james was going to make some changes, but got diverted to other things.07:57
stubBut I want the statsd stuff in, so sod waiting.07:57
lazyPower|Sprintack08:13
lazyPower|SprintI'll be loking into merging this massive change08:13
lazyPower|Sprinta bit preoccupied with charm store triage atm, but will circle back to this before i EOD08:14
=== uru_ is now known as urulama
=== uru_ is now known as urulama
zezomI'm currently running proxmox at home on a single server + NAS but I'm quite impressed with what I've seen juju do. Unfortunately I am unable to tell if I can run MAAS and JUJU on the same single server. I would like the ability to add more servers in the future but for now one server is all I have.11:02
zezomI've read through the MAAS install doco on ubuntu.com but it doesn't say if the MASS controller install can run any virtual's on it's self or if it's only used to bootstrap other nodes to run virtual's.11:05
lazyPower|Sprintstub: looks like there's an interestinng mismatch between this branch and trunk.11:05
stubreally? I thought I had them in sync11:05
lazyPower|SprintText conflict in hooks/hooks.py, Text conflict in test.py - i'm getting issues between the tests and hooks11:06
lazyPower|Sprintpossibly they were at one point, i haven't looked to see if anything has been merged recently,  i just pulled the source and started prepping my env to dive into this giant11:06
stublazyPower|Sprint: There was one revision unpushed... pushed it now.11:08
stublazyPower|Sprint: yeah, it was 'merge trunk'. Sorry about that.11:08
lazyPower|Sprintall good :)11:09
stubLaunchpad is supposed regenerate the diff and highlight this :-(11:09
stubThat would be the lint fixes I landed for Adam, that were also lurking in this integration branch.11:09
lazyPower|Sprintstub: i've seen a few braches so far that haven't highlighted merge issues11:12
lazyPower|Sprinti think that *only* works if its got an issue when it's first pushed11:13
lazyPower|Sprintit doesn't continue to track the behavior of the branch if its diverged from trunk11:13
stubI can't really recall now.11:13
stubI normally remember to push so it isn't an issue ;)11:14
lazyPower|Sprintstub: i'm making it an even shorter distance than my colleagues have11:19
lazyPower|Sprinthttp://paste.ubuntu.com/8532283/11:19
stublazyPower|Sprint: Do you have a JUJU_REPOSITORY set, with trusty/postgresql pointing to the right location? And you will need postgresql-psql too if you are testing with trusty11:21
stubNot that running tests is reviewing ;) That is something you do before landing.11:22
lazyPower|Sprintstub: we should add these to the dependency targets for making the tests. CI would fail miserably on this11:23
lazyPower|Sprintit expects whatever bundletester executes, to do everything it needs to successfully run the tests. I see what you've got in  here pre-dates amulet11:23
stubYes, I need to switch to amulet. I think it supports most of what I need now, and I can probably add the rest.11:23
lazyPower|Sprintoh i wasnt saying re-write in amulet, i just noticed you built a fixture harness11:24
lazyPower|Sprintwhich is kind of neat tbh11:24
stubNo, rewriting in amulet is a goal I have. I will then be able to put my fixture for driving amulet into the amulet package.11:24
lazyPower|Sprintstub: is postgresql-psql a pip package? I'm not seeing anything in apt to satisfy the dependency11:33
stublazyPower|Sprint: it is cs:precise/postgresql-psql11:34
lazyPower|Sprintoh! charm dependency11:34
lazyPower|Sprintack11:34
stublazyPower|Sprint: I've avoided getting it pushed to trusty, since I'd rather embed it into the PostgreSQL charm for testing like amulet does with its sentinals.11:34
stubCurrently, it pretends to be a general purpose cli, but it really mainly used by my test suite, and ends up sucking for both purposes.11:35
* lazyPower|Sprint grins11:46
lazyPower|SprintI've got a few projects along that vein11:46
lazyPower|Sprintyeah, i'm not getting any luck running the test suite11:46
lazyPower|Sprintthe code review portion looks fine, i dont see anything detrimental in this codeblock but i'm hesitant to rubberstamp the merge without the warm fuzzy feeling of getting test feedback11:47
stubSo we can fix that, or I can promise to confirm they pass before landing, or I can disable most of the tests ;)11:47
lazyPower|Sprintoption 3 sounds like a wash11:47
lazyPower|Sprintoption 2 is 'ok'ish but sets a precident11:47
lazyPower|Sprinti'd rather go with option 1 if at all possible11:47
stubCause even if I disable the integration tests, I still have more tests than most charms ;)11:48
lazyPower|Sprintand we <3 you for that stub11:48
lazyPower|Sprinthow do we go about fixing this? I'm not picking up where its failing, its like its not even attempting to make the deployment when i run make test (which is what bundletster is doing)11:48
lazyPower|Sprintjuju status shows i just have the bootstrap. my JUJU_REPOSITORY is exported, and i have the postgresql-psql charm in that local repo11:49
stubI'm already landing almost all the work on this charm. Just it would be evil to review and land my own work :)11:49
* lazyPower|Sprint nods11:49
stubYou have the same error as before? juju deploy fails?11:49
lazyPower|Sprinti want to make this process less painful for everyone involved, and i've got the time to do this. i don't have any other blocks today that i'm implicitly scheduled to be at11:49
lazyPower|Sprintyeh11:49
lazyPower|Sprintlet me capture that output for you11:49
lazyPower|Sprinthttp://paste.ubuntu.com/8532440/11:51
lazyPower|Sprintthere's a snippet of the error output11:51
stubCalledProcessError: Command '['juju', 'deploy', '--config=/tmp/tmpyCZBRK/config.yaml', 'local:trusty/postgresql', 'postgresql', '-n', '1']' returned non-zero exit status 111:51
lazyPower|Sprintwait11:51
lazyPower|Sprinti see it11:51
lazyPower|Sprintits looking for trusty/postgresql11:52
lazyPower|Sprinti have precise branched, and placed locally11:52
stubRight11:52
lazyPower|Sprintcheese and rice11:52
lazyPower|Sprintso this should target the trusty charm, as well as the MP then11:52
* lazyPower|Sprint facepalms11:52
lazyPower|Sprintwhat am i doing with my life11:52
stubSo lp:charms/precise/postgresql and lp:charms/trusty/postgresql are identical11:53
lazyPower|Sprintopportunity to make the tests more intuitive, as precise tests will *always* fail in this context11:53
lazyPower|Sprintbut thats a matter for another day11:53
stubmake test SERIES=precise will override the default series in your environment11:54
stubOtherwise, it will test the default series in your environment11:54
lazyPower|Sprinthmm11:55
lazyPower|Sprinti should make that a follow up item to investigate making test targets easier to work with, and sniffable from the path or something.11:55
lazyPower|Sprintthat way if set use env, otherwise look @ path and use that to determine the series11:56
stubYeah, it isn't pretty how I've got it11:56
lazyPower|Sprinti dont think this is just your problem. we have the same issue in amulet11:56
stubMaybe that will come out in the wash if I can get rid of this JUJU_REPOSITORY setup requirement?11:56
lazyPower|Sprintyou either define series=precise|trusty  when you declare amulet.deployment()11:56
lazyPower|Sprintwell, thats gone in amulet as well11:56
lazyPower|Sprintbut it doesn't support local: links, you pass it the name of the charm, and all others are defined with cs: url's11:57
lazyPower|Sprintso you can't test 2 modified targets at once, which is a limitation (or feature, depending on how you look at it) of the amulet workflow11:57
stubThe automatic test environment could just set some environment variables that amulet or other test suites sniff. No need to overengineer for this case.11:57
lazyPower|Sprintso for example, if these tests were amulet, it would just be d.add('postgresql') and thats enough for amulet to know to deploy the local copy of the charm.11:57
lazyPower|Sprintit uses the name of the charm from metadata.yaml and matches on that11:58
lazyPower|Sprinthowever, it looks more promising this time around now that i'm using the proper series11:58
lazyPower|Sprinti think this bit cory_fu as well looking over the feedback chain11:58
stubThis is why we make bots run the tests ;)11:58
stubYeah11:58
stubCause who the hell still uses precise?11:58
stub(I don't! I should get the default branch switched to trusty)11:59
stubSo generally, I'll see on average one test fail that works just fine when it is rerun alone, which I chalk up to the deployed services not being settled properly before the actual test is made12:02
stubAlthough I've improved that an awful lot on this branch (and slower now too T_T)12:02
lazyPower|Sprintwell its still not deploying the tests12:04
lazyPower|Sprintsame feedback, regardless of SERIES= during make test, or moving everything to trusty12:04
stubWhat does it tell you if you manually run the 'juju deploy' command, sans --config=/tmp/xxx12:05
lazyPower|Sprinti think the core issue here is the tests arent architected to be satisfied when run via bundletester. (however i'm getting the same result with make test)12:05
lazyPower|Sprintooo well thats interesting12:05
lazyPower|SprintAdded charm "local:precise/postgresql-10" to the environment. ERROR cannot add service "postgresql": environment is no longer alive12:05
stubHas anyone packaged bundletester yet?12:06
lazyPower|Sprintindeed!12:06
lazyPower|Sprintits pip installable12:06
lazyPower|Sprintpip install bundletester12:06
lazyPower|Sprintlooking into wth is going on with my local environment12:06
stubGah... pooping all over FS without apt12:06
stubI'll put together a recipe build if I get bored ;)12:07
lazyPower|Sprintstub: source ~/.venv/bin/activate12:08
lazyPower|Sprintpip install bundletester12:08
lazyPower|Sprintno longer polluting anything, or you can even pass --user to install to ~/.local12:08
stubWhere is ~/.venv coming from? Last I used virtualenv it was more manual12:09
lazyPower|Sprintyou initialize it12:09
lazyPower|Sprintvirtualenv ~/.venv12:09
stuboh, cool12:09
lazyPower|Sprintyeah, i do that when i want to test python modules - as its more of a 'bundler' experience12:09
lazyPower|Sprintand i'm the unpopular rails guy that got hired :)12:09
stubI've been using lxc, but had issues with juju inside lxc creating lxc containers...12:10
lazyPower|Sprintyikes. yeah12:10
lazyPower|Sprintit gets a bit angry about that12:10
lazyPower|Sprinthav eyou looked @ our vagrant story? after a networkin bugfix lands it'll be a nice alternative if you want isolated 'fresh cloud image' style testing12:11
stubNot yet. I'm backlogged on a couple of projects :-(12:11
stubSo many things to do, so little enthusiasm ;)12:11
lazyPower|Sprintpresently there's a route added that makes everything appear as if its coming from the gateway address - which unfortunately makes the postgresql ACL setting fail fantastically12:11
lazyPower|Sprinthurray for packet rewriting12:12
lazyPower|Sprint\o/12:12
lazyPower|Sprinti think aisrael actually pointed that one out a few weeks ago12:13
tvansteenburghlazyPower: amulet actually does support local: urls now12:13
lazyPower|Sprinttvansteenburgh: wat12:14
lazyPower|Sprintwhy aren't we shouting from mountain tops about the awesomeness now?12:14
tvansteenburghi added that recently-ish?12:14
tvansteenburghshouting to whom?12:14
stub  Analyzing links from page https://pypi.python.org/simple/lazr.authentication/12:14
stub    Skipping link https://launchpad.net/lazr.authentication (from https://pypi.python.org/simple/lazr.authentication/); unknown archive format: .authentication12:14
stub:-P12:14
tvansteenburghodd12:15
tvansteenburghis that with pip install?12:15
stubYes, but hmm.... might be mixing venv python with non-vm pip12:16
lazyPower|Sprinttvansteenburgh: in charmhelpers, the config object is read only yeah?12:16
tvansteenburghnope12:16
tvansteenburghyou can save arbitrary data in it12:17
lazyPower|Sprinti have evidence that it gets saved to the archive, but its not coming back out when referenced12:17
stubnah, carping on about --allow-external (which doesn't seem to be a supported option)12:17
tvansteenburghstub: valid option, is pip out of date?12:17
stubThe config object is writable, and should be saved on successful exit if you are using @hook or the services framework.12:17
tvansteenburghstub: e.g. https://github.com/juju-solutions/bundletester#installation12:18
lazyPower|Sprintwell we know its getting saved, but it doesn't appear to be getting loaded12:18
stubThis is from 'sudo apt-get install python-virtualenv; virtualenv ~/.venv; source ~/.venv/bin/activate; ~/.venv/bin/pip install bundletester'12:19
stubfresh trusty12:19
stubwell... old trusty, but fresh venv12:19
tvansteenburghstub: and you are passing the --allow- options ?12:21
rbasakmarcoceppi_: around? Any news on removing the sentries from amulet?12:22
tvansteenburghlazyPower: want extra eyes on the config stuff?12:22
tvansteenburghrbasak: yeah we're still planning to do it :)12:22
rbasaktvansteenburgh: OK, thanks. In the meantime I think I'll drive what I need from the shell instead. Do you have any idea how I might wait for a hook to finish firing after I trigger one with add-relation or destroy-relation or something?12:23
rbasakHow will amulet do that?12:23
tvansteenburghrbasak: there are some helpers for stuff like that in amulet, you may be able to reuse12:23
rbasakI thought they currently relied on the sentries?12:24
stubSo --allow-external lazr.authentication and --allow-unverified lazr.authentication required it seems, which is not good for me to do on this box :-(12:24
lazyPower|Sprinttvansteenburgh: not just yet, gimme another 5, doing env inspection12:24
tvansteenburghstub: it's the only way, unless you can convince someone from lazr to publish on pypi12:24
stubHeck, I might have rights to do that.12:24
tvansteenburghomg you would be my hero12:25
lazyPower|Sprinttvansteenburgh: its not loading the .charm_persistent_config12:25
tvansteenburghrbasak: memory is a little rusty on how it's done exactly, i'd just take a look and see12:25
lazyPower|Sprinttvansteenburgh: appears related to scope possibly?12:25
tvansteenburghscope of?12:25
lazyPower|Sprint@cached12:26
lazyPower|Sprintits saving this in $CHARM_DIR/CONFIG_FILE_NAME12:26
lazyPower|Sprintwhatever that gets init'd to, in this case, its .persistent_charm_config12:26
lazyPower|Sprintand the config() object is passed a dest_dir key/value, and it shows at end of install print12:27
stubleonard.... I might still have his email around here somewhere...12:27
lazyPower|Sprintwhen config() is loaded in config_changed() - it's missing a key/value that's present in the file.12:27
tvansteenburghlazyPower: i'm gonna come downstairs, having trouble following12:28
lazyPower|Sprintack12:28
stubhookenv.config_data() uses the @cached decorator, so it is returning a singleton. It will only be loaded on process startup.12:33
stubFirst invocation I mean12:33
lazyPower|Sprintstub: it was an interesting omission from get_item()12:43
tvansteenburgh1had to override keys() in Config, the data was there, you just couldn't see it in keys()12:44
=== tvansteenburgh1 is now known as tvansteenburgh
stubsounds like a bug. would never have suspected a bug ;)12:46
tvansteenburghhaha12:46
stubSo I just got a automatic test run result, and the same problem as lazyPower|Sprint . The test suite is detecting the default series is trusty, and attempting to deploy that.12:59
stubBut the branch being tested is precise, so everything fails.13:01
lazyPower|Sprintstub: yeah :( i'm working through this a layer at a time13:02
lazyPower|Sprinttrying to unwind complexity13:02
lazyPower|Sprinti got it to deploy by moving it  to trusty, however install is failing consistently as its not deploying with +x perms on hooks.py - and i'm confused as to how that happened13:03
stubeasy fix for me is to have the runner set the SERIES environment, cause then I don't need to update anythng ;)13:03
stubhuh - hooks.py doesn't have the execute bit here13:04
stublazyPower|Sprint: Try refreshing the branch. I just pushed a change to the perms to that file.13:06
lazyPower|Sprintstub: they're firing off like crazy atm i should have resutls shortly13:26
lazyPower|Sprintstub: this branch looks good in CR format, the tests need more work at the end of the day13:48
lazyPower|Sprinti'm going to make the associated comment on the MP and run the readme deployment tests. so long as everything passes standup i'm happy with whats here13:49
stubta13:52
lazyPower|Sprintnp, ty for taking me through the ropes of the unfamiliar territory of pgsql tests13:52
vtolstovhello14:29
vtolstovdoes somebody helps me and say: how can i provide website: interface: http and https ?14:30
jamespagecoreycb,14:30
jamespagehttps://code.launchpad.net/~corey.bryant/charms/trusty/cinder/remove-missing/+merge/23743914:30
vtolstovhow can i specify array or what i need to do?14:30
jamespageunit test failures14:30
coreycbjamespage, doh14:31
jamespagecoreycb, :-)14:32
jamespagecoreycb, if you have time - https://code.launchpad.net/~james-page/charms/trusty/neutron-openvswitch/disable-security-groups/+merge/23793314:36
jamespagewe're running that on serverstack :-)14:36
coreycbjamespage, sure I'll take a look - oh so it's being tested atm :)14:37
coreycbjamespage, btw, does it make sense to add openstack-origin and openstack upgrade support to that charm?14:37
jamespagecoreycb, no because its a suboridinate; it would get upgraded by its principle14:37
coreycbjamespage, ok, was wondering about that14:38
jamespagehowever having it twiddle is config might be a good idea14:38
jamespageright now its not an issue14:38
coreycbok14:38
coreycbjamespage, so nova-compute presumably reinstalls packages for it?14:38
jamespagecoreycb, yes it will14:39
coreycbjamespage, ok14:39
jamespagecoreycb, this would get tricky if the list of packages needed to change - but atm it does not14:39
jamespage...14:39
jamespageso we are OK for now14:39
jamespagenot ruling out altogether tho14:39
coreycbjamespage, ok, there are a decent amount of tempest failures after upgrade so I wanted to blame it on that.  I need to dig some more.14:40
jamespagecoreycb, icehouse -> juno?14:40
coreycbyes14:40
jamespagecoreycb, there will be because of that neutron ipset bug I just uploaded a fix for14:40
jamespagewithout it you can't spin up instances....14:41
coreycbjamespage, ah maybe that's it.  yeah that would do it.14:41
jamespagebasically the port creation fails because neutron can't setup the security correctly14:41
roadmrHello folks, I had trouble getting nested lxc working with juju; meaning, juju deploying to a local provider, and having the deployed workload start some containers (inside the local-instantiated container)14:43
roadmranybody had success with this?14:43
vtolstovanybody knows ?14:53
=== underyx is now known as Underyx
=== scuttle|afk is now known as scuttlemonkey
weblifeI am not able to bootstrap for some reason after I destroyed my last environment.  Boot strap will not load tool sets for my ubuntu distro 14.10.  My output: http://paste.ubuntu.com/8534392/17:44
stokachuweblife: yea its known issue17:45
stokachuweblife: they working on getting the certificate renewed i beleive17:45
weblifestokachu: thank you17:45
weblifestokachu: you hear about an eta?17:46
stokachuweblife: yea try it now just got word they updated the cert17:47
stokachuworks in browser  but please test juju17:47
stokachuweblife: any luck?17:51
weblifetrying now17:53
weblifelaunching17:54
weblifestokachu: launching17:54
stokachuweblife: cool17:55
weblifestokachu: success!17:58
weblifehas the issue with running juju on the local and running external environments been fixed?  I want to install the local version for some test.18:03
=== CyberJacob|Away is now known as CyberJacob
=== roadmr is now known as roadmr_afk
=== jcw4 is now known as jcw4|afk
=== danwest_ is now known as danwest
weblifeI just want the juju team to know I am using juju for my grad project.  Thanks for your work.18:39
=== roadmr_afk is now known as roadmr
=== jcw4|afk is now known as jcw4
zezomis it possible to run juju on a maas controller server?23:50

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