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

Fetch waagent logs for openshift e2e tests #3168

Merged
merged 2 commits into from
Jun 7, 2018
Merged

Fetch waagent logs for openshift e2e tests #3168

merged 2 commits into from
Jun 7, 2018

Conversation

0xmichalis
Copy link
Contributor

/assign jim-minter CecileRobertMichon

@0xmichalis
Copy link
Contributor Author

/label orchestrator/openshift

@jim-minter
Copy link
Member

/lgtm
/unassign jim-minter kargakis

@codecov
Copy link

codecov bot commented Jun 5, 2018

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3168   +/-   ##
=======================================
  Coverage   52.32%   52.32%           
=======================================
  Files         103      103           
  Lines       15452    15452           
=======================================
  Hits         8086     8086           
  Misses       6638     6638           
  Partials      728      728

@jim-minter
Copy link
Member

/lgtm

@acs-bot acs-bot added the lgtm label Jun 5, 2018
@jim-minter jim-minter removed their assignment Jun 5, 2018
@acs-bot
Copy link

acs-bot commented Jun 5, 2018

New changes are detected. LGTM label has been removed.

@acs-bot acs-bot removed the lgtm label Jun 5, 2018
@0xmichalis
Copy link
Contributor Author

Rebased on top of #3154; @jackfrancis please approve this one too

thanks

@jackfrancis
Copy link
Member

/approve

@acs-bot
Copy link

acs-bot commented Jun 5, 2018

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jackfrancis

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

@0xmichalis
Copy link
Contributor Author

These contexts will never let tide merge this - please merge

openshift flake is etcd

@CecileRobertMichon
Copy link
Contributor

@Kargakis can you please fix the merge conflict? (I just merged #3170)

@0xmichalis
Copy link
Contributor Author

rebased

@0xmichalis 0xmichalis added the lgtm label Jun 7, 2018
@acs-bot acs-bot merged commit 898fd14 into Azure:master Jun 7, 2018
@ghost ghost removed the in progress label Jun 7, 2018
@0xmichalis 0xmichalis deleted the grab-stderr-stdout branch June 7, 2018 00:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants