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

Updated validation GitHub Action #246

Merged
merged 13 commits into from
Jun 13, 2022
Merged

Updated validation GitHub Action #246

merged 13 commits into from
Jun 13, 2022

Conversation

juanangp
Copy link
Member

@juanangp juanangp commented Jun 13, 2022

juanangp Large: 386

Try to fix pipeline failure due to a bug in cache?

  • Added old flag to cache key in BuildOld job to avoid any colision with Build cache
    It seems that the cache issue was related with a problem in Github, however further upgrades have been added to this PR:
  • Renamed Validation.yml tovalidation.yml
  • Added description to the different workflows
  • Using global variables instead local paths
  • Removal of custom command for sourcing root, fixed on updated thisREST script to work when root/Geant4 is installed in /usr #243

@lobis lobis changed the title Added extra flag to cache in validation Updated validation GitHub Action Jun 13, 2022
@lobis
Copy link
Member

lobis commented Jun 13, 2022

juanangp Large: 393

Try to fix pipeline failure due to a bug in cache?

  • Added old flag to cache key in BuildOld job to avoid any colision with Build cache

Just for future reference, the problem with the cache was on GitHub's side, we were using the cache correctly.

@lobis lobis requested review from lobis, jgalan and nkx111 June 13, 2022 15:09
@lobis lobis self-assigned this Jun 13, 2022
@juanangp juanangp marked this pull request as ready for review June 13, 2022 16:21
@juanangp juanangp merged commit 750d2b0 into master Jun 13, 2022
@lobis lobis deleted the github-pipeline-cache-bug branch June 15, 2022 10:12
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