Re: Review Request 68785: Added jsonschema.py for managing json-based configs.

2019-02-22 Thread Mesos Reviewbot

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



Patch looks great!

Reviews applied: [68785]

Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' 
CONFIGURATION='--verbose --disable-libtool-wrappers 
--disable-parallel-test-execution' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; 
./support/docker-build.sh

- Mesos Reviewbot


On Dec. 5, 2018, 9:56 p.m., Eric Chung wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68785/
> ---
> 
> (Updated Dec. 5, 2018, 9:56 p.m.)
> 
> 
> Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Added jsonschema.py for managing json-based configs.
> 
> 
> Diffs
> -
> 
>   src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/68785/diff/2/
> 
> 
> Testing
> ---
> 
> Ref MESOS-7278
> 
> 
> Thanks,
> 
> Eric Chung
> 
>



Re: Review Request 68785: Added jsonschema.py for managing json-based configs.

2018-12-05 Thread Mesos Reviewbot Windows

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



FAIL: Some of the unit tests failed. Please check the relevant logs.

Reviews applied: `['68785']`

Failed command: `Start-MesosCITesting`

All the build artifacts available at: 
http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2679/mesos-review-68785

Relevant logs:

- 
[mesos-tests.log](http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2679/mesos-review-68785/logs/mesos-tests.log):

```
I1206 07:11:53.926291 12628 master.cpp:11060] Removing task 
b906611e-a0d8-42d3-8b17-f6a1eebc28f1 with resources cpus(allocated: *):4; 
mem(allocated: *):2048; disk(allocated: *):1024; ports(allocated: 
*):[31000-32000] of framework d492d358-74c4-41bf-b7de-2aa984a21adc- on 
agent d492d358-74c4-41bf-b7de-2aa984a21adc-S0 at slave(461)@192.10.1.6:57295 
(windows-02.chtsmhjxogyevckjfayqqcnjda.xx.internal.cloudapp.net)
I1206 07:11:53.929306 12628 master.cpp:1275] Agent 
d492d358-74c4-41bf-b7de-2aa984a21adc-S0 at slave(461)@192.10.1.6:57295 
(windows-02.chtsmhjxogyevckjfayqqcnjda.xx.internal.cloudapp.net) disconnected
I1206 07:11:53.929306 12628 master.cpp:3278] Disconnecting agent 
d492d358-74c4-41bf-b7de-2aa984a21adc-S0 at slave(461)@192.10.1.6:57295 
(windows-02.chtsmhjxogyevckjfayqqcnjda.xx.internal.cloudapp.net)
I1206 07:11:53.929306 12628 master.cpp:3297] Deactivating agent 
d492d358-74c4-41bf-b7de-2aa984a21adc-S0 at slave(461)@192.10.1.6:57295 
(windows-02.chtsmhjxogyevckjfayqqcnjda.xx.internal.cloudapp.net)
I1206 07:11:53.930291 14116 hierarchical.cpp:357] Removed framework 
d492d358-74c4-41bf-b7de-2aa984a21adc-
I1206 07:11:53.930291 14116 hierarchical.cpp:801] Agent 
d492d358-74c4-41bf-b7de-2aa984a21adc-S0 deactivated
I1206 07:11:53.930291 16004 containerizer.cpp:2463] Destroying container 
a038b849-a5d3-4c60-a9c9-6c021fc8e1d1 in RUNNING state
I1206 07:11:53.930291 16004 containerizer.cpp:3130] Transitioning the state of 
container a038b849-a5d3-4c60-a9c9-6c021fc8e1d1 from RUNNING to DESTROYING
I1206 07:11:53.931286 16004 launcher.cpp:161] Asked to destroy container 
a038b849-a5d3-4c60-a9c9-6c021fc8e1d1
W1206 07:11:53.932286 13540 process.cpp:1423] Failed to recv on socket 
WindowsFD::Type::SOCKET=2436 to peer '192.10.1.6:59096': IO failed with error 
code: The specified network name is no longer available.

W1206 07:11:53.933288 13540 process.cpp:838] Failed to recv on socket Window[   
OK ] IsolationFlag/MemoryIsolatorTest.ROOT_MemUsage/0 (685 ms)
[--] 1 test from IsolationFlag/MemoryIsolatorTest (704 ms total)

[--] Global test environment tear-down
[==] 1056 tests from 103 test cases ran. (502177 ms total)
[  PASSED  ] 1055 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] DockerFetcherPluginTest.INTERNET_CURL_FetchImage

 1 FAILED TEST
  YOU HAVE 231 DISABLED TESTS

sFD::Type::SOCKET=2764 to peer '192.10.1.6:59097': IO failed with error code: 
The specified network name is no longer available.

I1206 07:11:53.971282 14116 containerizer.cpp:2969] Container 
a038b849-a5d3-4c60-a9c9-6c021fc8e1d1 has exited
I1206 07:11:54.000337 14528 master.cpp:1117] Master terminating
I1206 07:11:54.001298 12184 hierarchical.cpp:643] Removed agent 
d492d358-74c4-41bf-b7de-2aa984a21adc-S0
I1206 07:11:54.312300 13540 process.cpp:927] Stopped the socket accept loop
```

