You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Armand Grillet <ag...@mesosphere.io> on 2017/09/25 14:26:50 UTC
Re: Review Request 61212: Added CLI utility functions to verify
addresses.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61212/
-----------------------------------------------------------
(Updated Sept. 25, 2017, 2:26 p.m.)
Review request for mesos and Kevin Klues.
Changes
-------
Resolved issues.
Summary (updated)
-----------------
Added CLI utility functions to verify addresses.
Bugs: MESOS-7840
https://issues.apache.org/jira/browse/MESOS-7840
Repository: mesos
Description
-------
This will be used by future plugins.
Diffs (updated)
-----
src/python/cli_new/lib/cli/util.py 7371f83543ed527bea8dbf2fe4e20d92ef8e4492
Diff: https://reviews.apache.org/r/61212/diff/4/
Changes: https://reviews.apache.org/r/61212/diff/3-4/
Testing
-------
Tested manually, PEP8 and Pylint used to make sure that the code style is correct.
Thanks,
Armand Grillet
Re: Review Request 61212: Added CLI utility function to verify
addresses.
Posted by Kevin Klues <kl...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61212/#review186254
-----------------------------------------------------------
src/python/cli_new/lib/cli/util.py
Lines 162 (patched)
<https://reviews.apache.org/r/61212/#comment262738>
Can we change this comment to make the usage of 'basestring' here more clear (how it is the base class for both unicode and string and depending on how this field is passed in it could be either of these).
src/python/cli_new/lib/cli/util.py
Lines 169 (patched)
<https://reviews.apache.org/r/61212/#comment262739>
s/respect/match/
src/python/cli_new/lib/cli/util.py
Lines 188 (patched)
<https://reviews.apache.org/r/61212/#comment262740>
Let's break this out into its own commit.
- Kevin Klues
On Sept. 26, 2017, 1:53 p.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61212/
> -----------------------------------------------------------
>
> (Updated Sept. 26, 2017, 1:53 p.m.)
>
>
> Review request for mesos and Kevin Klues.
>
>
> Bugs: MESOS-7840
> https://issues.apache.org/jira/browse/MESOS-7840
>
>
> Repository: mesos
>
>
> Description
> -------
>
> This will be used by future plugins.
>
>
> Diffs
> -----
>
> src/python/cli_new/lib/cli/util.py 7371f83543ed527bea8dbf2fe4e20d92ef8e4492
>
>
> Diff: https://reviews.apache.org/r/61212/diff/6/
>
>
> Testing
> -------
>
> Tested manually, PEP8 and Pylint used to make sure that the code style is correct.
>
>
> Thanks,
>
> Armand Grillet
>
>
Re: Review Request 61212: Added CLI utility function to verify
addresses.
Posted by Armand Grillet <ag...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61212/
-----------------------------------------------------------
(Updated Sept. 26, 2017, 2:15 p.m.)
Review request for mesos and Kevin Klues.
Changes
-------
Resolved issues.
Bugs: MESOS-7840
https://issues.apache.org/jira/browse/MESOS-7840
Repository: mesos
Description
-------
This will be used by future plugins.
Diffs (updated)
-----
src/python/cli_new/lib/cli/util.py 7371f83543ed527bea8dbf2fe4e20d92ef8e4492
Diff: https://reviews.apache.org/r/61212/diff/7/
Changes: https://reviews.apache.org/r/61212/diff/6-7/
Testing
-------
Tested manually, PEP8 and Pylint used to make sure that the code style is correct.
Thanks,
Armand Grillet
Re: Review Request 61212: Added CLI utility function to verify
addresses.
Posted by Armand Grillet <ag...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61212/
-----------------------------------------------------------
(Updated Sept. 26, 2017, 1:53 p.m.)
Review request for mesos and Kevin Klues.
Changes
-------
Cut patch.
Summary (updated)
-----------------
Added CLI utility function to verify addresses.
Bugs: MESOS-7840
https://issues.apache.org/jira/browse/MESOS-7840
Repository: mesos
Description
-------
This will be used by future plugins.
Diffs (updated)
-----
src/python/cli_new/lib/cli/util.py 7371f83543ed527bea8dbf2fe4e20d92ef8e4492
Diff: https://reviews.apache.org/r/61212/diff/6/
Changes: https://reviews.apache.org/r/61212/diff/5-6/
Testing
-------
Tested manually, PEP8 and Pylint used to make sure that the code style is correct.
Thanks,
Armand Grillet