mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Armand Grillet <>
Subject Re: Review Request 59177: CLI: Added Config class to manage the config file.
Date Mon, 15 May 2017 01:59:48 GMT

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

(Updated May 15, 2017, 1:59 a.m.)

Review request for mesos and Kevin Klues.


Patch modified to resolve the comments I have received offline.

Repository: mesos

Description (updated)

This new class simplifies the management of the configuration file
given by the user; it loads the TOML file on initialization and
has one method for each element that the user can set.

This new class and its associated content is also given to the plugins
at initialization so that they can read the user configuration and use

Diffs (updated)

  src/cli_new/bin/ 397c120eaaa8f21030dedb1ed552e65c704ee7da 
  src/cli_new/bin/ 0ef07cc67e8020be0424e939e5b19475fb998ac7 
  src/cli_new/lib/cli/ f4fc3f18af5641a4a87143adaba81e62334ccffb 
  src/cli_new/lib/cli/ PRE-CREATION 
  src/cli_new/lib/cli/plugins/ c10d70fb1b3d232008dd908ea747ec6782a9d47e 
  src/cli_new/lib/cli/plugins/config/ d95a36f4a66c66b4477c6816b7fa5a721f9212f7 
  src/cli_new/lib/cli/ 27c4f17e4b75f63f2fb31f0ad27a464227d29448 




Tested manually, PEP8 and Pylint used to make sure that the code style is correct.


Armand Grillet

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