Bug 1757689 - Remove memAvailable and memCommitted
Summary: Remove memAvailable and memCommitted
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Backend.Core
Version: 4.4.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ovirt-4.4.8
: ---
Assignee: Saif Abusaleh
QA Contact: Tamir
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-10-02 07:51 UTC by Liran Rotenberg
Modified: 2021-11-04 19:28 UTC (History)
4 users (show)

Fixed In Version: ovirt-engine-4.4.8.2, vdsm-4.40.80.3
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-08-19 06:23:09 UTC
oVirt Team: Virt
Embargoed:
pm-rhel: ovirt-4.4+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 115868 0 master MERGED engine: remove mem_available field 2021-07-27 14:13:55 UTC
oVirt gerrit 115877 0 master MERGED api: virt: remove memAvailable and memCommited from vdsm 2021-07-28 08:31:45 UTC

Description Liran Rotenberg 2019-10-02 07:51:00 UTC
memAvailable and memCommitted are used for the host statistics are deprecated and not in use for any logic. therfore it should be removed.

Comment 1 Arik 2021-05-31 11:45:30 UTC
Liran, can you please check if it was ever used by engine 4.2+ and if these fields appear in the DWH views? if it's a simple removal, we'll target it

Comment 2 Aviv Litman 2021-06-03 09:43:40 UTC
Hi Liran @lrotenbe 

I see both mem_commited and mem_available in the engine:
- mem_commited in table vds_dynamic
- mem_available in vds_statistics
We do not collect mem_commited and mem_available from engine to DWH.

Comment 3 Aviv Litman 2021-06-03 09:50:28 UTC
Note that the columns are also in the engine views:
- mem_commited, and mem_available both in vds, and vds_with_tags views.

Comment 4 Liran Rotenberg 2021-06-03 12:30:55 UTC
Yes mem_commited is calculated by the engine, shouldn't be removed. The 'memCommitted' should (VDSM). The 'memAvailable' should be dropped from both, as it seem unused and deprecated for years.
Thanks for the confirmation Aviv!

Comment 5 Tamir 2021-08-12 13:59:48 UTC
Verified on RHV 4.4.8-4. All looks good to me.

Env:
  - Engine instance with RHV 4.4.8-4 (ovirt-engine-4.4.8.3-0.10.el8ev) and RHEL 8.4 installed.
  - 3 hosts with RHV 4.4.8-4 and RHEL 8.4, vdsm-4.40.80.4-1.el8ev


Tests:

- Check that memAvailable and memCommited were removed from the host's stats in vdsm.

Steps:

1. Run the command in 1 of the hosts - "vdsm-client Host getStats | grep mem".
2. Check that memAvailable and memCommited attributes aren't present in the command's output.

Results (As Expected):

1. The command returned the host's stats.
2. memAvailable and memCommited attributes aren't present in the command's output.


- Check that memAvailable was removed from the engine.

Tests:

  - Check that the column memAvailable was removed from the views vds_with_tags and vds, the table vds_statistics and from the functions InsertVdsStatistics and UpdateVdsStatistics.
  Result (As Expected): The column memAvailable was removed from all those entities.


  - Do a manual regression. 

In Admin Portal:

1. Create a 4.5 data center and a 4.5 cluster.
2. Install the host and create a new NFS storage domain.
3. Create a RHEL 8.3 VM with NIC, 3GB memory and set it to start on the same host only.
4. Run the VM and check for it's memory usage with the "free" command.
5. Fill the memory on the host.
6. Check that the host's memory values was updated in the host's page.
7. Check that the VM is running normally and it's memory usage values didn't change drastically.


Results (As Expected):

1. The 4.5 data center and the 4.5 cluster were created.
2. The host was installed and the NFS storage domain was created.
3. The VM was created.
4. The VM is running and the memory values are normal.
5. The Host's memory is almost full.
6. The memory values were updated in the host's page.
7. The VM is running normally and it's memory usage values didn't change drastically.

Comment 6 Sandro Bonazzola 2021-08-19 06:23:09 UTC
This bugzilla is included in oVirt 4.4.8 release, published on August 19th 2021.

Since the problem described in this bug report should be resolved in oVirt 4.4.8 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.