Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .vscode/analyzersettings.psd1
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,13 @@
'AvoidProcessWithoutPipeline'
'AvoidSmartQuotes'
'AvoidThrowOutsideOfTry'
'AvoidWriteErrorStop'
<#
'AvoidWriteErrorStop' rule is disabled because it conflicts with
the use of 'Write-Error -ErrorAction Stop' pattern used in the module.
There are edge case issues with $PSCmdlet.ThrowTerminatingError that
being investigated in Pull Request https://github.com/dsccommunity/SqlServerDsc/pull/2364.
#>
#'AvoidWriteErrorStop'
'AvoidWriteOutput'
'UseSyntacticallyCorrectExamples'
)
Expand Down
15 changes: 11 additions & 4 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
Services configuration instances with consistent error handling. This function
is used by `Enable-SqlDscRsSecureConnection`, `Disable-SqlDscRsSecureConnection`,
and the `SqlRS` resource.
- Added private function `Get-HResultMessage` to translate common Windows HRESULT
error codes into human-readable messages. Used by `Invoke-RsCimMethod` to
provide actionable error messages when Reporting Services CIM methods fail
without detailed error information.
- Added parameters `RetryCount`, `RetryDelaySeconds`, and `SkipRetry` to provide
configurable retry behavior for transient CIM method failures. By default,
retries up to 3 times with 30-second delays. Handles both HRESULT failures
and exceptions from `Invoke-CimMethod`. Collects unique errors across retry
attempts with attempt number prefixes for comprehensive error reporting.
- `Invoke-ReportServerSetupAction`
- Now uses `Format-Path` with `-ExpandEnvironmentVariable` to expand environment
variables in all path parameters (`MediaPath`, `LogPath`, `InstallFolder`)
Expand Down Expand Up @@ -248,6 +249,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed

- SqlServerDsc
- Consolidated Reporting Services post-service-account-change integration tests
into three version-specific test files: `Post.ServiceAccountChange.SQL2017.RS`,
`Post.ServiceAccountChange.SQL2019-2022.RS`, and `Post.ServiceAccountChange.PowerBI.RS`.
SQL Server 2017 uses a workaround with `Remove-SqlDscRSEncryptedInformation`
and `Set-SqlDscRSDatabaseConnection` because the encryption key commands
fail with "Keyset does not exist" errors on SQL Server 2017.
- Split the `Test_HQRM` pipeline job into two parallel jobs (`Test_QA` and
`Test_HQRM`) to reduce overall pipeline execution time by approximately
15 minutes.
Expand Down
21 changes: 7 additions & 14 deletions azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -589,17 +589,13 @@ stages:
# Group 6 - Service account change
'tests/Integration/Commands/Set-SqlDscRSServiceAccount.Integration.Tests.ps1'
'tests/Integration/Commands/Get-SqlDscRSServiceAccount.Integration.Tests.ps1'
'tests/Integration/Commands/Post.DatabaseRights.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.EncryptedInformation.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.DatabaseConnection.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/New-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/Post.UrlReservationRecreate.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.Reinitialize.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.ServiceAccountChange.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.ServiceAccountChange.SQL2017.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.ServiceAccountChange.SQL2019-2022.RS.Integration.Tests.ps1'
# Group 8
'tests/Integration/Commands/Repair-SqlDscReportingService.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSUrlReservation.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/New-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSEncryptedInformation.Integration.Tests.ps1'
# Group 9
'tests/Integration/Commands/Uninstall-SqlDscReportingService.Integration.Tests.ps1'
Expand Down Expand Up @@ -690,15 +686,12 @@ stages:
# Group 6 - Service account change
'tests/Integration/Commands/Set-SqlDscRSServiceAccount.Integration.Tests.ps1'
'tests/Integration/Commands/Get-SqlDscRSServiceAccount.Integration.Tests.ps1'
'tests/Integration/Commands/Post.DatabaseRights.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/New-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/Post.UrlReservationRecreate.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.Reinitialize.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.ServiceAccountChange.RS.Integration.Tests.ps1'
'tests/Integration/Commands/Post.ServiceAccountChange.PowerBI.RS.Integration.Tests.ps1'
# Group 8
'tests/Integration/Commands/Repair-SqlDscPowerBIReportServer.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSUrlReservation.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/New-SqlDscRSEncryptionKey.Integration.Tests.ps1'
'tests/Integration/Commands/Remove-SqlDscRSEncryptedInformation.Integration.Tests.ps1'
# Group 9
'tests/Integration/Commands/Uninstall-SqlDscPowerBIReportServer.Integration.Tests.ps1'
Expand Down
102 changes: 0 additions & 102 deletions source/Private/Get-HResultMessage.ps1

This file was deleted.

Loading
Loading