Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: reduce verbosity of e2e test output #173

Merged
merged 1 commit into from
Apr 4, 2018

Conversation

nolanmar511
Copy link
Contributor

@nolanmar511 nolanmar511 commented Apr 4, 2018

This PR updates e2e test to be compatible with changes made to github.com/GoogleCloudPlatform/google-cloud-go/profiler/proftest here, and reduce verbosity of logs.

This should be merged in when Kokoro CI test is passing. At that point, Kokoro tests without this change will be expected to fail.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 4, 2018
@codecov
Copy link

codecov bot commented Apr 4, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #173   +/-   ##
======================================
  Coverage    94.7%   94.7%           
======================================
  Files           6       6           
  Lines         359     359           
  Branches       59      59           
======================================
  Hits          340     340           
  Misses         19      19

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 658e111...3489685. Read the comment docs.

@@ -54,14 +55,14 @@ set -eo pipefail
# Display commands being run
set -x
# Install git
apt-get update
apt-get -y -q install git build-essential
apt-get update > /dev/null

This comment was marked as spam.

This comment was marked as spam.

@@ -54,14 +55,14 @@ set -eo pipefail
# Display commands being run
set -x

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@nolanmar511
Copy link
Contributor Author

PTAL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants