Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

log errors in DCOS e2e test #3081

Merged
merged 1 commit into from
May 29, 2018

Conversation

CecileRobertMichon
Copy link
Contributor

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #3080

Special notes for your reviewer:

If applicable:

  • documentation
  • unit tests
  • tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

@acs-bot
Copy link

acs-bot commented May 24, 2018

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: brendandburns

Assign the PR to them by writing /assign @brendandburns in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@codecov
Copy link

codecov bot commented May 25, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #3081   +/-   ##
======================================
  Coverage    51.5%   51.5%           
======================================
  Files          99      99           
  Lines       15036   15036           
======================================
  Hits         7744    7744           
  Misses       6582    6582           
  Partials      710     710

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 d65c809...ece32a9. Read the comment docs.

@dmitsh
Copy link

dmitsh commented May 25, 2018

The error message of type "Error trying to install dcos client:Process exited with status 1" is not helpful. We need the actual error message produced by dcos.

@CecileRobertMichon
Copy link
Contributor Author

CecileRobertMichon commented May 25, 2018

@dmitsh the next line shows the exact error:

2018/05/25 20:43:38 Error while trying dcos cluster setup:Process exited with status 1
2018/05/25 20:43:38 Output:URL [http://localhost:80/pkgpanda/active.buildinfo.full.json] is unreachable.

@dmitsh
Copy link

dmitsh commented May 25, 2018

Thanks, missed that line.
I think we need to increase the VM size in the cluster. I will send out another PR

@dmitsh
Copy link

dmitsh commented May 29, 2018

@CecileRobertMichon are you considering merging this PR?

@CecileRobertMichon
Copy link
Contributor Author

@dmitsh sure, can I get your review?

@CecileRobertMichon CecileRobertMichon requested a review from dmitsh May 29, 2018 23:24
@CecileRobertMichon CecileRobertMichon merged commit 3a78439 into Azure:master May 29, 2018
@ghost ghost removed the in progress label May 29, 2018
@CecileRobertMichon CecileRobertMichon deleted the dcos-e2e-log branch June 6, 2018 19:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DCOS e2e test is unreliable
3 participants