Skip to content

Track hubs deployment options (#1466) #1471

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.

Sign up for GitHub

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

Jump to bottom
Closed
thecloudman wants to merge 51 commits into from
Closed

Track hubs deployment options (#1466) #1471

thecloudman wants to merge 51 commits into from

Conversation

@thecloudman
Copy link
Collaborator

@thecloudman thecloudman commented Apr 4, 2025

🛠️ Description

Fixes #

📷 Screenshots

📋 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?

  • ✅ Updated changelog (required for dev PRs)
  • ➡️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log 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 add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

flanakin and others added 30 commits March 5, 2025 21:14
@flanakin
Update version to 0.9-dev (#1354)
ea7b305
@MSBrett
Replace body {} with blank string to prevent errors raised by cost ma… (
62bd340 
#1365)

Co-authored-by: msbrett <msbrett@users.no-reply.github.com>
@flanakin
Add Power BI build script (#1328)
2bbe759
@allcontributors
add nsftwr as a contributor for code (#1378)
5b4ab64 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@flanakin
Fix scopes object array in settings.json (#1379)
3ea7b0d
@MSBrett
Update images per style guides (#1347)
748a258 
Update images per style guides (#1347)
@allcontributors
add philipstreet as a contributor for bug (#1393)
2d396aa 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@allcontributors
add japyjaap as a contributor for bug (#1397)
d89c1ef 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@allcontributors
add philipstreet as a contributor for review (#1398)
21836ba 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@flanakin
Fix Total savings and AHB reports (#1374)
fd47cf8
@flanakin
Fix FOCUS link in marketing site (#1375)
7f81f36
@flanakin
Update error doc (#1376)
06b42f9
@flanakin
Fix marketing site dark mode colors (#1377)
a73bb59
@flanakin
Remove the MI Contributor role assignment for hubs deployments (#1394)
1d5c03a
@flanakin
Add ConflictError doc (#1399)
acf1b3a
@flanakin
Apply tagsByResource to the stop triggers script (#1382)
e28de3b
@flanakin
Add Fluent 9 theme (#1282)
691d499
@flanakin
Clean up ADX dashboard functions (#1373)
4cec6d7
@flanakin
Handle spaces in tag keys (#1389)
5809868
@flanakin
Remove ADX trusted external tenants (#1405)
f47a28f
@flanakin
Only deploy key vault for remote hubs (#1406)
08f5c93
@flanakin
Governing board updates from Mar 19 board meeting (#1411)
3a0bcc9
@flanakin
Specify file type on resource types download buttons (#1422)
d85664a
@allcontributors
add am-jl as a contributor for review (#1445)
d2bc70e 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@DUeffing @flanakin
Update configure-dashboards.md to solve issue #1335 (#1442)
3be3af4 
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
@allcontributors
add DUeffing as a contributor for doc (#1446)
4b255ce 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@flanakin
Handle missing/incorrect costs in savings calculations (#1436)
51826a7
@picccard
Fix release year for v0.8 update 1 in changelog (#1434)
0a5feed
@allcontributors @flanakin
add picccard as a contributor for doc (#1438)
fce1f87 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
@DUeffing @flanakin
documentation update - steps to update from v0.7 to v0.8 (#1433)
dc95941 
Co-authored-by: msbrett <msbrett@users.no-reply.github.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
flanakin and others added 21 commits March 28, 2025 03:57
@flanakin
Fix AHB core count in storage reports (#1421)
4a8f32e
@helderpinto
[AOE] Fix issues with tag duplicates (case sensitive) after tag inher…
c7f1f2f 
…itance (#1431)

Co-authored-by: Hélder Pinto <helder.pinto@microsoft.com>
@allcontributors
add thecloudman as a contributor for code (#1426)
a0713e2 
Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
@MSBrett
Managed exports v2 (#1415)
82e4ec0 
Updated managed exports for EA scopes to include additional dataset types.
Fixed errors in config_RunExportJob
@flanakin
Disable extra storage query optimizations by default (#1381)
a721066
@flanakin
Advisory council updates (#1437)
6dc5e60
@flanakin
Hubs dashboard CD chargeback updates (#1319)
d8a8616
@ro100e @arthurclares @flanakin
Merge features/alerts into dev (#1450)
76f4250 
Co-authored-by: Arthur Clares <53261392+arthurclares@users.noreply.github.com>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
Co-authored-by: Michael Flanakin <micflan@microsoft.com>
@flanakin
Fix MCA reservation recommendation export ingestion (#1402)
e7ef307 
Co-authored-by: Open Data CI <finopstoolkit@users.noreply.github.com>
@flanakin
Update open data files for March 2025 (#1452)
6268273 
Co-authored-by: Open Data CI <finopstoolkit@users.noreply.github.com>
@flanakin @MSBrett
Address known cost/price data quality issues (#1453)
a93ee13 
Co-authored-by: Brett Wilson <MSBrett@users.noreply.github.com>
@nteyan @flanakin
Disk query update (#1440)
e3f68d4 
Co-authored-by: Nicolas Teyan <nteyan@outlook.fr>
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
@flanakin
Update ARG queries to use KQL instead of Power Query (#1454)
f1599cd
@ro100e @flanakin
Updates for FinOps alerts (#1459)
86260dd 
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
@arthurclares @flanakin
Improvements to Rate Optimization report (#1414)
bda6a54 
Co-authored-by: Michael Flanakin <flanakin@users.noreply.github.com>
Co-authored-by: Michael Flanakin <micflan@microsoft.com>
@flanakin
Update CODEOWNERS (#1458)
540baa2
@flanakin
Fix console errors in templates (#1461)
74a221a
@flanakin
Package 0.9 release (#1464)
500ff4f
@aromano2
Auto Comment on OpenData CI Failure (#1301)
9fb044e 
Co-authored-by: Open Data CI <finopstoolkit@users.noreply.github.com>
Co-authored-by: Anthony Romano <26576969+aromano2@users.noreply.github.com>
@flanakin
Update CODEOWNERS (#1462)
02c01f7
@flanakin
Track hubs deployment options (#1466)
82629e7
@microsoft-github-policy-service microsoft-github-policy-service bot added Status: ⛔ Blocked Progress blocked on another change Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity labels Apr 4, 2025
@microsoft-github-policy-service
Copy link

microsoft-github-policy-service bot commented Apr 4, 2025

@@thecloudman, thanks for the PR!

Unfortunately, you opened a PR against the main branch and we don't accept PRs into this branch.

Please create a new PR against either dev or a features/* branch.

@flanakin flanakin closed this Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@arthurclares arthurclares Awaiting requested review from arthurclares arthurclares will be requested when the pull request is marked ready for review arthurclares is a code owner

@nteyan nteyan Awaiting requested review from nteyan nteyan will be requested when the pull request is marked ready for review nteyan is a code owner

@flanakin flanakin Awaiting requested review from flanakin flanakin will be requested when the pull request is marked ready for review flanakin is a code owner

@Ivanmtta Ivanmtta Awaiting requested review from Ivanmtta Ivanmtta will be requested when the pull request is marked ready for review Ivanmtta is a code owner

@jamelachahbar jamelachahbar Awaiting requested review from jamelachahbar jamelachahbar will be requested when the pull request is marked ready for review jamelachahbar is a code owner

@helderpinto helderpinto Awaiting requested review from helderpinto helderpinto will be requested when the pull request is marked ready for review helderpinto is a code owner

@MSBrett MSBrett Awaiting requested review from MSBrett MSBrett will be requested when the pull request is marked ready for review MSBrett is a code owner

@ro100e ro100e Awaiting requested review from ro100e ro100e will be requested when the pull request is marked ready for review ro100e is a code owner

Assignees

No one assigned

Labels

Needs: Attention 👋 Issue or PR needs to be reviewed by the author or it will be closed due to no activity Status: ⛔ Blocked Progress blocked on another change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@thecloudman @flanakin @MSBrett @DUeffing @picccard @helderpinto @ro100e @nteyan @arthurclares @aromano2