-
-
Notifications
You must be signed in to change notification settings - Fork 808
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
Add replication support to dbatools! #8958
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @wsmelton thanks for going through this - I've addressed all your concerns (I think) except the naming of these four commands:
- Enable-DbaReplDistributor
- Disable-DbaReplDistributor
- Enable-DbaReplPublishing
- Disable-DbaReplPublishing
Also I can only apologise for the massive PR - it was hard to break into chunks since so many of the commands work together.
function Disable-DbaReplDistributor { | ||
<# | ||
.SYNOPSIS | ||
Disables replication distribution for the target SQL instances. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah makes sense - will wait for @potatoqualitee steer on this one.
public/Enable-DbaReplPublishing.ps1
Outdated
process { | ||
foreach ($instance in $SqlInstance) { | ||
|
||
$replServer = Get-DbaReplServer -SqlInstance $instance -SqlCredential $SqlCredential |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have gone through and made sure -EnableException:$EnableException
is included on all public functions - thanks
failed tests seem to be unrelated |
Login to appveyor using your GH creds and see if you can do the rerun incomplete button |
Hey @potatoqualitee I'd say enable\disable - that's the wording around replication in the SQL Server world. For appveyor I got logged in but don't see an option to rerun jobs 🤔 |
Hey @wsmelton \ @potatoqualitee - we happy to stick with enable\disable since it matches the experience and the terminology of replication, while still using approved verbs? I have a presentation next week so would love to be able to say it's in the dev branch ready for the next release if possible. :) |
YOOOO lez do this! Congrats!! <3 |
Thank you both so very much, what a huge amount of work. Looking forward to promoting this on socials 💯 |
Please read -- recent changes to our repo
On November 10, 2022, we removed some bloat from our repository (for the second and final time). This change requires that all contributors reclone or refork their repo.
PRs from repos that have not been recently reforked or recloned will be closed and @potatoqualitee will cherry-pick your commits and open a new PR with your changes.
Type of Change
.\tests\manual.pester.ps1
)Purpose
Adding replication support to dbatools!
Approach
There are many commands added with this PR to cover most of the replication functionality we can add - there is still work to do on these commands, more options and settings that can be added - and more code paths. However, they are all working and at this point I want to create this PR and start moving them towards the dbatools dev branch so we can start getting feedback.
I realise I've included the psd1 in this PR - we can remove it before we merge in - just makes testing this branch easier
Commands to test - New commands
Distribution
Publishing
Publications
Articles
Subscriptions
Commands to test - Existing commands that have been spruced up
Commands that still need some love
Testing
The following script takes you through building and destroying replication: