mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Armand Grillet <agril...@mesosphere.io>
Subject Re: Review Request 62067: Added 'mesos container list' command to CLI.
Date Wed, 27 Sep 2017 12:39:47 GMT

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

(Updated Sept. 27, 2017, 12:39 p.m.)


Review request for mesos and Kevin Klues.


Changes
-------

Improved code.


Bugs: MESOS-7285
    https://issues.apache.org/jira/browse/MESOS-7285


Repository: mesos


Description
-------

This command displays the containers running
on all the agents or a specific one.


Diffs (updated)
-----

  src/python/cli_new/bin/settings.py d42df04b0ff42bb6f466842e59223cd90a74d5c0 
  src/python/cli_new/lib/cli/plugins/container/__init__.py PRE-CREATION 
  src/python/cli_new/lib/cli/plugins/container/main.py PRE-CREATION 
  src/python/cli_new/lib/cli/tests/__init__.py 0daf28869e107263c51653ace39e3b1826871048 
  src/python/cli_new/lib/cli/tests/container.py PRE-CREATION 
  src/python/cli_new/tests/main.py 3e4d2e449a6485206700b4a490d325a393d31f90 


Diff: https://reviews.apache.org/r/62067/diff/3/

Changes: https://reviews.apache.org/r/62067/diff/2-3/


Testing
-------

To test with one master:

$ ./bootstrap
$ source activate
$ mesos-cli-tests

I also checked that the Python linter was still working.


Thanks,

Armand Grillet


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