-
Notifications
You must be signed in to change notification settings - Fork 225
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
Minor version upgrade of already installed SQL Server instance #2053
Comments
Look into |
It's exactly what I meant. Is there an open issue for that? |
It is now. 🙂 Happy to review a PR for this if someone sends one in. |
What happens when passing Is it possible to use |
Nothing.
Sounds good! That's how I expected it to work |
Hopefully someone wants to PR that, otherwise it won't happen. 🙂 |
I wouldn't mind taking a jab at it, but I have some questions.
|
I hope this helps. Great if you want to run with this.
Suggest adding public commands that
Those public commands together with All of this should be tested using
Please look at https://dsccommunity.org/guidelines/getting-started/ and then https://dsccommunity.org/guidelines/contributing/#create-or-update-a-pull-request-pr To test it you would probably move the built module from the dev machine to a lab machine in machine-wide path of PSMoudulePath where you test it easiest using
We should not have any versioninformation that needs to be maintained in this module. That is input parameters to the DSC resource and subsequently to the public commands. But I don't see the need to use versions as input parameters. Input parameters should probably specify the path to where the updated package is. Version is determined by the package and installed SQL Server instance. Unless you find another clever way. 🙂 |
Thanks! This helped me a lot. I will hopefully attempt doing that on Sunday. I have contributed once to dbatools in the past so I'm fairly confident in my abilities. |
Hi, I started working on this and I didn't quite get what you meant by
|
Is it possible to install a CU with sqlsetup?
I have been unable to do that
The text was updated successfully, but these errors were encountered: