/srv/irclogs.ubuntu.com/2017/02/06/#juju-dev.txt

anastasiamacwallyworld: btw, plz tag with "eda" whichever bug u decide is primary for "application in error, can't destroy model" to allow QA the ooportunity to add functional test for this scenario00:27
anastasiamacopportunity even :D00:27
anastasiamacwallyworld: also in addition to bug # 1654928, there is possibly related bug # 166193000:49
anastasiamacwallyworld: patience is not on this monday :) I have updated the bugs as per ^^01:06
axwwallyworld: can you please review https://github.com/juju/juju/pull/692403:46
axwwallyworld: ping? did you see my message?04:56
wallyworldaxw: sorry, missed it, i had my sound turned down so missed the ping05:56
wallyworldwtf, i already reviewed that pr and hit approve05:57
wallyworldmust have timed out, it does that when i bootstrap on aws and it floods by connection uploading agents05:58
axwwallyworld: thanks05:59
wallyworldanastasiamac: bug 1465317 isn't critical, the 2.1 binary works fine with current windows/macos and we will have released 2.2 before win 11 or macos 10.13 are close to being released. it's a fairly large fix so should be targetted at 2.206:01
mupBug #1465317: ubutu devel macos win: panic: osVersion reported an error: Could not determine series <osx> <packaging> <wily> <windows> <juju:Triaged> <juju-core:Won't06:01
mupFix> <juju-core 1.24:Won't Fix> <juju-core 1.25:Won't Fix> <juju-release-tools:Fix Released by sinzui> <https://launchpad.net/bugs/1465317>06:01
anastasiamacwallyworld: sure. plz adjust06:02
* wallyworld wishes bug reporters would assign bugs to the correct series06:14
anastasiamacwallyworld: if u r looking for bugs to do, this one is awesome - bug 165411606:19
mupBug #1654116: Attempts to write leadership settings when not the leader during relation-changed hooks <cdo-qa-blocker> <landscape> <uosci> <Autopilot Log Analyser:Fix Committed> <Charm Helpers:Invalid> <juju:Triaged by menno.smits> <Landscape Server:Confirmed for fginther> <juju (Ubuntu):Invalid>06:19
mup<ceilometer (Juju Charms Collection):Invalid> <rabbitmq-server (Juju Charms Collection):Invalid> <https://launchpad.net/bugs/1654116>06:19
wallyworldtis ok, i've found one06:20
=== frankban|afk is now known as frankban
anastasiamacwallyworld: did u test ur fix in https://github.com/juju/juju/pull/6906 for bug 1643430 with subordinate in error?08:50
mupBug #1643430: Unassigned units in an error state cannot be removed <juju:Fix Committed by wallyworld> <juju 2.1:Fix Committed by wallyworld> <https://launchpad.net/bugs/1643430>08:50
anastasiamacwallyworld: bug 1662018 seems to be in the same area08:50
mupBug #1662018: Cannot remove a deployed subordinate charm in a failed hook status <juju:Incomplete> <https://launchpad.net/bugs/1662018>08:50
mupBug #1661576 changed: show-status-log formatting contains unwanted pad lines <usability> <juju:Triaged> <juju 2.0:Won't Fix> <https://launchpad.net/bugs/1661576>09:00
mupBug #1661624 changed: show-status-log bogus timeline <usability> <juju:Triaged by hduran-8> <juju 2.0:Won't Fix> <https://launchpad.net/bugs/1661624>09:00
wallyworldanastasiamac: i didn't test with subordiante, fix should be same09:25
anastasiamacwallyworld: plz either comment on the bug with ur assertive conclusion or add a test \o/ thnx!09:52
rogpeppeanastasiamac: hiya10:37
rogpeppehere's a backport of the api dial spurious message fix to juju 2.1: https://github.com/juju/juju/pull/692610:44
rogpeppereviews appreciated10:44
perrito666morning10:52
perrito666jam: tx for the review on the ipv6 fix10:52
perrito666jam: does cheking on the glibc part of the error seems like enough to you?10:52
jamperrito666: "protocol not supported" ?10:57
perrito666"socket: address family not supported by protocol"10:57
* perrito666 is working at a bar and thinks people look weird at him because he is not using a mac10:59
jamperrito666: I'd be much happier if we had a ENOADDRSUPPORT with an official value rather than a random string that someone is going to think could be  better worded differently10:59
jamperrito666: but I can't think of anything better vs that10:59
perrito666jam: I dont think lxd library is going ot grace us with said code :p11:00
perrito666jam: also I would appreciate your input on how you think this should be tested other than actually invoking lxd (which seems like the wrong thing to do on a test)11:18
jamperrito666: can we mock out the thing we are calling SetServerConfig and have it return a custom error?11:19
=== jamespag` is now known as jamespage
perrito666jam: mot likely, but that woul not test what you want to test which is the fact that the string is still valid11:26
anastasiamacrogpeppe: hi \o/ m neck-deep in bootstrap failures but there is light \o/ m OCR tomorrow (it'll start here in couple of hrs), if noone else gets to it, I'll look then :D11:45
anastasiamacrogpeppe: i'll review in about 10hrs? :)11:45
rogpeppeanastasiamac: np - jam already hit $$merge$$ on it...11:45
anastasiamacrogpeppe: jam is awesome \o/11:48
jamrogpeppe: IIRC I approved it for 2.2, happy to rubber stamp it for 2.111:49
rogpeppejam: you added a $$merge$$ comment to https://github.com/juju/juju/pull/6926 which targets 2.111:49
rogpeppejam: and it's now merged11:50
jamrogpeppe: I mean I approved the work when you targetted develop a week or two ago11:50
rogpeppejam: i see a comment from an hour ago11:50
jamrogpeppe: and thus I approved the one you proposed today11:50
rogpeppejam: ah, ok great, i thought perhaps you hadn't realised what you were doing11:50
jamwell, I generally don't :). but this time I did'11:51
rogpeppejam: i'd read the future tense into "happy to rubber stamp it for 2.1" but it was actually the past tense :)11:51
rogpeppejam: :)11:51
jamI left off the "was" and "is" is a perfectly acceptable word to put in ther.11:52
anastasiamacrogpeppe: we tend to only approve one copy. thus back-ports and forward-ports r good to self-approve unless there r vast differences with original :D11:52
rogpeppeanastasiamac: ah, ok. i thought it's worth getting approval that it's worth backporting though.11:53
perrito666jam:  ptal https://github.com/juju/juju/pull/6916 ill add a pr to  2.1 too once this one is lgtmed11:54
hoenirHi, I'm tring to direct the juju client to insepct the index2.json but I'm keep getting this error in bootstrap invalid URL "file:///var/www/html/toolsdir/tools/streams/v1/index.sjson" not found12:10
hoenirin v1 i have just 2 json filex inde2.json and com.ubuntu.juju-devel-tools.json12:11
hoeniralso I'm runing juju with like this12:12
hoenirhttps://paste.ubuntu.com/23940708/12:12
hoenirany thoughts?12:12
perrito666abentley: could you shed some light on hoenir's problem?  I am not sure what --metadata-source expects12:14
hoenirI'm working on the cloud provider and in the process of the actual bootstrap the bootstrap config searches for images and tools in the default ubuntu sources and I specify the --metadata-source and customise it to pick the correct juju agent and image definitions.12:26
hoeniron the cloud provider I mean oracle cloud provider.12:26
hoenirSo I don't know why the simplestreams/tools are looking into index.sjson and not looking into index2.json12:27
hoeniralso when tring to validate the tools with the juju metadata validate-tools -d /var/www/html/toolsdir --debug I get the following errors/warnings.12:28
hoenir14:28:05 DEBUG juju.environs.simplestreams simplestreams.go:457 skipping index "file:///var/www/html/toolsdir/tools/streams/v1/index2.json" because of missing information: "content-download" data not found12:29
hoenir14:28:05 ERROR cmd supercommand.go:458 "content-download" data not found12:29
hoenirwhy?12:29
perrito666do you have said key?12:29
hoenirhttps://paste.ubuntu.com/23940769/ index2.json12:30
hoenirand the ubuntu.com one.12:31
hoenirhttps://paste.ubuntu.com/23940770/12:31
hoenirperrito666, what key?12:31
perrito666hoenir: I misinterpreted the error12:31
perrito666that error is really not informative :( we should fix that12:32
hoenirAlso I generate the tools with this cmd juju-metadata generate-tools --debug --stream devel12:32
perrito666jam: rogpeppe ever heard of it?12:32
rogpeppeperrito666: heard of what?12:32
hoenirAnd the documentation about the juju metadata is quite awful on the official site.12:33
perrito666rogpeppe: 14:28:05 DEBUG juju.environs.simplestreams simplestreams.go:457 skipping index "file:///var/www/html/toolsdir/tools/streams/v1/index2.json" because of missing information: "content-download" data not found12:33
perrito666never bumped into it12:33
perrito666hoenir: that is why I asked abentley or eve balloons they are more likely to have tinkered with simplestreams12:33
perrito666brb, need to relocate12:33
rogpeppehoenir, perrito666: is it actually allowed to give a file: URL as a metadata source?12:34
hoeniralso I'm on the tip of the develop branch12:34
rogpeppehoenir: have you done this kind of thing before?12:35
hoeniryeah you can spiecify the directory or you cand specify the url in the --config like --agent-metadata-url12:35
rogpeppehoenir: what files are inside that /var/www/html/toolsdir directory?12:35
jamhoenir: 'content-download' sounds like the image files, not the tools files12:35
jammaybe not, one sec12:35
hoenirYeah, when I was using the mass in conjunction with juju12:35
jamhoenir: http://streams.canonical.com/juju/tools/streams/v1/index2.json clearly has 'content-download'. I was thinking about the image binaries, but agent binaries also need content-download12:36
jam(vs AMI, etc)12:36
hoenirAlso when you guys will update the provider documentation in the juju/juju repo ? It's pretty outdated.12:36
hoenirI posted above pastes with my jsons.12:37
jamhoenir: offhand I'm wondering if your juju client isn't looking for 'devel' tools12:39
jamtry setting 'agent-stream: devel'12:39
hoeniralso the bootstrap --debug file https://pastebin.ubuntu.com/23940814/12:39
jamI see you are passing that in config12:39
rogpeppejam: if that is the case, then that error message should really be improved12:39
hoenirAfter the err returned I os.Exit(1) to stop the execution .. If you have better ways to debug the code/inspect analyze other tha fmt.Printf("%+v\n", that_var), fmt.Println(err), os.Exit(1) please tell me.12:40
hoenirlike?12:40
hoenir--config agent-stream: devel12:40
hoenir?12:40
hoenirlike this?12:40
rogpeppejam: to something like 'no appropriate tools found'12:40
jamhoenir: your bootstrap has --config agent-stream=devel which is the correct syntax for CLI (if you put it in a .yaml file it is : vs =)12:41
hoenirok I will try now with agent-stream thing.12:42
hoenirbut what about the --config development=true ?12:42
hoenirstill the same thing.12:43
hoenir"no tools available"12:43
hoenircan anyone point me into some friendly doc on how to generate the tools and images correctly? and what's the best method to point the juju config to it? Also keep in mind this is very experimental and I'm woking to add a new provider.12:46
hoenirAlso this is how I generated the images and escaped that "no oracle cloud found in default images error" juju metadata generate-image -a "amd64"  -r "us6" --stream "devel" -u "https://api-z52.compute.us6.oraclecloud.com" --debug -i 2016035612:48
hoenirIs this the correct way?12:48
hoenirthis is the tree structure https://pastebin.ubuntu.com/23940870/12:49
hoenirhow my metadata is laid out12:50
hoeniranyone?12:50
perrito666Hoenir I am one of the only ones online now and I don't know the answer sadly12:51
hoenir:(12:52
perrito666If you get a handle of sinzui he will know it12:52
hoenirperrito 666, I don't know why but this should have a simple solution but clearly the lack of documentation really bothers me.12:53
hoenirAt this point I don't have any solution to just fmt.Println() os.Exit(1) step line by line and know what is the problem.12:53
hoenirand it's painful but someone gotta do it.12:54
hoenirAlso I hate that the golang dosen't have a standardized compile like C has(gdb).12:54
hoenirdebugger*12:55
hoeniranyway perrito666 thanks.12:55
perrito666Hoenir I'll make sure to fwd your problem to the right people when they come online12:56
abentleyperrito666: I am not certain what it accepts, either.  Probably a path to a directory containing streams is permitted.  When I tinker with that, I use the agent-metadata-url config setting.14:07
perrito666Hey I just went through one of these eye exams where they dilate your pupils a lot in one eye so don't expect me to answer chats for an hour or so14:34
perrito666I'll use my good eye to answer emails though14:34
frobwareperrito666: I have these regualarly. in both eyes. It makes getting back from the hospital an interesting exercise...14:47
perrito666frobware: I feel like someone messed up with my white balance and my focus15:24
redirmorning juju-dev15:37
perrito666morning redir15:37
* perrito666 is starting to see again15:40
perrito666frobware: how long does this last?15:40
* perrito666 should have asked before leaving15:41
hoenirmorning.15:42
frobwareperrito666: for me around 4 hours.15:45
frobwareperrito666: but also complicated as my pupils are overly dilated due to RP.15:46
perrito666ghaaa 4 hs?15:46
perrito666it has been 2 and still dont see very well so its possible that for me its going to be 315:46
perrito666I have the fantastic convo of swollen eye and change in necessary correction15:47
=== mskalka is now known as mskalka|lunch
=== mskalka|lunch is now known as mskalka
mupBug #1662272 opened: Agents stuck with "dependency not available" <juju-core:New> <https://launchpad.net/bugs/1662272>17:49
mupBug #1662272 changed: Agents stuck with "dependency not available" <juju-core:New> <https://launchpad.net/bugs/1662272>17:55
mupBug #1662272 opened: Agents stuck with "dependency not available" <juju-core:New> <https://launchpad.net/bugs/1662272>17:58
=== menn0 is now known as menn0-otp
=== frankban is now known as frankban|afk
thumpermorning19:39
alexisbmorning thumper19:49
thumperhey19:49
babbageclunkthumper: do you know who needs to say $$merge$$ on this pr? https://github.com/go-goose/goose/pull/3921:15
babbageclunkthumper: can you do it?21:15
thumperbabbageclunk: I'll check21:16
babbageclunkthumper: clicking through some old PRs I only see axw and bz221:17
thumperI think the person needs write access to the repo21:18
thumperand I don't have that either21:18
thumperbabbageclunk: I have been told of three sprints this year, the next one next month.21:18
thumperhmm21:18
thumperthat isn't what I meant to paste21:18
thumperbabbageclunk: https://hangouts.google.com/hangouts/_/canonical.com/morning-chit-chat21:18
thumpernope21:18
thumpernot that either21:18
thumperwhere did all my clipboards go21:19
babbageclunklol21:19
thumperpleased I didn't have anything bad in there21:19
babbageclunkklajklj83jkhjs21:19
thumperbabbageclunk: https://github.com/orgs/go-goose/people21:19
thumperbabbageclunk: spicer's twitter password21:19
babbageclunkoh man, now I have to change all my passwords21:19
babbageclunkwallyworld: Can you merge this please? (or add me to the organisation I guess) https://github.com/go-goose/goose/pull/3921:22
=== menn0-otp is now known as menn0
redirdo we realy support puma, jaguar, panther?21:46
perrito666redir: ???21:49
redirperrito666: looking at series/supportedseries21:49
wallyworldbabbageclunk: you should have an invite to goose, see if you can now merge22:05
babbageclunkwallyworld: awesome, thanks22:06
thumperwallyworld: did you add babbageclunk to the go-goose org in github? doesn't look like it to me22:44
thumperhttps://github.com/orgs/go-goose/people22:45
wallyworldthumper: it said an invite was sent22:45
thumperbabbageclunk: did you accept invite?22:45
babbageclunkthumper: yeah, but I forgot that I needed to make my memebership public.22:56
thumperbabbageclunk: ah...23:12
wallyworldthumper: since at the moment you are looking at memory issues, it makes sense to assign bug 1660087 to you from menno - its probably already fixed in an earlier beta; easier to keep such things under the ine umbrella?23:23
mupBug #1660087: Controller and all models unresponsive <canonical-is> <juju:Triaged> <juju 2.1:Triaged by menno.smits> <https://launchpad.net/bugs/1660087>23:23
=== mramm_ is now known as mramm
=== iatrou_ is now known as iatrou
=== Dmitrii-Sh_ is now known as Dmitrii-Sh
=== petevg_ is now known as petevg
=== mpontillo_ is now known as mpontillo
=== plars_ is now known as plars
=== cargonza_ is now known as cargonza

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