Skip to content

fix: bumping ARO-Tools version so prow job executor respects dry-run#5370

Open
rachelvweber wants to merge 1 commit into
Azure:mainfrom
rachelvweber:main
Open

fix: bumping ARO-Tools version so prow job executor respects dry-run#5370
rachelvweber wants to merge 1 commit into
Azure:mainfrom
rachelvweber:main

Conversation

@rachelvweber
Copy link
Copy Markdown
Collaborator

@rachelvweber rachelvweber commented May 23, 2026

ARO-27224

What

bumping ARO-Tools version so prow job executor respects dry-run

Why

Prow job executor used to ignore the dry-run flag. This new version of ARO-Tools respects the flag

Testing

Tested by e2e-parallel gate

Special notes for your reviewer

PR Checklist

  • PR is scoped to a single task (no mixed concerns)
  • Title follows Conventional Commits format
  • Summary explains the "Why" behind the change
  • Linked to relevant ticket/issue
  • Screenshots included (if graph/UI/metrics changes)
  • Self-reviewed the diff
  • CI/CD checks are passing (ignore Tide)
  • Draft PR used for WIP (if applicable)
  • Commit history is clean (rebased/squashed)
  • Tricky code blocks are commented
  • Specific reviewers tagged
  • All comment threads resolved before merge

Copilot AI review requested due to automatic review settings May 23, 2026 00:13
@openshift-ci openshift-ci Bot requested review from bennerv and geoberle May 23, 2026 00:13
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

This PR bumps the github.com/Azure/ARO-Tools pseudo-version across the repo’s Go-based tooling and test modules so the prow-job-executor dependency includes the fix to respect the --dry-run flag.

Changes:

  • Updated github.com/Azure/ARO-Tools/* dependencies from v0.0.0-20260521134141-f6ebe44cb703 to v0.0.0-20260522235615-eaa4a26770c0 in affected modules.
  • Refreshed corresponding go.sum entries to match the new dependency versions/checksums.

Reviewed changes

Copilot reviewed 8 out of 16 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tooling/yamlwrap/go.mod Bumps ARO-Tools tools/yamlwrap (and indirect testutil) to the newer pseudo-version.
tooling/yamlwrap/go.sum Updates checksums for the bumped ARO-Tools dependencies.
tooling/templatize/go.mod Bumps ARO-Tools config/pipelines and related tool deps (including prow-job-executor) to the newer pseudo-version.
tooling/templatize/go.sum Updates checksums for the bumped ARO-Tools dependencies.
tooling/secret-sync/go.mod Bumps ARO-Tools tools/secret-sync and related indirect deps to the newer pseudo-version.
tooling/secret-sync/go.sum Updates checksums for the bumped ARO-Tools dependencies.
tooling/pipeline-documentation/go.mod Bumps ARO-Tools pipelines to the newer pseudo-version.
tooling/pipeline-documentation/go.sum Updates checksums for the bumped ARO-Tools dependency.
tooling/helmtest/go.mod Bumps ARO-Tools config/pipelines and related indirect deps to the newer pseudo-version.
tooling/helmtest/go.sum Updates checksums for the bumped ARO-Tools dependencies.
tooling/hcpctl/go.mod Bumps ARO-Tools tools/cmdutils to the newer pseudo-version.
tooling/hcpctl/go.sum Updates checksums for the bumped ARO-Tools dependency.
tooling/grafanactl/go.mod Bumps ARO-Tools tools/grafanactl (and indirect cmdutils) to the newer pseudo-version.
tooling/grafanactl/go.sum Updates checksums for the bumped ARO-Tools dependencies.
test/go.mod Bumps ARO-Tools config and tools/prow-job-executor to the newer pseudo-version used by tests.
test/go.sum Updates checksums for the bumped ARO-Tools dependencies.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 23, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: abiduke612, rachelvweber
Once this PR has been reviewed and has the lgtm label, please assign deads2k for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rachelvweber
Copy link
Copy Markdown
Collaborator Author

/retest

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.

3 participants