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

MAINTAINERS: add Aleksa Sarai #118

Merged
merged 1 commit into from
Feb 17, 2017
Merged

MAINTAINERS: add Aleksa Sarai #118

merged 1 commit into from
Feb 17, 2017

Conversation

cyphar
Copy link
Member

@cyphar cyphar commented Feb 16, 2017

Much of the work I've done in umoci is effectively rewriting parts of
image-tools so that they are better designed and stable. I feel very
strongly that most (if not all) of umoci should live within this
project.

As such, I feel that I'm sufficiently familiar with all of the many
implementation details necessary to correctly and sanely implement the
OCI image specification.

Signed-off-by: Aleksa Sarai asarai@suse.de

Much of the work I've done in umoci is effectively rewriting parts of
image-tools so that they are better designed and stable. I feel very
strongly that most (if not all) of umoci should live within this
project.

As such, I feel that I'm sufficiently familiar with all of the many
implementation details necessary to correctly and sanely implement the
OCI image specification.

Signed-off-by: Aleksa Sarai <asarai@suse.de>
@caniszczyk
Copy link
Contributor

Just as a note you have contributed to image-tools in the past:
https://github.com/opencontainers/image-tools/commits?author=cyphar

cc: @opencontainers/image-tools-maintainers

@cyphar
Copy link
Member Author

cyphar commented Feb 16, 2017

Before we merge this, there is some discussion about the future of umoci and image-tools that we should hash out on the mailing list first. I'll send a mail about it this afternoon/evening.

@vbatts
Copy link
Member

vbatts commented Feb 16, 2017 via email

@stevvooe
Copy link
Contributor

stevvooe commented Feb 16, 2017

@cyphar You beat me to it!

LGTM

Approved with PullApprove

@stevvooe
Copy link
Contributor

I've updated the commit to track the progress of the vote!

Good luck!

@vbatts
Copy link
Member

vbatts commented Feb 16, 2017

LGTM

Approved with PullApprove

4 similar comments
@jstarks
Copy link

jstarks commented Feb 16, 2017

LGTM

Approved with PullApprove

@philips
Copy link
Contributor

philips commented Feb 16, 2017

LGTM

Approved with PullApprove

@coolljt0725
Copy link
Member

coolljt0725 commented Feb 17, 2017

LGTM

Approved with PullApprove

@xiekeyang
Copy link
Contributor

xiekeyang commented Feb 17, 2017

LGTM

Approved with PullApprove

@jonboulle
Copy link
Contributor

jonboulle commented Feb 17, 2017 via email

@caniszczyk
Copy link
Contributor

caniszczyk commented Feb 17, 2017

7/9 votes is good enough to meet the 2/3 requirement.

Welcome @cyphar as an @opencontainers/image-tools-maintainers !

@caniszczyk caniszczyk merged commit 2817bc0 into opencontainers:master Feb 17, 2017
@cyphar cyphar deleted the add-maintainer branch February 17, 2017 06:24
wking added a commit to wking/oci-project-template that referenced this pull request Mar 20, 2017
Email-based voting seems to be a blocker for Michael, who prefers
PR-based voting for non-spec projects [1].  And the image-tools
maintainers had a PR-only vote to add Aleksa as a maintainer [2].

Specifications are still recommended to only use dev@ (for increased
visibility among folks with only peripheral involvement, e.g. spec
consumers).  But non-spec projects can backtrack more easily from
decisions which have negative impacts, so we don't have to push them
towards dev@ as firmly.

Adopting this more-relaxed approach should help reduce barriers to
non-spec projects adopting the vanilla governance procedures, which
may move us towards more consistency between OCI projects [3].

[1]: http://ircbot.wl.linuxfoundation.org/eavesdrop/%23opencontainers/%23opencontainers.2017-03-20.log.html#t2017-03-20T19:54:38
[2]: opencontainers/image-tools#118 (comment)
     Subject: MAINTAINERS: add Aleksa Sarai
[3]: opencontainers#4
     Subject: Apply consistent project template across OCI

Signed-off-by: W. Trevor King <wking@tremily.us>
wking added a commit to wking/oci-project-template that referenced this pull request Mar 20, 2017
Email-based voting seems to be a blocker for Michael, who prefers
PR-based voting for non-spec projects [1].  And the image-tools
maintainers had a PR-only vote to add Aleksa as a maintainer [2].

Specifications are still recommended to only use dev@ (for increased
visibility among folks with only peripheral involvement, e.g. spec
consumers).  But non-spec projects can backtrack more easily from
decisions which have negative impacts, so we don't have to push them
towards dev@ as firmly.

Adopting this more-relaxed approach should help reduce barriers to
non-spec projects adopting the vanilla governance procedures, which
may move us towards more consistency between OCI projects [3].

[1]: http://ircbot.wl.linuxfoundation.org/eavesdrop/%23opencontainers/%23opencontainers.2017-03-20.log.html#t2017-03-20T19:54:38
[2]: opencontainers/image-tools#118 (comment)
     Subject: MAINTAINERS: add Aleksa Sarai
[3]: opencontainers#4
     Subject: Apply consistent project template across OCI

Signed-off-by: W. Trevor King <wking@tremily.us>
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.

9 participants