/srv/irclogs.ubuntu.com/2014/07/18/#maas.txt

=== 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
rvbaallenap: 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
allenaprvba: That’s a good question :)10:57
allenaprvba: 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
rvbaallenap: yeah, I was thinking about a singleton too.10:58
rvbaI'll make the utility take a rpc_service argument for now.10:59
rvbaWe can figure out how to pass it around later.10:59
allenaprvba: Yeah, that sounds good.10:59
rvbaCool.  Ta.10:59
=== roadmr is now known as roadmr_afk
=== roadmr_afk is now known as roadmr
=== jfarschman is now known as MilesDenver
rvbablake_r: reviewing your add-boot-type-to-node branch now.13:58
blake_rrvba: oh, cool. thanks13:59
rvbaallenap: 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:37
allenaprvba: See ClusterRPCFixture (er, which may not yet have landed). That wires up RPC calls to the cluster without needing sockets and whatnot.14:50
rvbaallenap: right, not landed yet…14:53
=== sebas538_ is now known as sebas5384
allenaprvba: 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/22735815:58
allenapJust think, what a great way to finish a Friday.15:59
blake_rallenap: i can do it16:03
rvbaallenap: nice!16:03
allenapblake_r: What a wise man you are :) Thanks.16:03
rvbaallenap: 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:07
allenaprvba: Looking at that now.16:08
rvbaallenap: Thanks!16:08
allenapblake_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_rallenap: looking at it now17:08
allenapTa.17:08
blake_rdone17:16
allenapblake_r: Thanks. Have a tip top weekend.17:22
blake_rallenap: you to!17:22
=== 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

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