mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Mann <>
Subject Re: Review Request 43639: Allowed dynamic reservation without a principal.
Date Mon, 22 Feb 2016 18:39:20 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 22, 2016, 6:39 p.m.)

Review request for mesos, Michael Park and Neil Conway.


Added a new test case.

Bugs: MESOS-3940

Repository: mesos

Description (updated)

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. It may be desirable to remove this requirement;
this improvement 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 


Testing (updated)

A new test case was added to `ReservationTest.NoAuthentication`.

`make check` was used to test on OSX, both with and without SSL enabled.

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`

Inspecting `/master/state` before & after these operations confirmed that the reserve/unreserve
operations were successful.


Greg Mann

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