ONS Tutorial updates #263

dougbtv · 2018-10-25T22:06:28Z

This is... A big pull request. In general, it shouldn't have a super large impact on the core functionality. There's a few things we need from this to move forward as it fixes a couple bugs that we have noticed along the way too.

Happy to either work on fixes herein, or... Fix up as pull requests after the fact. I'll leave that to you guys if you have any preference.

… deps #250 #247

…rently working userspace CNI plugin run

…ocess

…tup with some shell scripts, only lightly tested

…ates to use non-bridge mode networking (e.g. NAT'd)

…id, includes way to ignore kube version during kubeadm init/join

…ice_id branch for api

…etwork-device-plugin

…in bootstrapping process

…rfaces and add custom Kubernetes configurations

… kube configs

…oxy issue for KUral

…ubernetes version pin on kubeadm tasks

…RNING: Could not initially verify that virt-net-device-plugin worked

… configs on the nodes as well as master

leifmadsen · 2018-10-25T22:16:28Z

@dougbtv somewhat related/unrelated, would you mind fixing up the .travis-ci.yml file and making sure the linting is hitting the appropriate files? It would be good to validate that your changes are ansible-lint valid as well...

leifmadsen

Needs a bunch of work before merging to master.

leifmadsen · 2018-10-25T22:17:36Z

README.md

@@ -195,6 +195,24 @@ kube-node-3   Ready     <none>    9m        v1.8.3

 Everything should be marked as ready. If so, you're good to go!

+## Creating a bootstrapped image
+
+Should you need to spin up multiple clusters or otherwise spin up a bunch of VMs for a cluster, it may behoove you to "bootstrap" your VM images so that you don't have to download the dependencies many times. You can create a sort of golden image to use by using the `./playbooks/create-bootstrapped-image.yml` playbook.


Documentation wrapped to 80 chars please...

leifmadsen · 2018-10-25T22:18:28Z

contrib/multi-cluster/all.sh

@@ -0,0 +1,5 @@
+#!/bin/bash


Suggested change

#!/bin/bash

#!/usr/bin/env bash

leifmadsen · 2018-10-25T22:18:49Z

contrib/multi-cluster/extravars-creator.sh

@@ -0,0 +1,68 @@
+#!/bin/bash


Suggested change

#!/bin/bash

#!/usr/bin/env bash

leifmadsen · 2018-10-25T22:19:33Z

contrib/multi-cluster/multi-init.sh

@@ -0,0 +1,12 @@
+#!/bin/bash


Suggested change

#!/bin/bash

#!/usr/bin/env bash

leifmadsen · 2018-10-25T22:19:42Z

contrib/multi-cluster/multi-spinup.sh

@@ -0,0 +1,13 @@
+#!/bin/bash


Suggested change

#!/bin/bash

#!/usr/bin/env bash

leifmadsen · 2018-10-25T22:40:02Z

roles/ehost-device-cni/tasks/main.yml

+    chdir: "{{ ansible_env.HOME }}/src/go/src/github.com/zshi-redhat/ehost-device-cni"
+
+- name: Copy Userspace CNI plugin to CNI bin dir
+  shell: >


Suggested change

shell: >

command: >

leifmadsen · 2018-10-25T22:44:52Z

roles/userspace-cni/tasks/main.yml

+    gpgcheck: no
+
+- name: Install VPP repo
+  yum:


Suggested change

yum:

package:

leifmadsen · 2018-10-25T22:45:26Z

roles/userspace-cni/tasks/main.yml

+
+# How do I install the proper VPP version?
+- name: Install VPP
+  yum:


As previously, this is cleaner with the vars approach to allow better dependency resolution with the added benefit that it'll be like 1000 times faster.

leifmadsen · 2018-10-25T22:45:44Z

roles/userspace-cni/tasks/main.yml

+
+
+- name: Start & Enable VPP
+  service:


Probably mean systemd here instead of service.

leifmadsen · 2018-10-25T22:46:07Z

roles/userspace-cni/tasks/main.yml

+    enabled: yes
+
+- name: Ensure git is installed
+  yum:


Suggested change

yum:

package:

This change allows to deploy userspace cni OVS with DPDK. Multus roles have been updated to support current kubernetes APIs. Docker proxy support has been added.

redhat-nfvpe/ansible-role-install-docker#20

Added userspace cni ovs dpdk

dougbtv added 30 commits August 14, 2018 16:07

[enhancement] Adds method to create a 'bootstrap image' which contain…

58b2682

… deps #250 #247

[userspace-cni] Has preliminary portions ready that result in an appa…

d5b3e08

…rently working userspace CNI plugin run

[userspace-cni] Adds userspace CNI to kube-install / bootstrapping pr…

fd5bbec

…ocess

[bootstrap] Adds kubeadm config images pull, pulls flannel image

a881bf6

[minor] Has instructions to install tmate in bootstrapping procedure

188209c

[minor][increment] Adds kube-niceties to bootstrap removes from init

27a40c0

[multi-cluster] Has all the basics for spinning up a multi-cluster se…

37f896c

…tup with some shell scripts, only lightly tested

[ons][docs] Updates docs to spell out multi-cluster install, also upd…

b8d147a

…ates to use non-bridge mode networking (e.g. NAT'd)

[ons][significant] Updates to use custom Kubernetes build for device_…

5b3f9ce

…id, includes way to ignore kube version during kubeadm init/join

[ons] Updates to use custom kube-api-server image to use dashpole dev…

f120ed2

…ice_id branch for api

[ons][stub][incomplete] Stubs in basics for ehost-device-cni + virt-n…

c947696

…etwork-device-plugin

[ons][ehost-device] Has ehost-device cni plugin working

86182c8

[ons][virt-network-device-plugin] Enables virt network device plugin …

a17605d

…in bootstrapping process

[ons][virt-network-device-plugin] Has portions to add additional inte…

366065f

…rfaces and add custom Kubernetes configurations

[ons][tmate] Has a generally working tmate script

71b3427

[ons][bugfix] Missing proper path for kubelet config when customizing…

c710e24

… kube configs

[ons][multi-cluster] Update readme to include galaxy requirements

f0d16de

[ons] Updates sizing to use one core per VM

ccfbf5a

[ons][minor][bugfix] replace erroneously removed ssh keygen

32fa3a8

[ons][quick] Updates to use curl instead of get_url to work around pr…

b74f3ce

…oxy issue for KUral

[ons][quick] add http_proxy for kural

d401ff2

[ons] Add download bootstrap image option

8e65c5b

[ons][significant] Adds static IP method for VM IP assignment, adds k…

8e5e8a8

…ubernetes version pin on kubeadm tasks

[ons] adds tmate role to create shell script

61a13cb

[ons][minor][docs] adds info about adding additional iface

6c85e2b

[ons][kural] adds ignore errors on the set bridge-nf-call-iptables

f863480

[ons][docs][minor] removes echo from example add interface command

3ab6449

[ons][tweak] remove kubectl proxy

a632fae

[ons][tweak] add sleep before init

79cfe1d

[ons][userspace-cni][WARNING] Updates to use latest userspace cni, WA…

5ef050f

…RNING: Could not initially verify that virt-net-device-plugin worked

[ons][bugfix] allows virt DP to use daemonset style by modifying kube…

7f72b90

… configs on the nodes as well as master

ghost assigned dougbtv Oct 25, 2018

ghost added the state:needs_review label Oct 25, 2018

dougbtv requested review from leifmadsen and s1061123 October 25, 2018 22:06

leifmadsen suggested changes Oct 25, 2018

View reviewed changes

pmossakx and others added 7 commits October 29, 2018 14:53

Added userspace cni ovs dpdk

9e161f0

This change allows to deploy userspace cni OVS with DPDK. Multus roles have been updated to support current kubernetes APIs. Docker proxy support has been added.

Moved specific options to ovs-dpdk vars

1e07721

Moved multus options to ovs-dpdk vars

1aaa190

Don't use proxy by default

ff7e1eb

Switched back to use default nfvpe install docker role

2ce59d8

redhat-nfvpe/ansible-role-install-docker#20

Added missing variables disabling customization

fa8e26c

Merge pull request #264 from pmossakx/feature/ons-tutorial-ovsdpdk

7993159

Added userspace cni ovs dpdk

s1061123 force-pushed the master branch 9 times, most recently from c75c368 to c8b4850 Compare February 28, 2021 18:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ONS Tutorial updates #263

ONS Tutorial updates #263

dougbtv commented Oct 25, 2018

leifmadsen commented Oct 25, 2018

leifmadsen left a comment

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

leifmadsen Oct 25, 2018

ONS Tutorial updates #263

Are you sure you want to change the base?

ONS Tutorial updates #263

Conversation

dougbtv commented Oct 25, 2018

leifmadsen commented Oct 25, 2018

leifmadsen left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment