fix: Append two additional facts to the graylist

src/rhsmlib/facts/collection.py

@@ -21,7 +21,14 @@
 class FactsDict(collections.abc.MutableMapping):
     """A dict for facts that ignores items in 'graylist' on compares."""
 
-    graylist = set(["cpu.cpu_mhz", "lscpu.cpu_mhz"])
+    # see bz #627962
+    # we would like to have this info, but for now, since it
+    # can change constantly on laptops, it makes for a lot of
+    # fact churn, so we report it, but ignore it as an indicator
+    # that we need to update
+    GRAYLIST = set(
+        ["cpu.cpu_mhz", "lscpu.cpu_mhz", "proc_cpuinfo.common.cpu_mhz", "lscpu.cpu(s)_scaling_mhz"]
+    )
 
     def __init__(self, *args, **kwargs):
         super(FactsDict, self).__init__(*args, **kwargs)
@@ -43,12 +50,12 @@ def __len__(self) -> int:
         return len(self.data)
 
     def __eq__(self, other) -> bool:
-        """Compares all of the items in self.data, except it ignores keys in self.graylist."""
+        """Compares all of the items in self.data, except it ignores keys in self.GRAYLIST."""
         if not isinstance(other, FactsDict):
             return NotImplemented
 
-        keys_self = set(self.data).difference(self.graylist)
-        keys_other = set(other.data).difference(self.graylist)
+        keys_self = set(self.data).difference(self.GRAYLIST)
+        keys_other = set(other.data).difference(self.GRAYLIST)
         if keys_self == keys_other:
             if all(self.data[k] == other.data[k] for k in keys_self):
                 return True

src/subscription_manager/facts.py

@@ -12,13 +12,14 @@
 from datetime import datetime
 import logging
 import os
-from typing import Dict, List, Optional, TYPE_CHECKING
+from typing import Dict, Optional, TYPE_CHECKING
 
 from subscription_manager.injection import PLUGIN_MANAGER, require
 from subscription_manager.cache import CacheManager
 from rhsm import ourjson as json
 
 from rhsmlib.facts.all import AllFactsCollector
+from rhsmlib.facts.collection import FactsDict
 
 if TYPE_CHECKING:
     from subscription_manager.plugins import PluginManager
@@ -41,13 +42,6 @@ class Facts(CacheManager):
     def __init__(self):
         self.facts = {}
 
-        # see bz #627962
-        # we would like to have this info, but for now, since it
-        # can change constantly on laptops, it makes for a lot of
-        # fact churn, so we report it, but ignore it as an indicator
-        # that we need to update
-        self.graylist: List[str] = ["cpu.cpu_mhz", "lscpu.cpu_mhz"]
-
         # plugin manager so we can add custom facts via plugin
         self.plugin_manager: PluginManager = require(PLUGIN_MANAGER)
 
@@ -70,7 +64,7 @@ def has_changed(self) -> bool:
         # In order to accurately check for changes, we must refresh local data
         self.facts = self.get_facts(True)
 
-        for key in (set(self.facts) | set(cached_facts)) - set(self.graylist):
+        for key in (set(self.facts) | set(cached_facts)) - FactsDict.GRAYLIST:
             if self.facts.get(key) != cached_facts.get(key):
                 return True
         return False