Skip to content

Comments

[WIP] [TEST] Fix this PR for me (ignore if you're a person)#41773

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/fix-41772
Closed

[WIP] [TEST] Fix this PR for me (ignore if you're a person)#41773
Copilot wants to merge 1 commit intomainfrom
copilot/fix-41772

Conversation

Copy link
Contributor

Copilot AI commented Jun 25, 2025

Fixed broken EventGrid SDK by addressing key issues from the system events regeneration:

Progress Report

  • Identify the main issues: The SDK regeneration incorrectly replaced all existing models with system events only
  • Fix main init.py: Restored proper client exports instead of incorrect __path__ extension
  • Fix tsp-location.yaml: Replaced placeholder values with proper commit and repo references
  • Restore missing ReleaseDelay enum: Added back the enum required by consumer client operations
  • Restore missing consumer client models: Added back essential models (CloudEvent, BrokerProperties, etc.)
  • Update model exports: Added all restored models to init.py imports and all lists
  • Verify imports: Confirmed EventGridPublisherClient, EventGridConsumerClient, and key models import successfully
  • Commit changes: Successfully committed all fixes to restore SDK functionality

What was Fixed

  1. Main init.py file: Was incorrectly changed to __path__ extension, now exports actual client classes
  2. Missing consumer client models: Restored CloudEvent, BrokerProperties, ReceiveDetails, PublishResult, etc.
  3. Missing ReleaseDelay enum: Added back for consumer client operations
  4. tsp-location.yaml: Fixed placeholder values with actual repo/commit references
  5. Import chain: Fixed all import errors in the dependency chain

Result

The EventGrid SDK now imports successfully and provides both:

  • Publisher Client: For sending events to Event Grid
  • Consumer Client: For receiving and managing events from Event Grid Namespaces
  • System Events Models: All the new system event models from the regeneration
  • Compatibility: Maintains backward compatibility with existing consumer client functionality

The PR is now ready for review and should pass all import validation checks.

Fixes #41772.


💡 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TEST] Fix this PR for me (ignore if you're a person)

2 participants