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

feat: read vm size from instance metadata service for windows cse telemetry #2663

Merged
merged 1 commit into from Feb 7, 2020

Conversation

marosset
Copy link
Contributor

@marosset marosset commented Feb 1, 2020

Reason for Change:

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Feb 1, 2020

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2663   +/-   ##
=======================================
  Coverage   71.77%   71.77%           
=======================================
  Files         134      134           
  Lines       25071    25071           
=======================================
  Hits        17995    17995           
  Misses       6021     6021           
  Partials     1055     1055

@marosset
Copy link
Contributor Author

marosset commented Feb 3, 2020

/cc @ksubrmnn

Copy link
Member

@devigned devigned left a comment

Choose a reason for hiding this comment

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

/lgtm

@acs-bot acs-bot added the lgtm label Feb 7, 2020
@acs-bot acs-bot merged commit 59c2ce0 into Azure:master Feb 7, 2020
@acs-bot
Copy link

acs-bot commented Feb 7, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: devigned, marosset

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

@marosset marosset deleted the win-cse-imds branch February 7, 2020 23:21
marosset added a commit to marosset/aks-engine that referenced this pull request Feb 12, 2020
…s cse telemetry (Azure#2663)"

This reverts commit 59c2ce0.
After this change e2e port forward tests started occasionally failing so
reverting to investigate.
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.

None yet

3 participants