<div><span style="font-size: 12px; ">Russell,</span></div><div><span style="font-size: 12px; "><br></span></div><div><span style="font-size: 12px; ">FYI, with the flags patch, it is no longer possible to set rpc-implementation dependent flags in fake_flags.</span></div><div><span style="font-size: 12px;"><br></span></div><div><span style="font-size: 12px;">Even Rabbit has a flag in there (fake_rabbit), but Rabbit flags are currently global, so it works. That won't be true for long… We're going to have to fix this.</span></div><div><span style="font-size: 12px;"><br></span></div><div><span style="font-size: 12px;">One option is to initialize the RPC layer from fake_flags.py and then set the options.</span></div><div><span style="font-size: 12px;"><br></span></div><div><span style="font-size: 12px;">Another option, for now, might just be to push this problem into the implementations and have a global "testing" flag that is implementation/backend independent. This is uglier on the implementation/driver side, but cleaner on the unit tests. This is basically what 'fake_rabbit' is now, anyway.</span></div><div><span style="font-size: 12px;"><br></span></div><div><span style="font-size: 12px;">Thoughts?</span></div>
<div><div><br></div>-- <br>Eric Windisch<div><br></div></div>