Skip to content

Revised Get-SQLDefaultPaths function#1

Merged
potatoqualitee merged 1 commit intodataplat:masterfrom
mrbru2:mrbru2-update_defaultpaths-1
Jun 9, 2015
Merged

Revised Get-SQLDefaultPaths function#1
potatoqualitee merged 1 commit intodataplat:masterfrom
mrbru2:mrbru2-update_defaultpaths-1

Conversation

@dougjjj
Copy link

@dougjjj dougjjj commented Jun 8, 2015

I had to remove the split-path($filepath), on systems with mount points and shallow directory structures it was reverting to folders which database engine lacked permissions (for instance if the data directory is d:\data it would change to d:\ and would fail due to lack of permissions). Also revised the switch which was replacing "data" with "mdf" and "log" with "ldf" and then bypassing the conditional test.

I had to remove the split-path($filepath), on systems with mount points and shallow directory structures it was reverting to folders which database engine lacked permissions (for instance if the data directory is d:\data it would change to d:\ and would fail due to lack of permissions).  Also revised the switch which was replacing "data" with "mdf" and "log" with "ldf" and then bypassing the conditional test.
@potatoqualitee
Copy link
Member

Woo! Thanks for the fix.

potatoqualitee pushed a commit that referenced this pull request Jun 9, 2015
Revised Get-SQLDefaultPaths function
@potatoqualitee potatoqualitee merged commit 90878aa into dataplat:master Jun 9, 2015
potatoqualitee pushed a commit that referenced this pull request Mar 19, 2016
…igration

Update Copy-SqlCentralManagementServer.ps1
potatoqualitee pushed a commit that referenced this pull request Jul 18, 2016
potatoqualitee pushed a commit that referenced this pull request Jul 19, 2016
dansqldba pushed a commit that referenced this pull request Aug 10, 2016
Development-2016-08-08-2033
dansqldba pushed a commit that referenced this pull request Oct 13, 2016
Development - refresh 2016-08-16
potatoqualitee pushed a commit that referenced this pull request Jan 28, 2017
Test sqlconnection port with colon, and others
potatoqualitee pushed a commit that referenced this pull request Feb 12, 2017
potatoqualitee pushed a commit that referenced this pull request Feb 13, 2017
potatoqualitee pushed a commit that referenced this pull request Jun 7, 2017
potatoqualitee pushed a commit that referenced this pull request Jun 30, 2017
FriedrichWeinmann pushed a commit that referenced this pull request Aug 2, 2017
potatoqualitee pushed a commit that referenced this pull request Sep 18, 2017
potatoqualitee pushed a commit that referenced this pull request Sep 28, 2017
…ptions

Pass SqlBulkCopyOptions to SqlBulkCopy
potatoqualitee pushed a commit that referenced this pull request Oct 3, 2017
niphlod added a commit to niphlod/dbatools that referenced this pull request Oct 23, 2017
@niphlod niphlod mentioned this pull request Oct 23, 2017
9 tasks
potatoqualitee pushed a commit that referenced this pull request Oct 30, 2017
#2116: Copy-DbaServerAudit: Test Current Audit Filepath for Null
potatoqualitee pushed a commit that referenced this pull request Dec 6, 2017
potatoqualitee pushed a commit that referenced this pull request Jan 20, 2018
potatoqualitee pushed a commit that referenced this pull request Jul 12, 2018
potatoqualitee pushed a commit that referenced this pull request Sep 7, 2018
add ability to only pull one proxy
potatoqualitee pushed a commit that referenced this pull request Oct 15, 2018
Fix bug with $sourceConnection variable
potatoqualitee pushed a commit that referenced this pull request Oct 29, 2018
potatoqualitee pushed a commit that referenced this pull request Oct 31, 2018
potatoqualitee pushed a commit that referenced this pull request Nov 9, 2018
* rdd - little cleaning

* fixing progress #1

* Update to progress
DorBreger pushed a commit to DorBreger/dbatools that referenced this pull request May 15, 2024
added an extra check for major version to be consistent with source
@claude claude bot mentioned this pull request Jan 23, 2026
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