Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

Guidance for legacy workspaces & alternate VM SKUs #429

Merged
merged 10 commits into from
Apr 5, 2024

Conversation

SenthuranSivananthan
Copy link
Contributor

@SenthuranSivananthan SenthuranSivananthan commented Apr 1, 2024

Overview/Summary

Added two Azure Databricks Guidance:

  • DBW-29 - Migrate Legacy Workspaces
  • DBW-30 - Define alternate VM SKUs

Related Issues/Work Items

N/A

This PR fixes/adds/changes/removes

Breaking Changes

None

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Performed testing and provided evidence (e.g. screenshot of output) for any changes associated to ARG queries and/or scripts
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@SenthuranSivananthan SenthuranSivananthan requested a review from a team as a code owner April 1, 2024 15:58
Copy link
Contributor

@rodrigosantosms rodrigosantosms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove all links from the "Guidance/Description", they should only exist in the "Reference Links" section, otherwise it could break some of our automations.

@Dost2010 Dost2010 added the Needs: Attention from aprl-admins 👋 Needs attention from aprl-admins GitHub team. label Apr 3, 2024
@oZakari oZakari added Needs: Author Feedback Area: Resource Graph Query(s) Area: Service Guidance Improvements or additions to documentation and removed Needs: Attention from aprl-admins 👋 Needs attention from aprl-admins GitHub team. labels Apr 5, 2024
@SenthuranSivananthan
Copy link
Contributor Author

@oZakari / @rodrigosantosms - links in the descriptions have been removed. Are there other changes to make?

Copy link
Contributor

@rodrigosantosms rodrigosantosms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've made some changes to avoid breaking our automation. I'll sync with Senthu later about the changes. PR Approved.

@rodrigosantosms rodrigosantosms merged commit 84be290 into Azure:main Apr 5, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Area: Resource Graph Query(s) Area: Service Guidance Improvements or additions to documentation
Projects
None yet
4 participants