Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Update-DbaInstance uses the same folder to extract the files #5447

Closed
garethnewman opened this issue May 1, 2019 · 4 comments · Fixed by #5450
Closed

[Bug] Update-DbaInstance uses the same folder to extract the files #5447

garethnewman opened this issue May 1, 2019 · 4 comments · Fixed by #5450
Assignees
Projects

Comments

@garethnewman
Copy link
Contributor

garethnewman commented May 1, 2019

dbatools 0.9.789
powershell 5.1.14409.1018

  • [ X] Verified errors are not related to permissions

When running against computers which can all see the cluster mounted volumes. It attempts to copy and extract the file to the same location causing issues such as files exist/can't cleanup.

It chooses the largest available disk for the temp location, in my case that is the same disk for more than 1 server as they can all see the cluster mounted volumes.

Being able to specify the temp location would be good, so I can just say put it in "C:\Temp" and then avoid this scenario, is this possible?

The actual function is Invoke-DbaAdvancedUpdate.ps1

"# Find a temporary folder to extract to - the drive that has most free space"

@potatoqualitee potatoqualitee added this to To do in 1.0 via automation May 1, 2019
@potatoqualitee
Copy link
Member

Thanks for the report! We'll take a look when we have a moment.

@nvarscar
Copy link
Contributor

nvarscar commented May 1, 2019

I'll look into it and either adopt your proposal or add some kind of UID to the temp folder name. Or both 🙂
Thanks for reporting!

@garethnewman
Copy link
Contributor Author

UID sounds good, thanks @nvarscar

1.0 automation moved this from To do to Done May 2, 2019
@garethnewman
Copy link
Contributor Author

Thanks @nvarscar - just installing more updates and noticed the folder name had a UID and was like hmm, he did it already 💃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
1.0
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants