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

Add orchestration stack my_zone. #15334

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

tinaafitz
Copy link
Member

Orchestration stacks can be retired individually and/or from Services. Retirement puts the retirement request event on the queue and supplies the zone if available. Service retirement for Orchestration stacks has a zone, but the individual Orchestration stack doesn't not.

https://bugzilla.redhat.com/show_bug.cgi?id=1458011

@tinaafitz
Copy link
Member Author

@miq-bot assign @gmcculloug
@miq-bot add_label bug, fine/yes

@@ -97,6 +97,10 @@ def raw_update_stack(_template, _options = {})
raise NotImplementedError, _("raw_update_stack must be implemented in a subclass")
end

def my_zone
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like you need to indent the method

@bzwei
Copy link
Contributor

bzwei commented Jun 8, 2017

LGTM

@tinaafitz tinaafitz force-pushed the orchestration_stack_add_zone branch from c9fc27d to d8ef167 Compare June 8, 2017 14:06
@miq-bot
Copy link
Member

miq-bot commented Jun 8, 2017

Checked commit tinaafitz@d8ef167 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🍰

@gmcculloug gmcculloug merged commit 8e6d2fb into ManageIQ:master Jun 8, 2017
@gmcculloug gmcculloug added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 8, 2017
simaishi pushed a commit that referenced this pull request Jun 13, 2017
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 52bbd3f46cec8ea74c5289b37764d40301b5c014
Author: Greg McCullough <[email protected]>
Date:   Thu Jun 8 10:43:30 2017 -0400

    Merge pull request #15334 from tinaafitz/orchestration_stack_add_zone
    
    Add orchestration stack my_zone.
    (cherry picked from commit 8e6d2fb05a36be637183b083efafbe7904f58e8d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1461143

@tinaafitz
Copy link
Member Author

tinaafitz commented Jun 19, 2017

@miq-bot add_label services

@miq-bot
Copy link
Member

miq-bot commented Jun 19, 2017

@tinaafitz unrecognized command 'services', ignoring...

Accepted commands are: add_label, assign, close_issue, move_issue, remove_label, rm_label, set_milestone

@simaishi
Copy link
Contributor

@tinaafitz @gmcculloug can this be euwe/yes (as per flag on the BZ)?

@tinaafitz
Copy link
Member Author

@miq-bot add_label euwe/yes
@simaishi Yes, I just added euwe/yes label.

simaishi pushed a commit that referenced this pull request Jun 30, 2017
@simaishi
Copy link
Contributor

Euwe backport details:

$ git log -1
commit fa669b4d02a4b334cded5644592d88cfa7feab2b
Author: Greg McCullough <[email protected]>
Date:   Thu Jun 8 10:43:30 2017 -0400

    Merge pull request #15334 from tinaafitz/orchestration_stack_add_zone
    
    Add orchestration stack my_zone.
    (cherry picked from commit 8e6d2fb05a36be637183b083efafbe7904f58e8d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1465415

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.

5 participants