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.

Sign up for GitHub

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

Jump to bottom

Perform instance name validation for google provider #10330

Merged
merged 3 commits into from
Aug 15, 2016
Merged

Perform instance name validation for google provider #10330

merged 3 commits into from
Aug 15, 2016

Commits on Aug 9, 2016

  1. Allow specifying list of required_methods 

    This allows a particular form to use multiple validations (for example,
vm_name and regex match).
    @selmanj
    selmanj committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    38da103 View commit details
    Browse the repository at this point in the history

  2. Allow regex validations to print detailed message on fail. 

    This allows for custom error reporting when the regex match is
complicated and not easily printable.
    @selmanj
    selmanj committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    0eca52a View commit details
    Browse the repository at this point in the history

  3. Add a regex validation on vm name for Google provider 

    Also enforced a min/max length. Fixes downstream bug
https://bugzilla.redhat.com/show_bug.cgi?id=1330982
    @selmanj
    selmanj committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    3ebc6cd View commit details
    Browse the repository at this point in the history