Skip to content
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

[cost] add billing reader role to api identity #2004

Merged

Conversation

guybartal
Copy link
Contributor

PR for issue #1898 (partial work)

What is being addressed

Allow API app to call Azure cost management API

How is this addressed

  • add Billing Reader role assignment to API app identity
  • next phase - implement cost report service

@guybartal
Copy link
Contributor Author

/test

@guybartal
Copy link
Contributor Author

/test 363904a

@github-actions
Copy link

github-actions bot commented Jun 8, 2022

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/2464200738 (with refid a7639696)

(in response to this comment from @guybartal)

@github-actions
Copy link

github-actions bot commented Jun 8, 2022

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/2464202580 (with refid a7639696)

(in response to this comment from @guybartal)

@tamirkamara tamirkamara changed the title [cost] add billing reader role to api identity #1898 [cost] add billing reader role to api identity Jun 9, 2022
@guybartal guybartal removed the request for review from marrobi June 9, 2022 06:14
@joalmeid joalmeid assigned joalmeid and guybartal and unassigned joalmeid Jun 9, 2022
@guybartal
Copy link
Contributor Author

/test 17b7db1

@github-actions
Copy link

github-actions bot commented Jun 9, 2022

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/2466902419 (with refid a7639696)

(in response to this comment from @guybartal)

@guybartal guybartal merged commit 559d061 into microsoft:main Jun 9, 2022
@guybartal guybartal deleted the guybartal/add-billing-reader-role branch June 9, 2022 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants