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

Fix nil linuxprofile #2621

Merged
merged 9 commits into from
Apr 10, 2018
Merged

Fix nil linuxprofile #2621

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
fc543e0
check if linuxprofile is nil
zqingqing1 Apr 6, 2018
bee38ce
add a test to check linuxprofile
zqingqing1 Apr 6, 2018
cde023d
add newline to json
zqingqing1 Apr 6, 2018
abd647f
Merge branch 'master' of https://github.com/Azure/acs-engine into fix…
zqingqing1 Apr 6, 2018
b7e1eee
assign empty when nil linuxprofile
zqingqing1 Apr 9, 2018
40fb4c2
assign ssh before convert
zqingqing1 Apr 9, 2018
ca3d4b7
remove the empty assignment
zqingqing1 Apr 9, 2018
09502b4
resorve dup code
zqingqing1 Apr 10, 2018
1e48478
trigger test
zqingqing1 Apr 10, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions pkg/api/agentPoolOnlyApi/v20180331/validate.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,12 @@ func (a *Properties) Validate() error {
}
}

if e := a.LinuxProfile.Validate(); e != nil {
return e
if a.LinuxProfile != nil {
if e := a.LinuxProfile.Validate(); e != nil {
return e
}
}

if e := validateVNET(a); e != nil {
return e
}
Expand Down