/srv/irclogs.ubuntu.com/2016/11/30/#juju.txt

petevgcory_fu: that bug you're running into is probably my fault. Was going to test some code more thoroughly and push in the morning, but take a look at https://github.com/petevg/python-libjuju/tree/bug/fixes-for-null-cases02:12
cory_fupetevg: I think preventing the crashes on our end is good, but that controller panic definitely shouldn't happen02:14
petevgTrue.02:14
cory_fupetevg: Also, there's some overlap with your branch and a PR I got in this evening: https://github.com/juju/python-libjuju/pull/2402:15
petevgcory_fu: I may also be wrong about it being my fault. Those fixes don't actually change anything (just ran a matrix test) :-/02:15
cory_fupetevg: Yeah, the directive change doesn't fix the controller panic.  I think we must be passing something in to the API as a None that it expects to have a value02:16
petevgLovely.02:17
cory_fupetevg: If you look at the panic log that I included, you can see which params are null (0x00) and compare that to the code signature to work backwards to what we should be passing02:17
cory_fuBut it's not going to be fun02:17
cory_fuAnyway, I'm done for the evening.  We can pair on it tomorrow, if you want.  Have a good night!02:18
petevgcory_fu: sounds good. You have a good night, too (undoing my placement fix fixes things, btw, so it is something to do with the Placement object.)02:20
petevgcory_fu: this works, btw `placement=[parse_placement(to)] if to else None`, for wiki-simple, and for hadoop-processing. I want to do some more banging on it before I check that in, though ... (for now, to bed for me)02:26
wxlis `juju set` not actually a thing? can't find it in the manpage and i get an error trying to run it in a fresh zesty server. and yet: https://jujucharms.com/wordpress/trusty/405:42
zeestratwxl: It's 'juju config' in Juju 2.006:30
=== frankban|afk is now known as frankban
=== iatrou_ is now known as iatrou
anitaHi12:09
=== anita is now known as Guest5875
Guest5875how to get the values from config.yaml file in juju 2.0. I searched and found config=hookenv.config(). then config.get('parameter'). but when I am giving as "juju config <service> paramter=value" its not accepting.12:10
Guest5875is it correct?12:11
marcoceppiGuest5875: everything you've mentioned is correct12:16
marcoceppiGuest5875: maybe a look at your charm code would be helpful12:16
Guest5875marcoceppi_:ok, thanks a lot12:18
BlackDexhello there12:43
=== freyes__ is now known as freyes
BlackDexI have an error "Incomplete relations: identity"12:44
BlackDexon all units of the same service12:44
BlackDexglance in this case12:44
BlackDexi tried to remove it and add it again, but it doesn't seem to work12:44
marcoceppiBlackDex: can you put the output of juju status into paste.ubuntu.com and share the link?12:51
BlackDexone moment12:59
BlackDexmarcoceppi: http://paste.ubuntu.com/23557948/13:01
marcoceppiBlackDex: so a few things, are you locked into 1.25? 2.0 has been out for a little over a month, second, there seems to be a few things still churning, finally you might get better assitance in #openstack-charms since this is really an openstack setup question13:05
BlackDexi'm locked to 1.25 for now13:09
SimonKLBis there a relation, similar to the special juju-info one for subordinate charms, which can be used with more or less every charm when the goal is simply to get the name of the related charm?13:20
SimonKLBthe application name, that is13:20
SimonKLBs/relation/interface13:22
marcoceppiSimonKLB: juju-info is that relation13:38
marcoceppiwell, that's the interface13:38
SimonKLBoooh, i thought that was only intended for subordinate charms13:38
marcoceppiSimonKLB: oh, you want this for primary charms?13:38
SimonKLBmarcoceppi: yes13:39
marcoceppiI've never tried, but it theoretically *should* work13:39
=== verterok` is now known as verterok
SimonKLBmarcoceppi: doesnt look like it works out of the box - ERROR charm "testcharm" using a reserved relation name: "juju-info"14:00
marcoceppiSimonKLB: relation name, what's the metadata look like?14:01
SimonKLBmarcoceppi: i guess this is the relevant part:14:01
SimonKLBsubordinate: false14:01
SimonKLBrequires:14:01
SimonKLB  juju-info:14:02
SimonKLB    interface: juju-info14:02
marcoceppiSimonKLB: don't call the relation juju-info14:02
marcoceppicall it anything else but that14:02
SimonKLBmarcoceppi: doesnt seem to make a difference14:03
petevgcory_fu: PR for you: https://github.com/juju/python-libjuju/pull/2814:03
SimonKLBmarcoceppi: nvm!14:04
petevg(I saw the code you merged, but I think that we still need to fix these two edge cases, too.)14:04
SimonKLBmarcoceppi: shouldve tried that first :) thanks man!14:05
marcoceppicheers14:05
stubAnyone know what happened with https://review.jujucharms.com/reviews/13 ?14:12
stubI think it was supposed to be promulgated, but https://jujucharms.com/nagios/ is the trusty only charm owned by Tom14:13
marcoceppistub: there was a bit of a miscommunication then, I've reset things14:21
marcoceppimagicaltrout: there appears to already be a group of people taking up nagios, you may want to join them https://launchpad.net/~nagios-charmers14:21
stubMore the merrier14:21
marcoceppistub: can I request that more people be added to metadata.yaml maintainers?14:22
stubSure. Not sure what it has atm.14:22
marcoceppime and someone at nagios14:22
marcoceppidoes nagios-charmers have a mailing list?14:23
stubThat sounds like it needs fixing14:23
marcoceppithat might be the best for a "contact the maintainers"14:23
stubIt could have, but I suspect half the people would filter it :-(14:23
stubCan we use the main juju list, or is that naughty?14:23
marcoceppinaughty :)14:24
stubOtherwise I can click the 'create a mailing list' button here in Launchpad14:24
marcoceppieh14:24
marcoceppiit's a pretty low traffic list14:24
marcoceppias the previous maintainer, I only ever recieved 1 email14:24
stubLaunchpad has mailman builtin, but I suspect if I turned it on for something this low traffic emails would just get lost in mail filters.14:25
stubBut at least it is a non main-juju-mailing-list contact point14:26
marcoceppistub: as an additional maintainer14:26
marcoceppiI'm happy to stay on as a POC, but would hate to be the only POC14:26
stuboh, sure. I was thinking you would want out of there completely :)14:27
marcoceppinaw14:27
marcoceppiI am mildly interested in maintaining14:28
deanmanHi, any suggestion for an existing subordinate well maintained charm to study ?14:31
marcoceppideanman: filebeat?14:34
deanmanmarcoceppi: on it14:38
stubhttps://api.jujucharms.com/charmstore/v5/nagios-14/archive/metadata.yaml14:40
marcoceppi+114:40
=== cmars` is now known as cmars
skay_Hi, I ran `charm proof` on my new charm and I have warnings about missing hooks and amissing hook directory. when I generated the charm structure, it created a reactive directory. I've been putting hooks and reactive functions in my one file there.14:48
petevgcory_fu, bcsaller: I pushed another build of python-libjuju to matrix master's wheelhouse. Contains the fixes cory_fu and I pushed for edge cases.14:48
skay_based on running charm proof, should I split out the hooks in to a separate location?14:48
petevgskay_: if you're building a layered charm (which you probably are, if you're following the updated instructions), you need to do "charm build", and then run "charm proof" on the built charm.14:50
marcoceppiskay_: proof needs to be run after you do a charm build14:50
skay_ack, thanks14:50
petevgno worries :-)14:50
skay_much better :)14:50
skay_I have some hooks, like config-changed, that should switch status to 'maintenance' and then eventually switch to 'active'.14:53
skay_if an exception occurs between those, would the status be stuck in 'maintenance'?14:54
skay_and where can I look at how exceptions get handled by the framework so that I can determine if I should do some handling before allowing an exception to be raised14:54
skay_also, are there best practices for commenting on status and state transitions?14:55
skay_I didn't add comments on everything that transitions, but where something might not be clear I made a list of possible transitions14:55
petevgskay_: it should flip into an errored state if you have an unhandled Exception. Juju will then retry the hook, and stay in an errored state until the Exception goes away, or someone manually intervenes and fixes things.14:55
skay_thanks14:56
petevgskay_: Documenting transitions where they are not clear sounds like a best practice to me :-)14:56
petevgnp14:56
skay_thanks I am just guessing here14:56
skay_okay, another question. I've been deploying my charm to try it out and it gets listed in an 'unknown' state, and I don't know why that happens14:57
skay_do you have advice on how to dig in to that?14:57
skay_listed by juju status14:58
petevgskay_: if I were troubleshooting, I'd read code and think through why a state wasn't getting set. You can add logging statements, or take a look at the documentation on the 'debug-hooks' command here:  https://jujucharms.com/docs/1.25/authors-hook-debug14:59
petevgAnd feel free to file issues or make contributions to juju docs (https://github.com/juju/docs) -- they could definitely be clearer about some of this stuff.15:00
skay_petevg: thanks. (I'm using juju2 so I'd go to a different url)15:00
skay_petevg: :)15:00
skay_petevg: I'm never sure when something is supposed to be obvious or not :)15:00
petevgskay_: yep. The correct url is https://jujucharms.com/docs/stable/authors-hook-debug  Sorry about that.15:00
petevgskay_: if it's not obvious to you, it's probably not obvious to someone else. Asking questions is always good. Filing issues when you get a good answer that isn't in the docs is even better :-)15:01
petevgskay_: in other words, thank you for the excellent questions :-)15:02
rick_hjcastro: ping15:16
jcastroyo15:18
rick_hjcastro: got a sec for prep?15:19
jcastroyeah, fire it up!15:19
rick_hjcastro: wanted to make sure the agenda is good, you mentioned some adding yesterday15:20
mbruzekrick_h: Will the link be here so we can join the show?15:20
rick_hmbruzek: definitely15:20
mbruzeklooking forward to it, carry on with prep15:20
rick_hjcastro: can you use the one for the show just not hit record for prep?15:20
jcastrorick_h: I think marcoceppi wants to crash the party!15:20
rick_hjcastro: or a diff one ?15:20
jcastroright, working that now15:20
rick_hwoot! crashing is fun15:20
jcastrohttps://hangouts.google.com/hangouts/_/ytl/mmAQjIgU5Fj-06oIFgojUAUgmlXJO-VnVCiW-omyT_g=?eid=10318440595651078563015:21
jcastrogive it 30 seconds15:21
jcastrohttp://youtu.be/wo23ZXwa8ZU to listen in15:22
marcoceppijcastro: "that's an error" when accessing page15:22
marcoceppinvm15:22
rick_hloaded here15:22
=== icey is now known as Guest28168
marcoceppihttps://lists.ubuntu.com/archives/juju-dev/2016-November/006169.html15:43
jrwrensounds like keystone charms are so good that its the easiest identity management on juju15:44
mbruzekAre there any questions for the Juju show live on you tube right now? http://youtu.be/wo23ZXwa8ZU15:52
marcoceppihttp://summit.juju.solutions/15:52
CoderEuropeWich #channel should I be in for the Juju Show ?16:11
zeestratCoderEurope: This one.16:11
CoderEuropezeestrat: Cheers - wheres jcastro ?16:12
zeestratNot sure. Ping rick_h or mbruzek for questions16:12
mbruzekzeestrat: either16:13
CoderEuroperick_h, mbruzek I would like to chat with jcastro about a project that I am kicking around. Can I talk to him after the show ?16:13
arosalesjcastro: rick_h is the hangout still live?16:14
mbruzekCoderEurope: Absolutely you can16:14
rick_harosales: yes16:14
arosalesmind if I drop in?16:14
arosaleshttps://hangouts.google.com/hangouts/_/ytl/mmAQjIgU5Fj-06oIFgojUAUgmlXJO-VnVCiW-omyT_g=?eid=103184405956510785630 isn't working for me16:14
arosales40316:14
rick_harosales: :/ not sure16:14
mbruzekprobably because we are live now16:14
bdxhttps://hangouts.google.com/hangouts/_/ytl/mmAQjIgU5Fj-06oIFgojUAUgmlXJO-VnVCiW-omyT_g=16:15
jcastrohttps://hangouts.google.com/call/ymfbnowfynfabphu36fwwyk46ee16:15
jcastrotry this one16:15
jcastroyou can both drop in if you'd like!16:15
CoderEuropeme too ?16:15
mbruzeksure16:15
jcastroyeah!16:16
marcoceppijump in, we've got like 15-20 mins to hang out16:16
zeestratrick_h: Thanks for going through my questions. Much appreciated!16:26
rick_hzeestrat: np! thanks for the feedback/questions!16:26
CoderEurope@jcastro, Something went snap on my hangout.16:36
CoderEuropejcastro, thank-you should be downloaded in 5mins. Thanks again.16:39
vmorriswatching the playback on the hangout -- this discussion about openstack endpoints being on private networks is super relevant to my interests16:41
vmorrisbut isn't this the point of having internalurl and externalurl attributes in the endpoint?16:42
bdxvmorris: yea, but natting wasn't taken into consideration16:46
bdxvmorris: I'm quite sure that functionality is meant to be facilitated by having actual interfaces with ip addresses on the separate networks16:47
vmorrisbdx: ah i see now, ty16:50
beisnerhi bdx, marcoceppi - re: barbican, be aware that the it is intended to be used with an hsm.  when used without it, you'll want to be aware of the warning @ http://docs.openstack.org/developer/barbican/setup/dev.html which we've also echoed in the charm release notes @ http://docs.openstack.org/developer/charm-guide/1610.html#barbican17:02
wxlzeestrat: thank you kindly. is 2.0 docuemnted outside of the man page?17:04
=== icey_ is now known as icey
zeestratwxl: All of the updated docs for 2.0 is up on https://jujucharms.com/docs/stable/ (see https://jujucharms.com/docs/stable/commands for commands for example)17:23
zeestratSome charms might be a bit older and therefore refer to juju 1.x commands and concepts.17:24
=== frankban is now known as frankban|afk
bdxbeisner: thanks for that17:38
bdxbeisner: Note that this plugin DOES NOT WORK at present due to17:41
bdxbug#1611393.y17:41
bdxbeisner: so its not really usable to that end17:42
beisnerbdx, exactly.  the idea here is:  standalone barbican is a poc/test only scenario.  or, provide a hardware hsm and point barbican at it.  or eventually resume softhsm work when the stars align wrt versions.17:43
bdxI see17:43
bdxI don't want to believe it though17:47
bdxthis spoils all my fun17:47
bdxbeisner: jerk17:48
bdxjp :)17:48
beisnerbdx, haha. just looking out for ya :)17:49
bdxmuch appreciated17:49
beisnerbdx, i've not looked, but i would guess that for yakkety and onward, the necessary openssl library versions are in line with continuing the softhsm work.  but i don't think we've got that roadmapped atm.  an opportunity to collab/explore?17:50
bdxyeah ... yakkety has 1.0.2g-1ubuntu917:52
bdxI would love to get this working ..... I have devs from other teams super interested in implementing what I've showed them in there organizations too .... not sure how apt they are to actually helping move it forward17:54
bdxbeisner: is there a roadmap to get 1.0.2h in xenial at all?17:55
bdxtheir*^17:55
beisnerbdx, i'm not sure but short of a major vulnerability, xenial's default stance will be on version stability generally speaking.17:56
bdxbeisner: do you know who deals with, or would know more about the openssl packaging?17:57
beisnerbdx, ubuntu security team, i believe.  have a look through https://launchpad.net/ubuntu/xenial/+source/openssl/+changelog17:58
bdxthx17:59
bdxmdeslaur: how's it going?17:59
bdxmdeslaur: would you mind chiming in here?18:00
bdxxnox, jmbl: ^^18:02
bdxbeisner: possibly those guys aren't on #juju all the time, should I email the group of what looks like 4 or 5 maintainers concerning this you think, or openstack/juju lists?18:04
beisnerbdx, it's worth getting familiarized with https://wiki.ubuntu.com/StableReleaseUpdates - those are the folks who will likely err on the side of version stability, for many good reasons of course.  that is not to say that it cannot happen.18:07
beisnero/ bdx - i've gotta check out.  cheers18:08
bdxbeisner: oooh 1.0.2h isn't stable yet18:08
bdxbeisner: ok, thanks18:08
wxlzeestrat: thank you!18:18
xnoxbdx, beisner - i'm lost, what is the question?18:21
xnoxbdx, beisner - we do not take openssl point releases, we cherrypick CVE fixes only. If there is a specific CVE you are after, you can check the security CVE tracker.18:22
xnoxif you are after any other performance enchancement or fixes, it may need stand alone SRU.18:22
xnoxthat's my understanding.18:23
beisnerxnox, ack, same understanding here.18:23
beisnerthanks xnox18:23
jaceknhello. What's the process to get https://jujucharms.com/apache2/trusty/20 cham pushed to xenial series? It should work just fine form what I can see18:28
bdxxnox: thanks18:34
cory_fubcsaller: If an additional matrix suite wants to override a rule in a test, should we match the rules up by task like we do tests by name?  Also, how do we handle removing args from a rule, or removing a rule entirely?  Or is it always just additive?20:15
cory_fu(Current implementation is match by name and task and always add or override)20:15
cory_fuI suppose you could remove an arg by setting it to None, but no way to remove an entire rule20:17
cory_fubcsaller, petevg: What about "task" becoming a top-level rule key and "do" changing to "args"?20:17
cory_fuI think that might be clearer anyway20:17
cory_fuHrm.  Still doesn't really help with deleting rules (or tests)20:18
petevgcory_fu: it feels like the right thing to do is just to write a new test.20:18
cory_fupetevg: What if you want all but one of the default tests?20:18
petevgcory_fu: you're kind of moving toward writing an inheritance system for tests. That's interesting, but might be out of scope.20:19
petevgI can see wanting to avoid copypasta, though ...20:19
cory_fupetevg: Well, we originally talked about it as more of a type of inheritance, but that's hard to do with just yaml20:19
petevgYeah ... gets messy quickly.20:19
cory_fupetevg, bcsaller: Could support a special "delete: True" attribute, perhaps20:20
cory_futests: [{name: foo, delete: True}]20:20
cory_fuOr tests: [{name: foo, rules: {task: bar, delete: True}}]20:20
bcsallercory_fu: sorry, on another call. I'd suggest a new version of an existing name overwrites it, not extends it (though there might be real use cases for that...)20:21
cory_fubcsaller: Just at the rule level, or overwrite the entire test?20:21
bcsallerthe entire test20:22
cory_fuHrm.  That seems heavy handed.  I feel like something small like tweaking a timeout or period should be easier than that20:22
bcsallerthen you need a syntax for handling deletes and so on like you said20:23
cory_fuOr even just adding an additional rule to an existing test20:23
cory_fupetevg: Thoughts on only being able to add new tests or entirely overwrite existing ones, rather than being able to tweak it by adding a rule or changing an arg?20:25
cory_fubcsaller: Regardless of that, I feel like moving the task name up a level makes more sense to me.20:26
petevgcory_fu: My brain is mush right now, but I like the simplicity of just having to override an entire test.20:26
petevgIt means duplicated yaml.20:26
petevgBut less duplicated yaml than having to write a whole new .yaml20:27
cory_fu{do: deploy, args: {version: previous}}20:27
cory_fupetevg: Ok, I'm out voted, then I guess20:27
petevgcory_fu: sorry.20:28
cory_funo problem.  Less code to maintain in matrix.  :)20:28
cory_fubcsaller, petevg: https://github.com/juju-solutions/matrix/pull/26 updated for review21:03
=== natefinch is now known as natefinch-afk
beisneranyone have advice/workarounds for juju controller mem leak/usage issues? http://i.imgur.com/C1WpcK2.png  in just a couple of hrs with the lxd provider i've succeeded in OOMing and Calltracing the pretty beefy host.  juju 2.0.121:56
alexisbbeisner, move to 2.0.221:59
beisnerhi alexisb - wherefrom?22:00
alexisbit is in proposed atm22:00
alexisbbeisner, there may be other work arounds, but 2.0.2 release has some good fixes in it around controller exhaustion issues22:01
beisneralexisb, ack.  will tear down, upgrade, redeploy and stuff.  many thanks.22:01
cory_fupetevg: Usage examples added22:32
cory_fubcsaller: ^22:32
petevgcory_fu: yay docs!22:41
skay_for the 2nd time maybe I've had a crash report pop up from juju-deployer. has that happened to anyone else?22:42
cory_fupetevg: Did I mention that that PR has the newest libjuju with your fixes for the controller panics?  I thought those fixes were in master, but I was still getting them until I updated libjuju again22:43
petevgcory_fu: interesting. I thought that I had updated this morning. Maybe I forgot to pull python-libjuju master or something ...22:43
petevgcory_fu: are you sure that it's not just that you didn't rebuild your tox environment?22:43
cory_fuThat's possible, but I thought I did22:44
cory_fuI can test it real quick22:44
cory_fupetevg: Yeah, you're right.  It seems to be working on master now22:45
petevgcory_fu: the semantics of matrix in your PR are a little confusing. If I have a bundle that sits at, say ~/Code/bigtop.hadoop/bigtop-deploy/juju/hadoop-processing/, and it doesn't have a matrix.yaml, but I want to run the default matrix test suite, how do I do so?22:46
petevgskay_: I haven't had deployer pull up a crash report, if by "crash report", you mean Ubuntu's crash reporter. I have had it just plain crash, but not recently.22:47
cory_fupetevg: matrix  ~/Code/bigtop.hadoop/bigtop-deploy/juju/hadoop-processing/22:48
skay_petevg: yeah, it's trigger Ubuntu's crash reporter. surprised me22:48
skay_I'm running a mojo manifest22:48
petevgcory_fu: that's what I thought the docs were telling me, but that doesn't work.22:49
cory_fupetevg: tests/matrix.yaml is included if present, but can be excluded with -B.  The default suite is always included unless excluded with -D22:49
petevgcory_fu: My matrix log: http://paste.ubuntu.com/23560175/22:50
petevg(It looks like its trying to interpret the bundle as a matrix test.)22:50
cory_fupetevg: Sorry, that should have been: matrix -p  ~/Code/bigtop.hadoop/bigtop-deploy/juju/hadoop-processing/22:50
cory_fuCLI args are additional suites.  -p <path> is the path to the bundle.22:51
petevgcory_fu: Aha. That's it. It tried running -Dp from the example, but that only works if there's a matrix.yaml file, I think.22:51
cory_fupetevg: Added another usage example for that22:52
petevgcory_fu: awesome. Was just about to ask you to do that :-)22:53
cory_fupetevg: Yeah, if you use -D and there's no tests/matrix.yaml, you'll have no suites.  Should probably catch that and report more nicely22:53
petevgcory_fu: merged. This is awesome stuff -- saves me a lot of find-replacing when I test hadoop-processing :-)22:54
cory_fubcsaller: Where would I look to colorize log messages in matrix?22:57
bcsallercory_fu: easier to explain in a hangout22:58
cory_fubcsaller: Ok, I'm in matrix daily22:59
alexisbthumper, axw ping23:01
skay_juju-deployer stacktrace https://www.irccloud.com/pastebin/O28WeBHn/23:04
skay_not sure why the above happened23:05
cory_fupetevg, bcsaller: Quick PR and I'm EOD.  Have a good one23:15
cory_fupetevg, bcsaller: Helps if I include the link; https://github.com/juju-solutions/matrix/pull/3023:15
cory_fubcsaller: Removed the  unused block and merged23:29
bcsallercory_fu: thanks23:29
tvansteenburghskay_: i'd check the juju logs on that one. looks like something went wrong server-side23:35

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