Skip to content

chore: add debug steps for upgrade-extended test ci flow artifacts#1697

Closed
kristina-solovyova wants to merge 1 commit intomainfrom
09-16-chore_add_debug_steps_for_upgrade-extended_test_ci_flow_artifacts
Closed

chore: add debug steps for upgrade-extended test ci flow artifacts#1697
kristina-solovyova wants to merge 1 commit intomainfrom
09-16-chore_add_debug_steps_for_upgrade-extended_test_ci_flow_artifacts

Conversation

@kristina-solovyova
Copy link
Collaborator

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

TL;DR

Added debug commands to troubleshoot shell script permissions in the upgrade-extended test flow.

What changed?

  • Added debug commands in .dagger/src/operator_flows/main.py to:
    • Display the contents and permissions of the /test_artifacts/ directory
    • Verify permissions after the chmod operations
  • Added a debug step in .github/workflows/dagger_ci_on_merge_queue_plan.yaml to:
    • Check the test-artifacts directory in the GitHub workspace
    • List permissions of shell scripts in the test-artifacts directory

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


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 10:10
@graphite-app graphite-app bot requested review from assafgi and tigrawap September 16, 2025 10:10
@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
Copy link
Collaborator Author

addressing this failure: https://github.com/weka/weka-operator/actions/runs/17761715827/job/50475806531

2025-09-16T09:58:23Z ERR RunWithSelectedSteps.RunPostSetupHook > Error running step error="hook script execution failed: fork/exec /test_artifacts/hooks/hook_POST_SETUP_HOOK/hook.sh: no such file or directory" test_identifier=6bd7613c-92e2-11f0-9b48-6ef3d70e5ada

@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)

2 reviewers were added to this PR based on Anton Bykov's automation.

@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 16, 2025

Merge activity

  • Sep 16, 10:00 PM UTC: kristina-solovyova added this pull request to the Graphite merge queue.
  • Sep 16, 10:01 PM UTC: kristina-solovyova removed this pull request from the Graphite merge queue.
  • 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.

3 participants