Project

General

Profile

Actions

Bug #24541

closed

Can not store "Add cdrom drive" on compute profile

Added by Dirk Götz about 6 years ago. Updated about 6 years ago.

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

Description

On the Compute Profile using Compute Resource VMware I can select "Add cdrom drive" but when storing the Profile it is not saved, so opening the Compute Profile again the box is unchecked and also using it while creating a Host requires always to check the box again. Using boot disk based provisioning it requires to remember always to check this box or if you forget to delete and recreate the host.

Installation is Foreman 1.18.0, Compute Resource VMware, Bootdisk and Column View Plugin.

Actions #1

Updated by Marek Hulán about 6 years ago

  • Bugzilla link set to 1566543
Actions #2

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Marek Hulán
  • Pull request https://github.com/theforeman/foreman/pull/5924 added
Actions #3

Updated by Marek Hulán about 6 years ago

The issue is interestingly not in the fact we wouldn't save the add_cdrom value. It's cause by the fact the checkbox is named add_cdrom, which we then turn into "cdrom" in fog attributes. When we redraw the compute profile form, it does not detect add_cdrom is set to 1, therefore it remains unchecked. The same applies when we enter new host form, after choosing compute profile, the checkbox simply remains unchecked.

Dirk, it would be great if you could test the PR and let us know if it fixes the issue for you.

Actions #4

Updated by Dirk Götz about 6 years ago

I tested it and it works fine.

Thanks, Marek.

Actions #5

Updated by Tomer Brisker about 6 years ago

  • Fixed in Releases 1.20.0 added
Actions #6

Updated by Marek Hulán about 6 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF