----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11975/#review36710 -----------------------------------------------------------
Is this patch still relevant or can it be dropped? - Niklas Nielsen On Oct. 22, 2013, 12:03 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11975/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2013, 12:03 p.m.) > > > Review request for mesos. > > > Repository: mesos > > > Description > ------- > > Use hostname instead of IP for redirection. > > In the Web UI, the redirection does not work for hosts where the local > interface address is not publicly accessible. For example, with EC2 the > redirection will not work. > > This change writes the Master's hostname into ZooKeeper, thereby > allowing the redirection to work with private IP addresses. > > Review: https://reviews.apache.org/r/11975/ > > > Diffs > ----- > > src/detector/detector.hpp b0e66888050c1987b7200cdbf21ebe5e2e55e6c0 > src/detector/detector.cpp 12deefa0b9df3f4946d80f500caaa5199b8ea28e > src/local/local.cpp e4b5ec5b3dfae6dd89746353d3caaa8d7e117840 > src/master/http.cpp ca66d6782023d303c8acc89ef9cf421cfca8d0dc > src/master/main.cpp 19fcb9f09d8fc03a0719aada7d216b575eb3069b > src/master/master.hpp 8ec0c17c71b7b4679d4f712d0fb742d420c9152d > src/master/master.cpp b0a2757af3ec83ead53374504fe24d3a8f7673ad > src/messages/messages.proto 19d4b387b50884f9f4a70efb3e9b739f846abf57 > src/sched/sched.cpp 5718fe9280d7e1124ffef2de9e4d47b347f0600a > src/slave/main.cpp 750a12766bde64059bfd4635ea077cbd43cb4301 > src/tests/cluster.hpp f743bb3251af81fb9d8afd51de4df6efcf289bb9 > src/tests/master_detector_tests.cpp > 2d140ba1a364a7af4d643951d6016ac17dd10526 > src/webui/master/static/js/controllers.js > c553e358e9f7598033e55a2e8dfff167d8282f7b > > Diff: https://reviews.apache.org/r/11975/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Brenden Matthews > >
