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: optimize GitHub Actions workflow for simulations #3468

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

crStiv
Copy link

@crStiv crStiv commented Jan 4, 2025

Description

This PR optimizes the simulation workflow (.github/workflows/sims.yml) by implementing several CI/CD best practices:

  • Remove duplicate jobs (newbuild and install-runsim) to reduce redundancy
  • Add timeouts for long-running simulation tests to prevent hanging
  • Improve cache configuration with proper versioning and restore-keys
  • Fix job dependencies for better workflow efficiency

Changes

  • Removed redundant newbuild and install-runsim jobs
  • Added timeout-minutes for long-running simulation tests
  • Updated cache configuration to use hashFiles('**/go.sum')
  • Added restore-keys for better cache fallback
  • Updated actions/cache from v4.2.0 to v4
  • Fixed job dependencies to optimize workflow execution

Motivation

These changes will:

  • Reduce CI execution time
  • Prevent workflow hanging
  • Improve cache hit rates
  • Follow GitHub Actions best practices
  • Make the workflow more maintainable

Additional Notes

All changes are backward compatible and follow GitHub Actions best practices.

@crStiv crStiv requested review from a team as code owners January 4, 2025 23:07
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.

1 participant