move configuration

inspectIT · Oct 28, 2024 · f3329a1 · f3329a1
1 parent 97e2a58
commit f3329a1
Show file tree

Hide file tree

Showing 13 changed files with 20 additions and 21 deletions.
diff --git a/...ard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHook.java b/...ard-agent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHook.java
@@ -5,7 +5,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import rocks.inspectit.gepard.agent.instrumentation.hook.action.SpanAction;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.bootstrap.context.InternalInspectitContext;
 import rocks.inspectit.gepard.bootstrap.instrumentation.IMethodHook;
 

diff --git a/...nt/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookManager.java b/...nt/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookManager.java
@@ -8,8 +8,8 @@
 import net.bytebuddy.matcher.ElementMatcher;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration;
 import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.HookedMethods;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration;
 import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.resolver.ClassHookConfigurationResolver;
 import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.resolver.MethodHookConfigurationResolver;
 import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration;

diff --git a/...gent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookState.java b/...gent/src/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookState.java
@@ -10,9 +10,9 @@
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.stream.Collectors;
 import net.bytebuddy.description.method.MethodDescription;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration;
 import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.HookedMethods;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.agent.instrumentation.hook.util.MethodHookFactory;
 
 /** Stores the method hook configurations of all instrumented classes. */

diff --git a/...configuration/ClassHookConfiguration.java → ...uration/model/ClassHookConfiguration.java b/...configuration/ClassHookConfiguration.java → ...uration/model/ClassHookConfiguration.java
@@ -1,5 +1,5 @@
 /* (C) 2024 */
-package rocks.inspectit.gepard.agent.instrumentation.hook.configuration;
+package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model;
 
 import java.util.HashMap;
 import java.util.Map;
@@ -9,12 +9,11 @@
 import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration;
 
 /**
- * Stores the hook configuration of all methods for a specific class. Currently, there is no complex
- * hook configuration, thus we only use booleans.
+ * Stores the hook configuration of all methods for a specific class, which is used to update hooks.
  */
 public class ClassHookConfiguration {
 
-  /** Set of methods and their hook configuration. Currently, just true. */
+  /** Set of methods and their hook configuration. */
   private final Map<MethodDescription, MethodHookConfiguration> hookConfigurations;
 
   private final MethodHookConfigurationResolver hookResolver;

diff --git a/...onfiguration/MethodHookConfiguration.java → ...ration/model/MethodHookConfiguration.java b/...onfiguration/MethodHookConfiguration.java → ...ration/model/MethodHookConfiguration.java
@@ -1,12 +1,12 @@
 /* (C) 2024 */
-package rocks.inspectit.gepard.agent.instrumentation.hook.configuration;
+package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model;
 
 import java.util.Objects;
 import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration;
 
 /**
- * Configuration for one specific method hook. Currently just for tracing. Later we will add
- * entry-/exit-actions and metrics.
+ * Configuration for one specific method hook, which is used to update hooks. Currently just for
+ * tracing. Later we will add entry-/exit-actions and metrics.
  */
 public class MethodHookConfiguration {
 

diff --git a/...ard/agent/instrumentation/hook/configuration/resolver/ClassHookConfigurationResolver.java b/...ard/agent/instrumentation/hook/configuration/resolver/ClassHookConfigurationResolver.java
@@ -5,8 +5,8 @@
 import net.bytebuddy.description.method.MethodDescription;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration;
 
 /**

diff --git a/...c/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactory.java b/...c/main/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactory.java
@@ -3,7 +3,7 @@
 
 import rocks.inspectit.gepard.agent.instrumentation.hook.MethodHook;
 import rocks.inspectit.gepard.agent.instrumentation.hook.action.SpanAction;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 
 /** Creates method hook objects */
 public class MethodHookFactory {

diff --git a/.../src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookStateTest.java b/.../src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookStateTest.java
@@ -15,9 +15,9 @@
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.ClassHookConfiguration;
 import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.HookedMethods;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.ClassHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration;
 
 @ExtendWith(MockitoExtension.class)

diff --git a/...agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookTest.java b/...agent/src/test/java/rocks/inspectit/gepard/agent/instrumentation/hook/MethodHookTest.java
@@ -13,7 +13,7 @@
 import org.mockito.junit.jupiter.MockitoExtension;
 import rocks.inspectit.gepard.agent.instrumentation.hook.action.SpanAction;
 import rocks.inspectit.gepard.agent.instrumentation.hook.action.exception.CouldNotCloseSpanScopeException;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.bootstrap.context.InternalInspectitContext;
 
 @ExtendWith(MockitoExtension.class)

diff --git a/...iguration/ClassHookConfigurationTest.java → ...ion/model/ClassHookConfigurationTest.java b/...iguration/ClassHookConfigurationTest.java → ...ion/model/ClassHookConfigurationTest.java
@@ -1,5 +1,5 @@
 /* (C) 2024 */
-package rocks.inspectit.gepard.agent.instrumentation.hook.configuration;
+package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model;
 
 import static org.junit.jupiter.api.Assertions.*;
 import static org.mockito.Mockito.when;

diff --git a/...guration/MethodHookConfigurationTest.java → ...on/model/MethodHookConfigurationTest.java b/...guration/MethodHookConfigurationTest.java → ...on/model/MethodHookConfigurationTest.java
@@ -1,5 +1,5 @@
 /* (C) 2024 */
-package rocks.inspectit.gepard.agent.instrumentation.hook.configuration;
+package rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotEquals;

diff --git a/...gent/instrumentation/hook/configuration/resolver/MethodHookConfigurationResolverTest.java b/...gent/instrumentation/hook/configuration/resolver/MethodHookConfigurationResolverTest.java
@@ -13,7 +13,7 @@
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.agent.internal.instrumentation.model.ClassInstrumentationConfiguration;
 import rocks.inspectit.gepard.agent.internal.instrumentation.model.rules.InstrumentationRule;
 import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration;

diff --git a/...st/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactoryTest.java b/...st/java/rocks/inspectit/gepard/agent/instrumentation/hook/util/MethodHookFactoryTest.java
@@ -9,7 +9,7 @@
 import org.mockito.Mock;
 import org.mockito.junit.jupiter.MockitoExtension;
 import rocks.inspectit.gepard.agent.instrumentation.hook.MethodHook;
-import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.MethodHookConfiguration;
+import rocks.inspectit.gepard.agent.instrumentation.hook.configuration.model.MethodHookConfiguration;
 import rocks.inspectit.gepard.config.model.instrumentation.rules.RuleTracingConfiguration;
 
 @ExtendWith(MockitoExtension.class)