Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Oct 16, 2025

Summary

This PR removes obsolete manual system event swagger specifications from the EventGrid data-plane directory as requested in the issue. The manual swaggers have been superseded by TypeSpec-generated specifications.

Changes Made

Deleted Services (24 directories)

Removed all Microsoft.* service directories from specification/eventgrid/data-plane/ except for Microsoft.EventGrid:

  • Microsoft.AVS, Microsoft.ApiCenter, Microsoft.ApiManagement, Microsoft.AppConfiguration
  • Microsoft.Cache, Microsoft.Communication, Microsoft.ContainerRegistry, Microsoft.ContainerService
  • Microsoft.DataBox, Microsoft.Devices, Microsoft.Edge, Microsoft.EventHub
  • Microsoft.HealthcareApis, Microsoft.KeyVault, Microsoft.MachineLearningServices, Microsoft.Maps
  • Microsoft.Media, Microsoft.PolicyInsights, Microsoft.ResourceNotifications, Microsoft.Resources
  • Microsoft.ServiceBus, Microsoft.SignalRService, Microsoft.Storage, Microsoft.Web

Deleted from Microsoft.EventGrid

  • stable/2018-01-01/SystemEvents.json - obsolete manual system events swagger
  • stable/2018-01-01/examples/event-grid-schema/ - system event examples
  • stable/2018-01-01/examples/cloud-events-schema/ - system event examples

Retained Files

  • GeneratedSystemEvents.json (2018-01-01 and 2024-01-01 versions) - TypeSpec-generated system events
  • EventGrid.json files (all versions: 2018-01-01, 2023-11-01, 2024-06-01) - REST operation swaggers
  • REST operation examples - PublishEventsToEventGridTopic*.json and operation examples for newer versions

Configuration Updates

Updated readme.md to remove references to deleted files:

  • Simplified package tags to only reference remaining EventGrid and GeneratedSystemEvents files
  • Updated all-api-versions section to list only retained files
  • Removed obsolete suppression rules for deleted ApiManagement files

Impact

  • 395 files changed: 6 insertions(+), 18,783 deletions(-)
  • All AutoRest configuration now points to valid files
  • No breaking changes to REST operation specifications
  • System events now exclusively use TypeSpec-generated schemas

Fixes the issue by removing all obsolete manual system event specifications while preserving the generated system events and all REST operation definitions.

Original prompt

This section details on the original issue you should resolve

<issue_title>Delete obsolete manual system event swaggers</issue_title>
<issue_description>Everything under azure-rest-api-specs/specification/eventgrid/data-plane at main · Azure/azure-rest-api-specs can be deleted except for Microsoft.EventGrid. Under that directory we could delete azure-rest-api-specs/specification/eventgrid/data-plane/Microsoft.EventGrid/stable/2018-01-01/Syste… and the examples only.

The GeneratedSystemEvents.json swagger should be retained as well as all swaggers that describe rest operations.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #38235


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link

github-actions bot commented Oct 16, 2025

Next Steps to Merge

⌛ Please wait. Next steps to merge this PR are being evaluated by automation. ⌛

Comment generated by summarize-checks workflow run.

Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Delete obsolete manual system event swaggers Delete obsolete manual system event swaggers from EventGrid data-plane Oct 16, 2025
Copilot finished work on behalf of JoshLove-msft October 16, 2025 18:32
@Copilot Copilot AI requested a review from JoshLove-msft October 16, 2025 18:32
@JoshLove-msft JoshLove-msft marked this pull request as ready for review October 16, 2025 18:42
@JoshLove-msft JoshLove-msft added the PublishToCustomers Acknowledgement the changes will be published to Azure customers. label Oct 16, 2025
@lmazuel lmazuel merged commit 4c5f20e into main Oct 17, 2025
107 of 119 checks passed
@lmazuel lmazuel deleted the copilot/delete-obsolete-event-swaggers branch October 17, 2025 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data-plane Event Grid PublishToCustomers Acknowledgement the changes will be published to Azure customers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Delete obsolete manual system event swaggers

4 participants