-
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
SqlServerDsc: Add a new pipeline job to test SQL Server 2019 family #1310
Comments
@johlju - How quickly can SQL2019 be added to the integration tests (presumably now in Azure Pipelines)? - I guess more people/companies will be using this now and this will only increase (I'm starting to install these more so would be good to know this was being tested on that version). My thoughts are that it would just be an update to the It also looks like there are also ISOs for SQL2016 and SQL2017 specified in config within the |
Additionally, are there any plans to setup integration tests on resources (where relevant) against Azure SQL Database instances? |
I updated the issue title. To get a new job to run in Azure Pipelines:
SqlServerDsc/azure-pipelines.yml Lines 183 to 249 in 5ab05bd
2, For each integration tests that should run for SQL2019 change the list of categories to include if (-not (Test-BuildCategory -Type 'Integration' -Category @('Integration_SQL2016','Integration_SQL2017', 'Integration_SQL2019')))
Yes, a public link to the evaluation or developer ISO media is needed. Preferably a a link to a Microsoft CDN which probably download quicker in Azure Pipelines (if it uses the same CDN), but otherwise AWS is always helpful in the quickstart-microsoft-sql project which has a link to AWS cloud: https://github.com/aws-quickstart/quickstart-microsoft-sql/blob/1c13ae4bc55fd5ac13e407724a5f3b802ddb23fd/ci/sql-master-ds-defaults.json#L36 |
The problem here is that we need access to a tenant where we can create the instances, but if we have that then it would be great to test that too.. But I happily review integration tests that can be run manually by a user to their own tenants. |
The exe can be download with this link: Then run with this to download the iso:
Full help from the .exe:
|
Great. Thanks for those. I might try and pick this up when PR #1652 is merged/closed. Finally, are there any concerns/considerations around build time/minutes/cost with having 3 SQL versions running/building in parallel? i.e. There's not a requirement to replace SQL2017 with SQL2019, for example? |
No there is no problem running more jobs. I think we can have 6-8 jobs in parallel and then it starts queuing. Every job can run up to 6 hours. It’s all at no cost for public repos. So no need to replace any older OS. Would be awesome to have all current (of a Microsoft) supported versions of SQL in the pipeline. But adding SQL 2019 is high priority as it is the one starting to be used more as you mentioned. |
- SqlServerDsc - Added unit tests and integration tests for SQL Server 2019 (issue #1310). - SqlSetup - Added/corrected `InstallSharedDir`, property output when using SQL Server 2019.
…not load SMO (#1684) - SqlServerDsc - Added a new script analyzer rule to verify that `Import-SQLPSModule` or `Connect-SQL` (that implicitly calls `Import-SQLPSModule`) is present in each `Get-`, `Test-`, and `Set-TargetResource` function. If neither command is not needed then the analyzer rule should be overridden ([issue #1683](#1683)). - Added a new pipeline job that runs Script Analyzer on all PowerShell scripts in the source folder. The rules are defined by the Script Analyzer settings file `.vscode\analyzersettings.psd1` (which also the Visual Studio Code PowerShell extension uses). - Added unit tests and integration tests for SQL Server 2019 ([issue #1310](#1310)). - Suppressed new custom Script Analyzer rule `SqlServerDsc.AnalyzerRules\Measure-CommandsNeededToLoadSMO` for `Get-`, `Test-`, and `Set-TargetResource` functions in the resources. - Minor lint errors throughout the repository. - SqlSetup - Minor refactor due to source code lint errors. The loop what evaluates the configuration parameters `*FailoverCluster` was change to a `foreach()`.
Details of the scenario you tried and the problem that is occurring
Suggest we add another job that can be run in the AppVeyor build worker to test the SQL Server 2019 family. For example SQL Server 2019, Reporting Services 2019, Power BI Reporting Services 2019. Or at least for the last two, maybe different builds than what are installed on the two existing jobs.
SQL Server 2019 is not yet available, but as soon as they are we should look into this.
Verbose logs showing the problem
n/a
Suggested solution to the issue
Add a new job in appveyor.yml and add the necessary changes to the SqlSetup, SqlRSSetup, and SqlPBIRSSetup (when available).
The DSC configuration that is used to reproduce the issue (as detailed as possible)
n/a
SQL Server edition and version the target node is running
n/a
SQL Server PowerShell modules present on the target node
n/a
The operating system the target node is running
n/a
Version and build of PowerShell the target node is running
n/a
Version of the DSC module that was used ('dev' if using current dev branch)
Dev
The text was updated successfully, but these errors were encountered: