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

New folder targeted refresh [Depends on vmware/32] #14460

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Mar 22, 2017

This implements the MiqVimBrokerWorker::Runner#on_create_event by parsing the event sent by MiqVimUpdate and call EmsRefresh.queue_refresh_new_target.

ManageIQ/manageiq-providers-vmware#32 implements the event parser and folder inventory filter.

Depends:

https://bugzilla.redhat.com/show_bug.cgi?id=1378984

@miq-bot miq-bot added the wip label Mar 22, 2017
@chessbyte chessbyte requested a review from Fryguy March 22, 2017 19:24
@agrare agrare force-pushed the bz_1378984_new_folder_targeted_refresh branch 3 times, most recently from ea06087 to e3f93f6 Compare March 29, 2017 18:58
@agrare agrare changed the title [WIP] New folder targeted refresh New folder targeted refresh Apr 5, 2017
@miq-bot miq-bot removed the wip label Apr 5, 2017
@blomquisg blomquisg changed the title New folder targeted refresh New folder targeted refresh [Depends on vmware/32] Apr 6, 2017
@agrare agrare force-pushed the bz_1378984_new_folder_targeted_refresh branch from e3f93f6 to 98e6995 Compare April 24, 2017 13:17
@miq-bot
Copy link
Member

miq-bot commented Apr 24, 2017

Checked commits agrare/manageiq@82b1c04~...98e6995 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
5 files checked, 0 offenses detected
Everything looks good. 👍

@blomquisg blomquisg merged commit d199f44 into ManageIQ:master Apr 24, 2017
@blomquisg blomquisg added this to the Sprint 59 Ending Apr 24, 2017 milestone Apr 24, 2017
@agrare agrare deleted the bz_1378984_new_folder_targeted_refresh branch April 24, 2017 16:21
@agrare
Copy link
Member Author

agrare commented Apr 25, 2017

ManageIQ/manageiq-providers-vmware#32 must be backported before this PR

@agrare
Copy link
Member Author

agrare commented Apr 25, 2017

simaishi pushed a commit that referenced this pull request Apr 25, 2017
…refresh

New folder targeted refresh [Depends on vmware/32]
(cherry picked from commit d199f44)

https://bugzilla.redhat.com/show_bug.cgi?id=1445311
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 4d484dc4196f9ad35c1d75adf3f51feb03480d03
Author: Greg Blomquist <[email protected]>
Date:   Mon Apr 24 12:20:46 2017 -0400

    Merge pull request #14460 from agrare/bz_1378984_new_folder_targeted_refresh
    
    New folder targeted refresh [Depends on vmware/32]
    (cherry picked from commit d199f44cd9a1ee5202f14222a9c74ed3f5fc9c7f)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1445311

@agrare
Copy link
Member Author

agrare commented May 11, 2017

EUWE PR #15073

agrare pushed a commit to agrare/manageiq that referenced this pull request May 11, 2017
…argeted_refresh

New folder targeted refresh [Depends on vmware/32]
(cherry picked from commit d199f44)
@simaishi
Copy link
Contributor

Backported to Euwe via #15073

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.

5 participants