-
Notifications
You must be signed in to change notification settings - Fork 882
added production-users and integrations pages #3602
Conversation
Instead of libraries and tools it should just be "projects that integrate with rkt" |
Added links to README and tweaked verbiage per @philips |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks great. I've left a bunch of comment for minor improvements.
# Integrations | ||
This document tracks projects that integrate with rkt. [Join the community](https://github.com/coreos/rkt/), and help us keep the list up-to-date. | ||
|
||
## Projects |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to add cadvisor here.
Documentation/production-users.md
Outdated
## [Kumul.us](https://kumul.us/) | ||
Kumulus Technologies offers classes and services to optimize your Cloud. They've [blogged about](https://kumul.us/docker-youve-failed-me-again-rkt-please-save-me/) rkt's pod native features and Kubernetes support being a reason for their adoption. | ||
|
||
## [Kinvolk.io](https://kinvolk.io/blog/2016/02/welcome-rkt-1.0/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps this link should be switched to the naked domain, and a link to a real usecase (eg. https://kinvolk.io/blog/2017/02/using-custom-rkt-stage1-images-to-test-against-various-kernel-versions/) added in the long description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good notes! Just pushed an update with your suggestions @lucab
added cAdvisor and tweaked Kinvolk entry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Merging as I don't htink @pgburt knows the culture of once you get an LGTM you merge your own PR. |
Actually my fault, I thought I already merged this after the green light. |
Production-users and Integrations pages, per the request from @philips