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

RHOAIENG-2512: Added test to validate the DSC and DSCI when Serverless operator is not installed #1574

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

CFSNM
Copy link
Contributor

@CFSNM CFSNM commented Jun 28, 2024

Added a new test where we check the DSC and DSCI conditions when the Serverless operator is not installed. The result should be an error in the conditions saying the operator is needed for the kserve component

@CFSNM CFSNM self-assigned this Jun 28, 2024
@CFSNM
Copy link
Contributor Author

CFSNM commented Jun 28, 2024

Validating in rhods-ci-pr-test 3078

Copy link
Contributor

github-actions bot commented Jun 28, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
499 0 0 499 100

@CFSNM CFSNM force-pushed the RHOAIENG-2512 branch 3 times, most recently from e9992d5 to 61811c8 Compare July 1, 2024 12:48
bdattoma
bdattoma previously approved these changes Jul 1, 2024
@CFSNM
Copy link
Contributor Author

CFSNM commented Jul 1, 2024

Validating in rhods-ci-pr-test 3078

Verified

@CFSNM CFSNM added the verified This PR has been tested with Jenkins label Jul 1, 2024
@CFSNM CFSNM enabled auto-merge July 1, 2024 14:47
@CFSNM CFSNM requested review from jgarciao, FedeAlonso and lugi0 July 1, 2024 15:18
Wait For DataScienceCluster CustomResource To Be Ready timeout=600
Set Service Mesh State To Managed And Wait For CR Ready
... ${SERVICE_MESH_CR_NAME} ${SERVICE_MESH_CR_NS} ${SERVICE_MESH_OPERATOR_NS}

Copy link
Contributor

Choose a reason for hiding this comment

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

Service Mesh is Managed when creating DSCI with default values. May not need to attempt to do so again.

@CFSNM CFSNM merged commit 753f805 into red-hat-data-services:master Jul 2, 2024
7 checks passed
@CFSNM CFSNM deleted the RHOAIENG-2512 branch July 3, 2024 08:13
CFSNM added a commit that referenced this pull request Jul 3, 2024
…SCI when Serverless operator is not installed (#1591)

Backports #1574
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants