-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Wenxliu/split diagnostic rp to individual package new #36673
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Elina-L
wants to merge
57
commits into
Azure:main
Choose a base branch
from
Elina-L:wenxliu/SplitDiagnosticRPToIndividualPackageNew
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Wenxliu/split diagnostic rp to individual package new #36673
Elina-L
wants to merge
57
commits into
Azure:main
from
Elina-L:wenxliu/SplitDiagnosticRPToIndividualPackageNew
Conversation
This file contains hidden or 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
Rename SpotPlacementScores.json to GenerateSpotPlacementScores.json Remove openapi-subtype: rpaas from readme.md Adding suppression for RecommenderRP Update suppressions file name from diagnostic.json to RecommenderRP.json Add required APIVersion to GetSpotPlacementScores.json [Monitor] Rename Ingestion TypeSpec directory (Azure#35991) Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com> [DevTestLabs] Convert existing OpenAPI spec to TypeSpec spec (Azure#35346) Stable Version 2025-07-15 for Microsoft.Quota (Azure#35765) * Copy files from stable/2025-03-01 Copied the files in a separate commit. This allows reviewers to easily diff subsequent changes against the previous spec. * Update version to stable/2025-07-15 Updated the API version from stable/2025-03-01 to stable/2025-07-15. * Added tag for 2025-07-15 in readme file * creates a new stable version for the enforcement endpoints * Adds the enforcement examples * adds GroupQuotaUsages folder * Adds enforcement definitions * Modifies the api version in the enforcement examples * adds proper api version for usages example * changes the retry after type and fixes typos * changes another description * renames GroupQuotasEnforcementResponse to GroupQuotaEnforcementStatus * formats object * Changes GroupQuotaEnforcementResponse to GroupQuotaEnforcementStatus. reverts unneccessary change in 2023-06-01 preview * ran prettier on 2023-06-01-preview for prettier check violation * changes object definition in preview version as well * resolves retry after issue * reverts the preview file objects Adding api-version input into GetSpotPlacementScores.json Run prettier Use forward slash [spec-model] Add cache in ctor, to re-use objects for the same path (Azure#36008) [Monitor Query Logs] Add Java customizations (Azure#36007) * [Monitor Query Logs] Add Java customizations * format [EngSys] Remove unnecessary workflow_dispatch triggers (Azure#36019) [dashboard] Remove properties order fix and fix location to be optional (Azure#36017) * remove properties order fix and fix location to be optional * remove needless suppression fix the back compatible for csharp (Azure#35965)
…ps://github.com/Elina-L/azure-rest-api-specs into wenxliu/SplitDiagnosticRPToIndividualPackageNew
…ps://github.com/Elina-L/azure-rest-api-specs into wenxliu/SplitDiagnosticRPToIndividualPackageNew
…son title and description
…ps://github.com/Elina-L/azure-rest-api-specs into wenxliu/SplitDiagnosticRPToIndividualPackageNew
…menderRP/main.tsp Co-authored-by: catalinaperalta <9859037+catalinaperalta@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Alerts Management
Analysis Services
API Management
ArcReview
Authorization
Automation
Batch
BatchAI
Billing
Cognitive - Language
Cognitive Services
Compute
Container Instances
Container Registry
Container Service Fleet
identifies PRs touching Azure Kubernetes Fleet Manager service spec
Container Service
Cosmos
Customer Insights
Data Factory
Data Lake Analytics
Data Lake Store
Data Migration
Device Update
Device Update Service
Devtestlab
Event Grid
Event Hubs
HDInsight
Import Export
Insights
Intune
IoTHub
KeyVault
Load Test Service
Azure Load Test Service
Logic App
Machine Learning Compute
Machine Learning
Maps
Media Services
Mobile Engagement
Monitor - Exporter
Monitor OpenTelemetry Exporter
Monitor - Operational Insights
Monitor
Monitor, Monitor Ingestion, Monitor Query
MySQL
Network - CDN
Network - DNS
Network - Traffic Manager
Network
Notification Hub
Operations Management
Peering
Policy Insights
PostgreSQL
Profile
Azure API profile issues.
Purview
Recovery Services Backup
Recovery Services Site-Recovery
RecoveryServices
Redis Cache
Relay
Scheduler
Search
Service Bus
Service Fabric
Service Map
SQL
Storage
Storage Service (Queues, Blobs, Files)
StorageSync
Storsimple
Stream Analytics
Subscription
Synapses
TimeseriesInsights
Web Apps
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.
ARM (Control Plane) API Specification Update Pull Request
Tip
Overwhelmed by all this guidance? See the
Getting help
section at the bottom of this PR description.PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.
Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.
Additional information
Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the
Generated ApiView
comment added to this PR. You can use ApiView to show API versions diff.Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.
Getting help
Purpose of this PR
andDue diligence checklist
.write access
per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositoriesNext Steps to Merge
comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.and https://aka.ms/ci-fix.
queued
state, please add a comment with contents/azp run
.This should result in a new comment denoting a
PR validation pipeline
has started and the checks should be updated after few minutes.