timClicksis it possible for a controller to claim control of a pre-existing model? say the instance that is hosting the original controller is accidentally deleted00:39
rick_htimClicks:  no00:41
rick_htimClicks:  the only way is the very manual db restore/disaster recovery steps that are noted somewhere that jam put together00:41
timClicksunderstood, that's what I thought00:42
anastasiamacoh wallyworld PTAL https://github.com/juju/juju/pull/10784 - update-creds changes01:25
wallyworldanastasiamac: lgtm01:49
wallyworldhpidcock: i left a couple more questions02:04
hpidcockwallyworld: I think that's it. Thanks-you for all the reviewing02:42
wallyworldno worries, will look soon02:44
wallyworldbabbageclunk: for whenever https://github.com/juju/description/pull/6303:06
wallyworldhpidcock: let's land it. looks good, we can deal with any fallout when we test over the next few days etc03:11
babbageclunkwallyworld: approved04:14
wallyworldwill fix the import func as suggested04:14
wallyworldanastasiamac: that bug does look legit - destroy-model should cleanup everything before returning05:45
anastasiamacwallyworld: it's enclear whether the 2 given commands are run in parallel.. it did not look like there was a wait for command to come back05:48
anastasiamacwallyworld: anyway i need to talk to u when u can05:48
wallyworldhe says they run one after the other05:48
wallyworldand destroy-model blocks05:48
wallyworldhelping in #juju atm05:48
anastasiamacno he says that remove-cloud blocks05:49
anastasiamacwallyworld: ^^05:49
wallyworldremove-model blocks also05:50
wallyworldit counts down the stuff that is being removed05:50
anastasiamacwallyworld: let's talk - there is no 'remove-model' and he is not saying that destroy-model blocks but that the following 'remove-cloud' does not succeed.. m not sure u've read the bug05:51
wallyworldi meant destroy-model, i have read it, was a typo :-)06:04
wallyworldanastasiamac: can talk now until i get pinged again06:04
anastasiamacwallyworld: PTAL https://github.com/juju/juju/pull/10786 -add-cloud exclusivity and go1.13 'fix' as discussed06:42
wallyworldlgtm, ty06:45
anastasiamac\o/ u r a review ninja!06:47
nammn_deanyone for a really, really small one? https://github.com/juju/juju/pull/10782, additionally do we have other places to add them?08:29
manadartnammn_de: Does it need test changes? See https://discourse.jujucharms.com/t/adding-new-regions-to-clouds-yaml/174108:37
nammn_demanadart: ohhh, I keep forgetting to search in discourse, I will search for that :)08:38
nammn_demanadart: just looked at the test, no does not seem to be the case. Seems like we removed those tests from there. The linked e.g. PR adds test to a place which was removed in later commit.  Only need to update jenkins AFAICT08:58
manadartnammn_de: OK.09:02
nammn_demanadart: juju bot only failed for windows. Some flaky tests (?)09:08
nammn_denow that was confusing me for some time. https://github.com/juju/juju/pull/10787 2 link updates in the readme09:22
nammn_destickupkid: around?09:23
stickupkidnammn_de, always09:23
nammn_destickupkid: now thats the spirit :D!09:23
nammn_denammn_de: regarding adding support for windows and catalina. Should be done with the pr we added, right? Catalina was added before and windows by us both09:24
nammn_dejust to make sure I did not overlook something09:24
stickupkidnammn_de, correct09:24
nammn_destickupkid: cool09:24
stickupkidnammn_de, nope09:24
achilleasamanadart: I have been looking into the peergrouper code for the address equality code that you mentioned in 10750 but can't seem to find it. Can you help me track it down?10:29
manadartachilleasa: Looks like the old comparison was removed from the peergrouper and it just does reflect.DeepEqual. This is on SpaceAddresses anyway, so it's a different type to here.10:37
achilleasamanadart: I agree that the comparison code should proabbly live in core/network10:38
achilleasajam: I have pushed two new commits to 10750 which address some of your comments. Can you take a look?11:04
jamachilleasa: looking11:18
jamachilleasa: did you find anything for comparing lists of addresses?11:23
achilleasajam: not yet... the code in my PR is the same as the code used by the current implementation11:25
jamachilleasa: reviewd11:34
hmlachilleasa: review pls https://github.com/juju/juju/pull/1078312:19
achilleasahml: looking12:20
achilleasahml: got a few min for a quick ho?12:21
hmlachilleasa:  sure, meet you in daily12:21
nammn_deam i doing something wrong? How do I set my client to run on the same version as the one I am planning to deploy ? I am using build-agent and have compiled before. Still got the err msg. "agent binary 2.7-rc1 not compatible with bootstrap client 2.7-beta1"12:37
achilleasahml: approved12:40
hmlachilleasa: ty!12:40
nammn_denvm stupid me... I thought make would include go-install..12:41
hmlnammn_de: what QA steps are needed for this pr?13:46
cory_fujam: Hey, can I get confirmation from you that relation IDs are in fact unique integers and that the `{relation_name}:{id}` form is just for informational purposes on the CLI?  Will the API and CLI always accept a bare int?13:48
nammn_dehml: oh i forgot to add them. My bad, let me quick add them. but it's just deploying them into the new regions.14:00
nammn_dehml: updated descriptin14:01
achilleasahml: can you do a quick CR on https://github.com/juju/juju/pull/10788?14:01
hmlachilleasa:  sure, once i’m done with the current14:01
achilleasahml: no rush14:02
Fallenourgoooood morning14:02
stickupkidhml, CR'd 1078114:04
hmlstickupkid: ty!14:04
hmlnammn_de: i’m unable to bootstrap to one of the new regions14:15
hmlnammn_de: it should be showing up with juju regions <cloud>, but they are not.14:15
Fallenourhey guys, Im having issues with networking with LXD. I have NAT configured, and I have a network setup, with a interface configured as well. I can ping containers from the host machine, but I cant reach the same hosts with the same network from another node. Can anyone help me out? This doesnt make any sense.14:17
Fallenourpreferrably Id like to just let MaaS be the DHCP server, and let maas issue the lxd containers their IPs14:18
nammn_dehml: which region does not work? Both work for me. They should show if you dont have a local cloud.yaml file. Else they will not fallback to the yaml file provided by juju14:18
hmlnammn_de:  neither14:18
nammn_dehml: here updated description and expected output https://github.com/juju/juju/pull/10782. As long you have a clouds.yaml under /.local/share/juju , juju will not respect the fallbackoption and use the one defined there. Could that be the case?14:20
jamcory_fu: the CLI immediately strips out just the integer portion for the rest of the internal operations. I don't see that changing in the near future, at least.14:21
cory_fujam: Great, thanks.  That's what I thought.14:21
jam"unique" in that both sides of a relation see the same integer, but every relation is a different integer14:21
hmlnammn_de:  HO?14:21
nammn_dehml: sure comin in 1 min14:22
hmlnammn_de: approved14:58
nammn_dehml: thanks!14:58
nammn_dehml rick_h: related bug description https://bugs.launchpad.net/juju/+bug/1849509 I tried to make it a bit more elaborate15:13
mupBug #1849509: Juju cloud update and fallback precedence <juju:New> <https://launchpad.net/bugs/1849509>15:13
achilleasamanadart: the unit manifold allows my to fetch an environs.Environ instance. Is it somehow possible to get a NetworkingEnviron instead?15:37
achilleasaI want to access NetworkInterfaces like we do in networkingcommon and handle NotSupported for providers that don't expose it15:39
achilleasamanadart: nvm, I think I 've got it16:13
anastasiamacwallyworld: PTAL https://github.com/juju/juju/pull/10789 - last in series ;D23:38
wallyworldok, in a sec23:38
anastasiamacwallyworld: of course, none of this is 'look mmediately' :D23:38

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