aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zameer Manji <>
Subject Re: Review Request 57708: Populate `host` and `webURL`fields of FrameworkInfo.
Date Fri, 17 Mar 2017 22:42:42 GMT

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

(Updated March 17, 2017, 3:42 p.m.)

Review request for Aurora, Mehrdad Nurolahzade and Stephan Erb.



Bugs: AURORA-1905

Repository: aurora


This patch extracts out `FrameworkInfo` construction from the `DriverSettings` data class
to a factory class. This factory class combines the base info constructed via CLI arguments
with the HTTP server's host and port information. This allows us to populate the `host` and
`weburl` fields of framework info, which enhance the Mesos UI.

This is necessary for users of the `V1_DRIVER` as the new driver does not automatically populate
the `host` field. Further, by using our own host and port information, we ensure the information
in ZooKeeper, the information used for HTTP redirects and the information in the Mesos UI
are all in sync.

Note that in vagrant, the hostname and URL are `aurora.local` because we set the `--hostname`
argument of the scheduler. By default Java will set it to the FQDN or IP address of the host.

Diffs (updated)

  src/jmh/java/org/apache/aurora/benchmark/ 95496c1520413269ecb6237427720cc78f176a2d

  src/main/java/org/apache/aurora/scheduler/app/ 3665c4d45f1dd29b4f5c8d3bcaed4b0bf3225f69

  src/main/java/org/apache/aurora/scheduler/http/ 53ebc0b60e60a8e1800f9e8b4ddc292bb44b4330

  src/main/java/org/apache/aurora/scheduler/mesos/ ad6110a7d42ef37023a362e6da4ce35977a5ff69

  src/main/java/org/apache/aurora/scheduler/mesos/ f477404268f96e60dba21d58e90127dcaa0122aa

  src/main/java/org/apache/aurora/scheduler/mesos/ PRE-CREATION 
  src/main/java/org/apache/aurora/scheduler/mesos/ f65a29d7ad8bc49784e324e674f30a6728a9d4ae

  src/main/java/org/apache/aurora/scheduler/mesos/ 84e3f47636d95521600e9a4c4d5b8bc8bbbff8cf

  src/main/java/org/apache/aurora/scheduler/mesos/ d928d02cab087991a8cd8896d4366f6e5dca0913

  src/test/java/org/apache/aurora/scheduler/app/ f5799751489a2773d116375ee17ca9e24fc3b134

  src/test/java/org/apache/aurora/scheduler/app/local/ 7151b81ce23973437aca96d92be7e99948172fb6

  src/test/java/org/apache/aurora/scheduler/mesos/ PRE-CREATION

  src/test/java/org/apache/aurora/scheduler/mesos/ 38e913570dce06359ab2fcc56778073c3ba17277

  src/test/java/org/apache/aurora/scheduler/mesos/ 988ec501e62c0cf65114f4ee4d10cbc0bc85c2d7

  src/test/java/org/apache/aurora/scheduler/thrift/ 14218213bc92fe8c6f981003996c70ebb9370902




e2e tests.

File Attachments



Zameer Manji

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