babbageclunkright - that confirms that it's code running inside the container that can't get to the lxd port.00:00
babbageclunkdmitri-s: ^00:00
dmitri-sSo my LXD config is incorrect?00:01
babbageclunkI think so00:01
dmitri-sJust tried curl from inside a container and it times out (curl: (7) Failed to connect to port 8443: Connection timed out)00:02
kelvinliuwallyworld: got this PR to add pod addresses to binding addresses for CaaS, could u take a look? thanks! https://github.com/juju/juju/pull/1074100:02
wallyworldi can, just need to get some stuff done first00:03
kelvinliunp, not urgent at all00:04
babbageclunkdmitri-s: sorry, just trying to work out what config needs to change...00:05
dmitri-sHere is my current lxd config (lxd init --dump) https://github.com/dmitri-s/juju-learn/blob/master/lxd_config00:06
babbageclunkoh thanks00:07
babbageclunkcomparing that to mine...00:07
babbageclunkdmitri-s: the only difference I can see is that I have `core.proxy_ignore_hosts:,::1,localhost` in the config section.00:11
babbageclunkwhat does `route -n` show inside the container?00:11
dmitri-sroot@pleasant-bonefish:~# route -n00:12
dmitri-sKernel IP routing table00:12
dmitri-sDestination     Gateway         Genmask         Flags Metric Ref    Use Iface00:12
dmitri-s0.0.0.0         UG    100    0        0 eth000:12
dmitri-s10.208.133.0   U     0      0        0 eth000:12
dmitri-s10.208.133.1 UH    100    0        0 eth000:12
babbageclunkand obviously you can get to the internet from the container since we can see things installing in the juju log...00:13
babbageclunk(the route output looks the same as mine)00:14
babbageclunkSorry, this is about the point I'd ask in #lxcontainers00:16
dmitri-sthanks. I will do that. BTY, was I supposed to post this in discourse.jujucharms.com rather than ask here?00:17
dmitri-sjust so I know next time.00:17
dmitri-sJust from the server message "https://jujucharms.com, general chat on https://discourse.jujucharms.com"00:17
dmitri-sseems like it is where it point me00:18
babbageclunkNo, this is a good place too, especially for back-and-forth. Although the advantage of discourse is that it's persistent - if people aren't in the same timezone then they won't see irc chat but they can still answer posts.00:19
dmitri-scool. thanks for your help.00:20
babbageclunkno worries - sorry I couldn't sort it out fully00:21
babbageclunk(or at all)00:21
dmitri-shaha... movement in any direction is progress as I see it. I figured out that "x.x.133.1" is a gateway, so it is a progress.00:25
dmitri-sWith your help, nothing less. :)00:26
wallyworldkelvinliu: free now in standup00:56
wallyworldkelvinliu: lgtm with some suggested cleanup to make things more logical now that the behaviour has changed02:46
kelvinliuwallyworld: yeo thx02:47
anastasiamacwallyworld: for post-lunch digestion, PTAL https://github.com/juju/juju/pull/10746 - 'show-cloud' ask-or-tell03:11
wallyworldjust getting coffee03:12
anastasiamacoh good! o the timing is right for light reading :D03:12
wallyworldanastasiamac: i asked a question, see if i am being dumb03:33
wallyworldbabbageclunk: i pushed a change to address the race, see what you think?03:33
anastasiamacwallyworld: kind of ;) i'll reply on pr03:34
wallyworldanastasiamac: s/kind of/most definitely. lgtm03:42
anastasiamacwallyworld: :) it was existing code anyway that I have just prettied up :)03:43
wallyworldyeah, i didn't see that from the review diff, just mainly saw the comment etc03:43
anastasiamacwallyworld: thank you for the review tho \o/ really appreciate another set of eyes03:44
wallyworldmore eyes always good03:44
babbageclunkwallyworld: approved03:52
wallyworld\o/ tyvm03:52
wallyworldbabbageclunk: call?04:02
thumperbabbageclunk: another https://github.com/juju/juju/pull/1074704:47
thumperor wallyworld ^^04:47
wallyworldoooh instancepoller, my favourite04:47
* thumper EODs04:50
anastasiamacwallyworld: PTAL https://github.com/juju/juju/pull/10748 - 'show-credential' part05:42
stickupkidnammn_de, good catch last night, yes if you hold it differently it breaks, I fixed it so both ways works https://github.com/juju/juju/pull/1074908:56
nammn_destickupkid: cool, approved :D09:11
nammn_derick_h I just scrolled through the systemd code and found out that we do a verify an a basic level. So I don't think we need to add additional checks. So IMO the pr is ready to be reviewed https://github.com/juju/juju/pull/1069609:26
manadartwallyworld: Is there are a resolution for all apps in a CAAS model getting "already exists" errors for generating secrets?09:29
nammn_demanadart: regarding the bug you mentioned before with the charm version. I did add some additional test, which does not include the bug yet, but still include some other cases. https://github.com/juju/juju/pull/10744 , when we "fix the bug" we could add the other cases as well.09:29
nammn_deAdditionally I updated the bug description with my debugging output https://bugs.launchpad.net/juju/+bug/1848149 . I planned to take this on later09:29
mupBug #1848149: Charm Deployed or Upgrade from Dir Ignores "version" <juju:Triaged> <https://launchpad.net/bugs/1848149>09:29
manadartnammn_de: Ack.09:30
manadartwallyworld: for example: "creating secrets for container: tensorboard: secret "tensorboard-tensorboard-secret" already exists"09:30
nammn_deachilleasa stickupkid : im trying the qa steps from hml https://github.com/juju/juju/pull/10718 there are some things I don't know. Got some time and want to have fun walking me through a little with a little qa? Doesn't have to be now09:37
achilleasajam: shouldn't this (https://github.com/juju/juju/blob/develop/worker/instancepoller/updater.go#L140-L145) check use the instance status (which can be running) instead of the machine status (AFAICT would be created)? Or are statuses working differently for manual machines?09:49
nammn_demanadart: if you have some free time https://github.com/juju/juju/pull/10691 pr regarding spaces. Sadly I do have some weird network issues and my internal upgrade bootstraps got stuck (qa steps)10:22
manadartnammn_de: Reviewed. Upgrade step needs another look.10:47
nammn_demanadart: Just read it, good to know for me! Thanks11:48
achilleasanammn_de: for 10691 you probably also need to patch this: https://github.com/juju/juju/blob/develop/cmd/juju/application/upgradecharm.go#L75712:39
nammn_demanadart: added your suggestions, qa steps worked for me!12:40
nammn_deachilleasa, manadart :" ohh, nvm me than I ll take a look12:41
nammn_deachilleasa: are we fine to use the default if none is set, which should be set in any case with my patch?12:44
stickupkidanyone know where the simple streams url is in the source code? I know we have a model config value, but it's empty, so we must fill it in with something if it's empty.12:46
achilleasanammn_de: I 'd say leave it for now to be safe in case someone is testing the new client against a controller from a sha before your PR landed12:47
nammn_deachilleasa: not patching this function at all or you mean adding a check if it exists, use it?12:48
achilleasanammn_de: if the model (it's passed an an arg to that function) defines it use that or use the network.DefaultSpaceName as a fallback12:49
nammn_deachilleasa: thanks. Similiar to that? https://github.com/juju/juju/blob/ef08716950173d3a388d675b5c49c1210137c050/cmd/juju/application/upgradecharm.go#L761-L76012:51
jamachilleasa: IIRC, MachineStatus is the status of the machine agent, vs the status of the machine itself12:53
achilleasajam: that's my understanding as well. So can this be ever running?12:54
achilleasa(in running state)12:54
jamachilleasa: I'm really not sure what it is trying to check there. It may just be trying to say "set it to running if it isn't running"12:54
achilleasayes, but I would assume that the check would be more like "if my *instance* status is not running and I am a manual machine then set me as running"12:55
achilleasastickupkid: can you please check if this works on develop for you? go test -check.v -check.f TestAgentSetsToolsVersion (in cmd/jujud/agent)15:09
stickupkidachilleasa, why is that?15:10
achilleasastickupkid: https://pastebin.canonical.com/p/pf5wXMFmtD/15:11
stickupkidwtf is that15:11
achilleasaI will retry on current HEAD15:12
stickupkidachilleasa, https://pastebin.canonical.com/p/J86Vg3z53B/15:12
achilleasathis fails on HEAD for me: "FAIL: machine_test.go:607: MachineLegacyLeasesSuite.TestAgentSetsToolsVersionManageModel"15:13
achilleasacould it be deps? brb15:14
achilleasanope... still failing15:15
nammn_demanadart: time for a quick review? I added your and achilleasa suggestions https://github.com/juju/juju/pull/10691/files15:16
achilleasastickupkid: wonder if that test is actually pulling stuff from somewhere...15:16
stickupkidachilleasa, we should have a look at what the test does15:16
achilleasastickupkid: L59815:18
stickupkidachilleasa, ho?15:20
manadartnammn_de: Needs another change. I have to run.15:25
achilleasahml: are you on eoan?15:30
hmlachilleasa:  not yet15:30
stickupkidhml, go test -check.v -check.f TestAgentSetsToolsVersion15:30
stickupkidhml, ho in daily15:31
hmlstickupkid: omw15:31
achilleasarick_h: are you on eoan?15:34
achilleasaalso nammn_de are you on eoan?15:36
nammn_deachilleasa: yesss15:36
nammn_demanadart: thanks, damnit15:36
achilleasacan you join daily please?15:37
achilleasanammn_de: ^15:38
achilleasanammn_de: actually can you please run go test -check.v -check.f TestAgentSetsToolsVersion (in cmd/jujud/agent) and let me know if it fails for you?15:40
rick_hachilleasa:  yes, I'm on eoan15:40
achilleasarick_h: can you try the above command ^^15:41
rick_hachilleasa:  the test command?15:41
achilleasayeap. I think it fails on eoan15:41
rick_hachilleasa:   I'll see. I need to setup a dev environment on here. I've not done that yet.15:46
rick_hachilleasa:  heads up in case you didn't see it https://github.com/juju/juju/pull/10747#event-272007028215:46
rick_hachilleasa:  might be worth pulling and making sure that there's no issues in overlap/etc15:46
achilleasarick_h: thanks, already incorporated that in my PR15:47
rick_hachilleasa:  awesome15:47
nammn_deachilleasa: sorry wasnt there, still need me?15:58
nammn_deachilleasa: ah i see not needed anymore15:59
nammn_demanadart: reverted the change16:05
manadartachilleasa: Can you approve that^ one?16:06
achilleasamanadart: one sec16:07
manadartGot hands full with kids here. Not at PC.16:07
achilleasanammn_de: approved16:14
stickupkidachilleasa, any news?16:30
manadartNeed a review for https://github.com/juju/juju/pull/10751.16:33
achilleasamanadart: one for tomorrow: https://github.com/juju/juju/pull/1075016:56
nammn_dethanks manadart achilleasa16:56
mbeierlHaving some trouble getting the apt layer to install a ppa.  It seems to be ignoring whatever I put in.  Maybe a yaml level problem?  Are there any working examples that show a full layer.yaml with adding a ppa?17:15
mbeierlstub: where is the best place to go for more info on the apt-layer?  I am obviously missing something.17:34
rick_hmorning thumper19:58
rick_hthumper:  if you get time can you peek back at nam's branch for the log permissions please?21:21
thumperrick_h: will do21:21
stubmbeierl: https://github.com/stub42/layer-apt has the README rendered21:57
babbageclunkcan I get a review for this merge? https://github.com/juju/juju/pull/1075222:21
hpidcockbabbageclunk: on it22:25
babbageclunkmerci m'sieur22:28

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