wallyworldhpidcock: lgtm, ta00:08
wallyworldthumper: do you know the status of the open PRs for branches etc - are they due to land prior to rc1 being built?00:09
thumperwallyworld: rick_h said that they had landed what needed to be landed and had a hash to use00:10
thumperso I'd say that nothing should land that isn't a bug fix00:10
thumperuntil we are past RC100:10
wallyworldyup, no worries, just checking00:11
wallyworldwe're just landing the race fix and that's it from us00:11
wallyworldtimClicks: fyi i updated k8s release info in https://discourse.jujucharms.com/t/new-features-and-changes-in-juju-2-7/226802:21
wallyworldanastasia will need to do some cloud updates02:21
wallyworldas the examples are now out of date02:22
timClickswallyworld: awesome, thanks for adding details there02:22
wallyworldi also will add extra info the the referenced pod spec v2 post02:22
wallyworldas i need to add custom resources02:22
wallyworldthere's a lot in this release02:23
timClicksI'll get the cloud stuff updated tomorrow02:23
wallyworldi also made a slight correction tojuju-run02:23
wallyworldk8s doesn't support juju ssh02:23
wallyworldyou need to kubectl exec in directly02:23
timClicksah okay, ty02:23
wallyworldnot that hard to add but ran out of time :-(02:24
wallyworldyou would think as a user it would be there02:24
wallyworldbut not yet02:24
wallyworldthere is a valid workaround k8s folks know how to use02:24
* timClicks nods02:25
timClickswe could take out the help output and change it to something else..02:25
timClickswe could take out the help output and change the example to something else..02:25
hpidcockshould we also update documentation re action id not being uuids? https://jaas.ai/docs/working-with-actions02:26
timClickshpidcock: thanks for volunteering - scroll to the bottom of the page, click the link02:27
hpidcockhah should keep my mouth shut02:28
timClickssomewhat more seriously, if you would like to add a comment on that page i'll make sure that gets done02:28
timClicksI'm trying to block out some time this arvo for some other content02:29
timClicksi realise that it feels a little bit scary to directly edit the docs yourself02:29
hpidcockoh I'm happy to edit it02:29
hpidcockjust a shame we don't version the documents by release.02:30
timClicksi think there's history there02:31
thumperhpidcock: we used to02:31
thumperhpidcock: but not any more02:31
hpidcockwallyworld: did we want to remove the developer-model feature flag requirement for k8s actions and exec?02:47
wallyworldhpidcock: ah bollocks, i didn't realise we still had that turned on02:49
wallyworldyeah, let's get up a pr02:50
hpidcockI'll make a PR?02:50
wallyworldthumper: i have one to add for 2.7.1 https://github.com/juju/juju/pull/1084802:58
wallyworldthumper: also, we need to land this for rc1 as a feature flag was left in https://github.com/juju/juju/pull/1084902:59
wallyworldmake you wonder how much non dev testing we got03:00
thumperwallyworld: ok03:00
thumperland it and update the email03:00
wallyworldi guess it's just for juju exec and most people would have tried juju call03:00
hpidcockwallyworld: https://github.com/juju/juju/pull/10849 I think it was just exec/run that was checking for the ff.03:00
wallyworldit was03:00
hpidcockso I'm ok to land?03:01
wallyworldhpidcock: yeah, i just hit $$merge$$03:01
wallyworldgood catch03:01
hpidcockdouble $$merge$$03:02
wallyworldit won't mind03:02
hpidcockI guess we are testing jenkins then ;)03:02
wallyworldjam: one thing - application relation data isn't propagated between models for cmr i don't think, so that will need to be a known linitation for 2.7.005:06
wallyworldthere's params structs that need updating, plus apis etc05:06
jamwallyworld: sure. we can target that as a 2.7.1 if you like.05:07
wallyworldi think we should, same with remaining model migration work etc05:08
hpidcockwallyworld: how do I upgrade an IAAS controller using a local jujud?05:26
hpidcockdo I just pass --build-agent?05:26
wallyworldhpidcock: yup05:38
hpidcockwallyworld: I think I hit the bug kelvinli_ hit where the caas model is upgraded but the workloads aren't redeployed. I think the caasunitprovisioner's application worker doesn't fire when the model version changes, not sure what the solution to this would be. Possibly the caasunitprovisioner worker should pass a chan into the application worker so it can trigger on model version change?06:02
hpidcockI'll see if I can smash out a PR to fix it.06:04
wallyworldhpidcock: quick chat about it?06:05
kelvinli_yeah, we need to change the init container using the updated jujud version06:28
kelvinli_not sure if the init container will be re-run if the image path changed06:29
stickupkidmanadart, https://github.com/juju/juju/pull/10844 Fancy giving this a review?09:56
manadartstickupkid: Sure.09:56
manadartstickupkid: Couple of UX things.10:37
stickupkidmanadart, good shout on -n 010:38
nammn_destickupkid: wanna take a look at branch completion? Removed the Option flags and added `--branch < tab > --> branch  https://github.com/juju/juju/pull/1084510:46
stickupkidnammn_de, sure10:47
nammn_destickupkid: and this one? https://github.com/juju/juju/pull/10836 its about the charm dep we recently changed. With this some tests are failing because how we place and version those charm tests. Not important for now but might wanna take a look and discuss10:48
nammn_demore information and this issue with suggestions how to solve. But not that important for now10:48
nammn_destickupkid: ahhh  I need some feedback on this comment https://github.com/juju/juju/pull/10836#issuecomment-54841425610:51
nammn_dei reformated the comment, makes it more readable10:51
stickupkidah ok10:51
nammn_destickupkid: if read. Does it make sense to you?10:53
stickupkidnammn_de, thinking10:53
stickupkidi don't like the first or the last10:53
stickupkidthe test sucks10:54
stickupkidthat's what the test does10:54
nammn_destickupkid: haha10:54
nammn_deand additionally its difficult because that they are under git and using version regarding the charm thing. Makes all more complex as the tests in general already are :D10:54
stickupkidnammn_de, ho?10:55
stickupkidmanadart, got a sec as well?10:58
manadartstickupkid: Yeah.10:58
nammn_destickupkid: sure sorry wasnt here11:06
stickupkidmanadart, updated my PR12:06
manadartstickupkid: OK, gimme 2.12:12
nammn_destickupkid rick_h: coming back to the branch completion cli thingi https://github.com/juju/juju/pull/1084515:35
nammn_deNot even sure if we want those flags or not, just opened pr in case to have it15:35
stickupkidnammn_de, kewl15:36
rick_hnammn_de:  my feedback would be to make it looks like the rest of tab completion tbh15:41
nammn_derick_h: IMO I think that its would fit that way more with the rest of the tabl completion. E.g. juju switch <tab> has no flags.  --controller, --unit ect. as well not.  BUT juju expose and other similiar commands have both. The "flags/option" and the values15:44
hmlreview anyone?  quick pr to fix nw-export-bundle test: https://github.com/juju/juju/pull/1085216:32
hmlci test16:32
hmlstickupkid: ta!17:46
nammn_deam I looking at things wrong? This fails for me for now reason. https://github.com/juju/utils/blob/master/fs/copy.go#L27  Why do we expect there to be an error before we continue?17:47
nammn_deahh I see it creates the folder and does not want to exist, ufff17:49
rick_hguild 2.7 branch is created and pushed17:54
rick_hhave fun!17:54
rick_hguild to be clear, that means develop is "unlocked". I'll send an email17:55
stickupkidwe should retarget all our stuff to 2.7 right and merge forward to develop17:55
stickupkidrick_h, ^17:55
rick_hstickupkid:  hmmm...actually...so I was hoping to not touch the rc1 space in case we need a fix/release17:55
stickupkidrick_h, but at some point we'd need to back port?17:56
rick_hstickupkid:  yea, so thinking if the idea for now would be land on develop and keep track of what we need to backport17:56
rick_hafter we cut .017:56
stickupkidrick_h, sweet17:56
rick_hI'll put that in the email17:56
babbageclunkanyone knowledgeable about shell quoting want to help me with a shell quoting problem?23:45
babbageclunkmy heuristic of changing quotes and adding/removing backslashes has reached its limit23:46
hpidcockbabbageclunk: is this for templating a script or are you trying to process arguments passed in?23:54
wallyworldneed a +1 on version bump pr for 2.7 branch https://github.com/juju/juju/pull/1085323:54
hpidcockwallkworld done23:55
babbageclunkhpidcock: I'm trying to use the wait_for function in a shell ci test23:55
hpidcockuse heredoc?23:56
wallyworldand for develop https://github.com/juju/juju/pull/1085423:57
babbageclunkthe weird bit is that it works if I ask for the current workload status (in jq) but not if I ask for the message.23:57
babbageclunkdoing my head in23:58
hpidcockcan you paste it?23:58
babbageclunkhpidcock: jump in standup so I can talk you through it?23:58

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