> On May 21, 2019, 12:07 p.m., Benjamin Bannier wrote: > > src/tests/master_authorization_tests.cpp > > Lines 3317-3320 (patched) > > <https://reviews.apache.org/r/70686/diff/1/?file=2145964#file2145964line3317> > > > > If you get rid of the `vector` `evolve` function in r/70683/ we'd > > create a temporary `RepeatedPtrField` and convert when creating the accept > > call. > > > > ```` > > RepeatedPtrField<v1::Offer::Operation> operations_ = > > evolve<v1::Offer::Operation, Offer::Operation>( > > {operations.begin(), operations.end()}); > > > > mesos.send(v1::createCallAccept( > > subscribed->framework_id(), > > offers->offers(0), > > {operations_.begin(), operations_.end()})); > > ````
Instead of doing this, the operations are now created with v1 proto messages, since operation feeback is only supported in v1. As a result, we no longer need the vector evolve helper. - Chun-Hung ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70686/#review215405 ----------------------------------------------------------- On May 20, 2019, 10:37 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70686/ > ----------------------------------------------------------- > > (Updated May 20, 2019, 10:37 p.m.) > > > Review request for mesos, Benjamin Bannier, Gastón Kleiman, and Greg Mann. > > > Bugs: MESOS-9485 > https://issues.apache.org/jira/browse/MESOS-9485 > > > Repository: mesos > > > Description > ------- > > This test verifies that allowing or denying an action will only result > in a success or failure on specific operations but not other operations > in an accept call. This is a regression test for MESOS-9474 and > MESOS-9480. > > > Diffs > ----- > > src/slave/slave.hpp c740bf7c4af1699dd71a31f9169d29c1b4adb40d > src/tests/master_authorization_tests.cpp > f65b6210a8189ec5e9089bce845a46d325253058 > src/tests/mesos.hpp e70e0e1bbebef41557f9bb1f26c379357fc74e2a > src/tests/mock_slave.hpp 326a450eb2ae9912a19c6e0220e80f74d2953add > src/tests/mock_slave.cpp 1122c2a8b56b9d4e5deea4e892d81061f5e39a8d > > > Diff: https://reviews.apache.org/r/70686/diff/1/ > > > Testing > ------- > > make check > > Ran the unit test under stress for 500 iterations. > > > Thanks, > > Chun-Hung Hsiao > >
