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

Support downloading artifacts from release builds #667

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

wkz
Copy link
Contributor

@wkz wkz commented Sep 30, 2024

Description

The gh-dl-artifact script is now able to find artifacts in any run associated with a particular commit, and if the one it finds is the result of a release build, that is now correctly extracted.

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Previously, the assumption was that the artifact could be found in the
first run of "Bob the Builder". This does not hold in the case when:

1. An initial run is aborted, and is re-run at a later date
2. When a release is built (by "Release General")
Release tarball names contain the version information. Make sure we
find these files when extracting them from the artifact ZIP.
Copy link
Contributor

@troglobit troglobit left a comment

Choose a reason for hiding this comment

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

Nice catch, lgtm!

@troglobit troglobit merged commit 808a04f into main Sep 30, 2024
5 checks passed
@troglobit troglobit deleted the gh-dl-artifact-fixes branch September 30, 2024 15:02
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