You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Andrew Schwartzmeyer <an...@schwartzmeyer.com> on 2017/10/02 18:57:34 UTC

Review Request 62734: Added --markdown and --target-people to post-reviews.py.

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

Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Till Toenshoff.


Repository: mesos


Description
-------

These options are appropriately forwarded to `rbt` so that you can post
reviews where the commit messages contain Markdown, and have the target
people automatically filled out.


Diffs
-----

  support/post-reviews.py 5ecde40553c6b1ace9bfcf27a7268450b8bc2838 


Diff: https://reviews.apache.org/r/62734/diff/1/


Testing
-------

Used it to post this patch.


Thanks,

Andrew Schwartzmeyer


Re: Review Request 62734: Added --markdown and --target-people to post-reviews.py.

Posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com>.

> On Oct. 11, 2017, 3:29 p.m., Joseph Wu wrote:
> > support/post-reviews.py
> > Lines 142-144 (original), 142-146 (patched)
> > <https://reviews.apache.org/r/62734/diff/1/?file=1842242#file1842242line142>
> >
> >     The lack of help text in this argument parsing makes these new arguments basically undiscoverable :(
> >     
> >     https://www.reviewboard.org/docs/rbtools/dev/rbt/commands/post/#rbt-post
> >     
> >     In any case, it would be nice to have a `--bugs-closed` flag too, for mass-filling JIRAs.

Oh, yeah, we should do that too.


- Andrew


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


On Nov. 5, 2017, 5:35 p.m., Andrew Schwartzmeyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62734/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2017, 5:35 p.m.)
> 
> 
> Review request for mesos, Joseph Wu, Michael Park, and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> These options are appropriately forwarded to `rbt` so that you can post
> reviews where the commit messages contain Markdown, and have the target
> people automatically filled out.
> 
> 
> Diffs
> -----
> 
>   support/post-reviews.py 5ecde40553c6b1ace9bfcf27a7268450b8bc2838 
> 
> 
> Diff: https://reviews.apache.org/r/62734/diff/1/
> 
> 
> Testing
> -------
> 
> Used it to post this patch.
> 
> 
> Thanks,
> 
> Andrew Schwartzmeyer
> 
>


Re: Review Request 62734: Added --markdown and --target-people to post-reviews.py.

Posted by Joseph Wu <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62734/#review187712
-----------------------------------------------------------




