Project

General

Profile

Actions

Bug #22571

closed

Storage allocation web UI issue with LVM

Added by Alexandre Anriot about 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Hello,

I'm using Foreman 1.16 on a Debian Stretch (amd64) server, with several Foreman-proxies.

Some compute profiles are configured to use a LVM-based storage pull, with "Full" allocation.

When I change the "Size (GB)" setting on the host creation window, the "Allocation (GB)" field is not updated automatically and I have to:

  • choose a different allocation value like "None" or "Size" ;
  • switch back to "Full".

Could you tell me if it is the expected behaviour?

The documentation gave me no answer.

Thanks a lot.

Cheers,

Actions #1

Updated by Lukas Zapletal about 6 years ago

  • Subject changed from Storage allocation issue with LVM to Storage allocation web UI issue with LVM
  • Category set to Web Interface

Hey, this works for me with devel version (Chrome stable, Linux). Can you record a video to verify I understand the problem correctly? Any javascript errors in debug web browser console?

Actions #2

Updated by Alexandre Anriot about 6 years ago

Lukas Zapletal wrote:

Hey, this works for me with devel version (Chrome stable, Linux). Can you record a video to verify I understand the problem correctly? Any javascript errors in debug web browser console?

Hello Lukas,

To avoid storing this temporary file on Redmine, I did just sent a quick screencast to you by mail (Ubuntu + latest Firefox).

I see no JavaScript errors.

Cheers,

Actions #3

Updated by Lukas Zapletal about 6 years ago

A bug in COMPUTE PROFILE perhaps. I can confirm. Without COMPUTE PROFILE it works fine.

Actions #5

Updated by Liza Gilman about 6 years ago

Hey Alexandre, possibly you just have to click outside of the field after changing the value?
Could you send me the video as well?
Thanks

Actions #6

Updated by Alexandre Anriot about 6 years ago

Liza Gilman wrote:

Hey Alexandre, possibly you just have to click outside of the field after changing the value?

Hello Liza,

Even when clicking outside, the field doesn't get updated.

Could you send me the video as well?
Thanks

It should be in your inbox now.

Best regards,

Actions #7

Updated by Liza Gilman about 6 years ago

  • Assignee set to Liza Gilman
Actions #8

Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5298 added
Actions #9

Updated by Alexandre Anriot about 6 years ago

Liza,

To begin with, we have upgraded our Foreman platform (server + 2 proxies) to 1.17 RC1, and I can confirm that the same problem still occurs.

How can I apply the 2nd part of the patch as the webpack files are compiled? Could you provide a compiled bundle JS file?

Thanks.

Best regards,

Actions #11

Updated by Alexandre Anriot about 6 years ago

Liza,

Thanks but I don't see any foreman_compute_resource.js file on my server. It has tos compiled, right?

Actions #12

Updated by Alexandre Anriot about 6 years ago

Alexandre Anriot wrote:

Liza,

Thanks but I don't see any foreman_compute_resource.js file on my server. It has tos compiled, right?

s/tos/to be/

Actions #13

Updated by Liza Gilman about 6 years ago

It should be at webpack/assets/javascripts/

Actions #14

Updated by Alexandre Anriot about 6 years ago

Liza,

Here is the webpack files list I have:

$ > ls -al /usr/share/foreman/public/webpack
total 3636
drwxr-xr-x 2 foreman foreman    4096 Mar  8 16:03 .
drwxr-xr-x 5 foreman foreman    4096 Mar  7 10:27 ..
-rw-r--r-- 1 foreman foreman   70588 Jan 30 16:57 bundle-4dc8d271027ff6d9a2b7.css
-rw-r--r-- 1 foreman foreman   13400 Jan 30 16:57 bundle-4dc8d271027ff6d9a2b7.css.gz
-rw-r--r-- 1 foreman foreman  109200 Jan 30 16:57 bundle-4dc8d271027ff6d9a2b7.js
-rw-r--r-- 1 foreman foreman   26607 Jan 30 16:57 bundle-4dc8d271027ff6d9a2b7.js.gz
-rw-r--r-- 1 foreman foreman     216 Jan 30 16:57 manifest.json
-rw-r--r-- 1 foreman foreman 2736704 Jan 30 16:57 vendor-8aaef31828f86f16484f.js
-rw-r--r-- 1 foreman foreman  738509 Jan 30 16:57 vendor-8aaef31828f86f16484f.js.gz
Actions #15

Updated by Alexandre Anriot about 6 years ago

I forgot to mention that I'm using the 1.17.0~rc1-1 Debian packages provided on [[http://deb.theforeman.org/pool/stretch/1.17/f/foreman/]].

Actions #16

Updated by Lukas Zapletal about 6 years ago

  • translation missing: en.field_release set to 330
Actions #17

Updated by Liza Gilman about 6 years ago

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

Also available in: Atom PDF