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

Gather metrics from openshift e2e tests #3170

Merged
merged 3 commits into from
Jun 6, 2018
Merged

Gather metrics from openshift e2e tests #3170

merged 3 commits into from
Jun 6, 2018

Conversation

0xmichalis
Copy link
Contributor

/assign jim-minter pweil-

@ghost ghost assigned 0xmichalis Jun 5, 2018
@ghost ghost added the in progress label Jun 5, 2018
@acs-bot acs-bot added the size/M label Jun 5, 2018
@0xmichalis
Copy link
Contributor Author

/label orchestrator/openshift

@codecov
Copy link

codecov bot commented Jun 5, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@2012c1c). Click here to learn what that means.
The diff coverage is n/a.

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

@jim-minter
Copy link
Member

/lgtm
/assign CecileRobertMichon
/unassign jim-minter pweil- kargakis

@acs-bot acs-bot added the lgtm label Jun 5, 2018
@acs-bot acs-bot removed the lgtm label Jun 5, 2018
@ghost ghost assigned 0xmichalis Jun 5, 2018
@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

@0xmichalis
Copy link
Contributor Author

@jackfrancis can you approve this PR? I am going to babysit it tomorrow.

@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

@CecileRobertMichon CecileRobertMichon merged commit 1956e19 into Azure:master Jun 6, 2018
@ghost ghost removed the in progress label Jun 6, 2018
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.

6 participants