Store logger name instead of object.

Some of our unit tests check for equality of `ModuleEntitlements` objects,
and they are entitled to do so (no pun intended).

Author: prdoyle

libs/entitlement/src/main/java/org/elasticsearch/entitlement/runtime/policy/PolicyManager.java

@@ -80,7 +80,7 @@ record ModuleEntitlements(
         String componentName,
         Map<Class<? extends Entitlement>, List<Entitlement>> entitlementsByType,
         FileAccessTree fileAccess,
-        Logger logger
+        String loggerName
     ) {
 
         ModuleEntitlements {
@@ -98,6 +98,10 @@ public <E extends Entitlement> Stream<E> getEntitlements(Class<E> entitlementCla
             }
             return entitlements.stream().map(entitlementClass::cast);
         }
+
+        public Logger logger() {
+            return LogManager.getLogger(loggerName);
+        }
     }
 
     private FileAccessTree getDefaultFileAccess(String componentName, Path componentPath) {
@@ -573,9 +577,9 @@ private void notEntitled(String message, Class<?> callerClass, ModuleEntitlement
         throw exception;
     }
 
-    private static Logger getLogger(String componentName, String moduleName) {
+    private static String getLogger(String componentName, String moduleName) {
         var loggerSuffix = "." + componentName + "." + ((moduleName == null) ? ALL_UNNAMED : moduleName);
-        return LogManager.getLogger(PolicyManager.class.getName() + loggerSuffix);
+        return PolicyManager.class.getName() + loggerSuffix;
     }
 
     public void checkManageThreadsEntitlement(Class<?> callerClass) {