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

Add VM uuid as the vm_uid_ems to the event payload #179

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Jan 29, 2018

Add the UUID for the VM to the event payload so that it can be used to
find disconnected VMs when linking events.

Depends:

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

@agrare agrare added the bug label Jan 29, 2018
@agrare agrare force-pushed the bz_1538996_add_vm_uid_ems_to_event branch 2 times, most recently from 791feab to e742700 Compare January 29, 2018 20:58
@agrare agrare changed the title Add VM uuid as the vm_uid_ems to the event payload [Depends vmware_web_service/32] Add VM uuid as the vm_uid_ems to the event payload Jan 31, 2018
@agrare agrare force-pushed the bz_1538996_add_vm_uid_ems_to_event branch from e742700 to adc7f70 Compare January 31, 2018 23:42
Add the UUID for the VM to the event payload so that it can be used to
find disconnected VMs when linking events.

https://bugzilla.redhat.com/show_bug.cgi?id=1538996
@agrare agrare force-pushed the bz_1538996_add_vm_uid_ems_to_event branch from adc7f70 to 9a9249d Compare January 31, 2018 23:48
@miq-bot
Copy link
Member

miq-bot commented Jan 31, 2018

Checked commit agrare@9a9249d with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Yaml - missing config files

@agrare
Copy link
Member Author

agrare commented Feb 1, 2018

@blomquisg @roliveri dependent PR merged and new gem version released

@blomquisg blomquisg merged commit d8c04f7 into ManageIQ:master Feb 7, 2018
@agrare agrare deleted the bz_1538996_add_vm_uid_ems_to_event branch February 7, 2018 21:33
@simaishi
Copy link
Contributor

simaishi commented Feb 9, 2018

gaprindashvili/yes ?

@agrare agrare added this to the Sprint 79 Ending Feb 12, 2018 milestone Feb 13, 2018
simaishi pushed a commit that referenced this pull request Mar 7, 2018
@simaishi
Copy link
Contributor

simaishi commented Mar 7, 2018

Gaprindashvili backport details:

$ git log -1
commit 9c749b5396a08de50a42c31e32849eb6db41befa
Author: Greg Blomquist <blomquisg@gmail.com>
Date:   Wed Feb 7 16:31:55 2018 -0500

    Merge pull request #179 from agrare/bz_1538996_add_vm_uid_ems_to_event
    
    Add VM uuid as the vm_uid_ems to the event payload
    (cherry picked from commit d8c04f7b07d7d82d5767bab0b6be96150b8cd6af)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1552677

agrare pushed a commit to agrare/manageiq-providers-vmware that referenced this pull request Apr 15, 2019
…sh_methods

Remove input for new refresh method
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.

4 participants