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

fix: include model registry condition to wait for ready #1855

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

CFSNM
Copy link
Contributor

@CFSNM CFSNM commented Sep 24, 2024

No description provided.

Copy link

@@ -158,6 +158,14 @@
... label_selector=app.kubernetes.io/name=data-science-pipelines-operator
... timeout=400
END
${modelregistry} = Is Component Enabled modelregistry ${DSC_NAME}
IF "${modelregistry}" == "true"
Log To Console "Waiting for 1 pod in ${APPLICATIONS_NAMESPACE}, label_selector=app.kubernetes.io/part-of=model-registry-operator"

Check warning

Code scanning / Robocop

Line is too long ({{ line_length }}/{{ allowed_length }}) Warning

Line is too long (136/120)
Copy link
Contributor

Robot Results

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

@CFSNM CFSNM self-assigned this Sep 24, 2024
@CFSNM CFSNM added the verified This PR has been tested with Jenkins label Sep 24, 2024
@CFSNM CFSNM enabled auto-merge September 24, 2024 14:35
@CFSNM CFSNM merged commit 30c72fe into red-hat-data-services:master Sep 24, 2024
10 checks passed
@CFSNM CFSNM deleted the model_registry_condition branch September 24, 2024 14:42
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