Skip to content

chore: add bash to uv_base#1699

Closed
kristina-solovyova wants to merge 1 commit into09-16-chore_add_debug_steps_for_upgrade-extended_test_ci_flow_artifactsfrom
09-16-chore_add_bash_to_uv_base
Closed

chore: add bash to uv_base#1699
kristina-solovyova wants to merge 1 commit into09-16-chore_add_debug_steps_for_upgrade-extended_test_ci_flow_artifactsfrom
09-16-chore_add_bash_to_uv_base

Conversation

@kristina-solovyova
Copy link
Collaborator

@kristina-solovyova kristina-solovyova commented Sep 16, 2025

TL;DR

Add bash to the uv Alpine container to support script execution.

What changed?

Added a new with_exec command to the _uv_base() function that installs bash in the Alpine-based uv container using apk add --no-cache bash.

How to test?

Use hooks. Do basic cluster healthcheck (wekacluster status.status == "Ready") after cluster is setup.

Copy link
Collaborator Author

kristina-solovyova commented Sep 16, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add the label main-merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@kristina-solovyova kristina-solovyova marked this pull request as ready for review September 16, 2025 12:01
@graphite-app graphite-app bot requested a review from tigrawap September 16, 2025 12:01
@kristina-solovyova kristina-solovyova requested review from assafgi and removed request for assafgi and tigrawap September 16, 2025 12:01
@kristina-solovyova kristina-solovyova added the run_ci_on_merge_queue_plan Run upgrade-extended test with generated AI hooks label Sep 16, 2025 — with Graphite App
@graphite-app
Copy link

graphite-app bot commented Sep 16, 2025

Graphite Automations

"Add anton/matt/sergey/kristina as reviwers on operator PRs" took an action on this PR • (09/16/25)

1 reviewer was added to this PR based on Anton Bykov's automation.

@scalar-workers
Copy link

🤖 WekAI Remote Test Execution

Status: ❌ Failed
Worker: operator-ci

Error Details:

Wekai process failed with exit code 1

Last updated: 2025-09-16 12:28:01 UTC • Commit: 7005888

@kristina-solovyova kristina-solovyova removed the run_ci_on_merge_queue_plan Run upgrade-extended test with generated AI hooks label Sep 16, 2025
@kristina-solovyova kristina-solovyova added the skip_ci_on_merge_queue_plan Skip running upgrade-extended test with/without AI hooks label Sep 16, 2025 — with Graphite App
@graphite-app
Copy link

graphite-app bot commented Sep 17, 2025

Merge activity

  • Sep 17, 12:22 PM UTC: kristina-solovyova added this pull request to the Graphite merge queue.
  • Sep 17, 12:23 PM UTC: CI is running for this pull request on a draft pull request (#1710) due to your merge queue CI optimization settings.
  • Sep 17, 1:08 PM UTC: Merged by the Graphite merge queue via draft PR: #1710.

@graphite-app graphite-app bot closed this Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip_ci_on_merge_queue_plan Skip running upgrade-extended test with/without AI hooks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants