Skip to content

Add online disk check / repair script (powershell)#141

Merged
silversword411 merged 3 commits intoamidaware:mainfrom
lcsnetworks:disk_check_repair
Apr 17, 2023
Merged

Add online disk check / repair script (powershell)#141
silversword411 merged 3 commits intoamidaware:mainfrom
lcsnetworks:disk_check_repair

Conversation

@joeldeteves
Copy link
Copy Markdown
Contributor

This PR adds an online disk repair using PowerShell. By default, it runs a simple online check for volume errors. Optionally, you can include the spotfix parameter to attempt to repair the volume while it is mounted, or you can schedule an offline scan on the next reboot.

Comment thread scripts/Win_Disk_Check_Repair.ps1 Outdated
Copy link
Copy Markdown
Contributor

@silversword411 silversword411 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

chatGPT comments need to be tweaked to not include the ps1 name ;)

You tested this yet?

Comment thread scripts/Win_Disk_Check_Repair.ps1 Outdated
@joeldeteves
Copy link
Copy Markdown
Contributor Author

joeldeteves commented Apr 17, 2023

Yes, I did test it and I also made tweaks to it prior to finalizing. I have always used write-host in my TRMM scripts. Wasn't aware there was a difference since it runs in the background as the trmm user.

I can correct this as long as it doesn't break the output in TRMM (obv we want to see the output results).

Re: script name I wasn't aware the script name needs to be removed. Sure I can update that.

@silversword411
Copy link
Copy Markdown
Contributor

doesn't break the output in TRMM (obv we want to see the output results).

I never see output with Write-Host, only Write-Output not sure how it's been working for you.

@joeldeteves
Copy link
Copy Markdown
Contributor Author

@silversword411 either way I addressed it - should be fixed now

@joeldeteves joeldeteves reopened this Apr 17, 2023
@silversword411 silversword411 merged commit 242531b into amidaware:main Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants