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

Add new-contributors.md #226

Merged
merged 1 commit into from
May 31, 2017
Merged

Conversation

fabiand
Copy link
Member

@fabiand fabiand commented May 30, 2017

Small document to provide some introductionlinks to related projects.

But let's start with how they relate to each other.

* **Kubernetes** is the upstream container orchestration engine, to run containers on a cluster
* **OpenShift** **Origin** is a friendly fork of Kubernetes with additional features, and configured to meet more enterprise requirements (i.e. RBAC authentication) than stock Kubernetes
Copy link
Contributor

Choose a reason for hiding this comment

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

AFAIK, nothing in kubevirt relies on openshift specific features, so i'd rather not mention openshift at all - it'll just serve to confuse IMHO.

Copy link
Member Author

Choose a reason for hiding this comment

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

Hm, makes sense. Especially if I consider i.e. tectonic, which is yet another K8s distrobution, then there is no particular reason why it must be mentioned here.

Removing it.

@@ -0,0 +1,46 @@
# Introduction

Let's start with the realtionship between the two important components:
Copy link
Contributor

Choose a reason for hiding this comment

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

s/realtionship/relationship


Let's start with the realtionship between the two important components:

* **Kubernetes** is the upstream container orchestration engine, to run containers on a cluster
Copy link
Contributor

Choose a reason for hiding this comment

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

... engine that runs containers in a cluster?

Copy link
Member Author

Choose a reason for hiding this comment

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

A bit cumbersome, rephrasing


Even that KubeVirt is an add-on to Kubernetes, both of them have things in common:

* Mostly written in Go Lang
Copy link
Contributor

Choose a reason for hiding this comment

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

golang


## Topics

### Go Lang
Copy link
Contributor

Choose a reason for hiding this comment

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

golang

@fabiand
Copy link
Member Author

fabiand commented May 30, 2017

Thanks updated

* **Kubernetes** is a container orchestration system, and is used to run containers on a cluster
* **KubeVirt** is an add-on which is installed on-top of Kubernetes, to be able to add basic virtualization functionality to Kubernetes.

Even that KubeVirt is an add-on to Kubernetes, both of them have things in common:
Copy link
Contributor

Choose a reason for hiding this comment

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

Trivial typo s/that/though/

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

Let's start with the relationship between the two important components:

* **Kubernetes** is a container orchestration system, and is used to run containers on a cluster
* **KubeVirt** is an add-on which is installed on-top of Kubernetes, to be able to add basic virtualization functionality to Kubernetes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Where reasonable, try to keep line length below 80 chars. A few more cases below, but don't artificially break hyperlinks in the middle

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, that's tricky if the links get to long.
For now I shortened the lines, but even if the link exceeded the 80char limit, then I left it as it.

Patterns

* [Introducing Operators: Putting Operational Knowledge into Software](https://coreos.com/blog/introducing-operators.html "https://coreos.com/blog/introducing-operators.html")
* [Microservices](https://martinfowler.com/articles/microservices.html "https://martinfowler.com/articles/microservices.html") nice content by Martin Fowler
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatting in this block looks kind of odd. 'Getting started' is in a list bullet, but 'Patterns' is not ?!?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, fixed it.


Let's start with the relationship between the two important components:

* **Kubernetes** is a container orchestration system, and is used to run containers on a cluster
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit-pick - only need a single whitespace after the list bullet - repeated elsewhere in the file

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

@fabiand
Copy link
Member Author

fabiand commented May 31, 2017

Updated.

@fabiand fabiand force-pushed the new-contributors branch 2 times, most recently from ee9f3aa to ab73678 Compare May 31, 2017 08:11
@fabiand
Copy link
Member Author

fabiand commented May 31, 2017

Sigh - Now even with fixed indentation.

@berrange berrange merged commit 5d395f5 into kubevirt:master May 31, 2017
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.

None yet

3 participants