test: 50 packages Weighted 10m/12m (DO NOT MERGE) - #58080
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR appears to be a “batching validation” / workload test PR that touches many packages by adding a marker comment (explicitly “DO NOT MERGE”).
Changes:
- Added a trailing “Test comment for batching validation - DO NOT MERGE” marker to many files across the SDK.
- Introduced pure-comment diffs across multiple packages to force broad repo churn.
Reviewed changes
Copilot reviewed 45 out of 50 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/websites/Azure.ResourceManager.AppService/src/Customization/AppService/Models/AppCertificatePatch.cs | Adds a trailing test-only marker comment. |
| sdk/trafficmanager/Azure.ResourceManager.TrafficManager/src/Customization/MockableTrafficManagerSubscriptionResource.cs | Adds a trailing test-only marker comment. |
| sdk/synapse/Azure.ResourceManager.Synapse/src/Customized/SynapseSqlPoolResource.cs | Adds a trailing test-only marker comment. |
| sdk/storage/Azure.Storage.Queues/src/AssemblyInfo.cs | Adds a trailing test-only marker comment. |
| sdk/storage/Azure.Storage.Files.DataLake/src/AssemblyInfo.cs | Adds a trailing test-only marker comment. |
| sdk/storage/Azure.Storage.Blobs/src/AppendBlobClient.cs | Adds a trailing test-only marker comment. |
| sdk/sqlmanagement/Azure.ResourceManager.Sql/src/Custom/ArmSqlModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/signalr/Azure.Provisioning.SignalR/src/Generated/SignalRBuiltInRole.cs | Adds a trailing test-only marker comment. |
| sdk/servicebus/Azure.Messaging.ServiceBus/src/Constants.cs | Adds a trailing test-only marker comment. |
| sdk/securitycenter/Azure.ResourceManager.SecurityCenter/src/Custom/SecurityAssessmentData.cs | Adds a trailing test-only marker comment. |
| sdk/search/Azure.Search.Documents/src/AzureSearchDocumentsEventSource.cs | Adds a trailing test-only marker comment. |
| sdk/relay/Azure.ResourceManager.Relay/src/Customized/RelayNamespaceData.cs | Adds a trailing test-only marker comment. |
| sdk/redis/Azure.Provisioning.Redis/src/Generated/RedisBuiltInRole.cs | Adds a trailing test-only marker comment. |
| sdk/recoveryservices-backup/Azure.ResourceManager.RecoveryServicesBackup/src/Custom/ArmRecoveryServicesBackupModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/provisioning/Azure.Provisioning.Deployment/src/AzCliBicepTool.cs | Adds a trailing test-only marker comment. |
| sdk/privatedns/Azure.ResourceManager.PrivateDns/src/Customization/PrivateDnsAaaaRecordCollection.cs | Adds a trailing test-only marker comment. |
| sdk/postgresql/Azure.ResourceManager.PostgreSql/src/Custom/PostgreSql/Custom/Models/PostgreSqlMinimalTlsVersionEnum.cs | Adds a trailing test-only marker comment. |
| sdk/operationalinsights/Azure.ResourceManager.OperationalInsights/src/Custom/ArmOperationalInsightsModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/network/Azure.ResourceManager.Network/src/Customization/ArmNetworkModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/netapp/Azure.ResourceManager.NetApp/src/Custom/ArmNetAppModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/mysql/Azure.ResourceManager.MySql/src/Custom/MySql/ArmMySqlModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/monitor/Azure.ResourceManager.Monitor/src/Customized/ArmMonitorModelFactory.cs | Adds a trailing test-only marker comment (also alters file ending formatting). |
| sdk/maps/Azure.Maps.Search/src/MapsSearchClient.cs | Adds a trailing test-only marker comment. |
| sdk/machinelearningservices/Azure.ResourceManager.MachineLearning/src/Customized/MachineLearningCodeVersionCollection.cs | Adds a trailing test-only marker comment. |
| sdk/logic/Azure.ResourceManager.Logic/src/Customized/LogicWorkflowResource.cs | Adds a trailing test-only marker comment. |
| sdk/kusto/Azure.Provisioning.Kusto/src/Generated/KustoAttachedDatabaseConfiguration.cs | Adds a trailing test-only marker comment. |
| sdk/keyvault/Azure.Security.KeyVault.Secrets/src/DeletedSecret.cs | Adds a trailing test-only marker comment. |
| sdk/keyvault/Azure.Security.KeyVault.Keys/src/CreateEcKeyOptions.cs | Adds a trailing test-only marker comment. |
| sdk/keyvault/Azure.Security.KeyVault.Certificates/src/AdministratorContact.cs | Adds a trailing test-only marker comment. |
| sdk/iothub/Azure.ResourceManager.IotHub/src/Customization/Models/IotHubCertificateProperties.cs | Adds a trailing test-only marker comment. |
| sdk/hdinsight/Azure.ResourceManager.HDInsight/src/Generated/Models/HDInsightClusterGatewaySettings.cs | Adds a trailing test-only marker comment. |
| sdk/frontdoor/Azure.ResourceManager.FrontDoor/src/Customized/FrontDoorExperimentResource.cs | Adds a trailing test-only marker comment. |
| sdk/eventhub/Azure.Messaging.EventHubs/src/EventData.cs | Adds a trailing test-only marker comment. |
| sdk/eventgrid/Azure.ResourceManager.EventGrid/src/Customized/Extensions/EventGridExtensions.cs | Adds a trailing test-only marker comment. |
| sdk/dns/Azure.ResourceManager.Dns/src/Customization/ArmDnsModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/devtestlabs/Azure.ResourceManager.DevTestLabs/src/Customization/DevTestLabArmTemplateData.cs | Adds a trailing test-only marker comment. |
| sdk/datafactory/Azure.ResourceManager.DataFactory/src/Customized/ArmDataFactoryModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/cosmosdb/Azure.ResourceManager.CosmosDB/src/Custom/RestorableCosmosDBAccountResource.cs | Adds a trailing test-only marker comment. |
| sdk/containerservice/Azure.ResourceManager.ContainerService/src/Custom/AgentPoolSnapshotData.cs | Adds a trailing test-only marker comment. |
| sdk/containerregistry/Azure.Provisioning.ContainerRegistry/src/Generated/ConnectedRegistry.cs | Adds a trailing test-only marker comment. |
| sdk/containerapps/Azure.ResourceManager.AppContainers/src/Custom/AppContainersSkuName.cs | Adds a trailing test-only marker comment. |
| sdk/compute/Azure.ResourceManager.Compute/src/Customize/ArmComputeModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/communication/Azure.Communication.Email/src/EmailClient.cs | Adds a trailing test-only marker comment. |
| sdk/communication/Azure.Communication.Chat/src/ChatClient.cs | Adds a trailing test-only marker comment. |
| sdk/cognitiveservices/Azure.ResourceManager.CognitiveServices/src/CognitiveServicesContext.cs | Adds a trailing test-only marker comment. |
| sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/ArmCdnModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/billing/Azure.ResourceManager.Billing/src/GlobalSuppressions.cs | Adds a trailing test-only marker comment. |
| sdk/batch/Azure.ResourceManager.Batch/src/Custom/ArmBatchModelFactory.cs | Adds a trailing test-only marker comment. |
| sdk/automation/Azure.ResourceManager.Automation/src/Customized/AutomationCertificateData.cs | Adds a trailing test-only marker comment. |
| sdk/apimanagement/Azure.ResourceManager.ApiManagement/src/Customized/ApiManagementProductResource.cs | Adds a trailing test-only marker comment. |
Comments suppressed due to low confidence (2)
sdk/websites/Azure.ResourceManager.AppService/src/Customization/AppService/Models/AppCertificatePatch.cs:1
- These test-only marker comments should not be checked into the main codebase; they create permanent churn/noise and don’t provide product value. Please remove this comment (and the identical ones added across the repo) before merge; if the goal is CI batching validation, prefer using a dedicated test branch, CI-only file(s), or benign metadata (e.g., pipeline variables) rather than modifying many source files.
sdk/storage/Azure.Storage.Queues/src/AssemblyInfo.cs:1 - The PR description mentions timing targets (Direct=10m, Indirect=12m), but the actual changes are only adding marker comments. Consider updating the PR description to explicitly state that the intent is to create broad, no-op diffs for batching validation (or link to the CI job/run plan), so reviewers and automation can correctly interpret why these files are being modified.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019fb458-7e27-73e2-90e8-6b15436827d2 Co-authored-by: Amp <amp@ampcode.com>
|
Hi m-nash. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days. |
461261f to
b941898
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5e0d1d6-f002-4bd1-9156-55ec649d89cf
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5e0d1d6-f002-4bd1-9156-55ec649d89cf
121736d to
eb71650
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 19db450c-9af7-474e-b3f1-640d1f968126
Direct=10m, Indirect=12m targets.