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.

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

Rename whitelist to allowlist #2338

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

tecarter94
Copy link
Contributor

@tecarter94 tecarter94 commented Jan 13, 2025

Rename whitelist to allowlist. Restore ability to override domain proxy version.

https://issues.redhat.com/browse/JBS-80

@tecarter94 tecarter94 requested a review from rnc January 13, 2025 05:30
@openshift-ci openshift-ci bot requested a review from vibe13 January 13, 2025 05:30
@openshift-ci openshift-ci bot added the lgtm label Jan 13, 2025
@rnc
Copy link
Collaborator

rnc commented Jan 13, 2025

I presume we need project-ncl/domain-proxy#7 merged first?

@tecarter94
Copy link
Contributor Author

/retest

1 similar comment
@rnc
Copy link
Collaborator

rnc commented Jan 13, 2025

/retest

@rnc rnc force-pushed the rename-domain-proxy-whitelist branch from f50496f to cb42e4c Compare January 13, 2025 11:24
@openshift-ci openshift-ci bot removed the lgtm label Jan 13, 2025
Copy link
Contributor

openshift-ci bot commented Jan 13, 2025

New changes are detected. LGTM label has been removed.

Copy link
Contributor

openshift-ci bot commented Jan 13, 2025

@tecarter94: The following test 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/jvm-build-service-in-repo-e2e 8fcd8d6 link true /test jvm-build-service-in-repo-e2e

Full PR test history. Your PR dashboard.

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.

@rnc rnc force-pushed the rename-domain-proxy-whitelist branch from 8fcd8d6 to 4589889 Compare January 13, 2025 13:19
@rnc rnc merged commit 5a95d39 into redhat-appstudio:main Jan 13, 2025
21 of 22 checks passed
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.

2 participants