Skip to content

ci(telemetry): pin the shared workflow to its main merge commit - #827

Merged
ParidelPooya merged 1 commit into
mainfrom
ci/telemetry-repin-shared-workflow
Oct 24, 2026
Merged

ParidelPooya merged 1 commit into
mainfrom
ci/telemetry-repin-shared-workflow

Conversation

@ParidelPooya

Copy link
Copy Markdown
Contributor

Summary

The telemetry conformance workflow on main fails before any job starts. Run 40127496650 reports:

error parsing called workflow ... telemetry-suite.yml@3e4fdaf0dda35ed8f64fb0d18365ef05a6a447fe: workflow was not found.

3e4fdaf0 was a commit on the branch of aws/aws-durable-execution-conformance-tests#150. That PR was squash-merged and its branch was deleted, so the commit is no longer reachable from a branch, and GitHub Actions cannot resolve it.

Changes

Testing

  • The workflow file parses as YAML.
  • d4b7df92 contains .github/workflows/telemetry-suite.yml, and the compare API reports it as identical to conformance main.
  • This PR touches the workflow file, so the telemetry job runs on it.

@ParidelPooya
ParidelPooya requested a review from a team October 24, 2026 19:57
@ParidelPooya
ParidelPooya deployed to ai-pr-review-runtime October 24, 2026 19:57 — with GitHub Actions Active
@ParidelPooya
ParidelPooya deployed to ai-pr-review-runtime October 24, 2026 19:58 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown
Contributor

Codex AI review

No actionable findings in the PR changes. Residual test risk: static inspection could not verify the external workflow at the new SHA or its cloud execution.

Reviewed commit a4b976b67355fbba1c120c2f3c95b379084cf8c4. Workflow run

@ParidelPooya
ParidelPooya merged commit 541ea40 into main Oct 24, 2026
25 checks passed
@ParidelPooya
ParidelPooya deleted the ci/telemetry-repin-shared-workflow branch October 24, 2026 20:07

This branch was successfully deployed

1 active deployment
ai-pr-review-runtime — a4b976b6 Deployed Oct 24, 2026 by ParidelPooya via ai-pr-review / Codex review / Generate Codex review #2689
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