Skip to content

Refractor the script to work in both python2 and python3#4559

Merged
estesp merged 1 commit intocontainerd:masterfrom
bsdnet:master
Sep 12, 2020
Merged

Refractor the script to work in both python2 and python3#4559
estesp merged 1 commit intocontainerd:masterfrom
bsdnet:master

Conversation

@bsdnet
Copy link
Copy Markdown
Contributor

@bsdnet bsdnet commented Sep 11, 2020

Signed-off-by: Roy Yang [email protected]

@bsdnet
Copy link
Copy Markdown
Contributor Author

bsdnet commented Sep 11, 2020

@bsdnet
Copy link
Copy Markdown
Contributor Author

bsdnet commented Sep 11, 2020

@dims

@bsdnet
Copy link
Copy Markdown
Contributor Author

bsdnet commented Sep 11, 2020

/assign @dims @Random-Liu

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 11, 2020

Build succeeded.

Comment thread contrib/gce/configure.sh Outdated
Comment thread contrib/gce/configure.sh Outdated
@dims
Copy link
Copy Markdown
Member

dims commented Sep 11, 2020

@bsdnet one nit! looks good to me otherwise.

@bsdnet
Copy link
Copy Markdown
Contributor Author

bsdnet commented Sep 11, 2020

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 11, 2020

Build succeeded.

@MHBauer
Copy link
Copy Markdown
Contributor

MHBauer commented Sep 11, 2020

@bsdnet
Copy link
Copy Markdown
Contributor Author

bsdnet commented Sep 11, 2020

Will this be merged automatically like k8s repo or I need to something @dims @Random-Liu

@dims
Copy link
Copy Markdown
Member

dims commented Sep 11, 2020

@mikebrow @dmcgowan can you please look?

LGTM 👍 from me!

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit bb6590d into containerd:master Sep 12, 2020
@bsdnet
Copy link
Copy Markdown
Contributor Author

bsdnet commented Sep 12, 2020 via email

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.

4 participants