- Mesos Reviewbot Windows


On Dec. 6, 2018, 5:56 a.m., Eric Chung wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68785/
> ---
> 
> (Updated Dec. 6, 2018, 5:56 a.m.)
> 
> 
> Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Added jsonschema.py for managing json-based configs.
> 
> 
> Diffs
> -
> 
>   src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/68785/diff/2/
> 
> 
> Testing
> ---
> 
> Ref MESOS-7278
> 
> 
> Thanks,
> 
> Eric Chung
> 
>



Re: Review Request 68785: Added jsonschema.py for managing json-based configs.

2018-12-05 Thread Eric Chung

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

(Updated Dec. 6, 2018, 5:56 a.m.)


Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.


Repository: mesos


Description (updated)
---

Added jsonschema.py for managing json-based configs.


Diffs (updated)
-

  src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 


Diff: https://reviews.apache.org/r/68785/diff/2/

Changes: https://reviews.apache.org/r/68785/diff/1-2/


Testing
---

Ref MESOS-7278


Thanks,

Eric Chung



Re: Review Request 68785: Added jsonschema.py for managing json-based configs.

2018-09-20 Thread Mesos Reviewbot Windows

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



FAIL: Some of the unit tests failed. Please check the relevant logs.

Reviews applied: `['68785']`

Failed command: `Start-MesosCITesting`

All the build artifacts available at: 
http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2363/mesos-review-68785

Relevant logs:

