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

Auto approval for Cloud Vm reconfigure. #236

Merged

Conversation

billfitzgerald0120
Copy link
Contributor

@billfitzgerald0120 billfitzgerald0120 commented Jan 4, 2018

Missing method yaml from PR #205.

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

@miq-bot add_label enhancement
@miq-bot assign @gmcculloug

@miq-bot
Copy link
Member

miq-bot commented Jan 4, 2018

@billfitzgerald0120 Cannot apply the following label because they are not recognized: rfe

@billfitzgerald0120
Copy link
Contributor Author

@miq-bot add_label enhancement

@billfitzgerald0120
Copy link
Contributor Author

@tinaafitz Please review

@miq-bot
Copy link
Member

miq-bot commented Jan 4, 2018

Checked commit billfitzgerald0120@5a56426 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Yaml - missing config files

Copy link
Member

@tinaafitz tinaafitz left a comment

Choose a reason for hiding this comment

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

@billfitzgerald0120 Looks good.
@mkanoor Please review.

version: 1.0
object:
attributes:
name: vmcloudreconfigure_request_auto_approve
Copy link
Contributor

Choose a reason for hiding this comment

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

@billfitzgerald0120
Is there a corresponding PR for a method (internal or external) that would take advantage of this new attribute?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This method yaml should have been included in #205.

@gmcculloug gmcculloug merged commit 06e0cab into ManageIQ:master Jan 4, 2018
@gmcculloug gmcculloug added this to the Sprint 77 Ending Jan 15, 2018 milestone Jan 4, 2018
simaishi pushed a commit that referenced this pull request Jan 4, 2018
@simaishi
Copy link
Contributor

simaishi commented Jan 4, 2018

Gaprindashvili backport details:

$ git log -1
commit 4df59687a7e7a7cd797c05607a77ca6774aac98e
Author: Greg McCullough <gmccullo@redhat.com>
Date:   Thu Jan 4 16:43:53 2018 -0500

    Merge pull request #236 from billfitzgerald0120/cloudvmreconfig_autoapproval
    
    Auto approval for Cloud Vm reconfigure.
    (cherry picked from commit 06e0cabc0f36692eb81bfa8b38147f8af4a5a9d1)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1531304

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.

6 participants