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

Add kubeflow/kubeflow repo into third-party-bots #425

Merged
merged 1 commit into from
Jan 14, 2021

Conversation

PatrickXYS
Copy link
Member

As part of kubeflow/kubeflow#5482

We discussed with @kubeflow/wg-notebooks-leads in the issue, and agreed to add kubeflow/kubeflow repo write access to third-party-bots, by merging this PR, we'll be able to set up CI for kubeflow/kubeflow on Optional-Test-Infra.

/cc @Bobgy
/cc @kubeflow/wg-notebooks-leads

@PatrickXYS
Copy link
Member Author

/cc @Bobgy

Can you help review? Let me know if any concerns

@PatrickXYS
Copy link
Member Author

@kubeflow/wg-notebooks-leads Can anyone comment here to say we're in an agreement to move forward with the current PR?

@thesuperzapper
Copy link
Member

/lgtm
Look good, we have talked about this in the WG-Notebook meeting, can one of @james-jwu or @Bobgy /aprove

@Bobgy
Copy link
Contributor

Bobgy commented Jan 14, 2021

/lgtm
/approve

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Aut0R3V, Bobgy, PatrickXYS

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit 0234f7e into kubeflow:master Jan 14, 2021
zijianjoy pushed a commit to zijianjoy/internal-acls that referenced this pull request Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants