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

Replace Jason Bouzane with Jon Johnson in image-spec maintainers #831

Merged
merged 1 commit into from
Apr 24, 2021

Conversation

estesp
Copy link
Contributor

@estesp estesp commented Mar 25, 2021

Switch approved by Jason Bouzane via email to OCI TOB mailing list: https://groups.google.com/a/opencontainers.org/g/tob/c/Q08hiintwY4/m/tMreB9LgCwAJ

Text:

From: Jason Bouzane <jbouzane@google.com>
Date: Thu, 25 Mar 2021 11:34:51 -0600
Message-ID: <CAFi6z1Hb+dC-=0g3rceGSyZ+dYHR=F8XNQvd4=0E=y9RaQ-pwA@mail.gmail.com>
Subject: Re: [oci-tob] image-spec maintainer gap
To: Vincent Batts <vbatts@hashbangbash.com>
Cc: Phil Estes <estesp@gmail.com>, Technical Oversight Board <tob@opencontainers.org>

I'm on PTO for a few days, so I'm not in a position to make the swap
myself. But I'm fine with this if someone else wants to do it.

This PR must get at least 6 LGTMs before merging:

Signed-off-by: Phil Estes estesp@gmail.com

Switch approved by Jason Bouzane via email to OCI TOB mailing list.

Signed-off-by: Phil Estes <estesp@gmail.com>
@estesp
Copy link
Contributor Author

estesp commented Mar 25, 2021

I think we want a threshold of approval from current maintainers as well as ack from @jonjohnsonjr (he verbally expressed interest on the weekly call); @vbatts can clarify on any existing TDC process.

@jonjohnsonjr
Copy link
Contributor

jonjohnsonjr commented Mar 26, 2021 via email

@dlorenc
Copy link

dlorenc commented Mar 26, 2021

Giant (non-binding) +1!

@vbatts
Copy link
Member

vbatts commented Mar 31, 2021

LGTM

Phil beat me to it, but I'll scratch out my personal TODO :-D

Approved with PullApprove

@cyphar
Copy link
Member

cyphar commented Apr 20, 2021

LGTM, but yeah we'd need to have the usual 6-LGTM threshold if we're going to do it by the book. We can take Jason's email as an LGTM.

Approved with PullApprove

@stevvooe
Copy link
Contributor

stevvooe commented Apr 21, 2021

LGTM

Approved with PullApprove

@caniszczyk
Copy link
Contributor

friendly ping @opencontainers/image-spec-maintainers

@brendandburns
Copy link

brendandburns commented Apr 21, 2021

LGTM

Approved with PullApprove

@vbatts
Copy link
Member

vbatts commented Apr 21, 2021 via email

@jstarks
Copy link
Contributor

jstarks commented Apr 23, 2021

LGTM

Approved with PullApprove

@cyphar
Copy link
Member

cyphar commented Apr 24, 2021

6-LGTM threshold reached, merging. Thanks all!

@cyphar cyphar merged commit 4518cd4 into opencontainers:master Apr 24, 2021
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