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 for k8s.io/[email protected] #14

Merged

Conversation

runakash
Copy link
Collaborator

@runakash runakash commented Aug 3, 2022

Fixes #9

@runakash runakash requested a review from chrisohaver August 3, 2022 00:14
@runakash runakash force-pushed the fix-apimachinery-api-for-coredns-193 branch from 192b845 to dc68c30 Compare August 3, 2022 00:15
Copy link
Member

@chrisohaver chrisohaver left a comment

Choose a reason for hiding this comment

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

Since Go modules are not used in this repo, we may want to implement both methods, so the Object interface would be implemented in apimachinery versions before and after v0.24.0.

@runakash
Copy link
Collaborator Author

runakash commented Aug 3, 2022

Since Go modules are not used in this repo, we may want to implement both methods, so the Object interface would be implemented in apimachinery versions before and after v0.24.0.

Makes sense. Seems like it will be simpler to onboard go module. What do you think?

@chrisohaver
Copy link
Member

Makes sense. Seems like it will be simpler to onboard go module. What do you think?

Simpler than not deleting two functions? Maybe I'm not seeing the whole picture.

@runakash runakash force-pushed the fix-apimachinery-api-for-coredns-193 branch 2 times, most recently from 123b3bc to d296d03 Compare August 3, 2022 19:39
…ame/SetZZZ_DeprecatedClusterName.

Signed-off-by: Akash Rungta <[email protected]>
@runakash runakash force-pushed the fix-apimachinery-api-for-coredns-193 branch from d296d03 to 215c3b9 Compare August 3, 2022 19:39
Copy link
Member

@chrisohaver chrisohaver left a comment

Choose a reason for hiding this comment

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

LGTM Thanks!

@runakash runakash merged commit 3afa45c into coredns:main Aug 3, 2022
@runakash runakash deleted the fix-apimachinery-api-for-coredns-193 branch August 3, 2022 19:42
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.

error: cannot use e (type *Endpoints) as type "k8s.io/apimachinery/pkg/apis/meta/v1"
2 participants