chris-short / rak8s

Stand up a Raspberry Pi based Kubernetes cluster with Ansible
MIT License
365 stars 112 forks source link

K8S patch and other clean up items #51

Closed jimhopkinsjr closed 5 years ago

jimhopkinsjr commented 5 years ago

Description

Updates to rak8s for a new installation on freshly installed b+'s Commit notes are pretty lengthy, but here is the synopsis:

  1. Updated base k8s version to something that isn't vulnerable.
  2. Removed control path setting as it wasn't working on my ubuntu 16.04 vm /shrug
  3. Updated some flannel logic
  4. moved a few things around in how apt-get is handled.

Testing

3 new raspberry pi b+'s

  1. ran installation -> successful
  2. ran cleanup
  3. ran installation again

Issue Number

My change fixes issue #50

StoicPerlman commented 5 years ago

Just confirming this installs v1.13.1 without issue for me

chris-short commented 5 years ago

python-apt probably needs to be available prior to calling apt with python. Also, there are some conflicts.

jimhopkinsjr commented 5 years ago

@chris-short I'll get this cleaned up. I saw the PR's and assumed it would have conflicts.

Thanks

sbaeurle commented 5 years ago

hey @jimhopkinsjr what's the current status of your clean-up? do you need some help or are you mostly done?`

(I'm planning to use rak8s but want to wait till version 1.13.1)

jimhopkinsjr commented 5 years ago

@sbaeurle picking this up again now. I'll updated when its there

chris-short commented 5 years ago

Thank you Jim.

jimhopkinsjr commented 5 years ago

@chris-short - I'm not super happy with how the changes I've introduced are laid out are so I'll likely submit another PR (not idempotent, PI restarts are always firing). Also a this point when adding a new node the GPU check failed, so I need to fix that since it responded with an rc 1.

.edit. tagging right chris.... :fail:

On Tue, Apr 16, 2019 at 7:19 AM Chris Short notifications@github.com wrote:

Thank you Jim.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rak8s/rak8s/pull/51#issuecomment-483635771, or mute the thread https://github.com/notifications/unsubscribe-auth/AFkOC359EB-1I5qEEgIkE18zOb-8mlYjks5vhb_lgaJpZM4ZlOV5 .