RHOAIENG-905 - Create a PR check for testing README file (without gitops)

opendatahub-io / ai-edge

ODH integration with AI at the Edge usecases

Apache License 2.0

9 stars 18 forks source link

RHOAIENG-905 - Create a PR check for testing README file (without gitops) #191

Closed MarianMacik closed 8 months ago

MarianMacik commented 9 months ago

…ops)

JIRA: RHOAIENG-905

Description

Adds 2 scripts:

common.sh which acts as a library
pipeline-test.sh currently testing just the build-container-image-pipeline and test-mlflow-image-pipeline with housing (TensorFlow) version of the pipeline run

How Has This Been Tested?

Locally using a separate cluster.

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

openshift-ci[bot] commented 9 months ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

openshift-ci[bot] commented 9 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: MarianMacik

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: - ~~[OWNERS](https://github.com/opendatahub-io/ai-edge/blob/main/OWNERS)~~ [MarianMacik] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

MarianMacik commented 9 months ago

/test all

MarianMacik commented 9 months ago

/retest

LaVLaS commented 9 months ago

@MarianMacik Can you add a link to the Jira issue in the description for anyone who want to check for additional context in the jira issue?

MarianMacik commented 9 months ago

/retest

MarianMacik commented 9 months ago

/retest

MarianMacik commented 9 months ago

/retest

MarianMacik commented 9 months ago

/retest

winklerm commented 8 months ago

Looks good to me, not approving now in case @LaVLaS has additional comments.

LaVLaS commented 8 months ago

/lgtm

Changes look reasonable to me.