Can create Audit with invalid parameters

[Vincent Françoise]

If you are making a POST to http://10.50.0.201:9322/v1/audits/ to create an audit, you can specify any audit_type and it will still create.

We should be able to validate the data that is sent upon creating an Audit to make sure all the fields are valid.

Also note that a wrong audit_template_uuid is only detected when inserting the data into the database

Note
====

There are Tempest tests that are currently skipped because of this bug. So they will have to be re-enabled as we get this fixed

[Hristo Paskalev (hpaskalev)]

Here is my first attempt at this one. Leaving it here for a review.

https://github.com/hpaskalev/watcher/commit/d19bd4f83b243853e305cd3fd0590d20008eabaa

[Vincent Françoise (vincent-francoise)]

Hi Hristo!

First of all, I would like to thank you for contributing to Watcher! Since your account on Launchpad is quite new, I believe you are not familiar the overall OpenStack review process. The actual way for you to contribute to any OpenStack project (including ours) is described at http://docs.openstack.org/infra/manual/developers.html#getting-started.
If you have any issue on how to set up this environment, please do not hesitate to contact us on IRC (irc.freenode.net) on the #openstack-watcher channel.

In the meantime I'll do a quick review onto your fork of Watcher :)

[Vincent Françoise (vincent-francoise)]

Hi Hristo,

I am fixing another bug in https://review.openstack.org/282395 and it is actually quite close from what I told you on you github commit. So you can use this a basis to include your fix.

[Hristo Paskalev (hpaskalev)]

Hello Vincent,

Thanks for the heads up. I actually ended doing something similar, as I figured that PostType validation would be needed.
I guess I can rebase on this one once that it's merged, and make some adjustments to my fix .

[Vincent Françoise (vincent-francoise)]

Hi Hristo,

Are you still working on this bug? If so, can you push your code onto review.openstack.org so we can review it ;)
If not, would you mind unassigning yourself?

If you need some help, feel free to come onto #openstack-watcher, we'll be more than happy to help you out.

[Hristo Paskalev (hpaskalev)]

Sorry, I haven't got back to this one yet. I guess it would be better if I un-assign myself from it as I don't think I'll have the time to finish it these days.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/302293

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/305846

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/302293
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=e52dc4f8aab349ae595eda0abf5e030f4fe59d5e
Submitter: Jenkins
Branch: master

commit e52dc4f8aab349ae595eda0abf5e030f4fe59d5e
Author: Alexander Chadin <email address hidden>
Date: Wed Apr 6 17:42:06 2016 +0300

    Add parameters verification when Audit is being created

    We have to check Audit Type and Audit State to make sure
    these parameters are in valid status.

    Also, we provide default states for the next attributes:

    - 'audit_template' is required and should be either UUID or text field
    - 'state' is readonly so it raises an error if submitted in POST
      and is set by default to PENDING
    - 'deadline' is optional and should be a datetime
    - 'type' is a required text field

    Change-Id: I2a7e0deec0ee2040e86400b500bb0efd8eade564
    Closes-Bug: #1532843
    Closes-Bug: #1533210

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/305846
Committed: https://git.openstack.org/cgit/openstack/python-watcherclient/commit/?id=0aaaf132789d977da0796b09b78a6d96a03d3a63
Submitter: Jenkins
Branch: master

commit 0aaaf132789d977da0796b09b78a6d96a03d3a63
Author: Alexander Chadin <email address hidden>
Date: Thu Apr 14 16:13:07 2016 +0300

    Add audit-template name checking in CLI

    This patch set allows to send audit_template_uuid as uuid type only.
    If audit_template argument given as name, watcherclient will send request
    to get audit template uuid.

    Change-Id: Idf5f07ca08f2e5d871dc2163c32fbda9ed338a99
    Related-Bug: #1532843

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/watcher 0.27.0 release.