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

Disable inventory collector worker by default #63

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Nov 6, 2017

Due to the large payload size generated by the watch notices disable the collector worker by default until payloads can be moved to BinaryBlobs table

@agrare
Copy link
Member Author

agrare commented Nov 6, 2017

cc @cben @Ladas @kbrock @simon3z

@miq-bot
Copy link
Member

miq-bot commented Nov 6, 2017

Checked commit agrare@646b05f with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@cben
Copy link
Contributor

cben commented Nov 6, 2017

@Fryguy Fryguy merged commit 177661d into ManageIQ:master Nov 6, 2017
@Fryguy Fryguy added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 6, 2017
@Fryguy Fryguy assigned Fryguy and unassigned moolitayer Nov 6, 2017
@kbrock
Copy link
Member

kbrock commented Nov 6, 2017

Adding cross reference: ManageIQ/manageiq-providers-kubernetes#160

@agrare agrare deleted the disable_inventory_collector_worker_by_default branch November 6, 2017 21:00
simaishi pushed a commit that referenced this pull request Nov 6, 2017
…_by_default

Disable inventory collector worker by default
(cherry picked from commit 177661d)
@simaishi
Copy link

simaishi commented Nov 6, 2017

Gaprindashvili backport details:

$ git log -1
commit 00ce38e0e31b6da41e5413f185fe75371b32592f
Author: Jason Frey <fryguy9@gmail.com>
Date:   Mon Nov 6 14:19:22 2017 -0500

    Merge pull request #63 from agrare/disable_inventory_collector_worker_by_default
    
    Disable inventory collector worker by default
    (cherry picked from commit 177661d8019e1a2d3d88689788e09e5c9e225ee1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants