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

test: add namespaced custom installation test #622

Merged
merged 2 commits into from
Sep 17, 2024

Conversation

osmman
Copy link
Contributor

@osmman osmman commented Sep 16, 2024

No description provided.

@openshift-ci openshift-ci bot requested review from bouskaJ and sallyom September 16, 2024 11:11
Copy link

openshift-ci bot commented Sep 16, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: osmman

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

@bouskaJ
Copy link
Collaborator

bouskaJ commented Sep 16, 2024

custom_install means custom operator installation (for example enhanced with proxy ENV). I think your test should go under integration test because it is executable on regular operator installation

@osmman osmman force-pushed the tturek/split-namespace-test branch 2 times, most recently from 1928dae to 50ebf3f Compare September 16, 2024 14:42
@osmman
Copy link
Contributor Author

osmman commented Sep 16, 2024

/retest

@osmman osmman force-pushed the tturek/split-namespace-test branch from 50ebf3f to 2c59468 Compare September 17, 2024 07:39
@bouskaJ
Copy link
Collaborator

bouskaJ commented Sep 17, 2024

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Sep 17, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 08f32a4 into main Sep 17, 2024
13 checks passed
@osmman osmman deleted the tturek/split-namespace-test branch November 7, 2024 16:13
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