-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Update garnetCache.json #38907
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
base: main
Are you sure you want to change the base?
Update garnetCache.json #38907
Conversation
Changing entity names to resolve conflicts in sdk generation
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
|
|
@mbhaskar - You will need to fix the [Required] Swagger BreakingChange check before ARM signoff. You should also put back the template comment at the top and fill out the intake form to help reviewers understand what your PR is for. I see there are no REST changes [currently] in this PR, so it doesn't really need ARM review, but it should stay in the review queue until the CI checks are fixed. |
Changing entity names to resolve conflicts in sdk generation.
This PR is in response to a conflict generated due to ClusterResrouce entity name being used already but duplicated in garnetCache.json. Entity names are appended with Garnet to resolve conflicts
** Was added to a conversation with Azure Dev Exp and this seems to be blocking the TSP Migration
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.
Click here to open a PR for only SDK configuration.