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

Adding Vishnu Kannan as a Maintainer. #223

Merged
merged 1 commit into from
Oct 13, 2015

Conversation

vishh
Copy link
Contributor

@vishh vishh commented Oct 12, 2015

Removing Victor Marmol and Rohit Jnagal from being Maintainers.

Signed-off-by: Vishnu kannan vishnuk@google.com

cc @crosbymichael @vmarmol @rjnagal

@wking
Copy link
Contributor

wking commented Oct 12, 2015

On Mon, Oct 12, 2015 at 03:53:00PM -0700, Vish Kannan wrote:

Removing Victor Marmol and Rohit Jnagal from being Maintainers.

I think they're there because they're listed in the draft charter [1](although that's not mentioned explicitly in #131). See also my
confusion about mapping TDC members to per-project maintainer lists
(opencontainers/web#5).

@vishh
Copy link
Contributor Author

vishh commented Oct 12, 2015

This PR was requested by @crosbymichael.

@wking
Copy link
Contributor

wking commented Oct 12, 2015

On Mon, Oct 12, 2015 at 04:06:58PM -0700, Vish Kannan wrote:

This PR was requested by @crosbymichael.

He proposed #131 too, so he can likely clarify why his reasoning
changed or what that reasoning was/is.

@LK4D4
Copy link
Contributor

LK4D4 commented Oct 13, 2015

Woot, what's up with travis?

@crosbymichael
Copy link
Member

@wking we know why they were there.

They have decided to step down because they don't have the time to devote to maintaining OCI because of other responsibilities.

@crosbymichael
Copy link
Member

@vishh FAIL - commit subject exceeds 90 characters

Signed-off-by: Vishnu kannan <vishnuk@google.com>
@vishh
Copy link
Contributor Author

vishh commented Oct 13, 2015

Trimmed down the subject.

@wking
Copy link
Contributor

wking commented Oct 13, 2015

On Mon, Oct 12, 2015 at 05:39:05PM -0700, Michael Crosby wrote:

@wking we know why they were there.

They have decided to step down because they don't have the time to
devote to maintaining OCI because of other responsibilities.

Thanks for clarifing :). Probably update opencontainers/web and
either update the draft charter there or point it at this MAINTAINERS
file as the source of truth for the TDC.

@tianon
Copy link
Member

tianon commented Oct 13, 2015

Sad to see them have to go 😢

LGTM

@rjnagal
Copy link

rjnagal commented Oct 13, 2015

LGTM and goodbye 😢

We are still watching you, though!

@mrunalp
Copy link
Contributor

mrunalp commented Oct 13, 2015

LGTM

mrunalp pushed a commit that referenced this pull request Oct 13, 2015
Adding Vishnu Kannan as a Maintainer.
@mrunalp mrunalp merged commit 0936a9c into opencontainers:master Oct 13, 2015
@@ -1,7 +1,6 @@
Michael Crosby <michael@docker.com> (@crosbymichael)
Alexander Morozov <lk4d4@docker.com> (@LK4D4)
Rohit Jnagal <jnagal@google.com> (@rjnagal)
Victor Marmol <vmarmol@google.com> (@vmarmol)
Vishnu Kannan <vishnuk@google.com> (@vishnuk)
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, is this actually the right GitHub ID? I expect it to be @vishh, not vishnuk (who doesn't seem to have any public activity).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants