-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43639/
-----------------------------------------------------------
(Updated Feb. 20, 2016, 1:53 a.m.)
Review request for mesos, Michael Park and Neil Conway.
Changes
-------
Added backticks.
Bugs: MESOS-3940
https://issues.apache.org/jira/browse/MESOS-3940
Repository: mesos
Description
-------
Allowed dynamic reservation without a principal.
The `ReservationInfo.principal` field has been migrated to `optional`, which
means we can now allow dynamic reservation and unreservation without a
principal. This allows the use of the `/reserve` and `/unreserve` HTTP
endpoints when HTTP authentication is disabled.
Note that we still require that frameworks/operators set the
`ReservationInfo.principal` field to match their own principal, if present.
This isn't necessary, and removing that requirement is tracked in MESOS-4696.
Diffs (updated)
-----
src/master/validation.cpp 66898e914c7b4ab83c4580be67530f355cfb05ca
src/tests/master_validation_tests.cpp
6fae01fa1833ae05ec82618a4ae28ac5bd275bd5
src/tests/reservation_endpoints_tests.cpp
afe81b1d38a1b3a82583720f26482ddcde8f5e85
src/tests/reservation_tests.cpp d2ef15934556cb879f31850d52712aec77231fc7
Diff: https://reviews.apache.org/r/43639/diff/
Testing
-------
`make check`
Also manually reserved/unreserved resources using curl, with a command like
this: `curl -i -d slaveId="8288b2f0-e33d-4547-a2b4-5230ba6e5279-S0" -d
resources='[ { "name": "cpus", "type": "SCALAR", "scalar": { "value": 3 },
"role": "ads", "reservation": { } } ]' -X POST
http://127.0.0.1:5050/master/reserve`
Inspecting `/master/state` before & after these operations confirmed that the
reserve/unreserve operations were successful.
Thanks,
Greg Mann
