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

feat: support the necessary configurability for testing odh-nightly builds #148

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

jiridanek
Copy link
Collaborator

I can't fully try this because operator in odh-nightlies gets stuck after deleting dsci, but these should be all the changes necessary.


The expected settings for running odh-nightly build, which is ODH packaged as if it was RHOAI, are something like

PRODUCT=odh
DEFAULT_DSCI_NAME=default-dsci

OPERATOR_INSTALL_TYPE=olm
OLM_SOURCE_NAME=rhoai-source
OLM_OPERATOR_NAME=rhods-operator
OLM_OPERATOR_CHANNEL=odh-nightlies
OLM_SOURCE_NAMESPACE=openshift-marketplace
OLM_OPERATOR_VERSION=1.18.0

@jiridanek jiridanek requested a review from jstourac June 27, 2024 11:36
…uilds

The expected settings for running odh-nightly build,
which is ODH packaged as if it was RHOAI, are something like

```
PRODUCT=odh
DEFAULT_DSCI_NAME=default-dsci

OPERATOR_INSTALL_TYPE=olm
OLM_SOURCE_NAME=rhoai-source
OLM_OPERATOR_NAME=rhods-operator
OLM_OPERATOR_CHANNEL=odh-nightlies
OLM_SOURCE_NAMESPACE=openshift-marketplace
OLM_OPERATOR_VERSION=1.18.0
```

* fixup, undo mistaken line before I realized I need to set this in `Environment`
@jiridanek jiridanek merged commit 1764396 into skodjob:main Jul 10, 2024
3 checks passed
@jiridanek jiridanek deleted the jd_odh_nightly branch July 10, 2024 11:09
jiridanek added a commit to jiridanek/odh-e2e that referenced this pull request Jul 19, 2024
…ate that if it already exists

This is a followup to skodjob#148 where I missed this.
jiridanek added a commit to jiridanek/odh-e2e that referenced this pull request Jul 19, 2024
…ate that if it already exists

This is a followup to skodjob#148 where I missed this.
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.

3 participants