Skip to content

Update to latest commit of argo-rollouts-manager '67002e569d7f7de26a60a1f5a8d0b4924931b4da' - #1269

Open
jgwest wants to merge 2 commits into
redhat-developer:masterfrom
jgwest:upgrade-rollouts-manager
Open

Update to latest commit of argo-rollouts-manager '67002e569d7f7de26a60a1f5a8d0b4924931b4da'#1269
jgwest wants to merge 2 commits into
redhat-developer:masterfrom
jgwest:upgrade-rollouts-manager

Conversation

@jgwest

@jgwest jgwest commented Aug 31, 2026

Copy link
Copy Markdown
Member

Update to most recent 'argo-rollouts-manager' commit: argoproj-labs/argo-rollouts-manager@67002e5

…0a1f5a8d0b4924931b4da'

Signed-off-by: Jonathan West <jgwest@gmail.com>
@openshift-ci
openshift-ci Bot requested review from anandrkskd and svghadi August 31, 2026 22:08
@openshift-ci

openshift-ci Bot commented Aug 31, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign olivergondza 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

@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 382c0d25-7ca8-4ae6-ae68-daa09eba8c23

📥 Commits

Reviewing files that changed from the base of the PR and between 2b95bb3 and 2147c80.

📒 Files selected for processing (1)
  • scripts/run-rollouts-e2e-tests.sh
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • argoproj-labs/argocd-operator (manual)
🚧 Files skipped from review as they are similar to previous changes (1)
  • scripts/run-rollouts-e2e-tests.sh

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.


📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated release metadata to reflect the latest publication timestamp.
    • Updated rollout components used by the project to incorporate the latest available improvements.
    • Aligned rollout tooling versions across release and validation workflows.
    • Improved consistency between packaged release information and automated rollout checks, supporting more reliable deployment validation.

Walkthrough

The changes update the Argo Rollouts Manager dependency and E2E test pins. The CSV creation timestamp is also updated.

Changes

Release alignment

Layer / File(s) Summary
Version and metadata updates
go.mod, scripts/run-rollouts-e2e-tests.sh, bundle/manifests/gitops-operator.clusterserviceversion.yaml
The Argo Rollouts Manager dependency and E2E pins reference newer revisions. The CSV createdAt timestamp is updated to 2026-08-31T22:08:00Z.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to 2147c

This change updates the argo-rollouts-manager dependency and its end-to-end test script without supplied evidence of a concrete production or correctness risk; no actionable merge-blocking risk remains.

Suggested reviewers: anandrkskd, svghadi

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the update to the latest argo-rollouts-manager commit and matches the stated objective.
Description check ✅ Passed The description directly explains the argo-rollouts-manager dependency update and includes the target commit link.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files.


Comment @coderabbitai help to get the list of available commands.

Signed-off-by: Jonathan West <jgwest@gmail.com>
@jgwest

jgwest commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

�[36mINFO�[0m[2026-08-31T22:20:53Z] Acquiring leases for test kuttl-sequential: [gitops-aws-quota-slice] 
�[31mERRO�[0m[2026-09-01T00:20:53Z] error: Failed to acquire resource, current capacity: 0 free, 10 leased 

/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.

1 participant