search

kubeflow / internal-acls

Repository used to main group ACLs used by Kubeflow developers
Apache License 2.0
15 stars 326 forks source link

REQUEST: New membership for droctothorpe #656

Closed droctothorpe closed 8 months ago

droctothorpe commented 8 months ago

/kind area/process

Please read the guide lines for joining the Kubeflow GitHub org before opening an issue

Please provide links to PRs or other contributions (2-3):

Please list 2 existing members who are sponsoring your membership: @connor-mccarthy @andreyvelich

Please test your PR

Run

cd github_orgs
pytest test_org_yaml.py

Include the output in the PR

===================================================================================== test session starts =====================================================================================
platform darwin -- Python 3.11.4, pytest-7.4.3, pluggy-1.0.0
rootdir: /Users/inn487/code/kubeflow/internal-acls/github-orgs
plugins: anyio-4.0.0, cov-4.1.0
collected 1 item

test_org_yaml.py .                                                                                                                                                                      [100%]

====================================================================================== 1 passed in 0.11s ======================================================================================

Additional Instructions

After your PR is merged please wait at least 1 hour for changes to propogate.

If after an hour you haven't recieved an invite to join the GitHub org please open an issue.

You can contact build copy in #buildcop in kubeflow.slack.com

google-oss-prow[bot] commented 8 months ago

@droctothorpe: The label(s) kind/area/process cannot be applied, because the repository doesn't have them.

In response to [this](https://github.com/kubeflow/internal-acls/issues/656): >/kind area/process > >Please read the [guide lines](https://www.kubeflow.org/docs/about/contributing/#joining-the-community) for joining the Kubeflow GitHub org before opening an issue > >**Please provide links to PRs or other contributions (2-3):** >- https://github.com/kubeflow/pipelines/pull/10471 >- https://github.com/kubeflow/katib/pull/2183 >- https://github.com/kubeflow/pipelines/pull/8517 > >** Please list 2 existing members who are sponsoring your membership:** >@connor-mccarthy >@andreyvelich > >** Please test your PR ** > >Run > >``` >cd github_orgs >pytest test_org_yaml.py >``` >Include the output in the PR >``` >===================================================================================== test session starts ===================================================================================== >platform darwin -- Python 3.11.4, pytest-7.4.3, pluggy-1.0.0 >rootdir: /Users/inn487/code/kubeflow/internal-acls/github-orgs >plugins: anyio-4.0.0, cov-4.1.0 >collected 1 item > >test_org_yaml.py . [100%] > >====================================================================================== 1 passed in 0.11s ====================================================================================== >``` > >**Additional Instructions** > >After your PR is merged please wait at least 1 hour for changes to propogate. > >If after an hour you haven't recieved an invite to join the GitHub org please open an issue. > >You can contact build copy in #buildcop in kubeflow.slack.com Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
droctothorpe commented 8 months ago

Hm the instructions here in the community repo and the readme in this repo diverge from the instructions here on the website. Betting the website is a more reliable source of truth given the various errors in this issue template so closing this issue and filing a PR per its instructions. We may want to bring the two into alignment.