Skip to content

Conversation

@rhuss
Copy link
Contributor

@rhuss rhuss commented Jun 4, 2025

If you don't have locally a remote named "downstream" then the push for syncing tags doesn't work.
SOlution: Ensure that "downstream" exists, much like for "upstream"

@openshift-ci openshift-ci bot requested review from cardil and pierDipi June 4, 2025 14:13
@rhuss rhuss force-pushed the pr/fix-push-remote branch from 4dbb10e to a4a6ca2 Compare June 4, 2025 16:02
@rhuss rhuss changed the title fix: Use remote URL not name for push fix: Ensure that remote "downstream" is available Jun 4, 2025
Copy link
Member

@cardil cardil left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Jun 5, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cardil, rhuss

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

@openshift-ci openshift-ci bot added the approved label Jun 5, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit ead21be into openshift-knative:main Jun 5, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants