Skip to content

Commit e794622

Browse files
[AMBARI-26134]Resolve Test case failures in Ambari Metrics host Monitoring
1 parent 395d653 commit e794622

File tree

8 files changed

+31
-15
lines changed

8 files changed

+31
-15
lines changed

ambari-metrics-host-monitoring/src/main/python/core/emitter.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,10 @@
2121
import logging
2222
import threading
2323

24-
from resource_monitoring.core.security import CachedHTTPSConnection, CachedHTTPConnection
25-
from resource_monitoring.core.blacklisted_set import BlacklistedSet
26-
from resource_monitoring.core.config_reader import ROUND_ROBIN_FAILOVER_STRATEGY
27-
from resource_monitoring.core.spnego_kerberos_auth import SPNEGOKerberosAuth
24+
from security import CachedHTTPSConnection, CachedHTTPConnection
25+
from blacklisted_set import BlacklistedSet
26+
from config_reader import ROUND_ROBIN_FAILOVER_STRATEGY
27+
from spnego_kerberos_auth import SPNEGOKerberosAuth
2828

2929
logger = logging.getLogger()
3030

ambari-metrics-host-monitoring/src/main/python/core/host_info.py

+1
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ def get_mem_info(self):
120120
swap_total = self.__host_static_info.get('swap_total')
121121

122122
bytes2kilobytes = lambda x: x / 1024
123+
mem_total = psutil.virtual_memory().total
123124

124125
return {
125126
'mem_total': bytes2kilobytes(mem_total) if mem_total else 0,

ambari-metrics-host-monitoring/src/main/python/core/metering.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import logging
2222
import time
2323
import json
24-
from resource_monitoring.core.instance_type_provider import HostInstanceTypeProvider
24+
from instance_type_provider import HostInstanceTypeProvider
2525

2626
logger = logging.getLogger()
2727

ambari-metrics-host-monitoring/src/main/python/core/metric_collector.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@
2020

2121
import logging
2222
from time import time
23-
from resource_monitoring.core.event_definition import HostMetricCollectEvent, ProcessMetricCollectEvent
24-
from resource_monitoring.core.metering import MeteringMetricHandler
23+
from event_definition import HostMetricCollectEvent, ProcessMetricCollectEvent
24+
from metering import MeteringMetricHandler
2525

2626
logger = logging.getLogger()
2727

ambari-metrics-host-monitoring/src/main/python/core/spnego_kerberos_auth.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
try:
2727
import kerberos
2828
except ImportError:
29-
from resource_monitoring.core import krberr as kerberos
29+
from krberr import krberr as kerberos
3030
logger.warn('import kerberos exception: %s' % str(ImportError))
3131
pass
3232

ambari-metrics-host-monitoring/src/test/python/core/TestEmitter.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import json
2222
import logging
2323
import time
24+
import distro
2425

2526
from unittest import TestCase
2627
from mock.mock import patch, MagicMock
@@ -30,7 +31,7 @@
3031

3132
os_distro_value = ('Suse','11','Final')
3233

33-
with patch("platform.linux_distribution", return_value = os_distro_value):
34+
with patch("distro.linux_distribution", return_value=os_distro_value):
3435
from application_metric_map import ApplicationMetricMap
3536
from config_reader import Configuration
3637
from emitter import Emitter

ambari-metrics-host-monitoring/src/test/python/core/TestHostInfo.py

+18-6
Original file line numberDiff line numberDiff line change
@@ -71,21 +71,33 @@ def testMemInfo(self, vm_mock, sw_mock):
7171
vm.free = 2312043
7272
vm.shared = 1243
7373
vm.buffers = 23435
74-
vm.cached = 23545
7574
vm.available = 2312043
75+
vm.total = 8192 * 1024
76+
vm.used = 4096 * 1024
77+
vm.cached = 1024 * 1024
7678

7779
sw = sw_mock.return_value
7880
sw.free = 2341234
81+
sw.used = 1024 * 1024
82+
sw.total = 2048 * 1024
83+
sw.sin = 512 * 1024
84+
sw.sout = 256 * 1024
7985

8086
hostinfo = HostInfo(MagicMock())
8187

8288
mem = hostinfo.get_mem_info()
8389

84-
self.assertAlmostEqual(mem['mem_free'], 2257)
85-
self.assertAlmostEqual(mem['mem_shared'], 1)
86-
self.assertAlmostEqual(mem['mem_buffered'], 22)
87-
self.assertAlmostEqual(mem['mem_cached'], 22)
88-
self.assertAlmostEqual(mem['swap_free'], 2286)
90+
self.assertAlmostEqual(mem['mem_free'], 2257, delta = 1)
91+
self.assertAlmostEqual(mem['mem_shared'], 1, delta=0.5)
92+
self.assertAlmostEqual(mem['mem_buffered'], 22, delta = 1)
93+
self.assertAlmostEqual(mem['mem_cached'], 1024, delta = 1)
94+
self.assertAlmostEqual(mem['swap_free'], 2286, delta = 1)
95+
self.assertAlmostEqual(mem['mem_total'], 8192, delta = 1)
96+
self.assertAlmostEqual(mem['mem_used'], 3072, delta = 1)
97+
self.assertAlmostEqual(mem['swap_used'], 1024, delta = 1)
98+
self.assertAlmostEqual(mem['swap_total'], 2048, delta = 1)
99+
self.assertAlmostEqual(mem['swap_in'], 512, delta = 1)
100+
self.assertAlmostEqual(mem['swap_out'], 256, delta = 1)
89101

90102

91103
@patch("psutil.process_iter")

ambari-metrics-host-monitoring/src/test/python/mock/mock.py

+2
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@
3131

3232
import pprint
3333
import sys
34+
import distro
35+
os_distro_value = distro.linux_distribution(full_distribution_name=False)
3436

3537
try:
3638
import inspect

0 commit comments

Comments
 (0)