vinowallyworld: addressed ur comments.00:40
vinocud u plz take a look.00:41
wallyworldvino: don't forget to change the PR description00:48
vinosure. i am doing that.00:48
wallyworldveebers: how far away are you from landing your PR?00:51
veeberswallyworld: I shouldn't be too far, was looking at the doc first. I can pivot now (just got back from lunch) and get that sorted.00:57
wallyworldveebers: that would be gr8 as i can't deploy k8s charms at the moment00:58
veebersah I see, that o' chestnut. Ok I'll get it sorted00:58
wallyworldDocker resource with ID: mariadb/mysql_image not found00:58
wallyworldi think that's the error that is fixed?00:58
veeberswallyworld: yep00:58
wallyworldanastasiamac: done01:01
anastasiamacwallyworld: ta01:01
anastasiamacwallyworld: model-constraints are indeed not inherited when new models are added... so --constraints on bootstrap does only set default and controller model constraints... i guess the reason is - how would u know from which model to inherit... i suspect that this is a desired behavior01:03
anastasiamacwallyworld: i will still add --model-constraints as an option to bootstrap01:03
anastasiamacwallyworld: we should probably consider --model-contraints as an option to add-model too...01:04
wallyworldthe pattern for model config is to store default sspecified at bootstrap time into a separate settings bucket01:04
wallyworldyes, it needs holistic thought01:04
anastasiamacwallyworld: yeah... maybe this is what we need to do with constraints too ...01:04
wallyworldshould do a "one pager" to propose a solution01:05
anastasiamacwallyworld: yes, i'll add it to my really-want-to-address-yesterday bucket :)01:05
anastasiamacwallyworld: +1 to one pager...01:05
wallyworldi have lots of those buckets01:05
* thumper goes to get food01:09
babbageclunkthumper: sonofy.co01:30
wallyworldveebers: won't you need to import the sha packages as well?01:46
thumperbabbageclunk: awesome!!!01:47
veeberswallyworld: it seems just once in for the 'binary' is fine. I did a manual test deploy using --resource and straight deploy and all is happy. I can add the import not too for completeness01:48
veebersI'm just sorting the deps01:48
veebersI realised at push that I hadn't done that yet01:48
wallyworldveebers: you can't rely on that01:48
veeberswallyworld: ack fair enough, I'll add the import at usage01:48
wallyworldespecially for say tests, they won't necessarily cause the import side effect01:49
wallyworldsince they only operate on that package they are in plus any transitive deps01:49
veebersack, makes sense01:50
veeberswallyworld: FYI I've pushed the updated branch w/ deps and imports01:54
wallyworldyay ty01:54
thumperbabbageclunk: installed it, doesn't seem to work properly02:00
babbageclunkthumper: ah well, sorry!02:01
veeberswallyworld: which operator image would the edge snap use out of interest? I recall you mentioned that we need to do an image operator push for the edge channel right?02:26
wallyworldveebers: it uses whatever the last person to push a copy has uploaded02:26
veebersah ok02:26
veebersIs there a way to force 'juju update-clouds' to update? Just removing $JUJU_DATA/clouds.yaml won't do the job02:47
wallyworldveebers: public-clouds.yaml02:48
wallyworldclouds.yaml is your personal one02:48
veeberswallyworld: where is it looking for public-clouds.yaml? Can't see it in $JUJU_DATA, nor any /snap/ dir02:50
wallyworldit's in $JUJU_DATA02:50
wallyworldsame as clouds.yaml02:50
veebersmy $JUJU_DATA doesn't have that file :-\ (I've set it to another dir for testing something). the only public-clouds.yaml I can find is ~/.local/share/juju/public-clouds.yaml, but moving that doesn't work (so juju isn't checking multiple places it seems)02:52
wallyworldveebers: sorry, i thought $JUJU_DATA was ~/.local/share/juju02:56
wallyworldupdate-clouds operates in that directory02:57
wallyworldkelvin_: your PR looks fairly complete? main issue I can see is we've lost the make target to update the deps file(s)02:58
veeberskelvin_: you may need to update the pr merge/check jobs with the deps changes. IIRC they explicitly call godeps etc.03:29
wallyworldkelvin_: there's no need to commit the lock file, just the toml03:35
wallyworlddep ensure will do what it needs to do03:35
wallyworldor whatever the cmd is to generate the lock from toml03:35
vinoveebers: i have a Pr for u to review03:43
vinoadding ci test for export bundle feature.03:44
vinocud u plz take a look when u r free ?03:44
kelvin_wallyworld, it should be no problem, we use `godeps` target to ensure deps but now we use `dep`03:44
veebersvino: sure can, link?03:45
kelvin_wallyworld, we should commit lock file with the .toml file together. even we try to use sha for revision but we can also just specify branch of the dep, in this case, the lock file has more detailed version control for the deps.03:46
kelvin_wallyworld, just like  package-lock.json/yarn.lock for node,03:47
wallyworldkelvin_: so the lock file is not simply generated from the toml file?03:47
babbageclunkwallyworld: The lock file should definitely be committed.03:47
kelvin_wallyworld, it's generated by toml file but it could be different later to use the same toml file03:48
wallyworldkelvin_: babbageclunk: ok, nps. whatever best practice is03:48
kelvin_wallyworld, for example, we changed the packages that imports a dep in juju03:48
wallyworldkelvin_: i think it would be useful to expand these use cases in the google doc and we can see what the workflow would be03:49
vinoveebers: went for a tea. link: https://github.com/CanonicalLtd/juju-qa-jenkins/pull/6403:50
kelvin_wallyworld, agreed, i will prepare a doc after i get all the things working correctly03:50
wallyworldyup, sgtm03:50
veebersvino: sweet, I'll hit that in a little bit03:51
babbageclunkare other people getting go vet errors on a format string in k8s.go?03:56
veebersbabbageclunk: I just ran the verify script manually now and get the error (2x errors lines 149x-ish)04:03
veebersbabbageclunk: that's really odd, the verify script is called in the check Make target, which should be what the PR/Build/Anything runs04:05
kelvin_after make add-patches, i got two more errors04:09
babbageclunkveebers: weird04:12
veebersbabbageclunk: yeah I'm a bit confused, I can see in the script where it calls "make check", which will call the verify script, and I'm pretty sure IGNORE_VET_WARNINGS isn't set anywhere04:12
babbageclunkveebers: maybe different versions of go vet?04:12
veebersbabbageclunk: jenkins machines have 1.10.3, I just tried with snap go1.10.3 an my other go1.10 and they both show the error04:14
veebersvino: LGTM04:18
babbageclunkkelvin_: those errors seem weird - if err is undefined how is it building after make add-patches is run.04:18
babbageclunkkelvin_: Is the setprogress one because a patch has been accepted upstream or something?04:23
kelvin_babbageclunk, yeah, it's weird. i need to take a look further.04:31
babbageclunkkelvin_: have you kept the same shas as in dependencies.tsv?04:32
kelvin_babbageclunk, yes, all sha are kept same04:33
babbageclunkhuh, then I don't get it.04:33
kelvin_babbageclunk, i translate the deps from our tsv file directly04:33
babbageclunkright, that sounds very sensible.04:34
vinoveebers: thank u. Have a question abt this node label. feature is for any node having lxd capabilities.05:11
vinothen wat is goodra for ?05:12
veebersvino: goodra is included in the 'features' tag, I imagine you may have seen an example that needed to use goodra expliclity05:13
vinoveebers: i am seeing in bootstrap caas yes.05:13
veebersvino: ah right, that would be due to the lxd version on goodra, we haven't yet been able to update the version on all machines05:14
vinoveebers: ok. ty.05:15
jaceknhello. Is it possible to add model with a specific, older version? I need to test bugfix for older juju versions08:29
rick_h_jacekn: https://docs.jujucharms.com/2.4/en/models-config you can use the --agent-version flag there I believe12:31
jaceknrick_h_: ERROR "agent-version"" must be set via "upgrade-model"12:48
jaceknrick_h_: ERROR cannot change version from 2.4.1 to lower version 2.3.112:48
jaceknI just bootstrapped from scratch12:49
rick_h_jacekn: right, but can you use that on an add-model command?12:49
rick_h_jacekn: I know you can't change it on the fly like other config12:49
rick_h_jacekn: yea, if add-model with the agent version specified doesn't work then yea a bootstrap will have to be the way to go12:49
jaceknah add-model might work (but now I'm getting no agent binaries found for version 2.3.1). I solved the probelm by re-bootstrapping anyway12:51
jaceknthanks for help though12:51
rick_h_jacekn: k, when all else fails go with the hammer heh12:54
magicaltrouthello folks anyone in this channel have half a clue about the lxd snap?13:07
pmatulismagicaltrout, question?13:12
magicaltrouti stuck it in #lxd but the migrate thing gets really confused for some reason and if you say no to removing the old lxc stack it still doesn't let you run the snap lxc comamnds but then something (juju or something else) is reinstalling the .deb lxc packages13:14
magicaltroutcause after a reboot they seem to magically reappear13:14
magicaltroutalso how do you reconfigure the lxdbr in snapworld cause whilst they have intermal ip addresses nothing seem to be able to talk to them now13:16
pmatulismagicaltrout, that's weird (first part)13:18
pmatulismagicaltrout, no idea (second part)13:20
pmatulisi'll test the first part though13:20
magicaltroutthanks pmatulis just for reference this is a manual cloud box13:20
magicaltroutwith a few different containers on, but they've been running in the lxd snap for weeks, the lxd deb stack has been empty for a long while13:21
pmatulismagicaltrout, so you're not using the 'lxd' cloud type. you're just creating containerised machines within a manual cloud node13:27
magicaltroutooooh fml13:34
magicaltroutwhen all your containers start13:34
magicaltroutthen it insists on a migration13:34
magicaltroutso they're up cause some services are responding13:35
magicaltroutbut i can't login to any of them13:35
magicaltroutlxd.migrate completely screws up my lxd bridge for some reason13:50
magicaltroutstuck it in there as well13:58
magicaltrouti would have thought, considering the removing of the lxc debs is optional14:05
magicaltroutthat whilst it wants you to migrate to the new snap14:05
magicaltroutlxd should still function without removing it14:05
magicaltroutso whatever does the detection stuff14:05
magicaltroutseems a bit screwed14:05
kwmonroemagicaltrout: is it possible in the deb env that you symlinked /var/lib/lxd/containers to somewhere else?  i did this once in a similar aws/manual machine to put my containers on the ephemeral /dev/sdb (ln -s /var/lib/lxd /mnt, or something like that).  anyway. i recall lxd.migrating booming on trying to stat the symlink'd containers.14:48
magicaltroutkwmonroe: as if you'd do something as hackish as that! ;)14:49
kwmonroei was trying a new gin&tonic recipe at the time.  hackies ensued.14:50
magicaltrouti think i'm getting stuff back together. The "force this deb not to install" solution from the forums seems to be taking me in the right direction14:51
kwmonroemagicaltrout: one other thing to check.. if you still have an /etc/default/lxd-bridge, that comes from the .deb and may be trying to start a bridge that should be handled by the lxd snap.  so if you have that file and you seem to have conflicting bridges, try moving that like the migrate script would have done:15:00
kwmonroe# ll /etc/default/lxd-bridge.migrated15:00
kwmonroe-rw-r--r-- 1 root root 1206 Jun 28 16:18 /etc/default/lxd-bridge.migrated15:00
manadartexternalreality: A small one: https://github.com/juju/juju/pull/9004. Preparatory patch for upgrade-series worker implementation.15:41
magicaltroutkwmonroe: we're wiring up Druid to the HDFS storage engine and rmcd says it says "Place your Hadoop configuration XMLs (core-site.xml, hdfs-site.xml, yarn-site.xml, mapred-site.xml) on the classpath of your Druid nodes. You can do this by copying them into conf/druid/_common/"16:05
magicaltroutworking on that logic could we build off of layer:hadoop-client or something and get the configs that way?16:05
magicaltroutor apache-bigtop-base?16:15
magicaltroutwhat part of the stack installs the configs?16:15
magicaltroutrick_h_: how do we get a big data category on discourse?16:30
rick_h_magicaltrout: you ask nicely and when I get back from lunch I add it in there for ya16:44
rick_h_Like magic!16:44
magicaltroutDearest Rick16:45
magicaltroutCould we possibly have16:45
magicaltrouta Big Data category in the discourse forum16:45
magicaltroutso that I may ask pertinent questions, like the one above, and their response be stored for all time, so that others may also benifit from Kevins infinite wisdom16:46
rick_h_LoL for you magicaltrout , anything16:49
kwmonroemagicaltrout: you want to build on layer:hadoop-client.  that will include the hadoop-plugin relation, so once your charm relates to hadoop-plugin, the system will automatically install all the hadoopy things (including ./conf files) from bigtop.17:17
kwmonroethen you make a druid reactive handler that says @when(hadoop.hdfs.ready), DO_THE_STUFF17:18
magicaltrout thanks kwmonroe17:18
kwmonroenp magicaltrout.  you can also include layer:apache-bigtop-base, but i would only do that if druid is a bigtop project.  the only thing that layer helps you do is setup puppet for bigtopy stuff to happen.17:20
rick_h_magicaltrout: https://discourse.jujucharms.com/c/charms/big-data17:27
rick_h_kwmonroe: ^17:27
magicaltroutwhy thanks rick_h_17:31
rick_h_magicaltrout: :)17:31
veebersMorning all o/20:44
rick_h_morning veebers20:46
rick_h_happy friday to you20:46
veeberswhy thank you rick_h_ :-) It's a foggy wet Friday but I won't complain :-)20:54
rick_h_hey, better than it being thurday! :P20:56
veebershow are things today  cory_fu, I haven't broken anything else for you yet? :-)20:56
veebersIndeed! Those poor suckers who are still stuck on Thursday20:57
cory_fuveebers: :)  All good, thanks21:03
veeberswhat's the level of repetition that makes sense for a table based test? i.e. if I'm doing to comparisons is it worth doing?22:53
thumperveebers: I'm moved away from table based tests mostly23:09
thumperInstead consider a helper function23:10

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