ManageIQ / manageiq

ManageIQ Open-Source Management Platform
https://manageiq.org
Apache License 2.0
1.35k stars 896 forks source link

Reports Fail When Using CPU or Memory Ratio Column [Najdorf-1 vSphere Image] #22059

Open derekleeth opened 2 years ago

derekleeth commented 2 years ago

We are having an issue with the latest stable version [Najdorf-1] we've deployed in one of our datacenters where reports fail to show any data at all when you add the "CPU Cores Virtual to Real Ratio" or "Memory Virtual to Real Ratio" columns to a report. We are running older versions of MIQ (Ivanchuk & Fine) in some datacenters that do not have this issue. If we remove those columns from the report, the report does not error out and shows data.

Report w/ Ratio Columns

image

Reports Failing w/ Ratio Columns

image

Report w/out Real Ratio Columns Showing Data

image

Fryguy commented 2 years ago

Sounds like a possible bug with aggregation?

derekleeth commented 2 years ago

I'm not 100% sure if it helps but it does show the "Virtual to Real Ratio" for memory and CPU when you pull up an individual cluster. Here's the screenshot below that shows the two fields.

image

miq-bot commented 1 year ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

miq-bot commented 1 year ago

This issue has been automatically closed because it has not been updated for at least 3 months.

Feel free to reopen this issue if this issue is still valid.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

kbrock commented 1 year ago

columns referenced:

general notes:

kbrock commented 1 year ago

@Fryguy I noticed that EmsCluster uses AggregationMixin.

class EmsCluster
  include AggregationMixin
  # include `RelationshipsAggregationMixin`
  has_many    :vms

  def direct_vms
    Relationship.resources(direct_vm_rels).sort_by { |v| v.name.downcase }
  end
end

That is used for creating virtual totals based upon a has_many :vms which sounds about right. But it also has the direct_vms, which make it sound like this should be a recursive type calculation and therefore using relationships

These seem to contradict each other. Ideas which one of these is correct?

Fryguy commented 1 year ago

Cluster has a normal vms relationship in the database...this is the same as "all_vms". direct_vms goes through resource_pools which is relationship based and only gets vms directly in the immediate resource pool.

kbrock commented 1 year ago

ok. cool. so direct_vms is an extra addon.

EmsFolder and ResourcePool definitely use relationships for children

miq-bot commented 11 months ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

miq-bot commented 7 months ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

miq-bot commented 4 months ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

miq-bot commented 1 month ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.