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

podex: initial import #1537

Merged
merged 14 commits into from
Oct 3, 2014
Merged

podex: initial import #1537

merged 14 commits into from
Oct 3, 2014

Conversation

proppy
Copy link
Contributor

@proppy proppy commented Oct 2, 2014

podex is a command line tool to bootstrap a kubernetes container manifests from docker image metadata.

Manifests can then be edited by a human to match deployment needs.

Example usage:

$ docker pull google/nodejs-hello
$ podex -yaml google/nodejs-hello > google/nodejs-hello/pod.yaml
$ podex -json google/nodejs-hello > google/nodejs-hello/pod.json

The podex/manifests subdirectory could also be used to host example pods for popular docker images.

//
// Example usage:
//
// $ docker pull google/nodejs-hello
Copy link
Member

Choose a reason for hiding this comment

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

No way to inspect without pulling?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe using the docker-registry/hub API, but I couldn't get it to work. So I gave up and used the remote API for now.

I will add a TODO.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@thockin
Copy link
Member

thockin commented Oct 2, 2014

First big issue - do we want to check stuff like this into kubernetes tree directly? If so, where? I'm OK with checking in small, highly coupled stuff in here, but there's a line somewhere. E.g. heapster is a separate project.

Assuming we're OK with checking it in, where? I'd say this goes in contrib/podex

You need a README.md and a MAINTAINERS.md so it is very clear that this is not a core piece (i.e. not guaranteed to stay functional)

@proppy
Copy link
Contributor Author

proppy commented Oct 2, 2014

@thockin Done. PTAL


## OS Distributions

* RedHat, Fedora: [Clayton Coleman](https://github.com/smarterclayton), [Derek Carr](https://github.com/derekwaynecarr), [Scott Collier](https://github.com/scollier)
Copy link
Member

Choose a reason for hiding this comment

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

Why did you import the whole top-level MAINTAINERS here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ahah because I wanted to reuse the structure, and forgot to clean it up :)

Copy link
Member

Choose a reason for hiding this comment

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

Ping me when done, I'll merge

On Thu, Oct 2, 2014 at 5:24 PM, Johan Euphrosine notifications@github.com
wrote:

In contrib/podex/MAINTAINERS.md:

@@ -0,0 +1,18 @@
+# Maintainers
+
+People responsible for ports of Kubernetes to different environments. CC at least one maintainer on relevant issues and PRs.
+
+## OS Distributions
+
+* RedHat, Fedora: Clayton Coleman, Derek Carr, Scott Collier

ahah because I wanted to reuse the structure, and forgot to clean it up :)

Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/1537/files#r18376338
.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@thockin
Copy link
Member

thockin commented Oct 3, 2014

LGTM after last comments

@proppy
Copy link
Contributor Author

proppy commented Oct 3, 2014

@thockin Thanks for the quick review.

thockin added a commit that referenced this pull request Oct 3, 2014
@thockin thockin merged commit 8fdae1d into kubernetes:master Oct 3, 2014
benluddy pushed a commit to benluddy/kubernetes that referenced this pull request Apr 14, 2023
…tion

UPSTREAM: <carry>: add shutdown annotation to response header
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.

3 participants