-
Notifications
You must be signed in to change notification settings - Fork 118
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
Email instances for VmMigrate and Reconfiguration. #311
Email instances for VmMigrate and Reconfiguration. #311
Conversation
@miq-bot add_label enhancement |
@tinaafitz @mkanoor Please review |
@@ -178,7 +178,7 @@ object: | |||
datatype: string | |||
priority: 9 | |||
owner: | |||
default_value: "/Infrastructure/VM/Migrate/Email/VmMigrateTask_Complete?event=vm_migrated" | |||
default_value: "/System/Notification/Email/CloudOrchestrationServiceReconfigureComplete?event=service_reconfigured" |
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.
@billfitzgerald0120
Is this value correct, isn't it supposed to be for VmMigrateTask instead of CloudOrchestration
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.
@mkanoor Yes, I need to fix this
Added 7 instances in System/Notification/Email class for VmMigrate and Reconfiguration. Modified EmailOwner value in 2 State Machine classes to use new instances. Modified 2 System/Request instances to use new instances. Fixed VmMigrate System/Request value.
91995d8
to
0921fda
Compare
Checked commit billfitzgerald0120@0921fda with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@mkanoor Fixed instance, please review |
Added 7 instances in System/Notification/Email class for VmMigrate and Reconfiguration.
Modified EmailOwner value in 2 State Machine classes to use new instances.
Modified 2 System/Request instances to use new instances.
https://bugzilla.redhat.com/show_bug.cgi?id=1314871
https://www.pivotaltracker.com/epic/show/3861726