fix containerd install, add checks for ubuntu version #84
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
xunholy
reviewed
Jan 12, 2021
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
xunholy
reviewed
Jan 24, 2021
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
xunholy
reviewed
Jan 25, 2021
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
Signed-off-by: Nick M <4718+rkage@users.noreply.github.com>
anthr76
reviewed
Jan 25, 2021
anthr76
added a commit
to carpenike/k8s-cluster-installation
that referenced
this pull request
Jan 25, 2021
This was dropped due to the latest merge and made it into the working tree raspbernetes#84 Signed-off-by: anthr76 <hello@anthonyrabbito.com>
xunholy
added a commit
that referenced
this pull request
Jan 30, 2021
* enable cilium CNI option
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* fix some variables
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* change variable name
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* set permissions on manifests folder and ignore curl warning
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* should have been inside args
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* add some creates pieces to commands
* add some creates pieces to commands
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* args needed to nest creates
Signed-off-by: Ryan Holt <ryan@ryanholt.net>
* add another pipefail option
* Updating variable role prefix and addressing some feedback
Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
* fix ansible lint issues
Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
* update helm values to be from a j2 template
Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
* Testing cilium on Kubic
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Add multi-arch envoyproxy/envoy
Opt to use newer offical image rather then 3rd party source.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Add quotes for endpoint routes
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Type check on inventory side
Need to read more about this
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Revised Values
* Moved masquerade into bpf block
* Trying tunnel
* Updated comments
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Disable tunnel
* Begin adding helm ansible galaxy module
Cilium is incomplete.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Revise cilium role to further use k8s modules
* Added check for kuberouter
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Set hostname to inventory hostname if not already set.
This will prevent haproxy from failing.
HA Proxy Daemon will fail to start if the hostnames are all the same.
```
fatal: [node-03]: FAILED! => changed=false
msg: |-
Unable to restart service haproxy: Job for haproxy.service failed because the control process exited with error code.
See "systemctl status haproxy.service" and "journalctl -xe" for details.
```
Instead of depending on the user setting the hostnames in cloudinit or
alike instead add a check.
This may have issues with SUSE. See
(ansible/ansible#42726)
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Add default pod subnet
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Try adding quotes on version.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* CNI Role Improvements
* Add meta for Ansible Kubernetes Collections
* Change `include_tasks` to `import_tasks` for ansible 2.7+
compatibility.
[1] (ansible/ansible#46177)
[2] (kubernetes-sigs/kubespray#3815)
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Change back to a include task
https://docs.ansible.com/ansible/latest/user_guide/playbooks_reuse.html
Aligns better with our current setup.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Began adding automating to some of cilium values.
Currently debugging URL split
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Use regex from `cluster_controlplane_endpoint`
To template values for Cilium.
1. Update Cilium values
2. Point K8s to right config
3. Add tests
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Bump tag values and template enovy version.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Drop cri_plugin for now
Also update inventory source and delegate to localhost.. also for now.
Change galaxy collection to new name.
https://github.com/ansible-collections/community.kubernetes/issues/221
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Install helm as a kubernete dependency
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Add openshift module as a dependency
This is required to leverage Kubernetes modules from ansible. Hopefully
this changes in the future.
According to git issues a .deb is planned. A RPM is available. There is
a package in the AUR though opted to install with PIP instead because I
would trust it better without testing.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Remove hostname
Set in a more elegant multi-os manner later.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Add community.general.modprobe as a requirement
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Drop FQCN, and test kubelt j2 logic
https://docs.cilium.io/en/v1.6/kubernetes/requirements/#enable-automatic-node-cidr-allocation-recommended
ansible/ansible#71824
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Enable automatic node CIDR allocation
https://docs.cilium.io/en/v1.6/kubernetes/requirements/#enable-automatic-node-cidr-allocation-recommended
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* allocate-node-cidrs is enabled by default
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Disable envoy version check
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Template out some variable values and disable BPF by default
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Remove old packages from Kubernetes role
This was dropped due to the latest merge and made it into the working
tree
#84
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Kube-Proxy revisions
Disable kube-proxy by default for cilium. Revise var handling.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Revise eBPF mount
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Change default variables
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* fix docker restart handler
https://github.com/raspbernetes/k8s-cluster-installation/pull/92/files
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Begin adding sysctl fix and enable hubble by default
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Some polishes
* Revise requirements to correctly point to modules.
* Disable hubble by default. Users can, and probably should do this on
their
[own](https://docs.cilium.io/en/v1.9/gettingstarted/k8s-install-kubeadm/#enable-hubble-for-cluster-wide-visibility)
in the meantime we'll not support but leave the option there.
* Add sysctl's for present systemd bug - cilium/cilium#10645
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Scope out cilium variables to examples.
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
* Small cleanups
* Added var twice
* `---`
Signed-off-by: anthr76 <hello@anthonyrabbito.com>
Co-authored-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
Co-authored-by: anthr76 <hello@anthonyrabbito.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Nick M 4718+rkage@users.noreply.github.com
Description
This PR updates the containers installation - rather than check for Debian, checks for ubuntu version.
Checklist