Skip to content

[DO NOT MERGE] CI measurement: 20pkg - Set B - #59401

Draft
m-nash wants to merge 13 commits into
mainfrom
test/loc-split-baseline-20pkg
Draft

[DO NOT MERGE] CI measurement: 20pkg - Set B#59401
m-nash wants to merge 13 commits into
mainfrom
test/loc-split-baseline-20pkg

Conversation

@m-nash

@m-nash m-nash commented May 21, 2026

Copy link
Copy Markdown
Member

Set B (baseline vs main) for measuring the standalone build/analyze LOC-split change.

  • Scale: 20pkg (20 packages touched)
  • Base: main
  • Purpose: measure CI build/analyze wall-clock and agent-min for a clean A/B against the LOC-split feature only (no other weighted-test-batching changes).

Companion to PR set in the other direction. Do not merge.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added Service Bus SQL Storage Storage Service (Queues, Blobs, Files) labels May 21, 2026
@m-nash
m-nash marked this pull request as draft May 21, 2026 22:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR is a CI measurement change-set (“Set B”) intended to touch 20 packages by introducing trivial, non-functional diffs, enabling A/B timing comparisons for the build/analyze LOC-split experiment.

Changes:

  • Appends a // Test comment for batching validation - DO NOT MERGE marker to the end of 20 source files across multiple SDK packages.
  • Introduces package-touch churn in both hand-authored and <auto-generated/> sources to force CI to process those packages.

Reviewed changes

Copilot reviewed 16 out of 20 changed files in this pull request and generated 20 comments.

Show a summary per file
File Description
sdk/storage/Azure.Storage.Blobs/src/AppendBlobClient.cs Adds a trailing test-only comment at EOF.
sdk/sqlmanagement/Azure.ResourceManager.Sql/src/Custom/ArmSqlModelFactory.cs Adds a trailing test-only comment at EOF.
sdk/signalr/Azure.Provisioning.SignalR/src/Generated/SignalRBuiltInRole.cs Adds a trailing comment to an <auto-generated/> file.
sdk/servicebus/Azure.Messaging.ServiceBus/src/Constants.cs Adds a trailing test-only comment at EOF.
sdk/securitycenter/Azure.ResourceManager.SecurityCenter/src/Custom/SecurityAssessmentData.cs Adds a trailing test-only comment at EOF.
sdk/search/Azure.Search.Documents/src/AzureSearchDocumentsEventSource.cs Adds a trailing test-only comment at EOF.
sdk/redis/Azure.Provisioning.Redis/src/Generated/RedisBuiltInRole.cs Adds a trailing comment to an <auto-generated/> file.
sdk/provisioning/Azure.Provisioning.Deployment/src/AzCliBicepTool.cs Adds a trailing test-only comment at EOF.
sdk/network/Azure.ResourceManager.Network/src/Customization/ArmNetworkModelFactory.cs Adds a trailing test-only comment at EOF.
sdk/monitor/Azure.ResourceManager.Monitor/src/Customized/ArmMonitorModelFactory.cs Adds a trailing test-only comment at EOF.
sdk/kusto/Azure.Provisioning.Kusto/src/Generated/KustoAttachedDatabaseConfiguration.cs Adds a trailing comment to an <auto-generated/> file.
sdk/keyvault/Azure.Security.KeyVault.Keys/src/CreateEcKeyOptions.cs Adds a trailing test-only comment at EOF.
sdk/eventhub/Azure.Messaging.EventHubs/src/EventData.cs Adds a trailing test-only comment at EOF.
sdk/cosmosdb/Azure.ResourceManager.CosmosDB/src/Custom/RestorableCosmosDBAccountResource.cs Adds a trailing test-only comment at EOF.
sdk/containerregistry/Azure.Provisioning.ContainerRegistry/src/Generated/ConnectedRegistry.cs Adds a trailing comment to an <auto-generated/> file.
sdk/compute/Azure.ResourceManager.Compute/src/Customize/ArmComputeModelFactory.cs Adds a trailing test-only comment at EOF.
sdk/communication/Azure.Communication.Email/src/EmailClient.cs Adds a trailing test-only comment at EOF.
sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/ArmCdnModelFactory.cs Adds a trailing test-only comment at EOF.
sdk/billing/Azure.ResourceManager.Billing/src/GlobalSuppressions.cs Adds a trailing test-only comment at EOF.
sdk/apimanagement/Azure.ResourceManager.ApiManagement/src/Customized/ApiManagementProductResource.cs Adds a trailing test-only comment at EOF.

Comment thread sdk/storage/Azure.Storage.Blobs/src/AppendBlobClient.cs Outdated
Comment on lines +115 to +116

// Test comment for batching validation - DO NOT MERGE
Comment on lines +114 to +115

// Test comment for batching validation - DO NOT MERGE
Comment on lines +61 to +62

// Test comment for batching validation - DO NOT MERGE
Comment on lines +354 to +355

// Test comment for batching validation - DO NOT MERGE
Comment thread sdk/communication/Azure.Communication.Email/src/EmailClient.cs Outdated
Comment thread sdk/cdn/Azure.ResourceManager.Cdn/src/Customization/ArmCdnModelFactory.cs Outdated
Comment thread sdk/billing/Azure.ResourceManager.Billing/src/GlobalSuppressions.cs
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
m-nash and others added 11 commits May 28, 2026 10:20
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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: 19db450c-9af7-474e-b3f1-640d1f968126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Communication Event Hubs KeyVault Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query Network - CDN Provisioning Search Service Bus SQL Storage Storage Service (Queues, Blobs, Files)

Projects

Status: Untriaged
Status: Untriaged

Development

Successfully merging this pull request may close these issues.

3 participants