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

Clean up after apimachinery dep deletion #3588

Closed
monopole opened this issue Feb 12, 2021 · 0 comments · Fixed by #3721
Closed

Clean up after apimachinery dep deletion #3588

monopole opened this issue Feb 12, 2021 · 0 comments · Fixed by #3721
Assignees

Comments

@monopole
Copy link
Contributor

The work leading up to #2506, and the residue of a previous approach to kubectl integration, resulted in interfaces and implementations of those interfaces that are no longer needed now that #2506 is closed.

Things to delete (maybe not a complete list)

This makes Resource a direct wrapper to RNode.

Longer term (file a new issue?), get rid of Resource.

Its only purpose was to hold ancillary, transient data about the resource being transformer, like its original (non-prefixed) name. Such things are now tracked in metadata annotations inside RNode.

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 a pull request may close this issue.

1 participant