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

test: set correct ssh private key for e2e get-logs #3824

Merged
merged 1 commit into from
Sep 14, 2020

Conversation

haofan-ms
Copy link
Contributor

@haofan-ms haofan-ms commented Sep 14, 2020

Reason for Change:

Issue Fixed:

Bug was introduced in #3747. When the uploaded ssh in e2e test was renamed to match the naming format of "${$RESOURCE_GRPUP}.ssh", the $PRIVATE_SSH_KEY_FILE no longer exist.

Requirements:

Notes:

@jadarsie jadarsie changed the title fix: Set correct ssh private key for e2e get-logs test: set correct ssh private key for e2e get-logs Sep 14, 2020
@codecov
Copy link

codecov bot commented Sep 14, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3824   +/-   ##
=======================================
  Coverage   73.19%   73.19%           
=======================================
  Files         148      148           
  Lines       25394    25394           
=======================================
  Hits        18587    18587           
  Misses       5671     5671           
  Partials     1136     1136           

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 615cb4f...5927def. Read the comment docs.

Copy link
Member

@jackfrancis jackfrancis 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 Sep 14, 2020
@jackfrancis jackfrancis merged commit 17f7698 into Azure:master Sep 14, 2020
@acs-bot
Copy link

acs-bot commented Sep 14, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: haofan-ms, jackfrancis, jadarsie

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

@haofan-ms haofan-ms deleted the fix-for-azs-changes branch October 9, 2020 17:07
penggu pushed a commit to penggu/aks-engine that referenced this pull request Oct 28, 2020
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

4 participants