Delete flavor by UI not working

Hello,
i think and i m sure if you try to delete a flavor by UI it’s not working
The flavor still in BDD
and the log :

/var/www/miq/vmdb/app/models/flavor.rb:101:in `raw_delete_flavor': raw_delete_flavor must be implemented in a subclass (NotImplementedError)
        from /var/www/miq/vmdb/app/models/flavor.rb:109:in `delete_flavor'
        from /var/www/miq/vmdb/app/models/miq_queue.rb:479:in `block in dispatch_method'
        from /usr/local/lib/ruby/2.5.0/timeout.rb:93:in `block in timeout'
        from /usr/local/lib/ruby/2.5.0/timeout.rb:33:in `block in catch'
        from /usr/local/lib/ruby/2.5.0/timeout.rb:33:in `catch'
        from /usr/local/lib/ruby/2.5.0/timeout.rb:33:in `catch'
        from /usr/local/lib/ruby/2.5.0/timeout.rb:108:in `timeout'
        from /var/www/miq/vmdb/app/models/miq_queue.rb:477:in `dispatch_method'
        from /var/www/miq/vmdb/app/models/miq_queue.rb:454:in `block in deliver'
        from /var/www/miq/vmdb/app/models/user.rb:290:in `with_user_group'
        from /var/www/miq/vmdb/app/models/miq_queue.rb:454:in `deliver'
        from /var/www/miq/vmdb/app/models/miq_queue_worker_base/runner.rb:104:in `deliver_queue_message'
        from /var/www/miq/vmdb/app/models/miq_queue_worker_base/runner.rb:137:in `deliver_message'
        from /var/www/miq/vmdb/app/models/miq_queue_worker_base/runner.rb:155:in `block in do_work'
        from /var/www/miq/vmdb/app/models/miq_queue_worker_base/runner.rb:149:in `loop'
        from /var/www/miq/vmdb/app/models/miq_queue_worker_base/runner.rb:149:in `do_work'
        from /var/www/miq/vmdb/app/models/miq_worker/runner.rb:329:in `block in do_work_loop'
        from /var/www/miq/vmdb/app/models/miq_worker/runner.rb:326:in `loop'
        from /var/www/miq/vmdb/app/models/miq_worker/runner.rb:326:in `do_work_loop'
        from /var/www/miq/vmdb/app/models/miq_worker/runner.rb:153:in `run'
        from /var/www/miq/vmdb/app/models/miq_worker/runner.rb:127:in `start'
        from /var/www/miq/vmdb/app/models/miq_worker/runner.rb:22:in `start_worker'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:398:in `block in start_runner_via_fork'
        from /usr/local/lib/ruby/gems/2.5.0/gems/nakayoshi_fork-0.0.4/lib/nakayoshi_fork.rb:23:in `fork'
        from /usr/local/lib/ruby/gems/2.5.0/gems/nakayoshi_fork-0.0.4/lib/nakayoshi_fork.rb:23:in `fork'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:396:in `start_runner_via_fork'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:386:in `start_runner'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:437:in `start'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:269:in `start_worker'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:147:in `block in sync_workers'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:147:in `times'
        from /var/www/miq/vmdb/app/models/miq_worker.rb:147:in `sync_workers'
        from /var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:54:in `block in sync_workers'
        from /var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:50:in `each'
        from /var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:50:in `sync_workers'
        from /var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:22:in `monitor_workers'
        from /var/www/miq/vmdb/app/models/miq_server.rb:346:in `block in monitor'
        from /usr/local/lib/ruby/gems/2.5.0/bundler/gems/manageiq-gems-pending-e280ad44830f/lib/gems/pending/util/extensions/miq-benchmark.rb:11:in `realtime_store'
        from /usr/local/lib/ruby/gems/2.5.0/bundler/gems/manageiq-gems-pending-e280ad44830f/lib/gems/pending/util/extensions/miq-benchmark.rb:28:in `realtime_block'
        from /var/www/miq/vmdb/app/models/miq_server.rb:346:in `monitor'
        from /var/www/miq/vmdb/app/models/miq_server.rb:387:in `block (2 levels) in monitor_loop'
        from /usr/local/lib/ruby/gems/2.5.0/bundler/gems/manageiq-gems-pending-e280ad44830f/lib/gems/pending/util/extensions/miq-benchmark.rb:11:in `realtime_store'
        from /usr/local/lib/ruby/gems/2.5.0/bundler/gems/manageiq-gems-pending-e280ad44830f/lib/gems/pending/util/extensions/miq-benchmark.rb:35:in `realtime_block'
        from /var/www/miq/vmdb/app/models/miq_server.rb:387:in `block in monitor_loop'
        from /var/www/miq/vmdb/app/models/miq_server.rb:386:in `loop'
        from /var/www/miq/vmdb/app/models/miq_server.rb:386:in `monitor_loop'
        from /var/www/miq/vmdb/app/models/miq_server.rb:248:in `start'
        from /var/www/miq/vmdb/lib/workers/evm_server.rb:27:in `start'
        from /var/www/miq/vmdb/lib/workers/evm_server.rb:48:in `start'
        from /var/www/miq/vmdb/lib/workers/bin/evm_server.rb:4:in `<main>'

Thanks
Bruno

This looks like a bug in that I wouldn’t expect you to be able to delete a flavor. Or perhaps you can delete flavors but only on certain providers that also support creating flavors. @agrare?

Delete flavor is only implemented by OpenStack, there is a supports feature flag (supports_create? and supports_delete?) that can be used to check if this feature is implemented on a flavor.

If the UI isn’t checking this and allowing you to issue a delete on a different flavor that sounds like a UI bug.