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

Updating Cilium to 1.15.2 #259

Merged
merged 1 commit into from
Apr 5, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion build-scripts/hack/update-gateway-api-chart.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash

VERSION="v0.7.1"
VERSION="v1.0.0"
DIR=`realpath $(dirname "${0}")`

CHARTS_PATH="$DIR/../../k8s/components/charts"
Expand All @@ -16,6 +16,7 @@ rm -rf gateway-api/templates/*
rm -rf gateway-api/charts
cp gateway-api-src/config/crd/standard/* gateway-api/templates/
cp gateway-api-src/config/crd/experimental/gateway.networking.k8s.io_tlsroutes.yaml gateway-api/templates/
cp gateway-api-src/config/crd/experimental/gateway.networking.k8s.io_grpcroutes.yaml gateway-api/templates/
sed -i 's/^\(version: \).*$/\1'"${VERSION:1}"'/' gateway-api/Chart.yaml
sed -i 's/^\(appVersion: \).*$/\1'"${VERSION:1}"'/' gateway-api/Chart.yaml
sed -i 's/^\(description: \).*$/\1'"A Helm Chart containing Gateway API CRDs"'/' gateway-api/Chart.yaml
Expand Down
Binary file removed k8s/components/charts/cilium-1.14.1.tgz
Binary file not shown.
Binary file added k8s/components/charts/cilium-1.15.2.tgz
Binary file not shown.
4 changes: 2 additions & 2 deletions k8s/components/charts/ck-loadbalancer/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.1.0
version: 0.1.1

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: "0.1.0"
appVersion: "0.1.1"
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ metadata:
{{- include "ck-loadbalancer.labels" . | nindent 4 }}
spec:
{{- with .Values.ipPool.cidrs }}
cidrs:
blocks:
{{- toYaml . | nindent 4 }}
{{- end }}
{{- end }}
Binary file removed k8s/components/charts/gateway-api-0.7.1.tgz
Binary file not shown.
Binary file added k8s/components/charts/gateway-api-1.0.0.tgz
Binary file not shown.
4 changes: 2 additions & 2 deletions src/k8s/pkg/component/images.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ const dnsImageRepository = "ghcr.io/canonical/coredns"
const dnsImageTag = "1.11.1-ck1"

const ciliumAgentImageRepository = "ghcr.io/canonical/cilium"
const ciliumAgentImageTag = "1.14.5-ck1"
const ciliumAgentImageTag = "1.15.2-ck0"

const ciliumOperatorImageRepository = "ghcr.io/canonical/cilium-operator"
const ciliumOperatorImageTag = "1.14.5-ck2"
const ciliumOperatorImageTag = "1.15.2-ck0"

const storageImageRepository = "ghcr.io/canonical/rawfile-localpv"
const storageImageTag = "0.8.0-ck5"
Expand Down
1 change: 1 addition & 0 deletions src/k8s/pkg/component/network.go
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ func UpdateNetworkComponent(ctx context.Context, s snap.Snap, isRefresh bool, po
"nodePort": map[string]any{
"enabled": true,
},
"disableEnvoyVersionCheck": true,
}

if s.Strict() {
Expand Down
4 changes: 2 additions & 2 deletions src/k8s/pkg/snap/snap.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ func (s *snap) Components() map[string]types.Component {
return map[string]types.Component{
"network": {
ReleaseName: "ck-network",
ManifestPath: path.Join(s.snapDir, "k8s", "components", "charts", "cilium-1.14.1.tgz"),
ManifestPath: path.Join(s.snapDir, "k8s", "components", "charts", "cilium-1.15.2.tgz"),
Namespace: "kube-system",
},
"dns": {
Expand All @@ -213,7 +213,7 @@ func (s *snap) Components() map[string]types.Component {
"ingress": {},
"gateway": {
ReleaseName: "ck-gateway",
ManifestPath: path.Join(s.snapDir, "k8s", "components", "charts", "gateway-api-0.7.1.tgz"),
ManifestPath: path.Join(s.snapDir, "k8s", "components", "charts", "gateway-api-1.0.0.tgz"),
Namespace: "kube-system",
},
"load-balancer": {
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/tests/test_cilium_e2e.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

ARCH = platform.machine()
CILIUM_CLI_ARCH_MAP = {"aarch64": "arm64", "x86_64": "amd64"}
CILIUM_CLI_VERSION = "v0.15.19"
CILIUM_CLI_VERSION = "v0.16.3"
CILIUM_CLI_TAR_GZ = f"https://github.com/cilium/cilium-cli/releases/download/{CILIUM_CLI_VERSION}/cilium-linux-{CILIUM_CLI_ARCH_MAP.get(ARCH)}.tar.gz" # noqa


Expand Down
8 changes: 3 additions & 5 deletions tests/integration/tests/test_gateway.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,6 @@ def test_gateway(instances: List[harness.Instance]):
)
gateway_http_port = p.stdout.decode().replace("'", "")

p = instance.exec(
["curl", f"localhost:{gateway_http_port}"],
capture_output=True,
)
assert "Welcome to nginx!" in p.stdout.decode()
util.stubbornly(retries=5, delay_s=5).on(instance).until(
neoaggelos marked this conversation as resolved.
Show resolved Hide resolved
lambda p: "Welcome to nginx!" in p.stdout.decode()
).exec(["curl", f"localhost:{gateway_http_port}"])
8 changes: 3 additions & 5 deletions tests/integration/tests/test_ingress.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,8 +95,6 @@ def test_ingress(instances: List[harness.Instance]):
]
)

p = instance.exec(
["curl", f"localhost:{ingress_http_port}", "-H", "Host: foo.bar.com"],
capture_output=True,
)
assert "Welcome to nginx!" in p.stdout.decode()
util.stubbornly(retries=5, delay_s=5).on(instance).until(
lambda p: "Welcome to nginx!" in p.stdout.decode()
).exec(["curl", f"localhost:{ingress_http_port}", "-H", "Host: foo.bar.com"])
Loading