Skip to content

"Upgrade AKS to 1.24" - Change helm install logic azdevops

Arturo Hernandez [EPAM] requested to merge 232-upgrade-aks-helm-azdevops into master

All Submissions:

  • [YES] Have you added an explanation of what your changes do and why you'd like us to include them? (Already explanation on the services side to install osdu through pipelines)
  • [YES] I have updated the documentation accordingly. - Used same documentation, however, added extra comments.
  • [YES] My code follows the code style of this project.

Current Behavior or Linked Issues

It is needed to upgrade AKS to the latest version, however, templates that are currently under charts/ directory of this repositories are obsolete, and most likely, will cause upgrade failure on aks v0.24.

  • Motivation is to change pipeline logic, adapt helm-charts-azure to be able to work with helm install instead of flux. Main idea it is to get rid of flux and adopt helm-charts-azure instead.
  1. Added optional conditional to use new helm-charts-azure approach, enabled by default, helm version upgraded.
  2. Flux wait and helm template got new conditional to uninstall instead of installing.
    • If condition migrationCleanup is enabled, pipeline will remove the templates generated by helm-template in the k8s-gitlab repository (Flux).
    • In case flux was disabled contains k8s deletion of each template, so it will try to remove all templates related to that service.
    • In this MR, we will reuse the osdu namespace, it might be convenient if clients want to migrate in the future to osdu-azure ns.
  3. In this MR, logic of pipelines will be changed:
    1. Pipeline chart-osdu-common:
      • Install osdu-base helm charts
      • Install osdu-azure/osdu-common (appgw ingress and other common stuff needed)
    2. Pipeline osdu-istio
      • Install istio-base
      • Install istio-operator
    3. Pipeline osdu-istio-auth
      • Install osdu-istio
      • (Temporary until full migration) Install entitlements along with envoy filters without check

In summary, the pipelines will get new stages, besides the helm template.

Screenshots:

chart-osdu-common

chart-osdu-istio

chart-osdu-istio-auth

In dev environment:

Common: https://dev.azure.com/osdu-demo/OSDU_Rx/_build/results?buildId=237191&view=results

istio-base https://dev.azure.com/osdu-demo/OSDU_Rx/_build/results?buildId=237202&view=results

osdu-istio https://dev.azure.com/osdu-demo/OSDU_Rx/_build?definitionId=1244&_a=summary

Upgraded helm charts as well in dev environment.

Does this introduce a breaking change?

  • [YES]

This MR introduces fix for a bigger change, which will be needed prior to upgrade AKS controlplane to the 0.24 version.

Other information

To be tested in dev.

Related to #232 (closed)

Edited by Arturo Hernandez [EPAM]

Merge request reports