Reviewed: https://review.openstack.org/528675 Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=154ddab1d4e1dfb46177027f7285ec4c92430966 Submitter: Zuul Branch: master
commit 154ddab1d4e1dfb46177027f7285ec4c92430966 Author: Ahmed Zaid <ahmed.z...@exalt.ps> Date: Mon Dec 18 04:34:07 2017 -0500 Drop flavor name validation in horizon side. This patch drop name validation in horizon and let nova check if the is name is valid or not. Instead of using pattern in the flavors form, pattern will be checked in the nova. Also I added limation for the name size with tests to check the max size since the min size has it's unit test. Closes-Bug: 1738328 Change-Id: Ib4f6f9b87560df43819d992254603e55ea8ff6a3 ** Changed in: horizon Status: In Progress => Fix Released -- You received this bug notification because you are a member of Yahoo! Engineering Team, which is subscribed to OpenStack Dashboard (Horizon). https://bugs.launchpad.net/bugs/1738328 Title: Cannot specify "#" as name of a flavor Status in OpenStack Dashboard (Horizon): Fix Released Bug description: I ran following command with a master of devstack: stack@indoh-test:~/devstack$ openstack flavor create --ram 2048 --disk 10 --vcpus 1 "#1" +----------------------------+--------------------------------------+ | Field | Value | +----------------------------+--------------------------------------+ | OS-FLV-DISABLED:disabled | False | | OS-FLV-EXT-DATA:ephemeral | 0 | | disk | 10 | | id | 308fccc5-b8ff-4435-b022-29b3422ff5f4 | | name | #1 | | os-flavor-access:is_public | True | | properties | | | ram | 2048 | | rxtx_factor | 1.0 | | swap | | | vcpus | 1 | +----------------------------+--------------------------------------+ stack@indoh-test:~/devstack$ openstack flavor list +--------------------------------------+-----------+-------+------+-----------+-------+-----------+ | ID | Name | RAM | Disk | Ephemeral | VCPUs | Is Public | +--------------------------------------+-----------+-------+------+-----------+-------+-----------+ | 1 | m1.tiny | 512 | 1 | 0 | 1 | True | | 2 | m1.small | 2048 | 20 | 0 | 1 | True | | 3 | m1.medium | 4096 | 40 | 0 | 2 | True | | 308fccc5-b8ff-4435-b022-29b3422ff5f4 | #1 | 2048 | 10 | 0 | 1 | True | | 4 | m1.large | 8192 | 80 | 0 | 4 | True | | 5 | m1.xlarge | 16384 | 160 | 0 | 8 | True | | c1 | cirros256 | 256 | 0 | 0 | 1 | True | | d1 | ds512M | 512 | 5 | 0 | 1 | True | | d2 | ds1G | 1024 | 10 | 0 | 1 | True | | d3 | ds2G | 2048 | 10 | 0 | 2 | True | | d4 | ds4G | 4096 | 20 | 0 | 4 | True | +--------------------------------------+-----------+-------+------+-----------+-------+-----------+ However, we couldn't create the flavor which includes "#" as 'name' of flavor. Please refer attached an image. I think it should be aligned with CLI behavior. To manage notifications about this bug go to: https://bugs.launchpad.net/horizon/+bug/1738328/+subscriptions -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : yahoo-eng-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp