[10:40] <nammn_de> achilleasa manadart is there a way to find out the fullname of the controllermodel?
[10:41] <manadart> nammn_de: What do you mean by full name? It is always "controller".
[10:42] <nammn_de> manadart: ohh, i meant with the fullname the owner name as well.
[10:43] <nammn_de> right now `upgrade-model` assumes the current account name for a upgrade. So does upgrade-controller. I want to find the controller owner to inject a correct model name
[10:47] <manadart> nammn_de: "juju show-model controller" has "name", which is the full name.
[10:49] <nammn_de> manadart: thanks!
[10:54] <achilleasa> manadart: is it possible to (temporarily) override the state clock inside a test?
[10:55] <achilleasa> I am trying to write a test using WaitAdvance but timers are being created by other parts of the test suite
[10:56] <manadart> achilleasa: Which suite?
[10:56] <achilleasa> MachineSuite; I am testing the changes to the model machine watcher that I am working on
[10:56] <achilleasa> (see comments in https://github.com/juju/juju/pull/10902)
[10:57] <achilleasa> I could make the watcher c-tor accept a clock (or make it take functional options) but that is ugly/overkill (for options)
[11:02] <manadart> Would it work to change StateSuite.SetUpTest to only instantiate the suite's Clock if nil, then you could derive from MachineSuite, and in that SetUpTest set the clock before calling the base SetUpTest?
[11:06] <achilleasa> for my use case, the watcher pulls the clock from st.clock(); so if the clock gets injected it will be used by other things (e.g. txnwatcher) which means that I get more waiters on the clock (2 or 3 more)
[11:06] <achilleasa> it wouldn't be a problem if the test was running in state instead of state_test...
[11:12] <manadart> achilleasa: Mock the clock?
[11:13] <manadart> achilleasa: I think it would be much nicer to do this everywhere and move away from the test clock.
[11:14] <achilleasa> but I am already using the test clock?
[11:14] <achilleasa> ah... as in gomock?
[11:14] <manadart> achilleasa: Yep.
[11:15] <achilleasa> not sure if it's worth the effort here.... maybe just add a method to give me the watcher with a clock.Clock in export_test?
[11:15] <achilleasa> (yuk!)
[11:22] <achilleasa> manadart: nah... I will just add a "SetClock" helper and try to patch it by checking the interface...
[11:27] <stickupkid> manadart, got a sec, quick question
[15:35] <hml> manadart: ready for review if you have a few: https://github.com/juju/juju/pull/10903
[15:36] <manadart> hml: Yep, taking a ride home this minute, but I will review once home.
[15:37] <hml> manadart:  rgr
[15:59] <stickupkid> anyone up for a CR on a backport? https://github.com/juju/juju/pull/10906
[16:01] <achilleasa> stickupkid: looking
[21:52] <hml> anastasiamac: review pls https://github.com/juju/juju/pull/10909
[21:53]  * anastasiamac looking
=== narindergupta is now known as narinderguptamac