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

Vnext/dotnet8 upgrade #489

Closed
wants to merge 21 commits into from
Closed

Vnext/dotnet8 upgrade #489

wants to merge 21 commits into from

Conversation

YaSuenag
Copy link
Contributor

Pull Request

#401

Summary

  • Migrate to .NET 8
  • Introduce cache feature for emission data
  • Introduce Prometheus exporter in WebAPI

Big thanks to @tiwatsuka

Checklist

  • Local Tests Passing?
  • CICD and Pipeline Tests Passing?
  • Added any new Tests?
  • Documentation Updates Made?
  • Are there any API Changes? If yes, please describe below.
  • This is not a breaking change. If it is, please describe it below.

Are there API Changes?

No

Is this a breaking change?

No

This PR Closes #401

tiwatsuka and others added 21 commits November 23, 2023 00:22
Migrating Azure Functions sample to isolated worker model
Signed-off-by: Takuya Iwatsuka <[email protected]>
Signed-off-by: Takuya Iwatsuka <[email protected]>
Signed-off-by: Takuya Iwatsuka <[email protected]>
Signed-off-by: Takuya Iwatsuka <[email protected]>
Signed-off-by: Takuya Iwatsuka <[email protected]>
Signed-off-by: Takuya Iwatsuka <[email protected]>
Add cache functionality for emissions data
Signed-off-by: Takuya Iwatsuka <[email protected]>
Co-authored-by: Vaughan Knight <[email protected]>
Signed-off-by: Takuya Iwatsuka <[email protected]>
@YaSuenag
Copy link
Contributor Author

This PR seems to conflict in casdk-docs/docs/overview/overview.md . I want to fix, but I haven't write permission on GSF repo, so I cannot fix. Can you help? @vaughanknight @danuw

I think we should copy casdk-docs/docs/overview/overview.md in dev branch into this PR without any changes - it means we can ignore changes in overview.md in this PR.

@vaughanknight
Copy link
Contributor

@danuw to resolve the conflict, as discussed on call ready to merge

@danuw
Copy link
Collaborator

danuw commented Mar 26, 2024

@YaSuenag we have removed the overview.md - would you be able to ensure any content that needs to be ported over has moved to enablement.md and then we delete from that branch (I saw there is your video at the bottom on helm I am not sure we have elsewhere, could you move that - create a new PR if needed - please?)

@YaSuenag YaSuenag mentioned this pull request Mar 27, 2024
@YaSuenag
Copy link
Contributor Author

I cannot modify this branch because it is located at GSF repo - I do not have write permission.
So I re-open another PR from my repo: #493, I'd like to continue this work in that PR.

would you be able to ensure any content that needs to be ported over has moved to enablement.md and then we delete from that branch (I saw there is your video at the bottom on helm I am not sure we have elsewhere, could you move that

All contents of previous overview.md (includes my video to instruct WebAPI installation with Helm) is available in enablement.md, so I think we can remove overview.md now. But I think it is an another issue. So I will open PR after this.

@YaSuenag YaSuenag closed this Mar 27, 2024
@danuw danuw deleted the vnext/dotnet8-upgrade branch May 1, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Contribution]: Upgrade .NET version to .NET 8
4 participants