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

client: Update kubevirt.io/client-go to v1.0.0 #24

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

orelmisan
Copy link
Member

@orelmisan orelmisan commented Sep 27, 2023

Currently, we are using kubevirt.io/[email protected].

Update to v1.0.0 in order to:

  1. Take advantage of context.Context parameter that was added to VMI methods.
  2. Eliminate the following warning from the checkup's logs:
W0702 08:56:11.138067 1 warnings.go:70] kubevirt.io/v1alpha3
is now deprecated and will be removed in a future release.

Pin K8s deps to v0.26.3 to match definitions set in kubevirt.io/client-go.

Copy link
Collaborator

@RamLavi RamLavi left a comment

Choose a reason for hiding this comment

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

could ypou also mention in the commit desc that ou are pinning to kubernetes-0.26.3 vendor?

@orelmisan
Copy link
Member Author

could ypou also mention in the commit desc that ou are pinning to kubernetes-0.26.3 vendor?

There was already a pin to an older version.
This PR is similar to kiagnose/kubevirt-dpdk-checkup#114

@RamLavi
Copy link
Collaborator

RamLavi commented Sep 27, 2023

could ypou also mention in the commit desc that ou are pinning to kubernetes-0.26.3 vendor?

There was already a pin to an older version. This PR is similar to kiagnose/kubevirt-dpdk-checkup#114

It is. I'm just saying that it's worth mentioning. won't insist on it though.

Currently, we are using kubevirt.io/[email protected].

Update to v1.0.0 in order to:
1. Take advantage of context.Context parameter
   that was added to VMI methods.
2. Eliminate the following warning from the checkup's logs:
```
W0702 08:56:11.138067 1 warnings.go:70] kubevirt.io/v1alpha3
is now deprecated and will be removed in a future release.
```
Pin K8s deps to v0.26.3 to match definitions set in
kubevirt.io/client-go.

Signed-off-by: Orel Misan <[email protected]>
@orelmisan
Copy link
Member Author

Updated commit message.

Copy link
Collaborator

@RamLavi RamLavi left a comment

Choose a reason for hiding this comment

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

/lgtm

@orelmisan orelmisan merged commit 1ab3a72 into kiagnose:main Oct 9, 2023
5 checks passed
@orelmisan orelmisan deleted the kubevirt-client-go branch October 9, 2023 09:15
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