build(capms): v1.33 and v1.34

[vknabel]

Description

Adds additional images for kubernetes v1.33 and v1.34

[mac641]

Perhaps, it makes sense to add another os to matrix to reduce code duplication.

- name: ubuntu-capms # this is the name of the docker-bake target
  version:
  - { apt_branch: v1.32, kube_version: 1.32.9 }
  - { apt_branch: v1.33, kube_version: 1.33.5 }
  - { apt_branch: v1.34, kube_version: 1.34.1 }

This would also allow us to remove if: ${{ matrix.os.name == 'ubuntu' }} for capms which is duplicate as well, since this' already been specified in docker-bake.hcl.

Not sure though, if there are conflicts with version: having an array assigned as opposed to having only a number assigned 🤔

[vknabel]

@mac641 good idea, but I'd vote for this to be a separate PR or issue as there is much more involved into this.
We also need to keep some complexities in mind like that capms images are built on top of ubuntu images.

[mac641]

@mac641 good idea, but I'd vote for this to be a separate PR or issue as there is much more involved into this. We also need to keep some complexities in mind like that capms images are built on top of ubuntu images.

@vknabel Fair, let's do a separate PR.
Nonetheless, I'm curious: What complexities do you have in mind?
The dependency to ubuntu is specified in docker-bake.hcl and will be respected every time an image is build.