Skip to content

improve create-k3d-cluster gh action with properly working network-policies#2219

Merged
anoipm merged 4 commits intokyma-project:mainfrom
anoipm:k3d-with-calico
Jan 29, 2026
Merged

improve create-k3d-cluster gh action with properly working network-policies#2219
anoipm merged 4 commits intokyma-project:mainfrom
anoipm:k3d-with-calico

Conversation

@anoipm
Copy link
Contributor

@anoipm anoipm commented Jan 22, 2026

Description

Changes proposed in this pull request:

  • improve create-k3d-cluster gh action with properly working network-policies

Related issue(s)

@anoipm anoipm requested a review from a team as a code owner January 22, 2026 12:38
@anoipm anoipm enabled auto-merge (squash) January 28, 2026 09:20
run: |
k3d cluster create kyma \
--agents 1 \
--image rancher/k3s:v1.31.7-k3s1 \
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the k3s1 version is decreased?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rancher/k3s:v1.31.7-k3s1 is used because newer versions hang during cluster startup when the following arguments are applied:

--k3s-arg "--flannel-backend=none@server:*"
--k3s-arg "--disable=traefik@server:0"

The selected version is based on a proven setup from another team:
https://github.tools.sap/xf-goat/knowledge-hub/blob/master/istio/k3d-network-policies/deploy-calico-k3d.sh

Copy link
Contributor Author

@anoipm anoipm Jan 29, 2026

Choose a reason for hiding this comment

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

I checked it again and it seems to be working with the newer version.

@anoipm anoipm disabled auto-merge January 28, 2026 10:11
@anoipm anoipm added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 28, 2026
@anoipm anoipm removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 29, 2026
@anoipm anoipm merged commit 7715543 into kyma-project:main Jan 29, 2026
26 checks passed
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.

2 participants