Skip to content
Merged
Show file tree
Hide file tree
Changes from 8 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