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

add commit hash to S3 object identifier for artifacts on S3 #8161

Merged
merged 4 commits into from
Jun 26, 2024

Conversation

Bodobolero
Copy link
Contributor

In future we may want to run periodic tests on dedicated cloud instances that are not GitHub action runners.
To allow these to download artifact binaries for a specific commit hash we want to make the search by commit hash possible and prefix the S3 objects with artifacts/${GITHUB_SHA}/${GITHUB_RUN_ID}/${GITHUB_RUN_ATTEMPT}

@Bodobolero Bodobolero requested a review from bayandin June 25, 2024 16:00
.github/actions/download/action.yml Outdated Show resolved Hide resolved
.github/actions/upload/action.yml Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Jun 25, 2024

2910 tests run: 2793 passed, 0 failed, 117 skipped (full report)


Code coverage* (full report)

  • functions: 32.8% (6877 of 20995 functions)
  • lines: 50.2% (53657 of 106915 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
050dd70 at 2024-06-25T21:31:20.260Z :recycle:

Co-authored-by: Alexander Bayandin <alexander@neon.tech>
@Bodobolero Bodobolero requested a review from bayandin June 25, 2024 17:38
@Bodobolero Bodobolero enabled auto-merge (squash) June 25, 2024 21:03
@Bodobolero Bodobolero merged commit 9b623d3 into main Jun 26, 2024
59 of 62 checks passed
@Bodobolero Bodobolero deleted the bodobolero/add_commit_hash_to_S3_prefix branch June 26, 2024 05:46
conradludgate pushed a commit that referenced this pull request Jun 27, 2024
In future we may want to run periodic tests on dedicated cloud instances
that are not GitHub action runners.
To allow these to download artifact binaries for a specific commit hash
we want to make the search by commit hash possible and prefix the S3
objects with
`artifacts/${GITHUB_SHA}/${GITHUB_RUN_ID}/${GITHUB_RUN_ATTEMPT}`

---------

Co-authored-by: Alexander Bayandin <alexander@neon.tech>
bayandin added a commit that referenced this pull request Jul 9, 2024
## Problem

In #8161, we changed the path
to Neon artefacts by adding commit sha to it, but we missed adding these
changes to `promote-compatibility-data` job that we use for
backward/forward- compatibility testing.

## Summary of changes
- Add commit sha to `promote-compatibility-data`
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