<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;"><span><div><div><div><br></div><div>I guess another question is, why do you need to set ZeroMQ related flags</div><div>in fake_flags?  I think those should only be settings that apply for</div><div>*all* unit tests.  I would just register your flags in your unit tests.</div><div><br></div><div><a href="https://github.com/openstack/nova/blob/master/nova/tests/rpc/test_qpid.py#L69">https://github.com/openstack/nova/blob/master/nova/tests/rpc/test_qpid.py#L69</a></div><div></div></div></div></span></blockquote><div><span style="font-size: 12px;"><br></span></div><div><span style="font-size: 12px; ">The fake_rabbit flag doesn't, but this is otherwise a good point. </span> <span style="font-size: 12px; "> For now, I've just hard-coded the flag into the test.</span><div><span style="font-size: 12px;"><br></span></div></div><div><div><span style="font-size: 12px; ">The specific flag was to set the MatchMaker. I was forcing messages to use a "Localhost" matchmaker. </span><span style="font-size: 12px; ">I could override the flag from the test itself, but there might be reasons why someone would want to override this. In fact, I have overridden this to test the various matchmakers.  It is also more transparent.</span></div><div><span style="font-size: 12px; "><br></span></div><div><span style="font-size: 12px; ">To be honest, this specific requirement will be much lessened once I write independent tests for the matchmaker classes.</span></div><div><span style="font-size: 12px; "><br></span></div><div><span style="font-size: 12px; ">However, it would also be *nice* to be able to override various settings to test them without modifying the modules or tests themselves. I'd like if I could bump up/down the number of ZeroMQ IO threads to use during tests (it defaults to 1).   While modifying the test/module isn't much worse than modifying fake_flags, I find it cleaner.</span></div><div><span style="font-size: 12px;"><br></span></div><div>-- <br>Eric Windisch</div></div>