- 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.
- Add 14 Azure related rules and probes
- Change 3 severity levels to four: High, Medium, Low, Information
- Fix and improve some rules
- Improve messages and descriptions
The servuce endpoints are required for SQLMI to access storage account for ashok backup /restore , replication , external tables etc.
The code which I am asking to delete off is actually deleting such endpoints at the VNET.
Pieter
dependabot[bot]
Josh G
Marek Šuppa
Christopher Pearson
Hiroshi Yoshioka
Jordan Hays
Umachandar Jayachandran
Shanyu Zhao
Renat Bekbolatov
Ebru Ersan
TDzakhov
Henrik Huhtinen
Davis Mosenkovs
Warren Zhu
Aleksei Guzev
Melony QIN
cloudmelon
LuisSoaresMSFT
sehgalnamit
Daniel Coelho
Daniel Coelho
Jim Dale
Pedro Lopes