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
Store the event initiator in MiqEvent object. #16179
Conversation
1889933
to
344cdf0
Compare
Test failure is due to the dependency on ManageIQ/manageiq-automation_engine#86. |
344cdf0
to
eecbdec
Compare
@@ -11,16 +11,20 @@ def process_tasks(options) | |||
task_audit_event(:success, options, :message => msg) | |||
else | |||
assert_known_task(options) | |||
options[:userid] ||= "system" | |||
options[:userid] = User.current_user.try(:userid) || "system" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lfu
Won't this override the options[:userid] if it exists? Backward compatibility
eecbdec
to
940e569
Compare
Checked commit lfu@940e569 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0 |
To keep the info about the event initiator in the MiqEvent object.
Depends on ManageIQ/manageiq-automation_engine#86.
Depends on ManageIQ/manageiq-schema#94.
https://bugzilla.redhat.com/show_bug.cgi?id=1487749
@miq-bot assign @gmcculloug
@miq-bot add_label enhancement