- 
[mesos-tests.log](http://dcos-win.westus2.cloudapp.azure.com/artifacts/mesos-reviewbot-testing/2363/mesos-review-68785/logs/mesos-tests.log):

```
I0920 16:45:21.030398  3872 executor.cpp:693] Forked command at 8412
I0920 16:45:21.054435  2012 exec.cpp:445] Executor asked to shutdown
I0920 16:45:21.055395  3872 executor.cpp:182] Received SHUTDOWN event
I0920 16:45:21.055395  3872 executor.cpp:796] Shutting down
I0920 16:45:21.055395  3872 executor.cpp:909] Sending SIGTERM to process tree 
at pid 841981f-e5af0dcf2a43-S0 at slave(462)@192.10.1.8:56497 
(windows-05.aa0q4n2kgcyefckmv0xukjvy4f.xx.internal.cloudapp.net)
I0920 16:45:21.057394   708 master.cpp:1251] Agent 
6ba22670-85be-4eb0-981f-e5af0dcf2a43-S0 at slave(462)@192.10.1.8:56497 
(windows-05.aa0q4n2kgcyefckmv0xukjvy4f.xx.internal.cloudapp.net) disconnected
I0920 16:45:21.057394   708 master.cpp:3267] Disconnecting agent 
6ba22670-85be-4eb0-981f-e5af0dcf2a43-S0 at slave(462)@192.10.1.8:56497 
(windows-05.aa0q4n2kgcyefckmv0xukjvy4f.xx.internal.cloudapp.net)
I0920 16:45:21.058388   708 master.cpp:3286] Deactivating agent 
6ba22670-85be-4eb0-981f-e5af0dcf2a43-S0 at slave(462)@192.10.1.8:56497 
(windows-05.aa0q4n2kgcyefckmv0xukjvy4f.xx.internal.cloudapp.net)
I0920 16:45:21.058388  6872 hierarchical.cpp:359] Removed framework 
6ba22670-85be-4eb0-981f-e5af0dcf2a43-
I0920 16:45:21.058388  6872 hierarchical.cpp:795] Agent 
6ba22670-85be-4eb0-981f-e5af0dcf2a43-S0 deactivated
I0920 16:45:21.059412  6872 containerizer.cpp:2455] Destroying container 
b3971218-5537-4[   OK ] IsolationFlag/MemoryIsolatorTest.ROOT_MemUsage/0 
(582 ms)
[--] 1 test from IsolationFlag/MemoryIsolatorTest (603 ms total)

[--] Global test environment tear-down
[==] 1051 tests from 103 test cases ran. (492565 ms total)
[  PASSED  ] 1050 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] DockerFetcherPluginTest.INTERNET_CURL_InvokeFetchByName

 1 FAILED TEST
  YOU HAVE 231 DISABLED TESTS

6e0-8a1e-6e8e734ee968 in RUNNING state
I0920 16:45:21.059412  6872 containerizer.cpp:3118] Transitioning the state of 
container b3971218-5537-46e0-8a1e-6e8e734ee968 from RUNNING to DESTROYING
I0920 16:45:21.060408  6872 launcher.cpp:166] Asked to destroy container 
b3971218-5537-46e0-8a1e-6e8e734ee968
I0920 16:45:21.065438   708 containerizer.cpp:2957] Container 
b3971218-5537-46e0-8a1e-6e8e734ee968 has exited
I0920 16:45:21.094424  5968 master.cpp:1093] Master terminating
I0920 16:45:21.096412  7812 hierarchical.cpp:637] Removed agent 
6ba22670-85be-4eb0-981f-e5af0dcf2a43-S0
I0920 16:45:21.318403  2212 process.cpp:926] Stopped the socket accept loop
```

- Mesos Reviewbot Windows


On Sept. 20, 2018, 3:43 p.m., Eric Chung wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68785/
> ---
> 
> (Updated Sept. 20, 2018, 3:43 p.m.)
> 
> 
> Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> [WIP] Added jsonschema.py for managing json-based configs.
> 
> 
> Diffs
> -
> 
>   src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/68785/diff/1/
> 
> 
> Testing
> ---
> 
> Ref MESOS-7278
> 
> 
> Thanks,
> 
> Eric Chung
> 
>



Re: Review Request 68785: Added jsonschema.py for managing json-based configs.

2018-09-20 Thread Eric Chung

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

(Updated Sept. 20, 2018, 3:43 p.m.)


Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.


Repository: mesos


Description
---

[WIP] Added jsonschema.py for managing json-based configs.


Diffs
-

  src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 


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


Testing (updated)
---

Ref MESOS-7278


Thanks,

Eric Chung



Review Request 68785: Added jsonschema.py for managing json-based configs.

2018-09-20 Thread Eric Chung

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

Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.


Repository: mesos


Description
---

Added jsonschema.py for managing json-based configs.


Diffs
-

  src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 


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


Testing
---


Thanks,

Eric Chung



Re: Review Request 68785: Added jsonschema.py for managing json-based configs.

2018-09-20 Thread Eric Chung

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

(Updated Sept. 20, 2018, 3:43 p.m.)


Review request for mesos, Armand Grillet, Jason Lai, and Kevin Klues.


Repository: mesos


Description (updated)
---

[WIP] Added jsonschema.py for managing json-based configs.


Diffs
-

  src/python/cli_new/lib/cli/jsonschema.py PRE-CREATION 


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


Testing
---


Thanks,

Eric Chung