Project

General

Profile

Actions

Bug #10779

open

Memory detection of Xen hosts is incorrect

Added by Florian Heigl over 7 years ago. Updated 5 months ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Web - Nodes & inventories
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Priority:
0
Name check:
Fix check:
Regression:

Description

Fusion Inventory seems to only look at /proc/meminfo to detect the memory a Xen host has.

That will reflect the memory assigned the dom0 - likely 2-16GB, whereas all the other ram is managed by the hypervisor.
The problem here is that Rudder uses that to fill the field named "Total physical memory (RAM)"

That is wrong. Total physical memory is obtained only via xl info | grep total_memory (newer systems) or xm info | grep total_memory (old systems)

It matters, really:
In my example server the difference between the two is just about 990GB.

That way you can't apply any rule-based tuning profiles (not that I have one yet, but not that I could have one, either ;-)

Actions #1

Updated by François ARMAND over 7 years ago

  • Related to Bug #10787: Xen domU detection issues with pvops kernels. added
Actions #2

Updated by François ARMAND over 7 years ago

  • Related to deleted (Bug #10787: Xen domU detection issues with pvops kernels.)
Actions #3

Updated by François ARMAND over 7 years ago

  • Priority changed from 0 to 35

Thanks for reporting.

What are the value(s) reported in hardware > memory?

Actions #4

Updated by Janos Mattyasovszky over 7 years ago

Nope.

Node Summary -> Node characteristics -> General -> Total physical memory (RAM): 9.08 GB

The dom0 has memory like:

# free -m
             total       used       free     shared    buffers     cached
Mem:          9298       5482       3815        457         28       4328
-/+ buffers/cache:       1125       8172
Swap:        32767          2      32765

The system has memory like:

# xl info 2>&1 | grep memory
total_memory           : 1048467
free_memory            : 998984

Here we see: 9298/1024 = 9.08, which is definetly not "Total physical memory", but "Total OS memory"

Actions #5

Updated by François ARMAND over 7 years ago

Yes, it is defenitly os memory, not physical memory. Fusion inventory is not correctly labeling the memory reported (it is in tag <HARDWARE><MEMORY>), and that we (dumbly) used the same term.

Do you have defferent information in Node Summary -> Node characteristics -> hardware > memory, where you should have "memory slots"? I'm almost sure it will be the same as os memory, though.

Actions #6

Updated by Benoît PECCATTE over 7 years ago

  • Priority changed from 35 to 50
Actions #7

Updated by Florian Heigl over 7 years ago

  • Priority changed from 50 to 49

Sorry I had missed your feedback request. Will check.

Actions #8

Updated by Florian Heigl over 7 years ago

Hi,

the memory slots are correctly detected...

Number of slots is correct (96)
and it also knows in which slot modules are installed.
A sum is not available.

I would much prefer if the correct number was obtained based on the xl/xm info output.
less effort on all ends and not relying on dmidecode (at least on ARM that matters!)

Actions #9

Updated by Benoît PECCATTE about 7 years ago

  • Priority changed from 49 to 47
Actions #10

Updated by François ARMAND almost 6 years ago

  • Target version set to 4.3.8
  • Priority changed from 47 to 0
Actions #11

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.3.8 to 4.3.9
Actions #12

Updated by Alexis Mousset almost 6 years ago

  • Target version changed from 4.3.9 to 4.3.10
Actions #13

Updated by François ARMAND over 5 years ago

  • Target version changed from 4.3.10 to 4.3.11
Actions #14

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.3.11 to 4.3.12
Actions #15

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.3.12 to 4.3.13
Actions #16

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.3.13 to 4.3.14
Actions #17

Updated by François ARMAND over 5 years ago

  • Translation missing: en.field_tag_list set to Sponsored
  • Severity changed from Minor - inconvenience | misleading | easy workaround to Major - prevents use of part of Rudder | no simple workaround
  • User visibility changed from Getting started - demo | first install | level 1 Techniques to Getting started - demo | first install | Technique editor and level 1 Techniques
  • Priority changed from 0 to 85
Actions #18

Updated by François ARMAND over 5 years ago

  • Severity changed from Major - prevents use of part of Rudder | no simple workaround to Minor - inconvenience | misleading | easy workaround
  • Priority changed from 85 to 69
Actions #19

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.3.14 to 587
Actions #20

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 587 to 4.3.14
Actions #21

Updated by Alexis Mousset over 5 years ago

  • Target version changed from 4.3.14 to 5.0.13

Probably still there

Actions #22

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions #23

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
Actions #24

Updated by Benoît PECCATTE about 5 years ago

  • Effort required set to Medium
  • Priority changed from 69 to 55
Actions #25

Updated by François ARMAND about 5 years ago

  • Effort required changed from Medium to Small
  • Priority changed from 55 to 82

We have an easy generic solution:

- rename "total memory" as "os memory"
- add a "total physical memory" as sum of all slots.

We must not change API keys, so we likely need to had one new (os memory), make it the default, and change current total physical to optionnal.

Actions #26

Updated by Alexis Mousset about 5 years ago

  • Category changed from Agent to Web - Nodes & inventories
Actions #27

Updated by Benoît PECCATTE about 5 years ago

For people who want to test we xen it is on ubuntu-18-04-64.labo.normation.com that is on orchestrateur-4.labo.normation.com

Actions #28

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 5.0.15 to 5.0.16
Actions #29

Updated by Alexis Mousset almost 5 years ago

  • Target version changed from 5.0.16 to 5.0.17
Actions #30

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions #31

Updated by François ARMAND over 4 years ago

  • User visibility changed from Getting started - demo | first install | Technique editor and level 1 Techniques to Operational - other Techniques | Rudder settings | Plugins
  • Effort required deleted (Small)
  • Priority changed from 82 to 55
Actions #32

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.18 to 5.0.19
Actions #33

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.19 to 5.0.20
  • Priority changed from 55 to 56
Actions #34

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 5.0.20 to 797
  • Priority changed from 56 to 57
Actions #35

Updated by Benoît PECCATTE over 3 years ago

  • Target version changed from 797 to 6.1.14
  • Priority changed from 57 to 60
Actions #36

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions #37

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.15 to 6.1.16
  • Priority changed from 60 to 61
Actions #38

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.16 to 6.1.17
Actions #39

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.17 to 6.1.18
  • Priority changed from 61 to 62
Actions #40

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 6.1.18 to 6.1.19
  • Priority changed from 62 to 63
Actions #41

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.19 to 6.1.20
  • Priority changed from 63 to 64
Actions #42

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.20 to 6.1.21
  • Priority changed from 64 to 65
Actions #44

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate
  • Priority changed from 65 to 32
Actions #45

Updated by Alexis Mousset 7 months ago

  • Target version changed from old 6.1 issues to relocate to 7.3.15
  • Priority changed from 32 to 0
Actions #46

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.15 to 7.3.16
Actions #47

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 7.3.16 to 7.3.17
Actions

Also available in: Atom PDF