Add milestone team, clean up teams in general

[zacharysarah]

For kubernetes/test-infra#11687 (comment).

This PR specifies who can use /milestone in the website repo.

Membership in this team requires an entry in a sig-docs-**-owners alias for a localization project in k/website. (See kubernetes/website#13287) Note that website-milestone-maintainers imports membership by reference to language owner groups.

This PR also updates sig-docs team membership based on k/website/OWNERS_ALIASES.

[zacharysarah]

/assign @nikhita

[Bradamant3]

Looks fine to me, with the possible exception of a stray whitespace character toward the top of the file. Not adding any commands because it looks like it's not ready to merge (conflicts, tests not passing), but content seems good.

[nikhita]

Unfortunately, teams can't be nested like this 😬

You'll either need to add this as a "parent team" to the sig-docs-* teams like https://github.com/kubernetes/org/blob/master/config/kubernetes/sig-cloud-provider/teams.yaml#L18 or expand all teams and list out members individually.

While the latter seems cumbersome, I'd mostly suggest that because GitHub says that (ref https://help.github.com/en/articles/about-teams#nested-teams):

Child teams inherit the parent's access permissions, simplifying permissions management for large groups. Members of child teams also receive notifications when the parent team is @mentioned, simplifying communication with multiple groups of people.

I'm not sure if this is a cause for concern here but it may lead to unexpected things with respect to permissions in the future.

[zacharysarah]

Understood! Listing folks out will be cumbersome for obvious reasons, but better safe than sorry.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chenopis, zacharysarah

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:

• config/kubernetes/sig-docs/OWNERS [chenopis,zacharysarah]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[zacharysarah]

@nikhita 👋 This PR is green now and ready to merge. 💚

[nikhita]

Since the link points to this file, maybe we drop it? :)

[nikhita]

All users need to defined as members instead of maintainers. Ref: #563

I have a PR to add a test for it #592, so this should be caught earlier in the future. 😬

[nikhita]

@zacharysarah added a couple of comments, but this looks good to go otherwise!

[nikhita]

Oh and one last thing...could you also sqaush the commits? :)

[zacharysarah]

@nikhita Two of the commits in this PR are merge commits and won't rebase properly for squashing. I'm going to close this PR and start fresh with a new one.

UPDATE: Rebooted as #644.