-
Notifications
You must be signed in to change notification settings - Fork 212
FAI-957: TrustyAI inclusion #740
FAI-957: TrustyAI inclusion #740
Conversation
Hi @RobGeada. Thanks for your PR. I'm waiting for a opendatahub-io member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
407c118
to
c13f963
Compare
e7d2d7f
to
ea10019
Compare
/retest |
@RobGeada @anishasthana Do you have a set up CI tests to add so we can verify Trusty AI is working |
@LaVLaS Working on those now |
b12c09d
to
acc1534
Compare
@LaVLaS CI tests are now included |
/ok-to-test |
/retest CI infra issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you accidentally delete the Makefle
we use to build and run CI tests
/retest |
Yep, that is now restored |
/retest |
43a20c7
to
8854780
Compare
Update: The CI failures are being caused by the v0.11.0 build of the |
fb34683
to
039fdf4
Compare
@RobGeada Can you add an OWNERS file to the root of the |
Done |
/ok-to-test |
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@RobGeada Thanks for your efforts on this!
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: anishasthana, ruivieira 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 |
Description
How Has This Been Tested?
The E2E test in the TrustyAI downstream repo
Merge criteria:
Testing instructions:
git clone https://github.com/red-hat-data-services/trustyai-explainability