-
Notifications
You must be signed in to change notification settings - Fork 896
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
Add an inventory_collector_worker base class #16198
Add an inventory_collector_worker base class #16198
Conversation
f2500de
to
63faf7c
Compare
36a77b9
to
8ee1bb2
Compare
This pull request is not mergeable. Please rebase and repush. |
8ee1bb2
to
12ee7ff
Compare
@Fryguy @blomquisg @Ladas can you guys review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesomely great. 👍
Haven't try to run the newest version, will try it tomorrow, but that is not a merge blocker. :-)
12ee7ff
to
2d75a49
Compare
Checked commits agrare/manageiq@49c8bb1~...2d75a49 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0 app/models/manageiq/providers/base_manager/inventory_collector_worker.rb
|
Add a basic inventory collector worker which can be used for sending update-driven inventory payloads to a RefreshWorker for saving.
The goal here is to use the Kubernetes watch_pods() API to ensure that we never miss a pod if it is created and deleted while a refresh is running (already real world issue with customers).
Required for ManageIQ/manageiq-providers-kubernetes#129 and ManageIQ/manageiq-providers-openshift#52