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

workflows: use safe-upload-artifacts throughout #608

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

szczys
Copy link
Contributor

@szczys szczys commented Sep 19, 2024

Update all workflows to use safe-upload-artifacts whenever upload artifacts that may contain secrets.

  • Binary artifact uploads from build steps do not contain secrets and still use upload-artifact directly
  • The merge reports workflows combine artifacts that have already been uploaded (and therefor have already run through the safe upload)
  • Remove a few erroneous ids and retention settings

resolves https://github.com/golioth/firmware-issue-tracker/issues/672

Update all workflows to use safe-upload-artifacts whenever upload artifacts
that may contain secrets.

- Binary artifact uploads from build steps do not contain secrets and still
  use upload-artifact directly
- The merge reports workflows combine artifacts that have already been
  uploaded (and therefor have already run through the safe upload)
- Remove a few erroneous ids and retention settings

Signed-off-by: Mike Szczys <mike@golioth.io>
Copy link

Visit the preview URL for this PR (updated for commit 99b9a35):

https://golioth-firmware-sdk-doxygen-dev--pr608-szczys-always-7oc5iqca.web.app

(expires Thu, 26 Sep 2024 16:16:42 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

Code Coverage (Zephyr)

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.zephyr 49% 14%
src 67% 27%
Summary 65% (1855 / 2847) 26% (763 / 2915)

@szczys szczys merged commit bdb5027 into main Sep 19, 2024
41 of 49 checks passed
@szczys szczys deleted the szczys/always-use-safe-upload branch September 19, 2024 19:50
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.

2 participants