-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
fix sagemaker user_profile r_studio_server_pro_app_settings drift #35477
base: main
Are you sure you want to change the base?
Conversation
Community NoteVoting for Prioritization
For Submitters
|
a92bbaf
to
6a7e1f0
Compare
67fd88c
to
482aecd
Compare
482aecd
to
aeafa05
Compare
Thank you for your contribution! 🚀 Please note that typically Go dependency changes are handled in this repository by dependabot or the maintainers. This is to prevent pull request merge conflicts and further delay reviews of contributions. Remove any changes to the Additional details:
|
Hi @DrFaust92 , Kindly review. This PR was split from #35416 |
Im concerned this is a breaking change, @ewbankkit wdyt? |
aeafa05
to
92eb5e8
Compare
Description
Add/Fix r_studio_server_pro_domain_settings update under domain_settings update to avoid drift.
Relations
Fixes #35382
References
Create User API
Update User API
Output from Acceptance Testing