Skip to content

Commit 46d5cbc

Browse files
committed
Merge branch 'onesdk-phase2' into onesdk-avirupch
2 parents e9c97ac + f55efc9 commit 46d5cbc

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

src/ServiceManagement/StorSimple/Commands.StorSimple/ServiceClients/StorSimpleDevicesClient.cs

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,8 +189,8 @@ private void UpdateDeviceDetailsObject(DeviceDetails deviceDetails, string newNa
189189
}
190190
}
191191

192-
// There are a bunch of details that this cmdlet never edits and the service considers null
193-
// values there to mean that there have been no changes.
192+
// There are a bunch of details that this cmdlet never edits and the service
193+
// considers null values for them to mean that there have been no changes.
194194
deviceDetails.AlertNotification = null;
195195
deviceDetails.Chap = null;
196196
deviceDetails.RemoteMgmtSettingsInfo = null;
@@ -215,6 +215,18 @@ public void UpdateVirtualDeviceDetails(DeviceDetails details, string newName, Ti
215215
var encryptedSecretKey = this.EncryptWithDevicePublicKey(details.DeviceProperties.DeviceId, sek);
216216

217217
details.VirtualApplianceProperties.EncodedServiceEncryptionKey = encryptedSecretKey;
218+
219+
// mark everything that we dont intend to modify as null - indicating
220+
// to the service that there has been no change
221+
details.AlertNotification = null;
222+
details.Chap = null;
223+
details.DnsServer = null;
224+
details.NetInterfaceList = null;
225+
details.RemoteMgmtSettingsInfo = null;
226+
details.RemoteMinishellSecretInfo = null;
227+
details.SecretEncryptionCertThumbprint = null;
228+
details.Snapshot = null;
229+
details.WebProxy = null;
218230
}
219231
}
220232
}

0 commit comments

Comments
 (0)