-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
refactor resource tracker to be usable across packages #3331
refactor resource tracker to be usable across packages #3331
Conversation
76ea7e9
to
d9fa618
Compare
/lgtm We've been wrestling with the tests today, I did bring up a cluster with the same k8s version (but not this patch) and it did work. But I'd still suspect the tests over an actual failure caused by this PR, at least for now :-) /retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewsykim, justinsb The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/test pull-kops-e2e-kubernetes-aws |
nice! 😛 |
/retest |
/test pull-kops-e2e-kubernetes-aws |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue |
Puts ResourceTracker into its own package
pkg/resources/tracker
so it can be called from different packges without hitting import cycle e.g.pkg/reources/digitalocean/resource.go
. This is so that we can group cloud specific code into their own packages (pkg/resources/aws/
.pkg/resources/gce/
, etc) rather than dump all the resource code from all clouds intopkg/resources