Skip to content

Comments

refactor: reduce ci.sh endpoint duplication#20700

Draft
ludamad wants to merge 2 commits intonextfrom
ad/refactor/reduce-ci-duplication
Draft

refactor: reduce ci.sh endpoint duplication#20700
ludamad wants to merge 2 commits intonextfrom
ad/refactor/reduce-ci-duplication

Conversation

@ludamad
Copy link
Collaborator

@ludamad ludamad commented Feb 19, 2026

Consolidates duplicate code across ci.sh endpoints by extracting common patterns into reusable utilities and helper functions. Proposes multiple approaches to reduce code duplication while maintaining clarity and maintainability. This refactoring decreases the maintenance burden and makes future CI changes easier to implement consistently.

@ludamad ludamad force-pushed the ad/refactor/reduce-ci-duplication branch 2 times, most recently from 6adbdf3 to 3bd6f02 Compare February 20, 2026 02:20
ci.sh:
- network_ec2(): dispatches network commands to EC2, consolidating
  CI_DASHBOARD/INSTANCE_POSTFIX/CPUS/bootstrap_ec2 boilerplate
- setup_denoise_run(): sets DENOISE env and defines the run() function
  used by grind, merge-queue, merge-queue-heavy, and release

bootstrap.sh:
- bench_upload(): replaces 4 identical bench merge+upload blocks
- resolve_docker_image(): builds and pushes image to aztecdev if none provided,
  used by the three network deploy/bench cases
@ludamad ludamad force-pushed the ad/refactor/reduce-ci-duplication branch from 3bd6f02 to 7dd14c3 Compare February 20, 2026 03:55
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.

1 participant