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

Divide CI in separate jobs #208

Merged
merged 1 commit into from
Apr 22, 2024
Merged

Divide CI in separate jobs #208

merged 1 commit into from
Apr 22, 2024

Conversation

dehanj
Copy link
Member

@dehanj dehanj commented Apr 19, 2024

Divide CI in separate jobs, categorized a bit better.

Cache: Fixed a bug and also fixed a potential issue with the cache strategy, where it could lead to the wrong bitstream being fetched if the original key was not found. This is solved now, now each and every run will create a unique key, and if the cache for some reason fails, the check-hash will fail instead of fetching a different bitstream.

@dehanj dehanj force-pushed the ci_checkhashes_in_new_job branch from d7fc75a to 77ee33e Compare April 19, 2024 09:46
@dehanj dehanj changed the title ci checkhashes in new job Divide CI in seperate jobs Apr 19, 2024
- Gives a better overview of CI and the different checks, without going
  into the logs too deeply.
- Cache: use a unique key for each run, and remove 'restore key' since it
  could potentially retrieve the wrong bitstream. The stragegy should be
  to fail if a cache is not present, not fetch a bitstream from a
  different build.
@dehanj dehanj force-pushed the ci_checkhashes_in_new_job branch from 77f0d13 to a5e30f1 Compare April 19, 2024 10:21
@dehanj dehanj requested a review from mchack-work April 19, 2024 10:37
@secworks secworks changed the title Divide CI in seperate jobs Divide CI in separate jobs Apr 22, 2024
@mchack-work mchack-work merged commit a5e30f1 into main Apr 22, 2024
5 checks passed
@dehanj dehanj deleted the ci_checkhashes_in_new_job branch June 11, 2024 09:25
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