mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashwin Murthy <ashwinmur...@gmail.com>
Subject Re: Review Request 45000: MESOS-3902: Fix in location header during redirect from non-leader.
Date Sun, 20 Mar 2016 21:57:23 GMT

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

(Updated March 20, 2016, 9:57 p.m.)


Review request for mesos and Vinod Kone.


Repository: mesos


Description (updated)
-------

MESOS-3902: Fix in location header during redirect from non-leader. Appended the path from
original request to the location header


Diffs (updated)
-----

  src/master/http.cpp bfea8fa811c24d37b2d7a8109728e1f40217e02f 
  src/slave/containerizer/mesos/containerizer.cpp ee7a265975323ca891114a286357c8e42901560c

  src/tests/containerizer/mesos_containerizer_tests.cpp f3ca32b2d8b8ef9dcfa8f20d9ceaff48b6598a66


Diff: https://reviews.apache.org/r/45000/diff/


Testing
-------

Was trying to write unit test in scheduler http api test but it turns out multi master tests
cannot be written at this point. Need to manually verify this by setting up multiple masters
with ZK.


Thanks,

Ashwin Murthy


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