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

Bump Microsoft.ApplicationInsights.AspNetCore from 2.20.0 to 2.21.0 #1794

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jul 20, 2022

Bumps Microsoft.ApplicationInsights.AspNetCore from 2.20.0 to 2.21.0.

Changelog

Sourced from Microsoft.ApplicationInsights.AspNetCore's changelog.

Version 2.21.0

  • no changes since beta.

Version 2.21.0-beta3

Version 2.21.0-beta2

Version 2.21.0-beta1

Version 2.20

Version 2.20.0-beta1

Version 2.19.0

  • Support for .Net6

Version 2.19.0-beta1

Version 2.18.0

Version 2.18.0-beta3

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [Microsoft.ApplicationInsights.AspNetCore](https://github.com/Microsoft/ApplicationInsights-dotnet) from 2.20.0 to 2.21.0.
- [Release notes](https://github.com/Microsoft/ApplicationInsights-dotnet/releases)
- [Changelog](https://github.com/microsoft/ApplicationInsights-dotnet/blob/main/CHANGELOG.md)
- [Commits](microsoft/ApplicationInsights-dotnet@2.20.0...2.21.0)

---
updated-dependencies:
- dependency-name: Microsoft.ApplicationInsights.AspNetCore
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added .NET Pull requests that update .net code dependencies Pull requests that update a dependency file labels Jul 20, 2022
@danigian danigian added the fullci Run full continuous integration label Aug 10, 2022
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 09:15 Inactive
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 09:15 Inactive
@danigian danigian temporarily deployed to CI_ALL_IN_ONE_ARM_GATEWAY August 10, 2022 09:17 Inactive
@danigian danigian temporarily deployed to CI_EFLOW_AZURE_VM August 10, 2022 09:17 Inactive
@danigian danigian temporarily deployed to CI_CONCENTRATOR_EFLOW_ARM32 August 10, 2022 09:17 Inactive
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 09:19 Inactive
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 09:22 Inactive
@github-actions github-actions bot added CloudDeploymentTest SensorDecodingTest Indicate SensorDecodingTest passed OTAAJoinTest Indicate OTAAJoinTest passed labels Aug 10, 2022
@danigian danigian added OTAATest Indicate OTAATest passed MacTest Indicate MacTest passed ClassCTest Indicate ClassCTest passed C2DMessageTest Indicate C2DMessageTest passed MultiGatewayTest Indicate MultiGatewayTest passed MultiConcentratorTest CupsTest LnsDiscoveryTest and removed SensorDecodingTest Indicate SensorDecodingTest passed labels Aug 10, 2022
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 10:03 Inactive
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 10:03 Inactive
@danigian danigian temporarily deployed to CI_ALL_IN_ONE_ARM_GATEWAY August 10, 2022 10:05 Inactive
@danigian danigian temporarily deployed to CI_EFLOW_AZURE_VM August 10, 2022 10:05 Inactive
@danigian danigian temporarily deployed to CI_CONCENTRATOR_EFLOW_ARM32 August 10, 2022 10:05 Inactive
@danigian danigian temporarily deployed to CI_AZURE_ENVIRONMENT August 10, 2022 10:09 Inactive
@github-actions github-actions bot added the SensorDecodingTest Indicate SensorDecodingTest passed label Aug 10, 2022
@danigian danigian requested a review from Mandur August 10, 2022 11:08
@danigian
Copy link
Collaborator

@Mandur, I made some changes (breaking changes that need to be reported in release notes)

Would be great if you could have a look at the changes and give your approval.
As validation, I ran the full ci and checked that both Azure Functions logs and Network Server logs are appearing in our e2e app insights

@Mandur
Copy link
Contributor

Mandur commented Aug 11, 2022

Looks good to me code wise.
We need to update docs on the files
Observability.md , lns-discovery.md,lns-configuration.md

@codecov-commenter
Copy link

Codecov Report

Merging #1794 (13902f0) into dev (e2658a6) will decrease coverage by 0.00%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##              dev    #1794      +/-   ##
==========================================
- Coverage   87.42%   87.41%   -0.01%     
==========================================
  Files         251      251              
  Lines        9572     9573       +1     
==========================================
  Hits         8368     8368              
- Misses       1204     1205       +1     
Impacted Files Coverage Δ
...BasicsStation/BasicsStationNetworkServerStartup.cs 59.68% <40.00%> (-0.47%) ⬇️
...oRaWan.NetworkServer/ManagedConnectionException.cs 0.00% <0.00%> (-25.00%) ⬇️
...kPreferredGateway/PreferredGatewayExecutionItem.cs 84.21% <0.00%> (+3.50%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@danigian danigian merged commit ca6375c into dev Aug 11, 2022
@danigian danigian deleted the dependabot/nuget/Microsoft.ApplicationInsights.AspNetCore-2.21.0 branch August 11, 2022 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C2DMessageTest Indicate C2DMessageTest passed ClassCTest Indicate ClassCTest passed CloudDeploymentTest CupsTest dependencies Pull requests that update a dependency file fullci Run full continuous integration LnsDiscoveryTest MacTest Indicate MacTest passed MultiConcentratorTest MultiGatewayTest Indicate MultiGatewayTest passed .NET Pull requests that update .net code OTAAJoinTest Indicate OTAAJoinTest passed OTAATest Indicate OTAATest passed SensorDecodingTest Indicate SensorDecodingTest passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants