Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(RELEASE-230): update release service ref to konflux ci org #3730

Merged
merged 1 commit into from
May 18, 2024
Merged

feat(RELEASE-230): update release service ref to konflux ci org #3730

merged 1 commit into from
May 18, 2024

Conversation

happybhati
Copy link
Contributor

This commit updates the release-service reference
as result of migration to konflux-ci from redhat-appstudio
more details parent EPIC:
https://issues.redhat.com/browse/RHTAPREL-800

@openshift-ci openshift-ci bot requested review from davidmogar and scoheb May 17, 2024 03:18
@redhat-appstudio-qe-bot
Copy link

❗ Detected an outage of the following critical component(s)❗

  • redhat: registry.connect.redhat.com

Due to this issue E2E tests will probably fail. Please keep an eye on the following status pages:

and add a comment /retest-required once the reported issues are solved

2 similar comments
@redhat-appstudio-qe-bot
Copy link

❗ Detected an outage of the following critical component(s)❗

  • redhat: registry.connect.redhat.com

Due to this issue E2E tests will probably fail. Please keep an eye on the following status pages:

and add a comment /retest-required once the reported issues are solved

@redhat-appstudio-qe-bot
Copy link

❗ Detected an outage of the following critical component(s)❗

  • redhat: registry.connect.redhat.com

Due to this issue E2E tests will probably fail. Please keep an eye on the following status pages:

and add a comment /retest-required once the reported issues are solved

@happybhati
Copy link
Contributor Author

/test appstudio-hac-e2e-tests

@happybhati
Copy link
Contributor Author

/retest

2 similar comments
@happybhati
Copy link
Contributor Author

/retest

@happybhati
Copy link
Contributor Author

/retest

@happybhati happybhati mentioned this pull request May 17, 2024
Closed
@johnbieren
Copy link
Member

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm label May 17, 2024
@happybhati
Copy link
Contributor Author

E2E time out :(
/retest

@happybhati
Copy link
Contributor Author

/test appstudio-hac-e2e-tests

@happybhati
Copy link
Contributor Author

/retest

2 similar comments
@happybhati
Copy link
Contributor Author

/retest

@happybhati
Copy link
Contributor Author

/retest

@happybhati
feat(RELEASE-230): update release service ref to konflux ci org
89518fa 
 This commit updates the `release-service` reference
 as result of migration to `konflux-ci` from `redhat-appstudio`
 more details parent EPIC:
 https://issues.redhat.com/browse/RHTAPREL-800

Signed-off-by: Happy Bhati <hbhati@redhat.com>
@openshift-ci openshift-ci bot removed the lgtm label May 18, 2024
@happybhati
Copy link
Contributor Author

/retest

@happybhati
Copy link
Contributor Author

/lgtm

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented May 18, 2024

@happybhati: you cannot LGTM your own PR.

In response to this:

/lgtm

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.

@johnbieren
Copy link
Member

/approve
/lgtm

@openshift-ci openshift-ci bot added the lgtm label May 18, 2024
@happybhati
Copy link
Contributor Author

happybhati commented May 18, 2024 •
edited

@scoheb could you please review/approve,Thanks!
FYI: Johnny already approved but I just saw he is not in the owners list.

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented May 18, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: happybhati, johnbieren, scoheb

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

The pull request process is described here

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

@scoheb
Copy link
Member

scoheb commented May 18, 2024

/lgtm

@happybhati happybhati merged commit 274651e into redhat-appstudio:main May 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scoheb scoheb scoheb approved these changes

@johnbieren johnbieren johnbieren approved these changes

@davidmogar davidmogar Awaiting requested review from davidmogar

Assignees

@scoheb scoheb

@johnbieren johnbieren

Labels
approved lgtm
Projects
None yet
Milestone
No milestone
4 participants
@happybhati @redhat-appstudio-qe-bot @johnbieren @scoheb