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.

Sign up for GitHub

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

Jump to bottom

Update Cilium v1.15.5 #150

Merged
merged 1 commit into from
May 27, 2024
Merged

Update Cilium v1.15.5 #150

merged 1 commit into from
May 27, 2024

Conversation

kvaps
Copy link
Member

@kvaps kvaps commented May 24, 2024 •
edited
Loading

This PR updates Cilium from 14 to 15

@kvaps
Update Cilium v1.15.5
3254cc7 
Signed-off-by: Andrei Kvapil <kvapss@gmail.com>
@kvaps kvaps changed the title upd cilium Update Cilium v1.15.5 May 24, 2024
themoriarti
themoriarti reviewed May 24, 2024
View reviewed changes
Copy link
Collaborator

@themoriarti themoriarti left a comment

Choose a reason for hiding this comment

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

This is a very big change, has functional testing or smoke testing been done on the main features, i.e. all deletion and editing objects have been implemented?

I've generally gone through the changes and everything looks good.

packages/system/cilium/charts/cilium/Chart.yaml Show resolved Hide resolved
@kvaps kvaps mentioned this pull request May 26, 2024
Merged
@kvaps kvaps requested a review from gecube May 26, 2024 16:11
themoriarti
themoriarti approved these changes May 27, 2024
View reviewed changes
Copy link
Collaborator

@themoriarti themoriarti left a comment

Choose a reason for hiding this comment

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

LGTM

@themoriarti themoriarti merged commit b5f8006 into main May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@themoriarti themoriarti themoriarti approved these changes

@gecube gecube Awaiting requested review from gecube

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kvaps @themoriarti