Project

General

Profile

Actions

Bug #3544

closed

Editing an oVirt compute resource allows changing the type, which is unsupported

Added by Dominic Cleal over 10 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Compute resources
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Description of problem:
When editing compute resources of certain types (e.g. EC2 and oVirt), the form allows the user to change the CR type to another. This is unsupported on existing CRs as VM UUIDs will change and CR attributes don't get updated properly. Some CR types (e.g. libvirt) correctly have it disabled. This option should be disabled consistently.

How reproducible:
Always

Steps to Reproduce:
1. More>Provisioning>Compute Resources, create a new oVirt compute resource
2. Edit the existing compute resource
3. Check if the type can be changed to another (e.g. EC2)

Actual results:
Type dropdown menu is not disabled, allows changing it (perhaps with an error on save).

Expected results:
Type dropdown is greyed out and disabled.

Additional info:
The dropdown is disabled based on "f.object.uuid.present?", however not all CR types have UUIDs. It should probably be .new_record?.

(https://bugzilla.redhat.com/show_bug.cgi?id=1024539)

Actions #1

Updated by Joseph Magen about 10 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Joseph Magen
  • Target version set to 1.8.2
Actions #2

Updated by Anonymous about 10 years ago

  • Target version changed from 1.8.2 to 1.8.1
Actions #3

Updated by Anonymous about 10 years ago

  • Target version changed from 1.8.1 to 1.8.0
Actions #4

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.8.0 to 1.7.5
Actions #5

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.7.5 to 1.7.4
Actions #6

Updated by The Foreman Bot almost 10 years ago

  • Pull request https://github.com/theforeman/foreman/pull/1476 added
Actions #7

Updated by Anonymous almost 10 years ago

  • Target version changed from 1.7.4 to 1.7.3
Actions #8

Updated by Dominic Cleal almost 10 years ago

  • Translation missing: en.field_release set to 21
Actions #9

Updated by Joseph Magen almost 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF