Skip to content

Conversation

idvoretskyi
Copy link
Member

@idvoretskyi idvoretskyi added committee/steering Denotes an issue or PR intended to be handled by the steering committee. area/community-management labels Feb 7, 2018
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 7, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: idvoretskyi

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 7, 2018
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 7, 2018
* [kubernetes-security](https://github.com/kubernetes-security)
* [kubernetes-sig-testing](https://github.com/kubernetes-sig-testing)
* [kubernetes-providers](https://github.com/kubernetes-providers)
* [kubernetes-addons](https://github.com/kubernetes-addons)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer we differentiate between orgs we are actively using, vs orgs which we are holding for potential future use

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the original list I was trying to only list the orgs we are using and not the ones we are name squatting on. For example, we aren't using kubernetes-charts. You'll noticed the section title is "Current Organizations In Use". If there are no repos on an org or no immediate plans to start adding them can we not list them?

Copy link
Member Author

@idvoretskyi idvoretskyi Feb 7, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

second list has kuberentes-providers twice and is non alphabetized, but otherwise LGTM

@spiffxp
Copy link
Contributor

spiffxp commented Feb 13, 2018

@idvoretskyi any update on this?

@cblecker
Copy link
Member

/uncc

@k8s-ci-robot k8s-ci-robot removed the request for review from cblecker February 13, 2018 19:21
idvoretskyi added a commit that referenced this pull request Mar 12, 2018
@idvoretskyi
Copy link
Member Author

Closed in favor of #1923

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/community-management cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. committee/steering Denotes an issue or PR intended to be handled by the steering committee. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants