-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
chore: fix CI by adding a dependency of python/sdk for the e2e-test #11221
Conversation
Suggestion to fix it properly.
The approach above is going to prevent this error from happening again. ( if it worked ) |
53576f4
to
ce90ec6
Compare
Signed-off-by: Ricardo M. Oliveira <[email protected]>
ce90ec6
to
b450b45
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chensun 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 |
/rerun-all |
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]> Signed-off-by: sefgsefg <[email protected]>
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]> Signed-off-by: sefgsefg <[email protected]>
too late now, but in the future add a note to the commit along the lines of |
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]>
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]>
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]> Signed-off-by: zed546213 <[email protected]>
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]> Signed-off-by: chris <[email protected]>
…ubeflow#11221) Signed-off-by: Ricardo M. Oliveira <[email protected]>
Description of your changes:
This PR is also reverting the files deletion that are required to run CI tests.
Checklist: