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

Feature Request - Application Insights #102

Open
gcormier opened this issue Jun 11, 2020 · 7 comments · May be fixed by microsoft/ga4gh-tes#670
Open

Feature Request - Application Insights #102

gcormier opened this issue Jun 11, 2020 · 7 comments · May be fixed by microsoft/ga4gh-tes#670
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed tobegroomed Add this label while creating new issues to get issues prioritized on the backlog
Milestone

Comments

@gcormier
Copy link

So our scientists can monitor and tune instance sizing accordingly (some workloads don't multithread well, might as well lower the instance sizes)

image

I believe we need to modify start tasks accordingly. Also the setup script could create application insights in the resource group.

@MattMcL4475 MattMcL4475 added the enhancement New feature or request label Jun 12, 2020
@MattMcL4475
Copy link
Contributor

MattMcL4475 commented Jun 12, 2020

Thanks @gcormier . The deployer already adds an Application Insights instance to the resource group, so the work here is to plumb it through for the Azure Batch tasks in the BatchScheduler class. This would also enable the CPU/memory/etc. to be viewed in the Azure Batch Explorer. We'd certainly appreciate the help if you and/or the team (or anyone else) can add this. Otherwise, this is definitely something we want to get done this year.

@MattMcL4475 MattMcL4475 added the help wanted Extra attention is needed label Jun 12, 2020
@MattMcL4475
Copy link
Contributor

@tracykard
Copy link
Contributor

UPDATE - The Genomics team is working on a way to capture and report max cpu, disk, and memory utilization, then upload the info into cosmosdb.

@tracykard tracykard added this to the 2.3 milestone Oct 14, 2020
@shanamatthews shanamatthews removed this from the 2.3 milestone Apr 19, 2021
@shanamatthews
Copy link

As of 2.2, you can get detailed metrics on disk utilization from cosmosdb. We'll use this item to track progress on feeding the rest of this information to application insights to make accessible after the run.

@shanamatthews shanamatthews added this to the backlog milestone Apr 28, 2021
@MattMcL4475 MattMcL4475 modified the milestones: backlog, next Sep 2, 2022
@MattMcL4475
Copy link
Contributor

MattMcL4475 commented Sep 2, 2022

TODO enable Batch and Application Insights integration. Since we don't use autopools anymore, Pools are longer-lived, and this feature makes more sense to implement.

@ngambani
Copy link

@MattMcL4475 this looks like a long pending issue open since 2020, can this be closed and reopened later when we decide to work on it or would you say this is something we intend to do in 2024?

@ngambani ngambani added the tobegroomed Add this label while creating new issues to get issues prioritized on the backlog label Feb 15, 2024
@BMurri
Copy link
Collaborator

BMurri commented Feb 29, 2024

I would love to see this in 2024, but I wonder how oversubscribed we might be. However, I fear that if we close this, it's more likely that we'd forget about it rather than remember to reopen/recreate it later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed tobegroomed Add this label while creating new issues to get issues prioritized on the backlog
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants