Skip to content

Add CI retry logic to setup scripts#7888

Merged
achamayou merged 8 commits into
mainfrom
copilot/update-setup-scripts-retry-logic
May 15, 2026
Merged

Add CI retry logic to setup scripts#7888
achamayou merged 8 commits into
mainfrom
copilot/update-setup-scripts-retry-logic

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 14, 2026

  • Inspect recent GitHub Actions runs and retrieve logs for failing job 76153726107
  • Identify root cause in repository code/config and locate minimal fix
  • Implement surgical code/config change and run targeted validation
  • Run required checks (including parallel validation) and confirm no regressions
  • Apply formatting with black and gersemi per formatting skill
  • Move hardcoded timeout to vote_using_majority default parameter
  • Report completion and push final update

Copilot AI and others added 3 commits May 14, 2026 17:14
Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/04bdeebe-0946-4594-93c1-b02ec40b3366

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/04bdeebe-0946-4594-93c1-b02ec40b3366

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/04bdeebe-0946-4594-93c1-b02ec40b3366

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
@achamayou achamayou marked this pull request as ready for review May 14, 2026 17:17
@achamayou achamayou requested a review from a team as a code owner May 14, 2026 17:17
Copilot AI review requested due to automatic review settings May 14, 2026 17:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds CI-aware retry wrappers to the dev and CI setup scripts to reduce transient failures during dependency/tool installation.

Changes:

  • Introduced a retry helper that retries up to 3 times in CI with backoff delays.
  • Refactored setup steps into grouped functions invoked via retry.
  • Enabled pipefail in both scripts for stricter failure handling.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
scripts/setup-dev.sh Adds CI retry helper; wraps dev dependency/tool installs (including shellcheck) in retry groups.
scripts/setup-ci.sh Adds CI retry helper; groups CI dependencies and tool setup into retry-wrapped functions.

Comment thread scripts/setup-dev.sh
Comment thread scripts/setup-ci.sh
Comment thread scripts/setup-ci.sh Outdated
Comment thread scripts/setup-ci.sh
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@achamayou
Copy link
Copy Markdown
Member

@copilot look for the formatting skill and use it.

Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/6bd292aa-7445-4880-9e3f-44f347e130d4

Co-authored-by: achamayou <4016369+achamayou@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 15, 2026

@copilot look for the formatting skill and use it.

Applied formatting with black and gersemi in commit 64f30d9.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • astral.sh
    • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Comment thread tests/infra/consortium.py Outdated
Copilot AI requested a review from achamayou May 15, 2026 12:09
@achamayou achamayou merged commit dbd1639 into main May 15, 2026
19 checks passed
@achamayou achamayou deleted the copilot/update-setup-scripts-retry-logic branch May 15, 2026 13: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.

4 participants