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

IndexOptimize command should exclude rdsadmin database when installed… #510

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lboozer
Copy link

@lboozer lboozer commented Apr 22, 2021

IndexOptimize command should exclude rdsadmin database when installed on RDS.

Sample of generated job step with change:

JobStep

@bojanderson
Copy link

bojanderson commented Oct 11, 2022

@lboozer beat me to this.

I know this PR is from 4/2021 but it would be nice if this PR got merged. As of October 11th 2022, this issue still persists, and this small change would fix things nicely.

My initial setup of these scripts and running of the job fails because this change is needed.

@darrenw-npi
Copy link

I know this PR is from 4/2021 but it would be nice if this PR got merged. As of October 11th 2022, this issue still persists, and this small change would fix things nicely.

Indeed! Sadly it seems likely Ola is the only one with write access to this repo. Hopefully he will find some time to merge this in. It's a trivial change so I guess if someone is desperate they could make the change manually or fork the repo. The https://github.com/lboozer/sql-server-maintenance-solution repo is a bit behind the main repo now otherwise I would use that.

@fill-e
Copy link

fill-e commented Sep 9, 2024

Lboozer are you still using this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants