[01:53] wallyworld: I put it on the calendar already, but just reminding you: I'm away this coming Fri-Mon [02:03] babbageclunk: do you have time to be a second reviewer for https://github.com/juju/juju/pull/7984? [02:04] axw: sure [02:04] axw: net line reduction automatic approval [02:05] :) [02:05] I mean, ok, I'll actually read it [03:33] wallyworld: what version of Juju was the relation bug found in? beta1? I'm failing to repro with wordpress-mysql. just noticed bug says mediawiki, will try that - can't see why it'd matter tho [03:39] https://github.com/juju/juju/pull/7987 fixes constraints handling with bundle deployment. [03:39] axw: i tested in tip of develop last week. found it by remove-relation from the offering model, and then add-relation again from the consuming model [03:40] axw: i'll try and repro [03:41] wallyworld: ah, I just removed it from the consuming model. I'll try that way [03:41] wallyworld: one of the migration test failures looked like a timeout, the other one looks like a genuine problem - we require a model type when importing a model, but older versions don't set one when exporting. [03:41] ah [03:41] i guess default to IAAS [03:41] creating a bug now [03:41] yu[ [03:41] p [03:41] babbageclunk: I don't think it's a timeout as the whole test is about 15 minutes, timeout is 50m. That being said I'm unsure why that log just stops [03:42] veebers: you're back! [03:42] veebers: hmm [03:42] wallyworld: hey o/ aye I am, first day back today :-) [03:42] wallyworld: seems to be some other bug though. I removed relation between hosted mysql and wordpress. something caused wordpress unit to restart, and now it's in an error loop complaining about the relation directory being non-empty [03:43] interesting. i wonder how mediawiki behaves [03:43] babbageclunk: let me have another quick look at those logs [03:45] thumper: lgtm [03:46] wallyworld: remove-relation on the offering side didn't trigger anything. I'll look into the other thing I found [03:46] righto, i'll try and reproduce [03:46] I mean didn't trigger any errors... re-adding relation just worked [03:46] axw: you tested with lxd? [03:47] wallyworld: yep [03:52] wallyworld: and now the remote relation worker seems to have stopped working: ERROR juju.worker exited "mysql": watching status for offer: verification failed: macaroon not found in storage [03:52] "juju offers" shows no relation, though the client thinks it has related [03:53] axw: hmmm, looks like macaroon has got cleaned up on offer side but we are not minting a new one [04:00] axw: i have a meeting now but will debug a bit when i am free [04:00] wallyworld: sure. I'll keep digging [04:01] wallyworld: there in a minute [04:02] so if I remove the remote app from the consuming side, and then re-relate, it fixes it up [04:05] axw: that would be because that causes a RegisterRelation which mints a new macaroon [04:08] wallyworld: the remote relation worker should be re-registering if the relation was removed. I suppose the worker isn't tidying up properly. will look more after lunch [04:09] axw: yeah, i'll look between meetings if i get time [04:30] axw: fixing this - https://bugs.launchpad.net/juju/+bug/1728486 [04:30] Bug #1728486: migration: allow model type to be blank when importing [04:30] axw: I should obviously make the reading side accept blank. [04:31] axw: Do you think I should make sure the exporting side also sets model type? [04:32] axw: I think the intention was that the 2.2 branch would export models as v3, so type would be set automatically. [04:32] babbageclunk: sorry just looking at the code, one minute [04:33] axw: but the description dependency was updated in 2.2 for some unrelated change and now it emits v4 models, but with blank type. [04:33] axw: thanks [04:35] babbageclunk: ugh, that's unfortunate. yes I think the exporter should be required to set the model type for v4 [04:35] babbageclunk: reading side shouldn't have needed to accept blank, but obviously will now [04:36] axw: do you think I should set the default in description or in migration_export.go? [04:38] babbageclunk: I think migration_export.go, I think description should be relatively dumb - not making choices, just checking the choices you made are correct [04:39] axw: cool, thanks! [05:13] later peeps [05:13] * thumper EODs [05:29] jam: did you forget to add a video call? [05:30] wallyworld: I did, but I just added "caas" [05:30] I will brt, need to use a restroom === frankban|afk is now known as frankban [07:36] wallyworld: the unitcount refcounting on relations for local vs. remote units looks dodgy. we never increment for remote units, but we sometimes decrement for remote [07:36] that might explain the "permission denied" error I saw [08:41] wallyworld: I think we should be maintaining relation.unitcount for both local and remote units. i.e. unitcount == number of units in scope, at all times [08:41] wallyworld: otherwise you can remove the relation without taking units out of scope [13:52] wallyworld: why does having a time-before caveat stop things from working? you should still get a verification error? [15:23] Bug #1726945 changed: [2.3] consume feature hard to manage

[15:29] Bug #1726945 opened: [2.3] consume feature hard to manage

[15:39] Bug #1726945 changed: [2.3] consume feature hard to manage

=== frankban is now known as frankban|afk [21:12] heyyyyy [21:12] I've got this model in a crazy stuck state on 2.3-beta2.5 [21:12] lol [21:12] 2.3-beta2.1 [21:13] whoops [21:13] ha [21:13] not sure if anyone wants to take a gander [21:13] http://paste.ubuntu.com/25854148/ [21:14] I can't get the units es-lxd-test/{0,1} to go away [21:14] I mean, its not really a big deal to me [21:14] but, possibly its a difficult situation to repro [21:15] and someone might want to hop in and take what they need [21:15] I'll leave it up for the next day or so [21:15] if anyone wants to take a closer look, ping me [21:15] I'll create a bug anyway [21:17] well, not that tough [21:19] https://bugs.launchpad.net/juju/+bug/1728721 [21:19] Bug #1728721: units stuck in model after failed deploy [21:29] bdx: do you have a build number for your juju 2.3-beta2.1? [21:33] hml: its whats in the beta snap channel [21:33] rev: 2640 [21:33] bdx: ty