> On Dec. 21, 2018, 4:34 a.m., Greg Mann wrote: > > src/tests/api_tests.cpp > > Lines 4885 (patched) > > <https://reviews.apache.org/r/69575/diff/3/?file=2115287#file2115287line4885> > > > > s/operationId.value()/operationId/
Dropping this, as would result in a compilation error: ``` In file included from ../../src/tests/containerizer.hpp:51: ../../src/tests/mesos.hpp:2027:10: error: no matching function for call to 'RESERVE' return common::RESERVE<mesos::v1::Resources, mesos::v1::Offer>( ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ../../src/tests/api_tests.cpp:4885:16: note: in instantiation of function template specialization 'mesos::internal::tests::v1::RESERVE<mesos::v1::Resource &, mesos::v1::OperationID &>' requested here {v1::RESERVE(reserved, /*operationId.value()*/operationId)})); ^ ../../src/tests/mesos.hpp:1302:35: note: candidate function template not viable: no known conversion from 'mesos::v1::OperationID' to 'const Option<std::string>' (aka 'const Option<basic_string<char> >') for 2nd argument inline typename TOffer::Operation RESERVE( ^ 1 error generated. ``` - Benno ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69575/#review211485 ----------------------------------------------------------- On Dec. 20, 2018, 12:19 p.m., Benno Evers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69575/ > ----------------------------------------------------------- > > (Updated Dec. 20, 2018, 12:19 p.m.) > > > Review request for mesos, Gastón Kleiman, Greg Mann, and Joseph Wu. > > > Bugs: MESOS-8782 > https://issues.apache.org/jira/browse/MESOS-8782 > > > Repository: mesos > > > Description > ------- > > When an agent is marked as gone through the operator api, > notify all frameworks with outstanding offer operations on > that agent that these operations have been transitioned > to `OPERATION_GONE_BY_OPERATOR`. > > > Diffs > ----- > > src/master/master.cpp b4b02d8b4d7d6d1aabda1f97b9bf824419f76a9e > src/tests/api_tests.cpp fdd9f871f75617fc26a28679e2a1e41f506c6133 > > > Diff: https://reviews.apache.org/r/69575/diff/3/ > > > Testing > ------- > > > Thanks, > > Benno Evers > >