Skip to content

Commit 76dfd03

Browse files
committed
Merge pull request #478 from dscsolutions/bugfixes
DSC Extension cmdlets bug fixes
2 parents 0513a42 + a1caa5e commit 76dfd03

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

src/ServiceManagement/Compute/Commands.ServiceManagement/IaaS/Extensions/DSC/PublishAzureVMDscConfiguration.cs

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -262,6 +262,12 @@ private string CreateConfigurationArchive()
262262
null));
263263
}
264264
List<string> requiredModules = parseResult.RequiredModules;
265+
//Since LCM always uses the latest module there is no need to copy PSDesiredStateConfiguration
266+
if (requiredModules.Contains("PSDesiredStateConfiguration"))
267+
{
268+
requiredModules.Remove("PSDesiredStateConfiguration");
269+
}
270+
265271
WriteVerbose(String.Format(CultureInfo.CurrentUICulture, Resources.PublishVMDscExtensionRequiredModulesVerbose, String.Join(", ", requiredModules)));
266272

267273
// Create a temporary directory for uploaded zip file
@@ -290,8 +296,8 @@ private string CreateConfigurationArchive()
290296
@"function Copy-Module([string]$module, [string]$tempZipFolder)
291297
{
292298
$mi = Get-Module -List -Name $module;
293-
$moduleFolder = Split-Path -Parent $mi.Path;
294-
Copy-Item -Recurse -Path $moduleFolder -Destination $tempZipFolder;
299+
$moduleFolder = Split-Path $mi.Path;
300+
Copy-Item -Recurse -Path $moduleFolder -Destination ""$tempZipFolder\$($mi.Name)""
295301
}"
296302
);
297303
powershell.Invoke();
@@ -329,7 +335,11 @@ private string CreateConfigurationArchive()
329335
}
330336
else
331337
{
332-
archive = Path.Combine(Path.GetTempPath(), configurationName + ZipFileExtension);
338+
string tempArchiveFolder = Path.Combine(Path.GetTempPath(), Guid.NewGuid().ToString());
339+
WriteVerbose(String.Format(CultureInfo.CurrentUICulture, Resources.PublishVMDscExtensionTempFolderVerbose, tempArchiveFolder));
340+
Directory.CreateDirectory(tempArchiveFolder);
341+
this._temporaryDirectoriesToDelete.Add(tempArchiveFolder);
342+
archive = Path.Combine(tempArchiveFolder, configurationName + ZipFileExtension);
333343
this._temporaryFilesToDelete.Add(archive);
334344
}
335345

0 commit comments

Comments
 (0)