Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Release 6.0.0 #2657

Merged
merged 21 commits into from
Dec 2, 2022
Merged

Release 6.0.0 #2657

merged 21 commits into from
Dec 2, 2022

Conversation

nharper285
Copy link
Contributor

@nharper285 nharper285 commented Nov 28, 2022

6.0.0

BREAKING CHANGES

Manual Deployment Step

When upgrading from version 5.20 a manual step is required. Before deploying 6.0 delete both Azure App Functions and the Azure App Service plan before upgrading. This is required because we have migrated the service from python to C#.

After deployment, there will be two App Functions deployed, one with the name of the deployment and a second one with the same name and a -net suffix. This is a temporary situation and the -net app function will be removed in a following release.

If you have not used the deployment parameters to deploy C# functions in 5.20, you can manually delete the -net app function immediately. Deploying the C# functions was not a default action in 5.20, for most deployments deleting the -net app function immediately is ok.

Deprecation of jinja templates

With this release we are moving from jinja templates to scriban templates. See the documentation for scriban here.

Version 6.0 will convert jinja templates on-the-fly for a short period of time. We do not guarantee that this will be successful for all jinja template options. These on-the-fly conversions are not persisted in the notifications table in this release. They will be in a following release. This will allow time for conversions of templates that are not handled by the current automatic conversion process.

CLI

The default value for the --container_type parameter to the container command has been removed. The container_type parameter is still required for the command. This change removes the ambiguity of the container information being returned.

Added

  • Agent: Making machine identity a parameter of the agent config. #2649
  • Agent: Pass the machine_id from the Agent to the Task. #2662

Changed

  • Service: Deployment enables refactored C# App Function. #2650
  • CLI: Attempt to use broker or browser login instead of device flow for authentication. Canceling the attempt with Ctrl-C will fall back to using the device flow. #2612
  • Service: Update to .NET 7. #2615
  • Service: Make Proxy TelemetryKey optional. #2619
  • Service: Update OMI to 1.6.10.2 on Ubuntu VMs. #2629
  • CLI: Make the --container_type parameter required when using the containers command. #2631
  • Service: Improve logging around notification failures. #2653
  • Service: Standardize HTTP Error Results. Better Rejection Message When Parsing Validated Strings. #2663
  • CLI: Retry on Connection Errors when acquiring autj token. #2668

Fixed

  • Service: Notification Template targetUrl parameter fix. Only use the filename instead of the absolute path in the URL. The makes the links created in ADO bugs work as expected. #2625
  • CLI: Fixed SignalR client code not reading responses correctly. #2626
  • Service: Fix a logic bug in the notification hook. #2627
  • Service: Bug fixes related to the unmanaged nodes (an unreleased feature). #2632
  • Service: Fix invocation of functionapp in the deployment script. Where the wrong value/parameter pair were used. #2645
  • Service: Fixing .NET crash report no-repro. #2642
  • Service: Check Extensions Status Before Transitioning to running state during VM setup. #2667

@codecov-commenter
Copy link

codecov-commenter commented Nov 28, 2022

Codecov Report

Merging #2657 (5e2beb2) into main (7fc6fc3) will decrease coverage by 0.97%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2657      +/-   ##
==========================================
- Coverage   30.26%   29.29%   -0.98%     
==========================================
  Files         254      289      +35     
  Lines       29604    35755    +6151     
==========================================
+ Hits         8961    10474    +1513     
- Misses      20643    25281    +4638     
Impacted Files Coverage Δ
src/agent/input-tester/src/lib.rs 2.94% <0.00%> (-97.06%) ⬇️
src/agent/win-util/src/lib.rs 18.51% <0.00%> (-81.49%) ⬇️
src/agent/onefuzz/src/memory.rs 25.00% <0.00%> (-36.12%) ⬇️
src/agent/onefuzz/src/fs.rs 54.57% <0.00%> (-9.66%) ⬇️
src/agent/onefuzz-agent/src/worker.rs 59.90% <0.00%> (-2.73%) ⬇️
...ent/onefuzz-task/src/tasks/report/dotnet/common.rs 50.44% <0.00%> (-1.38%) ⬇️
...c/agent/dynamic-library/src/bin/dynamic-library.rs 2.38% <0.00%> (-0.40%) ⬇️
src/agent/onefuzz/src/libfuzzer.rs 69.32% <0.00%> (-0.21%) ⬇️
src/agent/onefuzz/src/process.rs 1.32% <0.00%> (-0.14%) ⬇️
src/agent/onefuzz/src/asan.rs 0.00% <0.00%> (ø)
... and 47 more

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

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
nharper285 and others added 7 commits November 28, 2022 13:07
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
CHANGELOG.md Outdated Show resolved Hide resolved
nharper285 and others added 3 commits November 28, 2022 13:13
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
nharper285 and others added 2 commits November 28, 2022 13:21
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@nharper285 nharper285 marked this pull request as ready for review November 28, 2022 23:02
CHANGELOG.md Show resolved Hide resolved
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
CHANGELOG.md Outdated Show resolved Hide resolved
@nharper285 nharper285 merged commit af806a3 into main Dec 2, 2022
@Porges Porges deleted the release-6.0.0 branch December 8, 2022 21:45
@ghost ghost locked as resolved and limited conversation to collaborators Jan 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants