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

Automate - Add /System/Process/MiqEvent instance. #9336

Merged
merged 1 commit into from Jun 22, 2016

Conversation

tinaafitz
Copy link
Member

@tinaafitz tinaafitz commented Jun 21, 2016

The new MiqEvent instance enables a user to create custom buttons and use simulate for MiqEvents.
https://bugzilla.redhat.com/show_bug.cgi?id=1348605

screen shot 2016-06-21 at 4 46 28 pm

simulate

@tinaafitz tinaafitz changed the title Automate - Add /System/Process/MiqEvent instance. [WIP] Automate - Add /System/Process/MiqEvent instance. Jun 21, 2016
@tinaafitz tinaafitz changed the title [WIP] Automate - Add /System/Process/MiqEvent instance. Automate - Add /System/Process/MiqEvent instance. Jun 21, 2016
description:
fields:
- rel5:
value: "/System/Event/MiqEvent/Policy/${/#event_type}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tinaafitz I think you want the relationship to be more like the Request instance in the Process class which uses
/System/Request/${#request}

Here I would expect it to be: /System/Event/MiqEvent/Policy/${/#request}

Then in your example you would specify the request (which your screenshot shows) but you do not have to pass duplicate data in the Attribute/Value Pairs.

Does that work better here?

The new MiqEvent instance enables a user to create custom buttons for
our MiqEvents.

https://bugzilla.redhat.com/show_bug.cgi?id=1348605
@tinaafitz
Copy link
Member Author

@gmcculloug I used event_type to be consistent with the event instance, but agree that "request" makes more sense here. Change made.

@miq-bot
Copy link
Member

miq-bot commented Jun 22, 2016

Checked commit tinaafitz@fb5116a with ruby 2.2.4, rubocop 0.37.2, and haml-lint 0.16.1
0 files checked, 0 offenses detected
Everything looks good. 🍪

@gmcculloug gmcculloug merged commit 3ebe1ef into ManageIQ:master Jun 22, 2016
@gmcculloug gmcculloug added this to the Sprint 43 Ending July 11, 2016 milestone Jun 22, 2016
chessbyte pushed a commit that referenced this pull request Jul 5, 2016
Automate - Add /System/Process/MiqEvent instance.
(cherry picked from commit 3ebe1ef)
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.

None yet

4 participants