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] Cost Management ARM REST API call methods #2030

Merged
merged 39 commits into from
Jun 15, 2022

Conversation

guybartal
Copy link
Contributor

@guybartal guybartal commented Jun 12, 2022

Fixes #1898

What is being addressed

Implement cost service for querying TRE instance and workspace level cost reports

How is this addressed

  • Call Azure cost management REST API with python SDK
  • Query by custom dates and granularity of Daily and None
  • Filter by tag key & value and group by tags
  • Pending work - generate complete TRE cost report with hierarchy to cost service and merge with QueryResult

@guybartal guybartal changed the title Guybartal/cost service [cost] Cost Management ARM REST API call methods #1898 Jun 12, 2022
@guybartal guybartal changed the title [cost] Cost Management ARM REST API call methods #1898 [cost] Cost Management ARM REST API call methods Jun 12, 2022
@guybartal
Copy link
Contributor Author

/test-extended 76490ef

@github-actions
Copy link

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/2483675421 (with refid f9695c87)

(in response to this comment from @guybartal)

@guybartal guybartal marked this pull request as draft June 12, 2022 16:38
@guybartal guybartal marked this pull request as ready for review June 14, 2022 12:14
@guybartal
Copy link
Contributor Author

/test af8b2c9

@github-actions
Copy link

🤖 pr-bot 🤖

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

(in response to this comment from @guybartal)

Copy link
Collaborator

@LizaShak LizaShak left a comment

Choose a reason for hiding this comment

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

LGTM

@guybartal
Copy link
Contributor Author

/test 17b2956

@github-actions
Copy link

🤖 pr-bot 🤖

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

(in response to this comment from @guybartal)

@guybartal
Copy link
Contributor Author

/test a7275fa

@github-actions
Copy link

🤖 pr-bot 🤖

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

(in response to this comment from @guybartal)

@guybartal guybartal merged commit fa9e71a into microsoft:main Jun 15, 2022
@guybartal guybartal deleted the guybartal/cost-service branch June 15, 2022 05:01
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.

Cost Management ARM REST API call methods
2 participants