=== wgrant_ is now known as wgrant
=== wallyworld__ is now known as wallyworld
=== wallyworld__ is now known as wallyworld
=== CyberJacob|Away is now known as CyberJacob
=== CyberJacob is now known as CyberJacob|Away
[10:57] <rvba> allenap: I'm creating a small utility to send an event and create the event type if it doesn't exist.  How will that function (which will be called from the power-{on, off} tasks) get hold of the rpc_service object?
[10:57] <allenap> rvba: That’s a good question :)
[10:58] <allenap> rvba: I think we need to create a singleton to refer to all the services (or the application). I have some work towards that lying around...
[10:58] <rvba> allenap: yeah, I was thinking about a singleton too.
[10:59] <rvba> I'll make the utility take a rpc_service argument for now.
[10:59] <rvba> We can figure out how to pass it around later.
[10:59] <allenap> rvba: Yeah, that sounds good.
[10:59] <rvba> Cool.  Ta.
=== roadmr is now known as roadmr_afk
=== roadmr_afk is now known as roadmr
=== jfarschman is now known as MilesDenver
[13:58] <rvba> blake_r: reviewing your add-boot-type-to-node branch now.
[13:59] <blake_r> rvba: oh, cool. thanks
[14:37] <rvba> allenap: how can I easily unit test a method that uses other methods from the RPC region service?  Since I can't call call_responder directly… is there a better option or will I have to patch all the RPC methods?
[14:50] <allenap> rvba: See ClusterRPCFixture (er, which may not yet have landed). That wires up RPC calls to the cluster without needing sockets and whatnot.
[14:53] <rvba> allenap: right, not landed yet…
=== sebas538_ is now known as sebas5384
[15:58] <allenap> rvba: Got time for a very quick review before you finish for the weekend? https://code.launchpad.net/~allenap/maas/rpc-cluster-client-service-ipv4-only/+merge/227358
[15:59] <allenap> Just think, what a great way to finish a Friday.
[16:03] <blake_r> allenap: i can do it
[16:03] <rvba> allenap: nice!
[16:03] <allenap> blake_r: What a wise man you are :) Thanks.
[16:07] <rvba> allenap: thanks for the review!  Now, I need a review for https://code.launchpad.net/~rvb/maas/event-rpc-methods/+merge/227215 and https://code.launchpad.net/~rvb/maas/event-model/+merge/227156 and I'll be able to land this thing!
[16:08] <allenap> rvba: Looking at that now.
[16:08] <rvba> allenap: Thanks!
[17:08] <allenap> blake_r: Do you mind taking another look at https://code.launchpad.net/~allenap/maas/rpc-cluster-client-service-ipv4-only/+merge/227358?
[17:08] <blake_r> allenap: looking at it now
[17:08] <allenap> Ta.
[17:16] <blake_r> done
[17:22] <allenap> blake_r: Thanks. Have a tip top weekend.
[17:22] <blake_r> allenap: you to!
=== roadmr is now known as roadmr_afk
=== CyberJacob|Away is now known as CyberJacob
=== roadmr_afk is now known as roadmr
=== alexpilotti_ is now known as alexpilotti
=== CyberJacob is now known as CyberJacob|Away