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

BREAKING CHANGE: SqlRS: No need to run the method InitializeReportServer for any versions of SSRS #1721

Closed
johlju opened this issue Sep 1, 2021 · 0 comments · Fixed by #1550
Labels
breaking change When used on an issue, the issue has been determined to be a breaking change. enhancement The issue is an enhancement request.

Comments

@johlju
Copy link
Member

johlju commented Sep 1, 2021

Problem description

According to #1698 (comment) running the method InitializeReportServer might not be necessary.

$invokeRsCimMethodParameters = @{
CimInstance = $reportingServicesData.Configuration
MethodName = 'InitializeReportServer'
Arguments = @{
InstallationId = $reportingServicesData.Configuration.InstallationID
}
}
Invoke-RsCimMethod @invokeRsCimMethodParameters

Though according to the documentation ConfigurationSetting Method - InitializeReportServer the method InitializeReportServer do serve some purpose, but unclear when it must be run.

Verbose logs

Not available

DSC configuration

See integration tests for SqlRS in this repo.

Suggested solution

The following code should be run for all versions on SQL Server Reporting Services.

<#
When initializing SSRS 2019, the call to InitializeReportServer
always fails, even if IsInitialized flag is $false.
It also seems that simply restarting SSRS at this point initializes
it.
We will ignore $SuppressRestart here.
#>
if ($reportingServicesData.SqlVersion -ge 15)
{
Write-Verbose -Message $script:localizedData.RestartToFinishInitialization
Restart-ReportingServicesService -InstanceName $InstanceName -WaitTime 30
$restartReportingService = $false
}

SQL Server edition and version

Any

SQL Server PowerShell modules

Not applicable

Operating system

Any

PowerShell version

Not applicable

SqlServerDsc version

Version in main branch.
@johlju johlju added breaking change When used on an issue, the issue has been determined to be a breaking change. enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community. labels Sep 1, 2021
johlju added a commit to johlju/SqlServerDsc that referenced this issue Apr 14, 2022
@johlju johlju added in progress The issue is being actively worked on by someone. and removed help wanted The issue is up for grabs for anyone in the community. labels Apr 14, 2022
johlju added a commit to johlju/SqlServerDsc that referenced this issue Apr 28, 2022
johlju added a commit to johlju/SqlServerDsc that referenced this issue Jun 6, 2022
johlju added a commit to johlju/SqlServerDsc that referenced this issue Jun 6, 2022
@johlju johlju removed the in progress The issue is being actively worked on by someone. label Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change When used on an issue, the issue has been determined to be a breaking change. enhancement The issue is an enhancement request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant