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

deps: update dependencies to k8s 1.23.3. #766

Merged
merged 4 commits into from Feb 3, 2022
Merged

Conversation

klihub
Copy link
Contributor

@klihub klihub commented Feb 2, 2022

No description provided.

marquiz
marquiz previously approved these changes Feb 2, 2022
Copy link
Contributor

@marquiz marquiz left a comment

Choose a reason for hiding this comment

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

Let's merge.

One optional suggestion, though. We could update directly to v1.23.3 which is the latest afaiu.

On a sidenote we should really drop circleci...

@klihub
Copy link
Contributor Author

klihub commented Feb 2, 2022

Let's merge.

One optional suggestion, though. We could update directly to v1.23.3 which is the latest afaiu.

Ok, let me check if updating there goes nice and easy and if it does we can jump directly there...

On a sidenote we should really drop circleci...

I'm fine with dropping circleci... @kad WDYT ?

Update k8s dependencies to 1.23.3. Additionally, since some of
our dependencies use reflect.StructField.IsExported() which was
introduced in go1.17, update our go requirements accordingly.
Add recently introduced PodSandboxStats and ListPodSandboxStats
CRI gRPC handlers.
jukkar
jukkar previously approved these changes Feb 3, 2022
@klihub klihub dismissed stale reviews from jukkar and marquiz via a7a9044 February 3, 2022 12:25
@klihub klihub changed the title deps: update dependencies to k8s 1.23.1. deps: update dependencies to k8s 1.23.3. Feb 3, 2022
@klihub
Copy link
Contributor Author

klihub commented Feb 3, 2022

@marquiz @askervin @jukkar @kad Updated deps to 1.23.3. If CI tests pass, PsomeoneTAL...

Copy link
Contributor

@marquiz marquiz left a comment

Choose a reason for hiding this comment

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

👍 LGTM

@klihub klihub merged commit 6a0b312 into intel:master Feb 3, 2022
@klihub klihub deleted the k8s/1.23.1 branch February 3, 2022 12:37
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.

None yet

3 participants