support/post-reviews.py
Lines 142-144 (original), 142-146 (patched)
<https://reviews.apache.org/r/62734/#comment264759>

    The lack of help text in this argument parsing makes these new arguments basically undiscoverable :(
    
    https://www.reviewboard.org/docs/rbtools/dev/rbt/commands/post/#rbt-post
    
    In any case, it would be nice to have a `--bugs-closed` flag too, for mass-filling JIRAs.


- Joseph Wu


On Oct. 2, 2017, 11:57 a.m., Andrew Schwartzmeyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62734/
> -----------------------------------------------------------
> 
> (Updated Oct. 2, 2017, 11:57 a.m.)
> 
> 
> Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> These options are appropriately forwarded to `rbt` so that you can post
> reviews where the commit messages contain Markdown, and have the target
> people automatically filled out.
> 
> 
> Diffs
> -----
> 
>   support/post-reviews.py 5ecde40553c6b1ace9bfcf27a7268450b8bc2838 
> 
> 
> Diff: https://reviews.apache.org/r/62734/diff/1/
> 
> 
> Testing
> -------
> 
> Used it to post this patch.
> 
> 
> Thanks,
> 
> Andrew Schwartzmeyer
> 
>


Re: Review Request 62734: Added --markdown and --target-people to post-reviews.py.

Posted by Andrew Schwartzmeyer <an...@schwartzmeyer.com>.

> On Oct. 2, 2017, 4:50 p.m., Mesos Reviewbot Windows wrote:
> > FAIL: Some Mesos tests failed.
> > 
> > Reviews applied: `['62734']`
> > 
> > Failed command: `C:\mesos\src\mesos-tests.exe --verbose --gtest_filter="-ContentType/MasterAPITest.EventAuthorizationFiltering/1"`
> > 
> > All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62734
> > 
> > Relevant logs:
> > 
> > - [mesos-tests-stdout.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62734/logs/mesos-tests-stdout.log):
> > 
> > ```
> > [       OK ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/0 (297 ms)
> > [ RUN      ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/1
> > [       OK ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/1 (331 ms)
> > [ RUN      ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/2
> > [       OK ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/2 (346 ms)
> > [ RUN      ] Endpoint/SlaveEndpointTest.NoAuthorizer/0
> > [       OK ] Endpoint/SlaveEndpointTest.NoAuthorizer/0 (341 ms)
> > [ RUN      ] Endpoint/SlaveEndpointTest.NoAuthorizer/1
> > [       OK ] Endpoint/SlaveEndpointTest.NoAuthorizer/1 (367 ms)
> > [ RUN      ] Endpoint/SlaveEndpointTest.NoAuthorizer/2
> > [       OK ] Endpoint/SlaveEndpointTest.NoAuthorizer/2 (332 ms)
> > [----------] 9 tests from Endpoint/SlaveEndpointTest (3464 ms total)
> > 
> > [----------] 2 tests from ContainerizerType/DefaultContainerDNSFlagTest
> > [ RUN      ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/0
> > [       OK ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/0 (139 ms)
> > [ RUN      ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/1
> > [       OK ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/1 (168 ms)
> > [----------] 2 tests from ContainerizerType/DefaultContainerDNSFlagTest (347 ms total)
> > 
> > [----------] Global test environment tear-down
> > [==========] 714 tests from 72 test cases ran. (429332 ms total)
> > [  PASSED  ] 713 tests.
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] MesosContainerizer/DefaultExecutorTest.TaskWithFileURI/0, where GetParam() = "mesos"
> > 
> >  1 FAILED TEST
> >   YOU HAVE 161 DISABLED TESTS
> > 
> > ```
> > 
> > - [mesos-tests-stderr.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62734/logs/mesos-tests-stderr.log):
> > 
> > ```
> > I1002 23:50:07.312918 33332 containerizer.cpp:648] Recovering containerizer
> > I1002 23:50:07.326920 30612 provisioner.cpp:416] Provisioner recovery complete
> > I1002 23:50:07.326920 21552 slave.cpp:6310] Finished recovery
> > I1002 23:50:07.361922 21552 http.cpp:1185] HTTP GET for /slave(270)/monitor/statistics.json from 10.3.1.7:61843
> > I1002 23:50:07.372990 21552 http.cpp:976] Authorizing principal 'test-principal' to GET the '/monitor/statistics.json' endpoint
> > I1002 23:50:07.402922 31940 slave.cpp:869] Agent terminating
> > I1002 23:50:07.565924 33120 containerizer.cpp:292] Using isolation { windows/cpu, filesystem/windows, environment_secret }
> > I1002 23:50:07.569928 33120 provisioner.cpp:255] Using default backend 'copy'
> > I1002 23:50:07.587925 33120 cluster.cpp:448] Creating default 'local' authorizer
> > I1002 23:50:07.607926 21552 slave.cpp:254] Mesos agent started on (271)@10.3.1.7:60891
> > I1002 23:50:07.607926 21552 slave.cpp:255] Flags at startup: --acls="" --appc_simple_discovery_uri_prefix="http://" --appc_store_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\store\appc" --authenticate_http_readonly="true" --authenticate_http_readwrite="true" --authenticatee="crammd5" --authentication_backoff_factor="1secs" --authorizer="local" --container_disk_watch_interval="15secs" --containerizers="mesos" --credential="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\credential" --default_role="*" --disk_watch_interval="1mins" --docker="docker" --docker_kill_orphans="true" --docker_registry="https://registry-1.docker.io" --docker_remove_delay="6hrs" --docker_socket="//./pipe/docker_engine" --docker_stop_timeout="0ns" --docker_store_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\store\docker" --docker_volume_checkpoint_dir="/var/run/mesos/isolators/docker/volume" --enforce_container_disk_quota="false" --executor_registration_timeout="1mins" --executor_reregistration_timeout="2sec
 s" --executor_shutdown_grace_period="5secs" --fetcher_cache_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\fetch" --fetcher_cache_size="2GB" --frameworks_home="" --gc_delay="1weeks" --gc_disk_headroom="0.1" --hadoop_home="" --help="false" --hostname_lookup="true" --http_command_executor="false" --http_credentials="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\http_credentials" --http_heartbeat_interval="30secs" --initialize_driver_logging="true" --isolation="windows/cpu" --launcher="windows" --launcher_dir="C:\mesos\src" --logbufsecs="0" --logging_level="INFO" --max_completed_executors_per_framework="150" --oversubscribed_resources_interval="15secs" --port="5051" --qos_correction_interval_min="0ns" --quiet="false" --recover="reconnect" --recovery_timeout="15mins" --registration_backoff_factor="10ms" --resources="cpus:2;gpus:0;mem:1024;disk:1024;ports:[31000-32000]" --runtime_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j" --sandbox_directory="C:\mesos\sandbox" --strict="true" --versi
 on="false" --work_dir="C:\Users\mesos\AppData\Local\Temp\2\0R2kZH" --zk_session_timeout="10secs"
> > I1002 23:50:07.651927 21552 credentials.hpp:86] Loading credential for authentication from 'C:\Users\mesos\AppData\Local\Temp\2\735Q1j\credential'
> > I1002 23:50:07.652927 21552 slave.cpp:287] Agent using credential for: test-principal
> > I1002 23:50:07.652927 21552 credentials.hpp:37] Loading credentials for authentication from 'C:\Users\mesos\AppData\Local\Temp\2\735Q1j\http_credentials'
> > I1002 23:50:07.664933 21552 http.cpp:1045] Creating default 'basic' HTTP authenticator for realm 'mesos-agent-readonly'
> > I1002 23:50:07.665933 21552 http.cpp:1045] Creating default 'basic' HTTP authenticator for realm 'mesos-agent-readwrite'
> > I1002 23:50:07.677932 21552 slave.cpp:585] Agent resources: [{"name":"cpus","scalar":{"value":2.0},"type":"SCALAR"},{"name":"mem","scalar":{"value":1024.0},"type":"SCALAR"},{"name":"disk","scalar":{"value":1024.0},"type":"SCALAR"},{"name":"ports","ranges":{"range":[{"begin":31000,"end":32000}]},"type":"RANGES"}]
> > I1002 23:50:07.681936 21552 slave.cpp:593] Agent attributes: [  ]
> > I1002 23:50:07.682929 21552 slave.cpp:602] Agent hostname: mesos-bld-s2.zq4gs31qjdiunm1ryi1452nvnh.dx.internal.cloudapp.net
> > I1002 23:50:07.682929 30612 status_update_manager.cpp:177] Pausing sending status updates
> > I1002 23:50:07.693929 33388 state.cpp:64] Recovering state from 'C:\Users\mesos\AppData\Local\Temp\2\0R2kZH\meta'
> > I1002 23:50:07.697930 33332 status_update_manager.cpp:203] Recovering status update manager
> > I1002 23:50:07.708928 33756 containerizer.cpp:648] Recovering containerizer
> > I1002 23:50:07.715930 31940 provisioner.cpp:416] Provisioner recovery complete
> > I1002 23:50:07.716930 33524 slave.cpp:6310] Finished recovery
> > I1002 23:50:07.738930 33388 http.cpp:1185] HTTP GET for /slave(271)/containers from 10.3.1.7:61844
> > I1002 23:50:07.739930 33388 http.cpp:976] Authorizing principal 'test-principal' to GET the '/containers' endpoint
> > I1002 23:50:07.767931 32148 slave.cpp:869] Agent terminating
> > I1002 23:50:08.716954 33444 process.cpp:1068] Failed to accept socket: future discarded
> > ```

This test can be ignored.


- Andrew


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


On Oct. 2, 2017, 11:57 a.m., Andrew Schwartzmeyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62734/
> -----------------------------------------------------------
> 
> (Updated Oct. 2, 2017, 11:57 a.m.)
> 
> 
> Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> These options are appropriately forwarded to `rbt` so that you can post
> reviews where the commit messages contain Markdown, and have the target
> people automatically filled out.
> 
> 
> Diffs
> -----
> 
>   support/post-reviews.py 5ecde40553c6b1ace9bfcf27a7268450b8bc2838 
> 
> 
> Diff: https://reviews.apache.org/r/62734/diff/1/
> 
> 
> Testing
> -------
> 
> Used it to post this patch.
> 
> 
> Thanks,
> 
> Andrew Schwartzmeyer
> 
>


Re: Review Request 62734: Added --markdown and --target-people to post-reviews.py.

Posted by Mesos Reviewbot Windows <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62734/#review186918
-----------------------------------------------------------



FAIL: Some Mesos tests failed.

Reviews applied: `['62734']`

Failed command: `C:\mesos\src\mesos-tests.exe --verbose --gtest_filter="-ContentType/MasterAPITest.EventAuthorizationFiltering/1"`

All the build artifacts available at: http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62734

Relevant logs:

- [mesos-tests-stdout.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62734/logs/mesos-tests-stdout.log):

```
[       OK ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/0 (297 ms)
[ RUN      ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/1
[       OK ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/1 (331 ms)
[ RUN      ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/2
[       OK ] Endpoint/SlaveEndpointTest.UnauthorizedRequest/2 (346 ms)
[ RUN      ] Endpoint/SlaveEndpointTest.NoAuthorizer/0
[       OK ] Endpoint/SlaveEndpointTest.NoAuthorizer/0 (341 ms)
[ RUN      ] Endpoint/SlaveEndpointTest.NoAuthorizer/1
[       OK ] Endpoint/SlaveEndpointTest.NoAuthorizer/1 (367 ms)
[ RUN      ] Endpoint/SlaveEndpointTest.NoAuthorizer/2
[       OK ] Endpoint/SlaveEndpointTest.NoAuthorizer/2 (332 ms)
[----------] 9 tests from Endpoint/SlaveEndpointTest (3464 ms total)

[----------] 2 tests from ContainerizerType/DefaultContainerDNSFlagTest
[ RUN      ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/0
[       OK ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/0 (139 ms)
[ RUN      ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/1
[       OK ] ContainerizerType/DefaultContainerDNSFlagTest.ValidateFlag/1 (168 ms)
[----------] 2 tests from ContainerizerType/DefaultContainerDNSFlagTest (347 ms total)

[----------] Global test environment tear-down
[==========] 714 tests from 72 test cases ran. (429332 ms total)
[  PASSED  ] 713 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] MesosContainerizer/DefaultExecutorTest.TaskWithFileURI/0, where GetParam() = "mesos"

 1 FAILED TEST
  YOU HAVE 161 DISABLED TESTS

```

- [mesos-tests-stderr.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/62734/logs/mesos-tests-stderr.log):

```
I1002 23:50:07.312918 33332 containerizer.cpp:648] Recovering containerizer
I1002 23:50:07.326920 30612 provisioner.cpp:416] Provisioner recovery complete
I1002 23:50:07.326920 21552 slave.cpp:6310] Finished recovery
I1002 23:50:07.361922 21552 http.cpp:1185] HTTP GET for /slave(270)/monitor/statistics.json from 10.3.1.7:61843
I1002 23:50:07.372990 21552 http.cpp:976] Authorizing principal 'test-principal' to GET the '/monitor/statistics.json' endpoint
I1002 23:50:07.402922 31940 slave.cpp:869] Agent terminating
I1002 23:50:07.565924 33120 containerizer.cpp:292] Using isolation { windows/cpu, filesystem/windows, environment_secret }
I1002 23:50:07.569928 33120 provisioner.cpp:255] Using default backend 'copy'
I1002 23:50:07.587925 33120 cluster.cpp:448] Creating default 'local' authorizer
I1002 23:50:07.607926 21552 slave.cpp:254] Mesos agent started on (271)@10.3.1.7:60891
I1002 23:50:07.607926 21552 slave.cpp:255] Flags at startup: --acls="" --appc_simple_discovery_uri_prefix="http://" --appc_store_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\store\appc" --authenticate_http_readonly="true" --authenticate_http_readwrite="true" --authenticatee="crammd5" --authentication_backoff_factor="1secs" --authorizer="local" --container_disk_watch_interval="15secs" --containerizers="mesos" --credential="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\credential" --default_role="*" --disk_watch_interval="1mins" --docker="docker" --docker_kill_orphans="true" --docker_registry="https://registry-1.docker.io" --docker_remove_delay="6hrs" --docker_socket="//./pipe/docker_engine" --docker_stop_timeout="0ns" --docker_store_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\store\docker" --docker_volume_checkpoint_dir="/var/run/mesos/isolators/docker/volume" --enforce_container_disk_quota="false" --executor_registration_timeout="1mins" --executor_reregistration_timeout="2secs" -
 -executor_shutdown_grace_period="5secs" --fetcher_cache_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\fetch" --fetcher_cache_size="2GB" --frameworks_home="" --gc_delay="1weeks" --gc_disk_headroom="0.1" --hadoop_home="" --help="false" --hostname_lookup="true" --http_command_executor="false" --http_credentials="C:\Users\mesos\AppData\Local\Temp\2\735Q1j\http_credentials" --http_heartbeat_interval="30secs" --initialize_driver_logging="true" --isolation="windows/cpu" --launcher="windows" --launcher_dir="C:\mesos\src" --logbufsecs="0" --logging_level="INFO" --max_completed_executors_per_framework="150" --oversubscribed_resources_interval="15secs" --port="5051" --qos_correction_interval_min="0ns" --quiet="false" --recover="reconnect" --recovery_timeout="15mins" --registration_backoff_factor="10ms" --resources="cpus:2;gpus:0;mem:1024;disk:1024;ports:[31000-32000]" --runtime_dir="C:\Users\mesos\AppData\Local\Temp\2\735Q1j" --sandbox_directory="C:\mesos\sandbox" --strict="true" --version="
 false" --work_dir="C:\Users\mesos\AppData\Local\Temp\2\0R2kZH" --zk_session_timeout="10secs"
I1002 23:50:07.651927 21552 credentials.hpp:86] Loading credential for authentication from 'C:\Users\mesos\AppData\Local\Temp\2\735Q1j\credential'
I1002 23:50:07.652927 21552 slave.cpp:287] Agent using credential for: test-principal
I1002 23:50:07.652927 21552 credentials.hpp:37] Loading credentials for authentication from 'C:\Users\mesos\AppData\Local\Temp\2\735Q1j\http_credentials'
I1002 23:50:07.664933 21552 http.cpp:1045] Creating default 'basic' HTTP authenticator for realm 'mesos-agent-readonly'
I1002 23:50:07.665933 21552 http.cpp:1045] Creating default 'basic' HTTP authenticator for realm 'mesos-agent-readwrite'
I1002 23:50:07.677932 21552 slave.cpp:585] Agent resources: [{"name":"cpus","scalar":{"value":2.0},"type":"SCALAR"},{"name":"mem","scalar":{"value":1024.0},"type":"SCALAR"},{"name":"disk","scalar":{"value":1024.0},"type":"SCALAR"},{"name":"ports","ranges":{"range":[{"begin":31000,"end":32000}]},"type":"RANGES"}]
I1002 23:50:07.681936 21552 slave.cpp:593] Agent attributes: [  ]
I1002 23:50:07.682929 21552 slave.cpp:602] Agent hostname: mesos-bld-s2.zq4gs31qjdiunm1ryi1452nvnh.dx.internal.cloudapp.net
I1002 23:50:07.682929 30612 status_update_manager.cpp:177] Pausing sending status updates
I1002 23:50:07.693929 33388 state.cpp:64] Recovering state from 'C:\Users\mesos\AppData\Local\Temp\2\0R2kZH\meta'
I1002 23:50:07.697930 33332 status_update_manager.cpp:203] Recovering status update manager
I1002 23:50:07.708928 33756 containerizer.cpp:648] Recovering containerizer
I1002 23:50:07.715930 31940 provisioner.cpp:416] Provisioner recovery complete
I1002 23:50:07.716930 33524 slave.cpp:6310] Finished recovery
I1002 23:50:07.738930 33388 http.cpp:1185] HTTP GET for /slave(271)/containers from 10.3.1.7:61844
I1002 23:50:07.739930 33388 http.cpp:976] Authorizing principal 'test-principal' to GET the '/containers' endpoint
I1002 23:50:07.767931 32148 slave.cpp:869] Agent terminating
I1002 23:50:08.716954 33444 process.cpp:1068] Failed to accept socket: future discarded
```

- Mesos Reviewbot Windows


On Oct. 2, 2017, 6:57 p.m., Andrew Schwartzmeyer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62734/
> -----------------------------------------------------------
> 
> (Updated Oct. 2, 2017, 6:57 p.m.)
> 
> 
> Review request for mesos, Jeff Coffler, John Kordich, Joseph Wu, and Till Toenshoff.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> These options are appropriately forwarded to `rbt` so that you can post
> reviews where the commit messages contain Markdown, and have the target
> people automatically filled out.
> 
> 
> Diffs
> -----
> 
>   support/post-reviews.py 5ecde40553c6b1ace9bfcf27a7268450b8bc2838 
> 
> 
> Diff: https://reviews.apache.org/r/62734/diff/1/
> 
> 
> Testing
> -------
> 
> Used it to post this patch.
> 
> 
> Thanks,
> 
> Andrew Schwartzmeyer
> 
>