-
Notifications
You must be signed in to change notification settings - Fork 107
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
[DEMO] Add notifications around the toolbar and dialog demo for ansible operations #475
Conversation
+ gitignore node_modules
(cherry-picked from Hyperkid123/manageiq-ui-classic/b0c269da32fde4238475d1cc7f35bb3d6a36aa62)
and make it a string, we need to support all ids
Add ansible root and env_vars to ems
Replace vm start&stop by ansible call
Add role for changing VM state and start&stop playbooks using it
6a92f41
to
9618637
Compare
Checked commits skateman/manageiq-providers-amazon@9ce7e67~...9618637 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 app/helpers/manageiq/providers/amazon/toolbar_overrides/ems_cloud_center.rb
app/helpers/manageiq/providers/amazon/toolbar_overrides/ems_network_center.rb
app/helpers/manageiq/providers/amazon/toolbar_overrides/x_vm_cloud_center.rb
app/models/manageiq/providers/amazon/network_manager/security_group.rb
content_tmp/ansible/create_security_group.yml
content_tmp/ansible/roles/change_vm_state/defaults/main.yml
content_tmp/ansible/roles/change_vm_state/handlers/main.yml
content_tmp/ansible/roles/change_vm_state/meta/main.yml
content_tmp/ansible/roles/change_vm_state/tasks/main.yml
content_tmp/ansible/roles/change_vm_state/tests/test.yml
content_tmp/ansible/roles/change_vm_state/vars/main.yml
content_tmp/ansible/roles/create_security_group/defaults/main.yml
content_tmp/ansible/roles/create_security_group/handlers/main.yml
content_tmp/ansible/roles/create_security_group/meta/main.yml
content_tmp/ansible/roles/create_security_group/tests/test.yml
content_tmp/ansible/roles/create_security_group/vars/main.yml
content_tmp/ansible/start_vm.yml
content_tmp/ansible/stop_vm.yml
|
This pull request has been automatically closed because it has not been updated for at least 6 months. Feel free to reopen this pull request if these changes are still valid. Thank you for all your contributions! |
Demo around @martinpovolny's PR #468 with notifications.
It was necessary to pass ˙user_id˙ as an extra argument for the
create_security_group
method. The task initiation comes from the API, so the arguments had to be adjusted there in ManageIQ/manageiq-api#450.The new notification types and the notification mixin comes from ManageIQ/manageiq#17835