Hello everybody,
In the CU15 (15.0.4198.2), the mssql-conf tool was rewrited. With this sqlserver's update and the old version of mssql_conf, we get an error "usermod command not found" during the checkinstance.sh's execution. This fix adds a merge operation of setup_env dictionary with the MSSQL_SA_PASSWORD and MSSQL_PID variables
- Add a link for the ledger subfolder in sql-server-samples/README.md at master · Pietervanhove/sql-server-samples (github.com)
- Change the folder name from “sqlledger” to “ledger” (that’s the official feature name now).
- sql-server-samples/README.md at master · Pietervanhove/sql-server-samples (github.com)
o Fix grammar in “The demo in this folder showcases the ledger feature in Azure SQL Database. The demo uses the Contoso HR web application.”
o Are Az PS 5.6, SqlServer 21.1.18245, Bicep 0.4.63, SSMS 18.9.1 the versions you’ve tested with? These are older versions (the same we used when developing the AE demo). Should we recommend using newer versions?
o “The ContosoHR database using General Purpose compute tier, 2 vCores, and the Gen5 hardware generation.”
I’m wondering if there is a better/more economical SLO/SKU we can pick, instead of the 2-core Gen 5 database.
Of course, Basic/Standard would be very cheap, but we probably should promote the new v-core model.
Would serverless be a better option, e.g. “GP_S_Gen5_1”?
o If we change the folder name, change sqlledger to ledger in cd "C:\sql-server-samples\samples\features\security\sql-ledger\azure-sql-database\setup"
o “Open a new InPrivate browser window.” Are these instructions generic enough to accommodate browsers other than Edge?
- sql-server-samples/samples/features/security/contoso-hr-sql-db at master · Pietervanhove/sql-server-samples (github.com)
o This is unrelated to ledger. Could you also remove this folder? It was checked in by mistake when we added the AE demo. It is redundant/not needed.
CSS had a support ticket in which the customer had a password field provided by the data-driven subscription dataset query. The current GitHub script fails because it is trying to add a second instance of the PASSWORD parameter. The failure is:
Migrating data-driven subscription Monthly run of DATA0006116 (DataPortal) ... FAILURE:
System.Web.Services.Protocols.SoapException: An internal error occurred on the report server. See the error log for more details. ---> Microsoft.ReportingServices.Diagnostics.Utilities.InternalCatalogException: An internal error occurred on the report server. See the error log for more details. ---> System.ArgumentException: Item has already been added. Key in dictionary: 'PASSWORD' Key being added: 'PASSWORD'
at Microsoft.ReportingServices.Library.ReportingService2005Impl.CreateSubscription(String Report, ExtensionSettings ExtensionSettings
The updated code will check if the PASSWORD parameter already exists. If it does, then it will not add the PASSWORD parameter. The customer tested the updated script code and it successfully migrated all of his RS content to the new RS server.
The help link was pointing to wrong Microsoft Doc. I was unable to figure out how to find the LinkID(go.microsoft.com fwlink/ linkid=) that would be associated with the correct page so I use the one I knew.
dependabot[bot]
Perry Skountrianos
Umachandar Jayachandran
dblugeon
Pieter
Pedro Lopes
Josh G
Marek Šuppa
Christopher Pearson
Jordan Hays
Umachandar Jayachandran
Shanyu Zhao
Renat Bekbolatov
Henrik Huhtinen
Davis Mosenkovs