Validation for security groups can fail for valid groups
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Sahara |
Fix Released
|
Medium
|
Chad Roberts | ||
Juno |
Fix Released
|
Medium
|
Nikita Konovalov |
Bug Description
From validations/base.py
def check_security_
security_
allowed_groups = set(reduce(
for sg in security_groups:
if sg not in allowed_groups:
raise ex.InvalidExcep
The above fails with Security group '2' not found.
The input is security_groups = [u'2'] STRING 2
But allowed groups is [u'default', 2] INT 2
So, '2' is never found in the allowed_groups.
Based on an IRC conversation, it should be acceptable to cast the ids to str() so that we don't improperly report an invalid security group.
Changed in sahara: | |
assignee: | nobody → Chad Roberts (croberts) |
Changed in sahara: | |
importance: | Undecided → Medium |
Changed in sahara: | |
milestone: | none → kilo-1 |
Changed in sahara: | |
status: | Fix Committed → Fix Released |
Changed in sahara: | |
milestone: | kilo-1 → 2015.1.0 |
Fix proposed to branch: master /review. openstack. org/130576
Review: https:/