Skip to content

Conversation

ylmrx
Copy link

@ylmrx ylmrx commented Sep 1, 2025

Description

Adding the Cilium CNI basic/mandatory rules in the SKS examples

Checklist

(For exoscale contributors)

  • Changelog updated (under Unreleased block)
  • Acceptance tests OK (irrelevant here)
  • For a new resource, datasource or new attributes: acceptance test added/updated

Testing

(it's only comments in an example)

@kobajagi
Copy link
Contributor

kobajagi commented Sep 3, 2025

Thanks @ylmrx, can you also add the (commented-out) cni setting to make example complete:
https://registry.terraform.io/providers/exoscale/exoscale/latest/docs/resources/sks_cluster#cni-1
Goes here:

resource "exoscale_sks_cluster" "my_sks_cluster" {
zone = local.my_zone
name = "my-sks-cluster"
auto_upgrade = true
exoscale_csi = true
}

Copy link
Member

@sauterp sauterp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, it would be good if you could try out the example before merging. Thanks!

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.

4 participants