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 ems_uid and ems_type to event payload #20179

Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented May 15, 2020

The Event payload already containes the ems_id, we can add the ems_type and ems_uid to assist in identifying the source provider without having to hit the MIQ API.

Followup to: #19984

Fixes #20178

The Event payload already containes the ems_id, we can add the ems_type
and ems_uid to assist in identifying the source provider without having
to hit the MIQ API

Fixes ManageIQ#20178
@miq-bot
Copy link
Member

miq-bot commented May 15, 2020

Checked commit agrare@e0a5d20 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@chessbyte chessbyte assigned chessbyte and unassigned Fryguy May 16, 2020
@chessbyte chessbyte merged commit 5d73337 into ManageIQ:master May 16, 2020
@agrare agrare deleted the add_ems_uid_and_type_to_event_payload branch May 16, 2020 12:09
simaishi pushed a commit that referenced this pull request May 21, 2020
…ayload

Add ems_uid and ems_type to event payload

(cherry picked from commit 5d73337)
@simaishi
Copy link
Contributor

Jansa backport details:

$ git log -1
commit 5805d5ce2de636efcb1aa60be7627aeae5da420d
Author: Oleg Barenboim <chessbyte@gmail.com>
Date:   Sat May 16 08:02:58 2020 -0400

    Merge pull request #20179 from agrare/add_ems_uid_and_type_to_event_payload

    Add ems_uid and ems_type to event payload

    (cherry picked from commit 5d733373c1ca02872af71d4972ec75f7dc40d105)

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.

Add ems type and uid to event payload
6 participants