/srv/irclogs.ubuntu.com/2018/05/16/#juju.txt

=== cmars_ is now known as cmars
thumperquick PR for someone https://github.com/juju/bundlechanges/pull/4001:06
thumperand another https://github.com/juju/gomaasapi/pull/7301:17
veebersthumper: lgtm on the fist one01:17
veebersfirst*01:17
veebersthumper: reviewed the 2nd, one question and a suggestion ^_^01:19
thumperveebers: juju/names is the old one01:19
thumperand not used again01:19
veebersthumper: ack, makes sense01:19
thumperyes, many of the things I'm doing need merge jobs01:20
thumperveebers: there used to be a merge bot... was it the old one?01:21
veebersthumper: probably, many moons ago. All that old stuff is long gone01:24
* thumper sighs01:24
thumperI know we went through it last time...01:24
veebersthumper: it's for the best ;-) Have you done a merge job before? it's pretty straight forward01:24
veebersand also lots of fun01:24
thumperlies01:25
veebersand, uh, helps the environment?01:25
thumpercan you point me to the notes again?01:25
thumperhttps://github.com/juju/description/pull/3801:32
wallyworldveebers: tiny one https://github.com/juju/juju/pull/870602:15
veeberswallyworld: hey, no name calling, I might get taller one day. Oh you mean the PR right. What does ?= do, or is it a typo02:17
wallyworldveebers: it means set if empty02:17
veebersack02:17
wallyworldwe also use it for tje docker user name02:18
wallyworldallow env var to override02:18
veeberswallyworld: ack, quick question on the pr, looks good to me though02:19
wallyworldlooking02:19
wallyworldveebers: answer make sense?02:21
veeberswallyworld: indeed, LGTM02:21
wallyworldty02:21
wallyworldkelvinliu__: veebers: we need to talk at some point about k8s CI testing - the docker operator image needs to be built and uploaded as part of that whole process02:23
wallyworldi just did it manually for rc102:24
veeberswallyworld: ack, there was a brief email thread about that. I'm pretty sure we can fit that into our CI-Run bits easily enough (i.e. add an early step in the build process)02:24
wallyworldyup, no big issue, just something to add to the todo list02:24
veeberswallyworld: or is that for a release? (or both, something for each commit and something official for a release)02:25
wallyworldwe'll need to get the qa user to have perms on the jujusolutions repo02:25
wallyworldboth02:25
wallyworldwe need to push somewhere to test prior to release02:25
wallyworldwe can push to a qa namespace and override the source in juju for testing02:26
kelvinliu__yes, agreed, we could make it automated02:26
kelvinliu__or should we consider to create a separate dockerhub account for test to avoid some weird thing happened like test flow pushing image to the released image namespace. wallyworld veebers02:39
wallyworldkelvinliu__: a separate account is what i was thinking yes. there's a controller setting to override the default "caas-operator-image-path"02:46
kelvinliu__yeah02:48
veeberswallyworld, kelvinliu__ excuse my ignorance, what is getting pushed? can it be uniquely named? i.e. can I push something named ci-run-abc123?02:51
veebers(if so) We can then inject that as an env var so the test jobs can pick it up02:52
wallyworldveebers: it's a docker image for the jujud agent on k8s. it is tagged with the release number02:52
wallyworldtheoretically that would be sufficient02:52
wallyworldeg if we have released 2.3.7 and are testing 2.3.8 there's no clash02:53
wallyworldas they have different labels/tags02:53
wallyworldso they could all live in the jujusolutions namespace. depends on how paranoid we want to be about accidentally overwriting02:53
veeberswallyworld: but for ci runs where it's the same version but different builds it will need to be unique (ci runs can be run in parallel)02:53
wallyworldright, hence we'd use the controller config setting02:54
wallyworldpush with a tag and then use that path in the controller config02:54
wallyworldthe image path is <dockerusername>/<imagename>:<tag>02:55
veeberswallyworld: ah right, if <imagename> can be anything we're sorted02:55
wallyworldCI tests should not pollute the offcial user namespace with test images02:55
wallyworldso we'd want to use a qa namespace for that02:56
veebershaving a namespace for ci runs seems nice, de-clutters actual release stuff02:57
wallyworldyup02:57
wallyworldkelvinliu__: free for 5 minutes before next meeting, quick chat?03:53
kelvinliu__yup,03:53
wallyworldam in standup03:53
thumperreview anyone? kinda simple, just big https://github.com/juju/juju/pull/870703:58
thumper229 files03:58
thumpervery mechanical changes03:59
anastasiamaccould i plz have a review for https://github.com/juju/juju/pull/8709? removing distant relations in status... really simple and straightforward :D04:33
babbageclunkanastasiamac: looking04:49
blahdeblahanastasiamac: +1 to feature, and great function names like TestFilterOutRelationsForRelatedApplicationsThatDoNotMatchCriteriaDirectly04:49
blahdeblahI have no idea whether the code is good or not, though. :-)04:49
babbageclunkblahdeblah: I'll factor your enthusiasm into the review!04:52
blahdeblah 👍04:53
anastasiamacblahdeblah: \o/ i had to look at a test recently that i myself wrote in another life... it took me longer to figure out what it does than to actually write it originally... had to b very explicit here with status ;)04:54
anastasiamacanother simplistic review plz - https://github.com/juju/juju/pull/8710 ... this one just reduces loggin noise :D literally 4 lines of changes maybe...04:55
veebersanastasiamac: +104:56
anastasiamacveebers: ta :)04:57
anastasiamacbabbageclunk: fwiw, i wrote the test first to see it fail with current impl... then fixed the code :)05:04
babbageclunkanastasiamac: ooh, fancy! Approved.05:04
anastasiamacbabbageclunk: :)05:04
babbageclunkanastasiamac: hmm, looks like it still fails though?05:05
anastasiamacbabbageclunk: yep :( looking05:05
anastasiamacbabbageclunk: it seems to b racy \o/05:06
babbageclunkoh stink, I've got a few of those too.05:06
wallyworldkelvinliu__: tiny review? https://github.com/juju/juju/pull/871106:10
kelvinliu__wallyworld, looking now06:11
kelvinliu__wallyworld, looking all good to me.06:17
wallyworldty06:17
wallyworldi'm happy about the size reduction06:17
kelvinliu__wallyworld, the image size is much smaller, nice!06:17
wallyworldindee06:18
wallyworldd06:18
myrathello guys06:22
=== frankban|afk is now known as frankban
manadartSmall one for review: https://github.com/juju/juju/pull/871309:42
myrathow create a openstack project09:43
manadartexternalreality: Any chance you could take a look at that one? ^10:37
bobeo_kwmonroe: rick_h_ do either of you know what port the termination port for spiceclient proxy is for nova-cloud-controller? Im having issues with web based access to my project instances, and I know it has something to do with spice and the nova cloud controller, im assuming it needs to terminate on that server, and pass to the novaa servers from there.13:27
kwmonroebobeo_: beisner or admcleod_ are my openstack gotos.  one of them might know deats about spice/nova ^^.  you also might want to try asking in #openstack-charms.13:54
admcleod_bobeo_: you just enable the config option on nova-cloud-controller, then you use the api to get an access url (i.e. openstack console url show instance_id)15:06
bobeo_admcleod_: how do I "use the api" to get the access url? Via the web UI, it provides a token for the instance, how do I forward for that?15:09
admcleod_bobeo_: when you say webui what do you mean? openstack dashboard? horizon?15:10
admcleod_+ whats the token?15:10
bobeo_admcleod_: openstack dashboard (horizon)15:10
bobeo_it provides a project instance id via /horizon/project/instances/<id>15:11
bobeo_it also does it via /spice_auto.html?<token>15:11
bobeo_i figured I could use haproxy to forward for those15:11
admcleod_bobeo_: do be honest, i dont use horizon, and we typically dont use spice, so anything i could tell you i would read here: https://docs.openstack.org/nova/pike/admin/remote-console-access.html15:13
manadartAnother up for review: https://github.com/juju/juju/pull/871516:04
=== frankban is now known as frankban|afk
bobeo_kwmonroe: beisner admcleod_ I think i found the issue as to why spice isnt working. I noticed in the nova compute after reading the docs that the update for spice applies at the nova compute, but openstack via openstack dashboard (horizon) processes the connection with the nova cloud controller directly, but I dont see anything that also modifies the connection for spice in the18:59
bobeo_nova cloud controller. with that being said,a nd with spice expressly requiring vnc settings to be disabled as per openstack docs, this would mean that I dont believe there is a proxy pass to the nova compute nodes, which would create a break inside the config files correct? or am I missing something?19:00
bobeo_nova compute - http://paste.ubuntu.com/p/DsPF7wRbpW/  nova cloud controller - http://paste.ubuntu.com/p/TH9jNQ6s3B/ with spice configured to be used.19:01
bobeo_further review shows that the [spice] section inside the nova cloud controller isnt configured with anything for spice in the nova.conf file, even though inside the juju config file, for the nova cloud controller, according to juju, spice is configured.  nova cloud controller config: http://paste.ubuntu.com/p/7bx68YZCW4/   is this another bug?19:06
admcleod_bobeo_: is there anything for spice in nova.conf on the nova-compute nodes?19:09
bobeo_admcleod_: yes, which is what makes me think the issue is either with the nova cloud controller config since there is a section for spice in it, but its blank, and horizon forwards to the nova cloud controller19:13
bobeo_or its with nova cloud controller not proxy forwarding to the nova compute nodes directly, which would simply require the nova cloud controller to proxy to the nova computes. My only question would be how would it know how to do that, except through nova-consoleauth19:14
admcleod_bobeo_: if you think there is a bug you should definitely log it..19:14
bobeo_admcleod_: I dont think I would be the best person for that. I honestly dont understand the charms enough to speak well enough for it.19:14
admcleod_bobeo_: if you think there is a problem, thats understanding well enough - the docs should be clear enough, or things should 'just work'19:15
admcleod_bobeo_: unfortunately i have less experience with spice + horizon than you do19:15
bobeo_admcleod_: Ill definitely try then. its done through launchpad right?19:15
admcleod_yep - if you find the charm you think is at fault on the charm store, there is a link to 'submit a bug' on the right hand side19:16
kwmonroebobeo_: +1 to admcleod_'s suggestion for filing a bug -- you definitely don't need to know the inner workings of the charms to do that.  worst case, somebody comes along and says "you're doing it wrong; do it this way".  best case, there's an actual doc/bug that needs fixing.  either way, you'll get unstuck :)19:24
TheAbsentOnehey guys, is it possible in a bundle to have an option that says how many charms need to be deployed? For example I have charm a, b, c (all of them only once) that all have a relation to charm x. In some cases I only need 1 x but sometimes I need 5 x. Is this possible?19:28
TheAbsentOnekwmonroe you will probably know that x)19:28
bobeo_TheAbsentOne: I have seen this done19:29
TheAbsentOneOhn cool bobeo_ do you have any docs on that?19:29
bobeo_TheAbsentOne: I have you one better, https://api.jujucharms.com/charmstore/v5/~elasticsearch-charmers/elk-stack/archive/bundle.yaml19:30
bobeo_it deploys two seperate instance units of elasticsearch inside the ELK bundle, which if Im understanding correctly is what you are looking to do, except maybe not with ELK19:31
kwmonroeTheAbsentOne: you lost me on your use case19:31
bobeo_TheAbsentOne: is that what you were talking about?19:31
TheAbsentOnehaha xD I need to figure out if it's gonna work with multiple units but I really want different machines for it to work19:31
kwmonroewhat is 'x' where you would sometimes need 1 and other times need 5 in the same bundle?19:31
TheAbsentOnewait I will illustrate it with a pastebin!19:32
TheAbsentOnex is a charm!19:32
bobeo_so like -machine 1  ; -machine 2 ; -machine 319:32
bobeo_https://api.jujucharms.com/charmstore/v5/openstack-base/archive/bundle.yaml TheAbsentOne19:33
TheAbsentOnehttps://pastebin.com/LANAB7pN19:34
TheAbsentOneyeah bobeo_ that was my first thought too, with machines19:34
TheAbsentOneBut it would be cool if a user of the bundle could give the number of x's that need to be deployed19:35
bobeo_hmmm..I know after the bundle is deployed they can easily deploy additional units19:36
TheAbsentOneyeah exactly therefore it's no big deal, it was just a showerthought or rather question x)19:37
TheAbsentOnethanks for the help though guys!19:37
admcleod_num_unis19:38
kwmonroeTheAbsentOne: afaik, there is not a way to do dynamic bundles -- meaning you can't adjust num_units or applications at deploy time.19:38
bobeo_TheAbsentOne: I think its a great point, especially towards the point that I would like to have isolated instances of postgres in the same model. FOr instance, I have app A that I want to keep data from app B completely isolated from, and have a different config file for. Currently, im required to deploy an additional model so I can deploy additional postgres instances.19:38
kwmonroeTheAbsentOne: for your use case, you might be better served with multiple bundles.. mybundle-core could have 1 'x' deployed and related.  mybundle-scaled could have 5 'x' deployed and related per your paste.19:39
TheAbsentOnethat's clear kwmonroe, that was basicly my question. Sorry for the poor formulation and yeah multiple bundles or clear instructions for users to edit it to their needs19:39
kwmonroebobeo_: you can deploy the same postgres charm multiple times19:39
bobeo_kwmonroe: yea, id agree with what he said. ELK did something something simliar, and bdx did something like that as well.19:39
bobeo_kwmonroe: yea, but I want different deployments. for instance, app A, i dont want replication on, app b, I do.19:40
TheAbsentOnebobeo_: you can deploy the same charm multiple times without issue, just give them a name19:40
TheAbsentOneahn I see19:40
TheAbsentOneand kwmonroe was way ahead of me xD19:40
kwmonroeyeah bobeo_, what TheAbsentOne said.  you can just pick a different name for the pg deployment19:40
bobeo_TheAbsentOne: wait, I think I misread that, you mean I can have different charm instances of the same type of charm, like postgres? So I can have postgresA, which has no replicaton turned on in the config, and then postgresB, which does? in the same model?19:41
kwmonroeyup19:41
bobeo_kwmonroe: 8O19:41
bobeo_kwmonroe: can you provide me a command line on that?19:41
TheAbsentOnejuju deploy postgresql postgres-a ; juju deploy postgresql postgres-b19:42
TheAbsentOneand you will have 2 different machines19:42
TheAbsentOneavailable at your command x)19:42
kwmonroeyup ^^ that's it on the command line.. here it is in a bundle: http://paste.ubuntu.com/p/T2RxVsHJQ2/19:43
kwmonroebobeo_: ^19:43
bobeo_kwmonroe: TheAbsentOne with totally separate postgres configurations? so id do juju config postgres-a for config a and juju config postgres-b for config b?19:44
kwmonroeyup bobeo_19:44
TheAbsentOneyep completely different machines19:44
bobeo_kwmonroe: DATAFARM!!!! I can build replicated databases on the same MODEL!19:46
kwmonroeit's like i can see the lightbulb burning from here19:47
TheAbsentOneHaha x) good stuff guys19:48
kwmonroe:)19:48
bobeo_juju deploy postgres postgres-a --to lxd:0 && juju add-unit -n 2 postgres postgres-a --to lxd:0 && juju add-relation postgres:peer postgres:peer postgres-a19:50
bobeo_kwmonroe: TheAbsentOne yes?19:50
bobeo_replicated triplicate databases on a single machine, machine 0, in lxd.19:50
kwmonroebobeo_: once you provide that first name, postgres-a, that's how you'll be referring to it.  so it's not "juju add-unit postgres postres-a", but rather "juju add-unit postgres-a"19:52
TheAbsentOneyah wanted to say thing, once it gets a name always use the name and not the charm name anymore19:52
kwmonroebobeo_: and there's no need to explicitly add the peer relation -- juju does that for you19:52
bobeo_kwmonroe: oOOOH! So postgres effectively bcomes postgres-a or postgres-b, depending on which one I refer to19:52
kwmonroeyup bobeo_19:53
bobeo_so the charm name is not a permanent name, but more so a placeholder name, with the permenant name declared at deployment19:53
bobeo_OMG!19:53
bobeo_I CAN HAVE DIFFERENT TYPES OF NOVA COMPUTES!19:53
kwmonroeyeah bobeo_, postgres-a in your example is what you've chosen for your application name.  it's optional and defaults to the charm name if not specified, but you're welcome to give it whatever you want:19:54
kwmonroe$ juju deploy --help|head -119:54
kwmonroeUsage: juju deploy [options] <charm or bundle> [<application name>]19:54
kwmonroebobeo_: i just double checked, and your add-unit won't quite work as written.  juju add-unit -n 2 postgres-a --to lxd:0 would add 2 new postgres-a units, but only the first would go to a new lxd container on machine 0; the second will go to a new machine.20:00
kwmonroenot sure if that's by design or not.. but to get what you want, you'd have to do:20:00
kwmonroejuju deploy postgresql postgres-a --to lxd:0 && juju add-unit postgres-a --to lxd:0 && juju add-unit postgres-a --to lxd:020:01
TheAbsentOneohn did not know that20:01
kwmonroeyeah, that's new to me too20:03
bdxhttps://www.dropbox.com/s/g5e85591y1l4ahd/IMG_9161.JPG?dl=0 - just got these 40Gb switches in, looks like they run debian 😓 😓 😓20:18
bdxhave no idea what to do with this lol20:18
wpkbdx: install Ubuntu? :)20:18
bdxwpk: definitely crossed my mind20:19
bdxI'm wondering if its just straight linux networking going on here20:20
bdxeach port shows up as an interface or something20:21
bdxthats all I can think of ... unless there is some special networking application running20:21
bdxwell I know not the user name and password, so I'm going to have reinstall something on it20:22
zeestratbdx: Those Dell/Broadcom S6000?20:23
bdxya20:24
bdxzeestrat: you have experience with these?20:24
kwmonroebdx: did you try admin/admin?20:24
bdxfrom what I can gather, it looks like they just use they network stack of whatever os you install on it20:25
kwmonroeWindows For Workgroups!  make it happen bdx20:25
bdxyes20:25
bdxtried a bunch lol20:25
veebersMorning20:26
zeestratbdx: Nah, just seen them around. As you say, I think there are a couple of different stacks. Looks like there's some cumulus stuff and https://github.com/Azure/SONiC20:27
bdxtotally, I'm finding a bunch of stuff out there, just another new thing to learn thats all  :)20:28
zeestratAny special plans or just TOR?20:29
bdxahh, I was hoping these most inexpensive switches could lay the foundation for my SAN20:32
bdxpicked up 3 of them used for < 2k20:32
bdx:)20:32
bdxnow, now is when I pay the prie20:33
bdxprice20:33
bdxthe price of time20:33
zeestratHaha. Man, I do get jealous of the decent used and refurbished market in the US.20:35
zeestratWhat kind of storage you doing?20:35
bdxI've been grabbing used gear from https://unixsurplus.com and https://www.orangecomputers.com - possibly they will ship to you20:36
bdxzeestrat: I have a bunch of these https://unixsurplus.com/collections/4u-servers-1/products/freenas-server-36-bay-supermicro-4u-2x-e5-2680-2-8ghz-192gb-2-port-10gbe-sfp-nic20:39
bdxfilled with 6TB drives and ssds20:39
zeestratCool. Most outlets don't ship to eu, nevermind Norway which is outside. Then you get hit with vat and customs20:39
bdxahh20:39
bdxwell shoots20:40
zeestratYeah, I remember you showed them last time around. How much a pop? How many disks?20:41
kwmonroezeestrat: sounds like you need a 3d printer20:41
kwmonroethen bdx can just send you a few detail pics and BAM, free switches20:41
bdxzeestrat: we can devise a plan to smuggle you cheap used hardware from the US20:42
bdxhave  you ever seen the movie "blow"?20:42
zeestratNow we're talking. Can I be Penelope Cruz?20:43
bdxahhhh sure ....20:43
bdxhaha:)20:43
zeestratForgot how it ends so I figured Johnny Depp bought it in the end20:44
zeestratkwmonroe: I'll take my chances printing my own money20:44
kwmonroethat... that's a terrible idea.20:45
zeestratThat's exactly what Photoshop said when I tried to scan some bills when I was younger20:46
zeestratbdx: got 40g nics in there as well?20:53
bdxsure do20:58
bdx40Gb, 10Ggb, 1Gb, these things are stacked20:59
* thumper blows out21:22
thumpersetting up a merge from 2.3 -> develop21:22
thumperquite a few conflicts there... wasn't expecting any21:22
rick_h_thumper: surprise!21:22
thumperhmm...21:23
thumperI think these are me reordering and regrouping imports properly21:23
thumperdumb me21:23
babbageclunkthumper: no good deed yada yada21:28
admcleod_thumper: is that anything to do with 2.3.8? :}23:13
thumperadmcleod_: no... it was due to a branch I had which was breaking apart packages23:14
admcleod_bugger23:14
thumperadmcleod_: all fixed now though23:25
admcleod_\o/23:25
thumperwell... just checked and no it isn't23:26
thumperI think I have another conflict23:26
* thumper enfixorates23:26
wpkthumper: o/23:26
thumperhey wpk23:28
thumperok... I've had enough of this bollocks23:30
thumperwhy does 'make pre-check' on my machine fail differently to the merge bot?23:31
thumperthey are both running the verify script23:31
thumperand both should be using the same go version23:31
thumperwallyworld, veebers ^^ ? anyone23:32
rick_h_wpk: what's up?!23:32
veebersthumper: OTP will look shortly23:33
wpkrick_h_: not much, not much. watching natgeo, drinking an APA.23:33
rick_h_wpk: sound like a fine evening.23:33
rick_h_I'm waiting eagerly for the blue planet ii stuff to drop23:34
* thumper just doesn't understand why his local verify is failing...23:43
veebersthumper: did you do a make add-patches (or haven't added patches)23:45
thumperveebers: hadn't added patches23:45
veebersthumper: or are your local deps been patched when they shouldn't have?23:45
thumpernope23:45
thumperveebers: I'm about to head to class, can I grab you after lunch?23:46
thumperperhaps you can shed light on this23:46
veebersthumper: sure thing23:46

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