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

fix: validate signed powershell packages in Azure and China Cloud#3579

Merged
jsturtevant merged 2 commits intoAzure:masterfrom
jsturtevant:fix/validate-provisioning-scripts
Jul 7, 2020
Merged

fix: validate signed powershell packages in Azure and China Cloud#3579
jsturtevant merged 2 commits intoAzure:masterfrom
jsturtevant:fix/validate-provisioning-scripts

Conversation

@jsturtevant
Copy link
Contributor

Reason for Change:

The pr #3441 added signing of the powershell scripts for China Cloud but the validation script was not update to reflect the change.

Issue Fixed:

Requirements:

Notes:

@acs-bot acs-bot added the size/M label Jul 7, 2020
@codecov
Copy link

codecov bot commented Jul 7, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3579   +/-   ##
=======================================
  Coverage   73.16%   73.16%           
=======================================
  Files         147      147           
  Lines       25097    25097           
=======================================
  Hits        18362    18362           
  Misses       5604     5604           
  Partials     1131     1131           

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 c2d3abd...c52d588. Read the comment docs.

@mboersma mboersma changed the title Validate signed powershell packages in Azure and China Cloud chore: validate signed powershell packages in Azure and China Cloud Jul 7, 2020
@jsturtevant jsturtevant changed the title chore: validate signed powershell packages in Azure and China Cloud fix: validate signed powershell packages in Azure and China Cloud Jul 7, 2020
@jsturtevant jsturtevant requested a review from ksubrmnn July 7, 2020 18:50
@acs-bot
Copy link

acs-bot commented Jul 7, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jsturtevant, ksubrmnn

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

The pull request process is described here

Details 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

@jsturtevant jsturtevant merged commit 76bda65 into Azure:master Jul 7, 2020
penggu pushed a commit to penggu/aks-engine that referenced this pull request Oct 28, 2020
…ure#3579)

* validate packages in Azure and China Cloud
* update docs
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.

3 participants