Skip to content

chore: sync ARO-Tools to c3198b56b79a#5990

Closed
mmazur wants to merge 1 commit into
Azure:mainfrom
mmazur:sync-aro-tools
Closed

chore: sync ARO-Tools to c3198b56b79a#5990
mmazur wants to merge 1 commit into
Azure:mainfrom
mmazur:sync-aro-tools

Conversation

@mmazur

@mmazur mmazur commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

🤖 Generated with Claude Code

Updates:
- fix: enable Grafana viewers can edit (Azure#266)
- feat(graph): support multi-stamp execution graphs (Azure#250)
- Fix grafana schema (Azure#263)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 9, 2026 09:57
@openshift-ci

openshift-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mmazur

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

The pull request process is described 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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 syncs ARO-HCP’s Go tooling/test modules to a newer ARO-Tools revision (c3198b56b79a), picking up upstream changes (Grafana permissions/schema fixes and multi-stamp execution graph support).

Changes:

  • Bump ARO-Tools pseudo-versions across multiple internal modules from ...4612291d5420 to ...c3198b56b79a.
  • Update corresponding go.sum entries to match the new ARO-Tools revisions.

Reviewed changes

Copilot reviewed 9 out of 18 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tooling/yamlwrap/go.mod Bumps tools/yamlwrap (and indirect testutil) to the new ARO-Tools revision.
tooling/yamlwrap/go.sum Updates checksums for bumped ARO-Tools yamlwrap/testutil entries.
tooling/templatize/go.mod Bumps ARO-Tools dependencies used by templatize to the new revision.
tooling/templatize/go.sum Updates checksums for the bumped ARO-Tools dependencies.
tooling/secret-sync/go.mod Bumps tools/secret-sync and indirect ARO-Tools deps to the new revision.
tooling/secret-sync/go.sum Updates checksums for bumped ARO-Tools dependencies.
tooling/pipeline-documentation/go.mod Bumps pipelines to the new ARO-Tools revision.
tooling/pipeline-documentation/go.sum Updates checksums for bumped ARO-Tools pipelines entry.
tooling/helmtest/go.mod Bumps ARO-Tools dependencies used by helmtest to the new revision.
tooling/helmtest/go.sum Updates checksums for bumped ARO-Tools dependencies.
tooling/hcpctl/go.mod Bumps tools/cmdutils to the new ARO-Tools revision.
tooling/hcpctl/go.sum Updates checksums for bumped ARO-Tools cmdutils entry.
tooling/grafanactl/go.mod Bumps tools/grafanactl (and indirect cmdutils) to the new ARO-Tools revision.
tooling/grafanactl/go.sum Updates checksums for bumped ARO-Tools grafanactl/cmdutils entries.
test/go.mod Bumps direct and indirect ARO-Tools dependencies used by the test module to the new revision.
test/go.sum Updates checksums for bumped ARO-Tools dependencies (but still includes old testutil revision).
mgmt-agent/go.mod Bumps ARO-Tools testutil to the new revision.
mgmt-agent/go.sum Updates checksums for bumped ARO-Tools testutil entry.

Comment thread test/go.mod
Comment on lines 71 to +75
dario.cat/mergo v1.0.1 // indirect
github.com/Azure/ARO-Tools/pipelines v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/tools/cmdutils v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/tools/grafanactl v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/tools/helm v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/tools/registration v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/tools/secret-sync v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/tools/yamlwrap v0.0.0-20260703091318-4612291d5420 // indirect
github.com/Azure/ARO-Tools/pipelines v0.0.0-20260709090636-c3198b56b79a // indirect
github.com/Azure/ARO-Tools/tools/cmdutils v0.0.0-20260709090636-c3198b56b79a // indirect
github.com/Azure/ARO-Tools/tools/grafanactl v0.0.0-20260709090636-c3198b56b79a // indirect
github.com/Azure/ARO-Tools/tools/helm v0.0.0-20260709090636-c3198b56b79a // indirect
@openshift-ci

openshift-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

@mmazur: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/verify a700f8e link true /test verify
ci/prow/lint a700f8e link true /test lint
ci/prow/config-change-detection a700f8e link true /test config-change-detection
ci/prow/e2e-images a700f8e link true /test e2e-images
ci/prow/periodic-healthcheck-images a700f8e link true /test periodic-healthcheck-images
ci/prow/image-updater-images a700f8e link true /test image-updater-images
ci/prow/images a700f8e link true /test images
ci/prow/e2e-parallel a700f8e link true /test e2e-parallel

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants