Skip to content

Commit

Permalink
HEAT-230 hooking up app insights (#4)
Browse files Browse the repository at this point in the history
* Initial commit

* HEAT-230 Hooking up app insights

* HEAT-230 hooking up app insights call
  • Loading branch information
andrewrlee authored Mar 27, 2024
1 parent aea4013 commit 2a88ea6
Show file tree
Hide file tree
Showing 14 changed files with 1,113 additions and 111 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
node_modules/
dependencies.csv
components.json
.env
21 changes: 14 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@

# hmpps-component-dependencies

POC to look into surfacing app insight dependencies for the developer portal

## why?
Expand Down Expand Up @@ -111,8 +112,14 @@ It requires two sets of information:
(example curl/AppInsights queries can be found in `data-sources.ts`)

The data is joined by linking:
* AppInsights `cloud_RoleName` to the `name` of the component
* AppInsights `target` to an environments `hostname`

- AppInsights `cloud_RoleName` to the `name` of the component
- AppInsights `target` to an environments `hostname`


This allows us to build a graph of each component to both it's dependencies and the things it depends on.

This graph can be navigated in both directions.

How this would this would be integrated into the component dashboard would require a bit of a conversation.

Expand All @@ -129,11 +136,11 @@ We could also use this linked dependency information to support:

## Limitations

* This relies on the cloud role name matching the project name which may not be the case in all situations
(Should be able to write something to narrow down where there is a mismatch)
- This relies on the cloud role name matching the project name which may not be the case in all situations
(Should be able to write something to narrow down where there is a mismatch)

* Requires components to be in the service catalogue!
- Requires components to be in the service catalogue!

* Only works for applications using AppInsights
- Only works for applications using AppInsights

* Needs some thorough testing
- Needs some thorough testing
Loading

0 comments on commit 2a88ea6

Please sign in to comment.