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.

Sign up for GitHub

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

Jump to bottom

Patch/0.3.8 #327

Merged
merged 5 commits into from
Feb 6, 2023
Merged

Patch/0.3.8 #327

merged 5 commits into from
Feb 6, 2023

Conversation

mocsharp
Copy link
Collaborator

@mocsharp mocsharp commented Feb 5, 2023

Merge patch/0.3.8 into main

dependabot bot and others added 4 commits January 11, 2023 08:12
@dependabot
Bump actions/upload-artifact from 3.1.1 to 3.1.2 (#314)
97d1b53 
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.1.1 to 3.1.2.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v3.1.1...v3.1.2)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump actions/cache from 3.0.11 to 3.2.3 (#313)
c0e35e9 
Bumps [actions/cache](https://github.com/actions/cache) from 3.0.11 to 3.2.3.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v3.0.11...v3.2.3)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump Microsoft.NET.Test.Sdk from 17.4.0 to 17.4.1 (#309)
2831d3c 
Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 17.4.0 to 17.4.1.
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Changelog](https://github.com/microsoft/vstest/blob/main/docs/releases.md)
- [Commits](microsoft/vstest@v17.4.0...v17.4.1)

---
updated-dependencies:
- dependency-name: Microsoft.NET.Test.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@mocsharp
Clears payload at startup (#326)
7ad457a 
* Fix Mongodb configuration where GUIDs are not deserialized correctly

Remove pending payloads only when MachineName matches.

* Update changelog

Signed-off-by: Victor Chang <vicchang@nvidia.com>
@mocsharp mocsharp added the build Build & build workflow label Feb 5, 2023
@mocsharp mocsharp added this to the 0.3.8 milestone Feb 5, 2023
@mocsharp mocsharp self-assigned this Feb 5, 2023
@codecov
Copy link

codecov bot commented Feb 5, 2023

Codecov Report

Merging #327 (e044adf) into main (a412da2) will decrease coverage by 1.31041%.
The diff coverage is 7.49280%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##                main        #327         +/-   ##
===================================================
- Coverage   81.95948%   80.64907%   -1.31042%     
===================================================
  Files            323         325          +2     
  Lines          19495       19813        +318     
  Branches         919         921          +2     
===================================================
+ Hits           15978       15979          +1     
- Misses          3204        3518        +314     
- Partials         313         316          +3
Flag Coverage Δ
unittests 80.64907% <7.49280%> (-1.31042%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ork/Migrations/20230131233123_R3_0.3.8.Designer.cs 0.00000% <0.00000%> (ø)
...ityFramework/Migrations/20230131233123_R3_0.3.8.cs 0.00000% <0.00000%> (ø)
...grations/InformaticsGatewayContextModelSnapshot.cs 0.00000% <0.00000%> (ø)
...ay/Services/Connectors/PayloadMoveActionHandler.cs 70.54795% <23.07692%> (-7.74662%) ⬇️
src/Api/Storage/Payload.cs 97.14286% <100.00000%> (+0.17315%) ⬆️
...ityFramework/Configuration/PayloadConfiguration.cs 100.00000% <100.00000%> (ø)
.../EntityFramework/Repositories/PayloadRepository.cs 84.84848% <100.00000%> (ø)
...DB/Configurations/InferenceRequestConfiguration.cs 100.00000% <100.00000%> (ø)
...B/Configurations/MongoDBEntityBaseConfiguration.cs 100.00000% <100.00000%> (ø)
...ase/MongoDB/Configurations/PayloadConfiguration.cs 100.00000% <100.00000%> (ø)
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a412da2...e044adf. Read the comment docs.

dbericat
dbericat approved these changes Feb 6, 2023
View reviewed changes
Copy link
Member

@dbericat dbericat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mocsharp mocsharp merged commit 7d753e6 into main Feb 6, 2023
@github-actions
Copy link

github-actions bot commented Feb 6, 2023

🎉 This issue has been resolved in version 0.3.8 🎉

The release is available on:

mocsharp added a commit that referenced this pull request Apr 6, 2023
@mocsharp @dependabot
Patch/0.3.8 (#327)
3f4ee6a 
+semver: patch

* Bump actions/upload-artifact from 3.1.1 to 3.1.2 (#314)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.1.1 to 3.1.2.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v3.1.1...v3.1.2)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump actions/cache from 3.0.11 to 3.2.3 (#313)

Bumps [actions/cache](https://github.com/actions/cache) from 3.0.11 to 3.2.3.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v3.0.11...v3.2.3)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump Microsoft.NET.Test.Sdk from 17.4.0 to 17.4.1 (#309)

Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 17.4.0 to 17.4.1.
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Changelog](https://github.com/microsoft/vstest/blob/main/docs/releases.md)
- [Commits](microsoft/vstest@v17.4.0...v17.4.1)

---
updated-dependencies:
- dependency-name: Microsoft.NET.Test.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Clears payload at startup  (#326)

* Fix Mongodb configuration where GUIDs are not deserialized correctly

Remove pending payloads only when MachineName matches.

* Update changelog

Signed-off-by: Victor Chang <vicchang@nvidia.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Victor Chang <vicchang@nvidia.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
mocsharp added a commit that referenced this pull request Sep 19, 2023
@mocsharp
Include project references in the nuget package
700fc72 
Signed-off-by: Victor Chang <vicchang@nvidia.com>

Related Work Items: #118, #120, #121, #122, #123, #126, #130, #131, #132, #133, #135, #136, #139, #140, #142, #143, #144, #145, #146, #147, #148, #150, #151, #152, #156, #157, #158, #164, #166, #167, #169, #170, #177, #182, #185, #186, #187, #189, #190, #194, #196, #197, #198, #201, #203, #204, #207, #210, #211, #212, #213, #215, #217, #222, #223, #281, #293, #309, #313, #314, #319, #323, #326, #327, #328, #330, #332, #333, #334, #335, #336, #337, #338, #342, #343, #344, #345, #346, #348, #350, #352, #354, #355, #366, #371, #372, #378, #379, #386, #448, #455, #458, #464, #477
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbericat dbericat dbericat approved these changes

@rahul-imaging rahul-imaging Awaiting requested review from rahul-imaging

Assignees

@mocsharp mocsharp

Labels
build Build & build workflow
Projects
None yet
Milestone
0.3.8
Development

Successfully merging this pull request may close these issues.
2 participants
@mocsharp @dbericat