On Mon, Mar 7, 2016 at 4:27 PM, ramu <ramu.ramamur...@gmail.com> wrote:
> > >> Does this depend on your zone-ids fix? Do you have another rev of that >> coming? >> > > Russell, This test does not depend on the zone-id fix, because it replaces > zone-ids on flows > with a dummy string "xyz". It passes on the master as is. > > I will send the next revision of the zone-id fix soon, and as part of that > fix, this test > will also be updated to compare zone-ids on flows. > I see now. You even wrote: +# For now, zone-ids in flows are masked out because, they are not assigned +# consistently. This test is added to prepare for that code-change. What I would probably do is submit them together: the zone-ids fix + this test case that helps test it. If you take out your "xyz" change, will this test fail currently? -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
