In heartbeat log, set value for Virtual Cores

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: abicloud-1.5.0-PR
  • Fix Version/s: Abiquo-1.6.5
  • Component/s: Heartbeat / Register
  • Labels:
    None
  • Affect Build:
    2311

Description

Steps to Reproduce
-Enter as sysadmin
-go to configuration -> Heartbeating
-turn on the heartbeating and look at last heartbeat entry
-look at the Virtual Cores fields

Expected Results
-Virtual Cores = this is the virtual cores number REALS.
For example, we have 10 physical machine with 2 CPU with 4 cores/CPU -> virtual cores = 80
-Modify the literal since these are not Virtual Cores, these are "Real Cores"

Real Results
-Virtual Cores = 0, this does not exist in database
  1. File
    screenshot
    23/Jul/10 12:20 PM
    498 kB
    David Calavera
  1. virtual_cores.png
    54 kB
    21/Mar/11 1:44 PM

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
David Calavera added a comment -
Actually this is not a bug. The virtual cores used are stored in the database as cpuUsed in the physical machine. The query that collects this data is fine and I can see results in the page.

I'm attaching a screenshot to verify it works
Show
David Calavera added a comment - Actually this is not a bug. The virtual cores used are stored in the database as cpuUsed in the physical machine. The query that collects this data is fine and I can see results in the page. I'm attaching a screenshot to verify it works
Hide
Permalink
Valerie Kalusinski added a comment -
In this screenshot, the row shown is Virtual Cores used.
There are 3 information about virtual cores:
-virtual cores used
-virtual cores allocated
-virtual cores
This last one should be defined.
Show
Valerie Kalusinski added a comment - In this screenshot, the row shown is Virtual Cores used. There are 3 information about virtual cores: -virtual cores used -virtual cores allocated -virtual cores This last one should be defined.
Hide
Permalink
David Calavera added a comment -
I've also added total_virtual_memory and total_volume_space to those fields.

It requires changes into the heartbeat database as well as the kinton database.
Show
David Calavera added a comment - I've also added total_virtual_memory and total_volume_space to those fields. It requires changes into the heartbeat database as well as the kinton database.
Hide
Permalink
Valerie Kalusinski added a comment -
Xavi, could you verify and close this issue ?
thanks.
Show
Valerie Kalusinski added a comment - Xavi, could you verify and close this issue ? thanks.
Hide
Permalink
Xavier Fernandez added a comment -
This is still failing

1) Change label "Virtual Cores" with "Real CPUs"

2) On screen we are showing 0 for this value. On DB it's with the right value. Assure that you are not using another value or the server it's not sending the correct value

Regards
Show
Xavier Fernandez added a comment - This is still failing 1) Change label "Virtual Cores" with "Real CPUs" 2) On screen we are showing 0 for this value. On DB it's with the right value. Assure that you are not using another value or the server it's not sending the correct value Regards
Hide
Permalink
Xavier Thevenot added a comment -
Please check why values are retrieved with NULL values
Show
Xavier Thevenot added a comment - Please check why values are retrieved with NULL values
Hide
Permalink
Zain Malik added a comment -
Virtual Cores weren't assigned in any place


PD: the commit message accidently done with out '-'
https://github.com/abiquo/heartbeat/commit/43a8bca56260de5063dee70fc209ecad495f97f7

Show
Zain Malik added a comment - Virtual Cores weren't assigned in any place PD: the commit message accidently done with out '-' https://github.com/abiquo/heartbeat/commit/43a8bca56260de5063dee70fc209ecad495f97f7
Hide
Permalink
Valerie Kalusinski added a comment -
1) ok
2) Real cpu (total virtual cores in BD) still is 0

Reopened on 1.7.6 master.
Show
Valerie Kalusinski added a comment - 1) ok 2) Real cpu (total virtual cores in BD) still is 0 Reopened on 1.7.6 master.
Hide
Permalink
Valerie Kalusinski added a comment -
Verified on 1.7.6 master
Show
Valerie Kalusinski added a comment - Verified on 1.7.6 master

People

  • Assignee:
    Zain Malik
    Reporter:
    Valerie Kalusinski
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: