Skip to content
This repository has been archived by the owner. It is now read-only.

Updates #5 with resources #6

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

vielmetti
Copy link
Contributor

Add to resource-me per intent described in #5.

Add to resources per intent in #5.
@johnstudarus
Copy link
Contributor

If the goal is to stay focused on Kubernetes on Bare Metal, I recommend we omit the Packet-Terraform-Provider and Packet-Terraform-BGP demo from this guide. Kubespray on Packet encompasses the Terraform and MetalLB support.

@deitch
Copy link
Contributor

deitch commented Feb 13, 2019

@johnstudarus the goal is both. We definitely want to make this the "kube on metal" guide, i.e. platform agnostic. On the other hand, where there are resources that specifically focus on bare metal cloud providers (notably Packet, who is most kindly sponsoring this guide), then we want to highlight them as well.

So generally we want completely agnostic stuff in "Resource Me", and bare-metal-cloud-specific stuff in "Cloud Me".

Given that, I think it makes sense to have kubespray in "Resource Me" (the first half of the Kubespray part added in this PR, along with the title "Kubespray" or "Kubespray on Metal"), with reference to some of its cloud capabilities, but the Packet-specific stuff in "Cloud Me" (beginning with "Within the kubespray project").

Copy link
Contributor

@deitch deitch left a comment

Choose a reason for hiding this comment

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

Proposal:

  1. Rename "Kubespray with Packet..." title to just "Kubespray on Metal"
  2. Relocate the kubespray part beginning with "Within the kubespray project..." to "Cloud Me" doc.
  3. Have a reference to it here, something like, "kubespray has support for bare-metal clouds, see [link]"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants