Skip to content

Update to FinOps hubs documented permissions prerequisites#787

Merged
flanakin merged 2 commits into
microsoft:devfrom
helderpinto:dev
Jul 4, 2024
Merged

Update to FinOps hubs documented permissions prerequisites#787
flanakin merged 2 commits into
microsoft:devfrom
helderpinto:dev

Conversation

@helderpinto

Copy link
Copy Markdown
Member

🛠️ Description

Previously documented prerequisites w.r.t. permissions for hubs resources template deployment were insufficient for a successful deployment. The prerequisites were updated to reflect the minimum required permissions.

Fixes #783

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

  • ✅ Yes (required for dev PRs)
  • ➡️ Will cover in a future PR (feature branch PRs only)
  • ❎ Not needed (small/internal change)

📖 Did you update documentation?

  • ✅ Public docs in docs (required for dev)
  • ✅ Internal dev docs in src (required for dev)
  • ➡️ Will cover in a future PR (feature branch PRs only)
  • ❎ Not needed (small/internal change)

@helderpinto helderpinto requested a review from a team July 3, 2024 15:51
@microsoft-github-policy-service microsoft-github-policy-service Bot added Skill: Documentation Documentation updates Needs: Review 👀 PR that is ready to be reviewed labels Jul 3, 2024
@flanakin flanakin added this to the June 2024 milestone Jul 3, 2024
@flanakin flanakin merged commit 12efb0e into microsoft:dev Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed Skill: Documentation Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FinOps hubs documented permissions prerequisites are insufficient for a successful deployment

2 participants