Skip to content

Conversation

@mgoerens
Copy link
Contributor

@mgoerens mgoerens commented Nov 19, 2024

Installation of the python dependencies and scripts is done at the start of each job within the build.yaml workflow.

By caching the venv we can save up to 10 seconds on each job after the first.

NOTE FOR REVIEWERS: This builds upon #419 and should hence be merged after. Currently it also contains the commit of #419 but we should rebase before merging.

This was referenced Nov 19, 2024
This was referenced Mar 21, 2025
Copy link
Contributor

@komish komish left a comment

Choose a reason for hiding this comment

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

/lgtm

@mgoerens mgoerens merged commit 6bf6abd into openshift-helm-charts:main May 14, 2025
31 checks passed
@mgoerens mgoerens deleted the cache_venv branch May 14, 2025 19: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