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

use task queue for update stack operation #373

Merged
merged 1 commit into from
Feb 24, 2017

Conversation

sseago
Copy link
Contributor

@sseago sseago commented Feb 14, 2017

Update stack should use the task queue for direct operation on the
provider (used by scaling/scaledown UI in ems infra)

This commit includes the necessary UI changes.

Depends on model PR: ManageIQ/manageiq#13897

@sseago
Copy link
Contributor Author

sseago commented Feb 14, 2017

The travis failure is due to the dependency on ManageIQ/manageiq#13897 -- once that's merged, these tests should pass.

@sseago
Copy link
Contributor Author

sseago commented Feb 16, 2017

OK triggering Travis build again now that the dependent core PR is merged.

Update stack should use the task queue for direct operation on the
provider (used by scaling/scaledown UI in ems infra)

This commit includes the necessary UI changes.
@sseago sseago force-pushed the queue-update_stack branch from a9afa48 to 33194c6 Compare February 16, 2017 18:14
@miq-bot
Copy link
Member

miq-bot commented Feb 16, 2017

Checked commit sseago@33194c6 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks good. 👍

@sseago sseago closed this Feb 17, 2017
@sseago sseago reopened this Feb 17, 2017
@sseago
Copy link
Contributor Author

sseago commented Feb 23, 2017

@miq-bot remove_label "pending core"

@miq-bot
Copy link
Member

miq-bot commented Feb 23, 2017

@sseago Cannot remove the following label because they are not recognized: "pending core"

@sseago
Copy link
Contributor Author

sseago commented Feb 23, 2017

@miq-bot remove_label pending_core

@miq-bot
Copy link
Member

miq-bot commented Feb 23, 2017

@sseago Cannot remove the following label because they are not recognized: pending_core

@sseago sseago closed this Feb 23, 2017
@sseago sseago reopened this Feb 23, 2017
@mzazrivec mzazrivec self-assigned this Feb 24, 2017
@mzazrivec
Copy link
Contributor

@tzumainn Could you please confirm this has been tested to work successfully? Thanks.

@tzumainn
Copy link
Contributor

@mzazrivec yep!

@mzazrivec mzazrivec added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 24, 2017
@mzazrivec mzazrivec merged commit e72650b into ManageIQ:master Feb 24, 2017
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.

4 participants