[@azure/eventgrid] - Azure EventGrid Beta Release V4.13.0-beta.1 #25906
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sarangan12
changed the title
sarangan12
requested review from
ckairen,
deyaaeldeen,
jeremymeng,
mikeharder,
bterlson,
xirzec and
ellismg
as code owners
ellismg
reviewed
May 18, 2023
joheredi
reviewed
May 19, 2023
joheredi
reviewed
May 19, 2023
| @@ -446,6 +475,7 @@ export interface CloudEvent<T> { | |||
| extensionAttributes?: Record<string, unknown>; | |||
| id: string; | |||
| source: string; | |||
| specversion?: string | "1.0"; | |||
There was a problem hiding this comment.
any chance to name this specVersion?
There was a problem hiding this comment.
But it is actually specversion in tsp file
joheredi
approved these changes
May 19, 2023
ellismg
approved these changes
May 20, 2023
azure-sdk
pushed a commit
to azure-sdk/azure-sdk-for-js
that referenced
this pull request
Oct 4, 2023
Release machinelearningservices microsoft.machine learning services 2023 08 01 preview (Azure#24762) * Adds base for updating Microsoft.MachineLearningServices from version preview/2023-06-01-preview to version 2023-08-01-preview * Updates readme * Updates API version in new specs and examples * MFE 2023-08-01-preview mfe.json (Azure#24764) * add mfe.json * labeling jobs update descriptions * add examples for ServerlessEndpoint * update labelingjob examples * label job examples remove additional properties * adding sku to patch * add identity to patch * add in operationresult to examples --------- Co-authored-by: Kayla Ames <[email protected]> * Adding Resources at sweep level (Azure#25090) Co-authored-by: Anubha98 <[email protected]> * HoboVM Preview Features (Azure#24769) * Update Pricesheet properties for savingsPlan (Azure#24708) * Update consumption.json * Update PriceSheet.json * Update consumption.json * Update PriceSheet.json * Update consumption.json * Update PriceSheet.json * Update consumption.json * Update consumption.json * Update consumption.json * Update PriceSheet.json * elasticsan preview refresh microsoft.elasticsan 2022-12-01-preview (Azure#24494) * Adds base for updating Microsoft.ElasticSan from version preview/2021-11-20-preview to version 2022-12-01-preview * Updates readme * Updates API version in new specs and examples * Added PE related changes * Added Private Link resource API * Nit change to correct the path for privateLinkResource API * Updated Lint errors * prettier check * resolved comments & added publicNetworkAccess * removed publicNetworkAccess & NIT changes * Updated the list operations * resolved lintDiff * proper value for examples and removed value from PECollections * resolved comments * spell check resolved * removing volumeCreateParameter * [Hub Generated] Publish private branch 'InbalZim-storagemover-Microsoft.StorageMover-RPSaaSMaster-2023-07-01-preview' (Azure#24390) * add or modify files * Updated `credentials` models according to PR Azure#24318 + reverted unwanted changes to `readme.md` * Removed URI format from SMB credentials to support it receiving empty string in order to clean-up their value * Removed extra hierarchy of `AzureStorageFileShareEndpointBaseProperties` * improve NR Examples (Azure#24636) * improve NR Examples * addressing AutoRun Errors * Fixing Prettier Issue * Adding NextLink in Max Examples * Formatting Azure Resource Id * Updating Format of azure subscriptionIds * update code owner (Azure#24754) * Update readme.python.md (Azure#24758) * [TypeSpecValidation] Refactor package.json (Azure#24756) - Improves upon Azure#24745 - Creates eng/tools/package.json for all EngSys tools - Moves "postinstall" to eng/tools/TypeSpecValidation/package.json - Technically incorrect if the package were ever to be published to a registry - In practice, the best way I have found so far to use source-based tools in a TypeScript repo * Add resize APIs * workflow-bot comments: update breaking changes alternative C to point to alternative B (Azure#24770) * Add properties for os patching (Azure#24775) Co-authored-by: Chris Jiang <[email protected]> * [Communication] Job Router Public Preview v2022-07-18 (Azure#23804) * Updated JR swagger with maxNumberOfJobs * job scheduling swagger changes * Enforce DistributionMode.kind to be read-only * retry with removing Required tag from abstract property for DistributionMode * WorkerState stopped showing up in generated swagger * Add reofferTimeUtc to decline offer API * Prettier * Prettier * rename WorkerState to RouterWorkerState * more fixes in polymorphic types * Rename decline offer request * prettier fix * add example for scheduled job * cpsell fix * fix response schema validation error * address review feedback item round 1 * SDK review changes * More API changes * Update distribution mode required properties * Renamings * job matching mode added * prettier fixes * fix examples refs * update ref examples * update swagger spec * fix enum mismatch * change operator to labelOperator in classification policy get and update ops example * Update specification/communication/data-plane/JobRouter/preview/2022-07-18-preview/communicationservicejobrouter.json Co-authored-by: Liangying.Wei <[email protected]> * hide redundant communicationerror and communicationerrorresponse during codegen * revert commit * fix typo * test whether removing kind from request works * revert removing kind from request * make kind read-only * run prettier * try adding properties to empty classes --------- Co-authored-by: Rajarshi Sarkar 🦄 <[email protected]> Co-authored-by: Rajarshi Sarkar <[email protected]> Co-authored-by: Liangying.Wei <[email protected]> * [OpenAI] Rename properties for Java (Azure#24596) * renames * more renames * testing on the deploymentOrModelName * revert for no breaking changes * [TypeSpec] Upgrade to 0.46.0 (Azure#24776) * Release quota on stop (Azure#24778) * Release quota on stop * add ReleaseQuotaOnStop to computeinstance properties * update * rm * [OpenAI] Update generated autorest (Azure#24788) - Continuation of Azure#24596 * Dynatrace.Observability merge to public repo (Azure#24101) * Initial Commit * Updating version in all files * Removed version from readme * removing version which are not available from readme * Resolving PR comments (Azure#24205) * Features/users/abhargava/merge to public repo (Azure#24230) * Resolving PR comments * Features/users/abhargava/merge to public repo (Azure#24233) * Resolving PR comments * Responding to the comment in PR review * Fixed typo * Features/users/abhargava/merge to public repo (Azure#24253) * Resolving PR comments * Responding to the comment in PR review * Fixed typo * Added a suppression for Avacado issue * Adding the required field * Updating suppression reason * Updating path for suppression * Adding suppression for deprecated API * suppression * suppress * Update specification/dynatrace/resource-manager/readme.md Co-authored-by: Yuchao Yan <[email protected]> * updating path in suppression * updating suppression * updating suppression * Update specification/dynatrace/resource-manager/readme.md: WIP: fixing suppressions Iteratively work on fixing suppressions * Update readme.md: update suppressions * Update readme.md: update suppressions * Update readme.md: remove Avocado suppressions --------- Co-authored-by: kanupriyasinghh <[email protected]> Co-authored-by: Yuchao Yan <[email protected]> Co-authored-by: Konrad Jamrozik <[email protected]> * Double Header Fix (Azure#24760) * [OpenAI.Inference] Adding content filter support (Azure#24652) * first-pass adding annotation * FilterResult definitions * move Filter definitions to common * cleaning up naming and imports * adding version headers * added descriptions to objects * correct self_harm JSON projection * Correct to "prompt_annotations" * "prompt_annotations" for chat, too * Update specification/cognitiveservices/OpenAI.Inference/models/completions.common.tsp Co-authored-by: Shawn Fang <[email protected]> * Add per-choice content_filter_results to /completions * tsp compile after merge * recompile with 0.46.0 after today's update --------- Co-authored-by: Travis Wilson <[email protected]> Co-authored-by: Shawn Fang <[email protected]> * Update OpenAI PR assignment (Azure#24794) * Update OpenAI PR assignment * Update pull_request_assignment.yml * [typespec-ci] Automatically detect spec folders to validate (Azure#24629) * [fist] mgmt, update service name (Azure#24781) * [typespec-pr] Add pipeline (Azure#24802) * [typespec-ci] Remove path wildcard (Azure#24804) * Update engsys CODEOWNERS (Azure#24808) * Align optionality of content filter categories (Azure#24819) * Add spec ARM PR review workflow diagram to ARM PR description template. (Azure#24823) * Add spec PR review workflow diagram to ARM PR description template. * Update control_plane_template.md: fix link to diagram (Azure#24828) * Added generator config for OpenAI `Functions` for the Java SDK (Azure#24765) * Remove repeatability support for token revocation endpoint (Azure#24589) * Update clientTYpe (Azure#24503) * [Hub Generated] Publish private branch 'dev-maintenance-Microsoft.Maintenance-2023-04-01' (Azure#24048) * Revert "Revert "[Hub Generated] Publish private branch 'dev-maintenance-Microsoft.Maintenance-2023-04-01' (Azure#23832)" (Azure#23923)" This reverts commit e6c0e188a74d051a3a44728ce058e85447553392. * Removed 202 and Async supression. * Update documentation. * Updated documentation --------- Co-authored-by: Kalpesh Chavan <[email protected]> * [package-lock.json] Add names to local dependencies (Azure#24846) - Command "npm update" removes these names, but "npm install" adds them - It's probably better to keep the names, since users are more likely to run "npm install" * Update ci-fix.md: add API Doc Preview TSG (Azure#24843) * Update ci-fix.md: add API Doc Preview TSG * Remove unsupported SQL Pool Rename/move operation (Azure#24517) * Add enableOSPatching and address comments (Azure#24849) * Updated text for property isSoftDeleteFeatureStateEditable (Azure#24713) * Updating folder structure for Microsoft.Migrate (Azure#24550) * folders restructure and update readme * revert removed offazure in readme * Removing readonly flag from top level object (Azure#24848) * [ACS JobRouter] Add misssing descriptions (Azure#24842) * Updated JR swagger with maxNumberOfJobs * sync with upstream main * add descriptions * prettier fix * add more missing descriptions * add more description fixes --------- Co-authored-by: williamzhao87 <[email protected]> * a couple of representation tweaks for codegen (Azure#24847) * Removed pricing not relevant to CI * Added Resizing state * Change targetVMSize name * Revert "Merge branch 'main' of https://github.com/Azure/azure-rest-api-specs into teom-hobovm08-01-preview" This reverts commit 47adff637482e53956bcde7a4331f0638a1ddfbc, reversing changes made to 4b9165da959a09109ecf96e7e70a940ec3989732. * Resolve comments for os patching (Azure#24872) * Add properties for os patching * resolve comments * remove autorestartafterupdates * update description * rename property --------- Co-authored-by: Chris Jiang <[email protected]> * Rename osPatchingStatus to upper case (Azure#24954) * Add properties for os patching * resolve comments * remove autorestartafterupdates * update description * rename property * Change property osPatchingStatus to upper case * rename osPatchingStatus to upper case --------- Co-authored-by: Chris Jiang <[email protected]> * Fix some minor issues * Fix comments * Minor changes * Fix more builds * 200 and nextLink * Remove nextlink * Try 204 not 200 * Change GET call to POST * Remove 204 * Add 200 response * Change example state * Add enum for patch status (Azure#25208) * Add properties for os patching * resolve comments * remove autorestartafterupdates * update description * rename property * Change property osPatchingStatus to upper case * rename osPatchingStatus to upper case * add enum for patch status * add enum for patch status --------- Co-authored-by: Chris Jiang <[email protected]> * LRO does not return 200 * Format readme * Format readme * Format readme * Fix readme * Rename enum for patchStatus (Azure#25257) * Add properties for os patching * resolve comments * remove autorestartafterupdates * update description * rename property * Change property osPatchingStatus to upper case * rename osPatchingStatus to upper case * add enum for patch status * add enum for patch status * rename enum's name * rename enum --------- Co-authored-by: Chris Jiang <[email protected]> --------- Co-authored-by: Jayati Tehri <[email protected]> Co-authored-by: PreetPRG <[email protected]> Co-authored-by: InbalZim <[email protected]> Co-authored-by: VIPRAY JAIN <[email protected]> Co-authored-by: Dapeng Zhang <[email protected]> Co-authored-by: Yuchao Yan <[email protected]> Co-authored-by: Mike Harder <[email protected]> Co-authored-by: Teo Magnino Chaban <[email protected]> Co-authored-by: Konrad Jamrozik <[email protected]> Co-authored-by: Chris Jiang <[email protected]> Co-authored-by: Chris Jiang <[email protected]> Co-authored-by: williamzhao87 <[email protected]> Co-authored-by: Rajarshi Sarkar 🦄 <[email protected]> Co-authored-by: Rajarshi Sarkar <[email protected]> Co-authored-by: Liangying.Wei <[email protected]> Co-authored-by: Shawn Fang <[email protected]> Co-authored-by: mqwang <[email protected]> Co-authored-by: abhargavams <[email protected]> Co-authored-by: kanupriyasinghh <[email protected]> Co-authored-by: JainRah <[email protected]> Co-authored-by: Brandon Miller <[email protected]> Co-authored-by: Travis Wilson <[email protected]> Co-authored-by: Laurent Mazuel <[email protected]> Co-authored-by: Albert Cheng <[email protected]> Co-authored-by: Weidong Xu <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Jose Alvarez <[email protected]> Co-authored-by: Maxim Rytych <[email protected]> Co-authored-by: Daniel Hu <[email protected]> Co-authored-by: Kalpesh Chavan <[email protected]> Co-authored-by: Kalpesh Chavan <[email protected]> Co-authored-by: kashyap-kunal <[email protected]> Co-authored-by: ishanjindalms <[email protected]> Co-authored-by: athulvijayanmsft <[email protected]> Co-authored-by: Thomas Ricci <[email protected]> Co-authored-by: williamzhao87 <[email protected]> * Adding new query param on list job api (Azure#25375) Co-authored-by: Shail Paragbhai Shah <[email protected]> * Remove list materialization jobs api (Azure#25372) * Update mfe.json * prettify swagger * Remove example * Update readme.md * Update readme.md * Add backfill api contract changes * Fixing json * Add property bag * Modifying example --------- Co-authored-by: Shail Paragbhai Shah <[email protected]> * sweep over pipeline component (Azure#25616) * Add SSO and RootAccess (Azure#25754) Co-authored-by: Teo Magnino Chaban <[email protected]> * add EnvironmentVariables in sparkjob (Azure#25814) * Add model monitor 2023-08-01-preview changes (Azure#25785) * Add model monitor 2023-08-01-preview changes * update * run prettier * update comments * add * Add WS Vnet properties for Serverless GA (Azure#25780) * add serverless ws vnet properties * add to workspace update properties * update format * update description --------- Co-authored-by: Aaheli Chattopadhyay <[email protected]> * Resource to ProxyResource for preview version (Azure#25906) Co-authored-by: Shail Paragbhai Shah <[email protected]> * August-2023-preview fix circular reference (Azure#25870) * TriggerType->ComputeTriggerType * RecurrenceFrequency->ComputeRecurrenceFrequency * RecurrenceSchedule->ComputeRecurrenceSchedule --------- Co-authored-by: Kayla Ames <[email protected]> * Update ServerlessEndpoints design (Azure#25593) * first draft redesign of ServerlessEndpoints * add getStatus.json * update examples * add LRO location header to example * add CapacityReservationGroups API * spellcheck fix * example fix * capacity reservation groups should be under resource groups * fix examples * inferenceUri -> uri * CapacityReservation -> ServerlessEndpointCapacityReservation * update description * remove ExcessCapacityReservation concept * remove parentGroupId concept * rename /status api to /getStatus * label metrics as readOnly * lintdiff GET fix * more fixes * add listBySubscription * add/fix examples * add suppression for lint error (Azure#25941) * minor fixes (Azure#25940) * Update 2023-08-01-preview swagger for workspaceRP (Azure#25532) * Update swagger * Bug fix * update * Update testconnection and example * Fix format * update custom words and example * Update missing custom words * reflect comment remove LRO opetions * make metadata a string dictionary * reflect comment * Revert "reflect comment" This reverts commit 9b85252250fd442dd891b27509f5d2e315d8aa2a. * Update with managedNetwork change included * Update managed network schema * revert metadata change * remove the RBAC properties related with shared connection * Update readme * Update suppression reason * Updated preview API to add Kernel Config (Azure#25361) * Updated preview API to add Kernel Config and LRO Headers * Fixing formatting issue * Updated as per comments * Updated as per comments * Reverting changes * Reverted header changes * Reverting change * Reverting example * Added version parameter and removed addition properties as per guidelines --------- Co-authored-by: Srivatsa Sinha <[email protected]> * Thtodoro/serverless endpoints 2023 08 01 preview fix (Azure#25993) * Capacity Reservation Groups is not an async resource, and thus does not have 202 as a response type * PATCH response must have same resource as GET/PUT APIs * add missing description * Adding Inference Pools, Groups and Endpoints recreated for Azure/azure-rest-api-specs#25783 for rebased issue (Azure#25819) * Adding Inference Pools, Groups and Endpoints * PR comments * remove EventLog and change data * remove EventLog and change data * moving endpoint as sibling to group * Moving back to the tracked resources * force change * fix the missing field --------- Co-authored-by: Raushan Kumar <[email protected]> * Make suppression more specific (Azure#26031) Co-authored-by: Teo Magnino Chaban <[email protected]> * add serverless compute settings (Azure#26007) Co-authored-by: Aaheli Chattopadhyay <[email protected]> --------- Co-authored-by: Kayla Ames <[email protected]> Co-authored-by: Anubha Jain <[email protected]> Co-authored-by: Anubha98 <[email protected]> Co-authored-by: teochaban <[email protected]> Co-authored-by: Jayati Tehri <[email protected]> Co-authored-by: PreetPRG <[email protected]> Co-authored-by: InbalZim <[email protected]> Co-authored-by: VIPRAY JAIN <[email protected]> Co-authored-by: Dapeng Zhang <[email protected]> Co-authored-by: Yuchao Yan <[email protected]> Co-authored-by: Mike Harder <[email protected]> Co-authored-by: Teo Magnino Chaban <[email protected]> Co-authored-by: Konrad Jamrozik <[email protected]> Co-authored-by: Chris Jiang <[email protected]> Co-authored-by: Chris Jiang <[email protected]> Co-authored-by: williamzhao87 <[email protected]> Co-authored-by: Rajarshi Sarkar 🦄 <[email protected]> Co-authored-by: Rajarshi Sarkar <[email protected]> Co-authored-by: Liangying.Wei <[email protected]> Co-authored-by: Shawn Fang <[email protected]> Co-authored-by: mqwang <[email protected]> Co-authored-by: abhargavams <[email protected]> Co-authored-by: kanupriyasinghh <[email protected]> Co-authored-by: JainRah <[email protected]> Co-authored-by: Brandon Miller <[email protected]> Co-authored-by: Travis Wilson <[email protected]> Co-authored-by: Laurent Mazuel <[email protected]> Co-authored-by: Albert Cheng <[email protected]> Co-authored-by: Weidong Xu <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Jose Alvarez <[email protected]> Co-authored-by: Maxim Rytych <[email protected]> Co-authored-by: Daniel Hu <[email protected]> Co-authored-by: Kalpesh Chavan <[email protected]> Co-authored-by: Kalpesh Chavan <[email protected]> Co-authored-by: kashyap-kunal <[email protected]> Co-authored-by: ishanjindalms <[email protected]> Co-authored-by: athulvijayanmsft <[email protected]> Co-authored-by: Thomas Ricci <[email protected]> Co-authored-by: williamzhao87 <[email protected]> Co-authored-by: shail2208 <[email protected]> Co-authored-by: Shail Paragbhai Shah <[email protected]> Co-authored-by: ghsutirtha <[email protected]> Co-authored-by: ycheng35xo <[email protected]> Co-authored-by: Mathieu St-Louis <[email protected]> Co-authored-by: ac923 <[email protected]> Co-authored-by: Aaheli Chattopadhyay <[email protected]> Co-authored-by: Teddy Todorov <[email protected]> Co-authored-by: ZhidaLiu <[email protected]> Co-authored-by: srivatsasinha <[email protected]> Co-authored-by: Srivatsa Sinha <[email protected]> Co-authored-by: Raushan Kumar <[email protected]> Co-authored-by: Raushan Kumar <[email protected]>
3 tasks
sarangan12
added a commit
that referenced
this pull request
Nov 10, 2023
) ### Packages impacted by this PR @Azure/eventgrid ### Issues associated with this PR NA ### Describe the problem that is addressed by this PR 1. The EventGrid Service team has made changes to their [CADL specification](https://github.com/Azure/azure-rest-api-specs/tree/main/specification/eventgrid/Azure.Messaging.EventGrid). This PR regenerates the SDK using the latest CADL specification. 2. Important changes in this PR: i. A new API `renewCloudEventLocks` has been added. ii. In the ` publishCloudEvent` API, a new option `binaryMode` has been added. This enables the users to send in the binary encoded event. iii. There are some other minor changes and some breaking changes also in this PR. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? One of the design decisions is around the `binaryMode`. Currently, the API accepts only binary encoded data for this API. If there is a requirement in the future that the user prefers that the encoding be done by the SDK, we could consider it in future, ### Are there test cases added in this PR? _(If not, why?)_ Yes. New test cases have been added to test the `binaryMode` and the new `renewCloudEventLocks` API. ### Provide a list of related PRs _(if any)_ 1. #26307 2. #25906 ### Checklists - [X] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [X] Added a changelog (if necessary) @xirzec Please review and approve the PR & API Views. ## API View to approve 1. [Link 1](https://apiview.dev/Assemblies/Review/0ead6d0c15c64cd7a007a11f514cfc4c/b94b6c83619f4dcfb04e476365bf7e19?diffRevisionId=fb754e995af24441901461c1c6ada906&doc=False&diffOnly=True) (This compares this Beta 3 version with Beta 2 version which gives you a clear view of the changes) 2. [Link 2](https://apiview.dev/Assemblies/Review/b6a1a4040a1f46528f095df8589bfa3e/80e3111615cc47f49b25a4f16d9e1322?diffRevisionId=20bf158c9f2341eba08c706d55a0aa4c&doc=False&diffOnly=True) (This is generated automatically by the PR Bot wgich compares the Beta 3 version with latest GA which is not correct. But, in order to release the SDK, we need the approval for the same. So, kindly review the Link 1 and approve both). Thanks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Packages impacted by this PR
@azure/eventgrid
Issues associated with this PR
NA
Describe the problem that is addressed by this PR
The EventGrid Service team has introduced a new Client called
EventGridClientwith 4 apis -publishCloudEvent,publishCloudEvents,receiveCloudEvents,acknowledgeCloudEvents,releaseCloudEventsandrejectCloudEvents. Now, the service is represented using the new CADL specification. This SDK, in this PR, has been generated using the new JS SDK CADL Generator.During the generation process, I had a few issues in the generation process. Seperate issues have been filed for the same and many of them have already been resolved. For the remaining issues, I have provided manual patches in this PR, so the release schedule will not be impacted. Jose is already working on fixing any remaining issues. A complete list of issues can be found in the related PRs/Issues section below.
This is a beta release. So, this code is not merged to the main branch. The code changes are merged to a private branch feature/eventgrid/4_13_beta_1.
APIView To Approve
https://apiview.dev/Assemblies/Review/de09872b51a24f489981db711a73c430/2e0e731c80a840faab6df4c51acb6e95?diffRevisionId=2a4fb5a9265847c09428acb9b0581b0e&doc=False&diffOnly=True
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
No special design considerations for the generation process.
Are there test cases added in this PR? (If not, why?)
This version of the eventgrid service has not been released to public and not available through azure portal. So, the test cases are not added to this PR for now. Locally, I have tested the code changes with private test resources and confirmed that the apis are working fine.
Provide a list of related PRs/Issues (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
npx tsp compile main.tsp
Checklists
@joheredi @ellismg Please review and approve the PR.
@xirzec FYI....