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

fix: upgrade cilium to a stable version #1183

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ritwikranjan
Copy link
Contributor

@ritwikranjan ritwikranjan commented Jan 2, 2025

Description

Moving cilium from v1.16.0-pre.1.0.20240403152809-b9853ecbcaeb to v1.16.4 as it is blocking other package upgrades. As a result of that there is a major refactoring introduced in the codebase to accommodate the changes in Cillium.

Dependency Updates:

  • Updated the import path for hive/cell from github.com/cilium/cilium/pkg/hive/cell to github.com/cilium/hive/cell across multiple files. [1] [2] [3] [4] [5] [6] [7] [8] [9]

go.mod Updates:

  • Updated various dependencies to their latest versions, including cel.dev/expr, github.com/Azure/go-autorest/autorest/adal, github.com/cncf/xds/go, github.com/containerd/containerd, github.com/docker/docker, github.com/fatih/color, github.com/hashicorp/consul/api, github.com/spiffe/go-spiffe/v2, go.etcd.io/etcd/api/v3, google.golang.org/genproto/googleapis/api, and google.golang.org/grpc. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]

Related Issue

#1036 #1010 #1009 #31

Checklist

  • I have read the contributing documentation.
  • I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
  • I have correctly attributed the author(s) of the code.
  • I have tested the changes locally.
  • I have followed the project's style guidelines.
  • I have updated the documentation, if necessary.
  • I have added tests, if applicable.

Screenshots (if applicable) or Testing Completed

Was able to do quick-build and quick-deploy with the changes.

retina-agent-sqx4t                      1/1     Running   0          2m26s   10.224.0.6     aks-nodepool1-15192462-vmss000000   <none>           <none>
retina-agent-t9pfh                      1/1     Running   0          2m26s   10.224.0.5     aks-nodepool1-15192462-vmss000001   <none>           <none>
retina-agent-wvp2f                      1/1     Running   0          2m26s   10.224.0.4     aks-nodepool1-15192462-vmss000002   <none>           <none>
retina-operator-657b5968d5-gjrwv        1/1     Running   0          2m26s   10.244.2.145   aks-nodepool1-15192462-vmss000001   <none>           <none>

Additional Notes

Add any additional notes or context about the pull request here.


Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.

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.

1 participant