Bug 1435780
Summary: | invalid values supplied when creating new policies using /api/policies doesn't result in error | ||
---|---|---|---|
Product: | Red Hat CloudForms Management Engine | Reporter: | Martin Kourim <mkourim> |
Component: | API | Assignee: | Joe Vlcek <jvlcek> |
Status: | CLOSED ERRATA | QA Contact: | Parthvi Vala <pvala> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 5.8.0 | CC: | jhardy, jvlcek, obarenbo, pvala, simaishi, smallamp |
Target Milestone: | GA | ||
Target Release: | 5.10.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | api:rest | ||
Fixed In Version: | 5.10.0.18 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2019-02-07 23:02:25 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Martin Kourim
2017-03-24 18:14:10 UTC
Sorry Šimon, didn't notice you took that one. Reassigning https://github.com/ManageIQ/manageiq/pull/14519 https://github.com/ManageIQ/manageiq-ui-classic/pull/851 So the validations for mode was done in https://github.com/ManageIQ/manageiq/pull/14519 However, we are still missing validations for towhat attribute. New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/d344e4307ed69e095d2c7869f12448d0b6e240a3 commit d344e4307ed69e095d2c7869f12448d0b6e240a3 Author: Joe VLcek <jvlcek> AuthorDate: Fri Sep 28 09:07:03 2018 -0400 Commit: Joe VLcek <jvlcek> CommitDate: Fri Sep 28 09:07:03 2018 -0400 Validate towhat policy field https://bugzilla.redhat.com/show_bug.cgi?id=1435780 app/models/miq_policy.rb | 11 + spec/models/miq_policy_spec.rb | 21 + 2 files changed, 32 insertions(+) New commit detected on ManageIQ/manageiq/hammer: https://github.com/ManageIQ/manageiq/commit/28ab0177beb315a791af3d60a7ca5ae1cfc5c29a commit 28ab0177beb315a791af3d60a7ca5ae1cfc5c29a Author: Brandon Dunne <brandondunne> AuthorDate: Fri Sep 28 14:38:24 2018 -0400 Commit: Brandon Dunne <brandondunne> CommitDate: Fri Sep 28 14:38:24 2018 -0400 Merge pull request #18032 from jvlcek/bz_1435780_tow_hat_validations Validate towhat policy field (cherry picked from commit 52704166f4dc48ccc2ced8c6bac0e8001e8d786f) https://bugzilla.redhat.com/show_bug.cgi?id=1435780 app/models/miq_policy.rb | 11 + spec/models/miq_policy_spec.rb | 10 + 2 files changed, 21 insertions(+) FIXED. Verified on 5.10.0.30. Request: POST /api/policies Query: { "action": "create", "resources": [{ "name": "test policy foo", "description": "Desc foo", "mode": "bar", "towhat": "baz", "conditions_ids": [ 2000, 3000 ], "policy_contents": [{ "event_id": 2, "actions": [ {"action_id": 1, "opts": { "qualifier": "failure" } } ] }] }] } Response: { "error": { "kind": "bad_request", "message": "Could not create the new policy - Validation failed: MiqPolicy: Mode is not included in the list, MiqPolicy: Towhat is not included in the list", "klass": "Api::BadRequestError" } } Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2019:0212 |