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

feat: Add IsCredentialAutoGenerated for WindowsProfile #2804

Merged
merged 1 commit into from
Mar 2, 2020

Conversation

AbelHu
Copy link
Member

@AbelHu AbelHu commented Feb 28, 2020

Reason for Change:

AKS plans to auto generate Windows username and password if Windows agent pools can be added to this cluster later. We need to distinguish that the Windows credential is set by users from that the Windows credential is auto generated.

Issue Fixed:

Requirements:

Notes:

@mikkelhegn mikkelhegn requested a review from marosset February 28, 2020 11:30
@marosset
Copy link
Contributor

/azp run pr-e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov
Copy link

codecov bot commented Feb 28, 2020

Codecov Report

Merging #2804 into master will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2804      +/-   ##
==========================================
+ Coverage   72.35%   72.43%   +0.08%     
==========================================
  Files         137      140       +3     
  Lines       25339    25552     +213     
==========================================
+ Hits        18333    18508     +175     
- Misses       5949     5976      +27     
- Partials     1057     1068      +11     
Impacted Files Coverage Δ
pkg/armhelpers/azurestack/httpMockClient.go 61.46% <0.00%> (-0.25%) ⬇️
pkg/armhelpers/mockclients.go 22.61% <0.00%> (-0.14%) ⬇️
pkg/armhelpers/httpMockClient.go 58.71% <0.00%> (-0.13%) ⬇️
pkg/helpers/azure_locations.go 100.00% <0.00%> (ø) ⬆️
pkg/armhelpers/azurestack/subscriptions.go 60.00% <0.00%> (ø)
pkg/armhelpers/subscriptions.go 60.00% <0.00%> (ø)
cmd/get_locations.go 87.34% <0.00%> (ø)
pkg/api/addons.go 97.61% <0.00%> (+0.01%) ⬆️
pkg/armhelpers/azurestack/azureclient.go 29.46% <0.00%> (+0.64%) ⬆️
pkg/armhelpers/azureclient.go 35.27% <0.00%> (+0.69%) ⬆️
... and 1 more

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 3dd077c...7c0846d. Read the comment docs.

@marosset
Copy link
Contributor

marosset commented Mar 2, 2020

/azp run pr-e2e

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants