Skip to content

Provide per-suite vic-machine-server logs in CI#7978

Merged
zjs merged 1 commit intovmware:masterfrom
zjs:topic/organize-server-logs
May 22, 2018
Merged

Provide per-suite vic-machine-server logs in CI#7978
zjs merged 1 commit intovmware:masterfrom
zjs:topic/organize-server-logs

Conversation

@zjs
Copy link
Member

@zjs zjs commented May 22, 2018

Leverage the recent change which introduced per-suite logging from vic-machine-server to provide per-suite log files. These log files will be included in the CI log bundle, and they will be written to the local filesystem when running CI locally.

[specific ci=Group23-VIC-Machine-Service]

Leverage the recent change which introduced per-suite logging from
vic-machine-server to provide per-suite log files. These log files
will be included in the CI log bundle, and they will be written to
the local filesystem when running CI locally.
@zjs zjs added this to the Sprint 32 Lifecycle milestone May 22, 2018
@zjs zjs self-assigned this May 22, 2018
@zjs zjs requested review from AngieCris and rogeliosanchez May 22, 2018 17:39
@zjs
Copy link
Member Author

zjs commented May 22, 2018

Example of local execution:

~/go/src/github.com/vmware/vic (topic/organize-server-logs)$ ls Test*
ls: cannot access 'Test*': No such file or directory
~/go/src/github.com/vmware/vic (topic/organize-server-logs)$ tests/local-integration-test.sh -t *23-01*
[vic-integration-test-on-pr:L0:0s] + pybot -L DEBUG --debugfile debug.log --exclude skip -t "*23-01*" tests/test-cases
[vic-integration-test-on-pr:L1:1s] ==============================================================================
[vic-integration-test-on-pr:L2:1s] Test-Cases                                                                    
[vic-integration-test-on-pr:L3:1s] ==============================================================================
[vic-integration-test-on-pr:L4:1s] Test-Cases.Group23-VIC-Machine-Service                                        
[vic-integration-test-on-pr:L5:1s] ==============================================================================
[vic-integration-test-on-pr:L6:2s] Test-Cases.Group23-VIC-Machine-Service.23-01-Metadata :: Test 23-01 - Version 
[vic-integration-test-on-pr:L7:2s] ==============================================================================
[vic-integration-test-on-pr:L8:7s] sh: 1: netstat: not found
[vic-integration-test-on-pr:L9:7s] Get Version                                                           | PASS |
[vic-integration-test-on-pr:L10:7s] ------------------------------------------------------------------------------
[vic-integration-test-on-pr:L11:7s] Get Hello                                                             | PASS |
[vic-integration-test-on-pr:L12:7s] ------------------------------------------------------------------------------
[vic-integration-test-on-pr:L13:7s] Test-Cases.Group23-VIC-Machine-Service.23-01-Metadata :: Test 23-0... | PASS |
[vic-integration-test-on-pr:L14:7s] 2 critical tests, 2 passed, 0 failed
[vic-integration-test-on-pr:L15:7s] 2 tests total, 2 passed, 0 failed
[vic-integration-test-on-pr:L16:7s] ==============================================================================
[vic-integration-test-on-pr:L17:7s] Test-Cases.Group23-VIC-Machine-Service                                | PASS |
[vic-integration-test-on-pr:L18:7s] 2 critical tests, 2 passed, 0 failed
[vic-integration-test-on-pr:L19:7s] 2 tests total, 2 passed, 0 failed
[vic-integration-test-on-pr:L20:7s] ==============================================================================
[vic-integration-test-on-pr:L21:7s] Test-Cases                                                            | PASS |
[vic-integration-test-on-pr:L22:7s] 2 critical tests, 2 passed, 0 failed
[vic-integration-test-on-pr:L23:7s] 2 tests total, 2 passed, 0 failed
[vic-integration-test-on-pr:L24:7s] ==============================================================================
[vic-integration-test-on-pr:L25:7s] Debug:   /go/src/github.com/vmware/vic/debug.log
[vic-integration-test-on-pr:L26:7s] Output:  /go/src/github.com/vmware/vic/output.xml
[vic-integration-test-on-pr:L27:7s] Log:     /go/src/github.com/vmware/vic/log.html
[vic-integration-test-on-pr:L28:7s] Report:  /go/src/github.com/vmware/vic/report.html
removing temp file .tmp.drone.4l13U.yml
~/go/src/github.com/vmware/vic (topic/organize-server-logs)$ ls Test*
Test-Cases.Group23-VIC-Machine-Service.23-01-Metadata-vic-machine-server.log
~/go/src/github.com/vmware/vic (topic/organize-server-logs)$ cat Test-Cases.Group23-VIC-Machine-Service.23-01-Metadata-vic-machine-server.log 
May 22 2018 17:42:53.071Z INFO  Starting Service. Version: "v1.5.0-dev-0-940ca22"
May 22 2018 17:42:53.140Z INFO  Serving pprof at http://localhost:6060
May 22 2018 17:42:53.141Z INFO  Serving vic machine at http://127.0.0.1:41357
May 22 2018 17:42:57.907Z DEBUG op=17.1: [NewOperation] op=17.1 [vic/lib/apiservers/service/restapi.addLogging.func1:213]
May 22 2018 17:42:57.907Z INFO  op=17.1: Request: GET /container/version HTTP/1.1
May 22 2018 17:42:57.907Z INFO  op=17.1: Response: 200 OK
May 22 2018 17:42:57.958Z DEBUG op=17.2: [NewOperation] op=17.2 [vic/lib/apiservers/service/restapi.addLogging.func1:213]
May 22 2018 17:42:57.958Z INFO  op=17.2: Request: GET /container/hello HTTP/1.1
May 22 2018 17:42:57.959Z INFO  op=17.2: Response: 200 OK

@codecov-io
Copy link

codecov-io commented May 22, 2018

Codecov Report

Merging #7978 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7978   +/-   ##
=======================================
  Coverage   26.07%   26.07%           
=======================================
  Files          37       37           
  Lines        5243     5243           
=======================================
  Hits         1367     1367           
  Misses       3769     3769           
  Partials      107      107

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58effdf...9d1f200. Read the comment docs.

Copy link
Contributor

@rogeliosanchez rogeliosanchez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@zjs zjs merged commit 346982a into vmware:master May 22, 2018
zjs added a commit to zjs/vic that referenced this pull request Jul 27, 2018
Leverage the recent change which introduced per-suite logging from
vic-machine-server to provide per-suite log files. These log files
will be included in the CI log bundle, and they will be written to
the local filesystem when running CI locally.

(cherry picked from commit 346982a)
zjs added a commit to zjs/vic that referenced this pull request Aug 2, 2018
…#8162)

Leverage the recent change which introduced per-suite logging from
vic-machine-server to provide per-suite log files. These log files
will be included in the CI log bundle, and they will be written to
the local filesystem when running CI locally.

(cherry picked from commit 346982a)
zjs added a commit that referenced this pull request Aug 7, 2018
Leverage the recent change which introduced per-suite logging from
vic-machine-server to provide per-suite log files. These log files
will be included in the CI log bundle, and they will be written to
the local filesystem when running CI locally.

(cherry picked from commit 346982a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants