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

Move to v4 of the upload/download artifact actions. #1240

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

philliphoff
Copy link
Contributor

Description

The handoff between the build step and the publish step of the CI/CD pipeline does not seem to be working. I noticed that we were using the master version of the upload action but the v2 version of the download action. The action repos mention that the latest versions (v4) represent significant architectural changes from prior versions, which I speculate may be the reason for the break (i.e. the upload is using the new architecture while the download is using the old).

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Phillip Hoff <phillip@orst.edu>
Copy link

codecov bot commented Feb 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (3cab20a) 67.28% compared to head (ebde350) 67.28%.

Additional details and impacted files
@@              Coverage Diff              @@
##           release-1.13    #1240   +/-   ##
=============================================
  Coverage         67.28%   67.28%           
=============================================
  Files               174      174           
  Lines              6025     6025           
  Branches            671      671           
=============================================
  Hits               4054     4054           
  Misses             1802     1802           
  Partials            169      169           
Flag Coverage Δ
net6 67.26% <ø> (ø)
net7 67.26% <ø> (ø)
net8 67.27% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@philliphoff philliphoff merged commit 17b504c into release-1.13 Feb 16, 2024
25 checks passed
@philliphoff philliphoff deleted the philliphoff-artifacts branch February 16, 2024 23:09
philliphoff added a commit that referenced this pull request Mar 8, 2024
* Update protos and related use for Dapr 1.13. (#1236)

* Update protos and related use.

Signed-off-by: Phillip Hoff <phillip@orst.edu>

* Update Dapr runtime version.

Signed-off-by: Phillip Hoff <phillip@orst.edu>

* Init properties.

Signed-off-by: Phillip Hoff <phillip@orst.edu>

---------

Signed-off-by: Phillip Hoff <phillip@orst.edu>

* Update artifact action versions. (#1240)

Signed-off-by: Phillip Hoff <phillip@orst.edu>

* Make recursive true as default (#1243)

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

* Make final 1.13 changes.

Signed-off-by: Phillip Hoff <phillip@orst.edu>

---------

Signed-off-by: Phillip Hoff <phillip@orst.edu>
Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
Co-authored-by: Shivam Kumar <shivamkm07@gmail.com>
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.

None yet

2 participants