Skip to content

Commit

Permalink
Dep graph improvements (#27099)
Browse files Browse the repository at this point in the history
* Improve the dependency graph and add arc graph

* Add AzureFileCopy step to pipeline

* Add AzureFileCopy step to pipeline

* Fix broken link in TROUBLESHOOTING.md

* Fix positions of AzureFileCopy

* Add ability to Skip JaCoCo Coverage Reports
  • Loading branch information
chidozieononiwu authored Feb 22, 2022
1 parent 9475bb1 commit 21f4ee2
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
4 changes: 2 additions & 2 deletions eng/DependencyGraph/gen-dep-graph-data.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
# Use case: Generate the dependency graph data file (data.js) for track 2 (client) libraries.
#
# Output:
# This scipt will create a data.js file which contains the data entries that the dependency graph html
# This script will create a data.js file which contains the data entries that the dependency graph html
# file needs to correctly graph the interdependency data
# This script can be run locally from the root of the repo. .\eng\DependencyGraph\gen-dep-graph-data.ps1

Expand Down Expand Up @@ -53,7 +53,7 @@ class Library {
$retString += ",`n"
}
$first = $false
$retString += " { name: `"$($item.Value.name)`", version: `"$($item.Value.version)`" }"
$retString += " { `"name`": `"$($item.Value.name)`", `"version`": `"$($item.Value.version)`" }"
}
$retString += "`n"
$retString += " ]`n"
Expand Down
2 changes: 0 additions & 2 deletions eng/pipelines/aggregate-reports.yml
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,6 @@ stages:
Copy-Item -Path eng/jacoco-test-coverage/target/site/test-coverage -Destination $(Build.ArtifactStagingDirectory)/staging/test-coverage -Force -Recurse
Copy-Item eng/code-quality-reports/src/main/resources/index.html $(Build.ArtifactStagingDirectory)
Copy-Item eng/code-quality-reports/src/main/resources/dependency-allowlist.html $(Build.ArtifactStagingDirectory)/staging
Copy-Item eng/common/InterdependencyGraph.html $(Build.ArtifactStagingDirectory)/staging
Copy-Item data.js $(Build.ArtifactStagingDirectory)/staging
ls -R $(Build.ArtifactStagingDirectory)/staging
displayName: 'Copy reports to artifact staging'
Expand All @@ -133,7 +132,6 @@ stages:
displayName: 'Copy BOM report to artifact staging'
condition: succeededOrFailed()
- publish: $(Build.ArtifactStagingDirectory)
condition: succeededOrFailed()
displayName: 'Publish Report Artifacts'
Expand Down
3 changes: 1 addition & 2 deletions sdk/monitor/azure-monitor-query/TROUBLESHOOTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -115,8 +115,7 @@ com.azure.core.exception.HttpResponseException: Status code 400, "{"error":{"mes
```

The error message may include the line number and position where the Kusto query has an error (line 2, position 244
in the above example). You may also refer to the [Kusto Query Language](https://docs.microsoft.
com/en-us/azure/data-explorer/kusto/query/) reference docs to learn more about querying logs using KQL.
in the above example). You may also refer to the [Kusto Query Language](https://docs.microsoft.com/azure/data-explorer/kusto/query) reference docs to learn more about querying logs using KQL.

### Troubleshooting empty log query results

Expand Down

0 comments on commit 21f4ee2

Please sign in to comment.