mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrei Sekretenko <asekrete...@apache.org>
Subject Review Request 72955: Consolidated creation and validation of `allocator::Framework` options.
Date Mon, 12 Oct 2020 20:12:32 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72955/
-----------------------------------------------------------

Review request for mesos and Benjamin Mahler.


Bugs: MESOS-10176
    https://issues.apache.org/jira/browse/MESOS-10176


Repository: mesos


Description
-------

This merges three near-identical pieces of scattered code in SUBSCRIBE
and UPDATE_FRAMEWORK execution paths in the Master that validate
and construct parts of `allocator::FrameworkOptions` (the set of
suppressed roles and the offer constraints filter) into a single
function.

This is a prerequisite to adding validation of offer constraint roles.


Diffs
-----

  src/master/master.hpp 83d8190702ff25b5e3e9607d8250aa331aaad028 
  src/master/master.cpp d6d3ea7e27539a65b601783eca2ac4e3b688f188 


Diff: https://reviews.apache.org/r/72955/diff/1/


Testing
-------

make check


Thanks,

Andrei Sekretenko


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message