Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix kuma issues and heartbeat monitoring #104

Open
wants to merge 81 commits into
base: main
Choose a base branch
from

Conversation

rlratcliffe
Copy link
Contributor

@rlratcliffe rlratcliffe commented Oct 17, 2023

This is based off the K8s upgrade branch, so #102 needs to be merged before this. For ease of view before merge, relevant commits are here.

Due to an issue with duplicate listeners/uptime of pods in the cluster, some changes are also necessary to Tavros.

  • Update Flux because Kuma update requires a helm update
  • Update Kuma, some changes related to moving sidecar injection label instead of annotation
  • Update Heartbeat instances to fix issue with monitoring

  • Requires Jenkins Tavros library merge/release - Yes While testing, can test with v0.5-alpha.5
  • Requires Tavros chart release - Yes, switch 0.2.8 from prerelease to release
  • Requires archetype release - No
  • Requires update to tavros-collection image - Yes (because of flux version)

fix deprecated version + issue with missing quay image
old chart no longer existed, and values didn't match and needed to be reconfigured. old values are here: https://github.com/bitnami/charts/blob/4c72712f8d0c79ca3b573c87b993bfe80fc99b01/bitnami/postgresql/values.yaml
maybe not a good idea if it deletes IP stuff but trying it out
https://cert-manager.io/docs/installation/supported-releases/

prod-kong-ingress-certs & sandbox-kong-ingress-certs weren't being created, but are now
NOTE: this keycloak-operator is deprecated and no more fixes are available
was suddenly failing after many repeat runs
@rlratcliffe
Copy link
Contributor Author

rlratcliffe commented Oct 21, 2023

this requires a change to the tavros-collection docker image, but if that can't be done, here are the steps to use/test this:

# clone tavros, cd to directory

mkdir tmp
curl -sSL0 "https://github.com/fluxcd/flux2/releases/download/v0.41.2/flux_0.41.2_linux_amd64.tar.gz" | tar -zx -C tmp
chmod +x tmp/flux

docker run -v $PWD:$PWD -e TAVROS_DIR=$(pwd) -it ghcr.io/ms3inc/tavros-collection bash

cd /usr/local/bin && rm -rf flux
cd $TAVROS_DIR
cp tmp/flux /usr/local/bin/flux
flux -v #should equal 0.41.2

cd $TAVROS_DIR && make install

# IF using AKS
python -m ensurepip --upgrade
cd /root/.ansible/collections/ansible_collections/azure/azcollection
pip3 install -r requirements-azure.txt
# login to AKS
az login --tenant <tenant-id>

# provision
cd / && ansible-playbook $TAVROS_DIR/playbooks/provision_playbook.yaml \
--inventory "$TAVROS_DIR/playbooks/provision_playbook/default_vars.yaml"

@rlratcliffe rlratcliffe marked this pull request as ready for review October 21, 2023 21:05
@rlratcliffe rlratcliffe changed the title Fix kuma issues Fix kuma issues and heartbeat monitoring Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant