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

CI: Upgrade GHA artefact upload action #1793

Merged
merged 2 commits into from
Jun 10, 2020

Conversation

edalex-ian
Copy link
Member

Checklist
Description of change

Version 2 is now out of preview and has some fixes, so time to start using the official release. Hopefully will also address the upload issues as mentioned in #1792 .

Version 2 is now out of preview and has some fixes, so time to start
using the official release.
@edalex-ian edalex-ian requested a review from a team June 5, 2020 01:58
Copy link
Contributor

@PenghaiZhang PenghaiZhang left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@PenghaiZhang
Copy link
Contributor

PenghaiZhang commented Jun 9, 2020

Checklist
Description of change

Version 2 is now out of preview and has some fixes, so time to start using the official release. Hopefully will also address the upload issues as mentioned in #1792 .

The latest version 2.0.1 has this feature: Now if a file fails to upload after hitting a retry limit, any remaining uploads stop and the step fails. But we are using v2(2.0.0?) so maybe we should upgrade to 2.0.1?

Version 2.0.1 contains a fix and logic change for when uploads fail -
which is a main issue we're having.
@edalex-ian
Copy link
Member Author

Nicely spotted @PenghaiZhang !

If I understand how the versioning works with GHA actions, it's just tags so indeed we have to ensure we explicitly state the version to match a tag. I've made that change, let's see how the build goes. 🤞

@ChristianMurphy
Copy link
Member

ChristianMurphy commented Jun 9, 2020

FWIW, Renovate can also help keep github actions up-to-date
https://github.com/renovatebot/renovate/tree/c5f51853c8447b327624ebc66ad1f6a1cdad7bd6/lib/manager/github-actions

But this isn't a part of the hosted service due to

For security reasons, GitHub has blocked integrations/apps from editing GitHub Actions workflow files in any branch, so this only works on GitHub if using a Personal Access Token instead of using the WhiteSource Renovate app.

edit: Digging through the issue tracker is appears an admin may be able to request workflow access for renovate, enabling this feature for hosted as well renovatebot/renovate#5733 (comment)

@edalex-ian
Copy link
Member Author

Thanks @ChristianMurphy ! I was actually wondering about that the other day, but didn't have time to look it up. Will look into enabling that when it all stablises. 👍

@edalex-ian edalex-ian merged commit 2146e49 into openequella:develop Jun 10, 2020
@edalex-ian edalex-ian deleted the ci/upgrade-artefact-upload branch June 10, 2020 01:46
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.

3 participants