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

chore(ci): remove space reclamation steps for GH runners #430

Merged

Conversation

vprashar2929
Copy link
Collaborator

This commit removes the steps previously used to reclaim space on GH runners. Since we have moved to self-hosted runners with more resources available, these steps are no longer necessary

Addresses: #429

@vprashar2929 vprashar2929 requested a review from sthaha October 24, 2024 18:02
@@ -165,7 +165,7 @@ jobs:
- name: Run Operator Upgrade
shell: bash
run: |
./tests/run-e2e.sh --ci
./tests/run-e2e.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

Lets preserve --ci since the flag can be used to decide on other things than run container rm later.

- name: Run e2e tests
run: |
./tests/run-e2e.sh --ci --no-upgrade
./tests/run-e2e.sh --no-upgrade
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here .. lets keep --ci 🙏

Copy link
Collaborator

@sthaha sthaha left a comment

Choose a reason for hiding this comment

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

Requesting changes to keep the --ci .. If docker images rm isn't needed, lets remove that from the script instead

This commit removes the steps previously used to reclaim space on GH
runners. Since we have moved to self-hosted runners with more resources
available, these steps are no longer necessary

Signed-off-by: vprashar2929 <[email protected]>
@sthaha sthaha merged commit 3442439 into sustainable-computing-io:v1alpha1 Oct 28, 2024
12 checks passed
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