using ports 5442/5443 (#422)

Author: thugrock7

README.md

@@ -43,7 +43,11 @@ The final artifact is in `javaagent/build/libs/hypertrace-agent-<version>-all.ja
 Download the [latest version](https://github.com/hypertrace/javaagent/releases/latest/download/hypertrace-agent-all.jar).
 
 ```bash
-HT_REPORTING_ENDPOINT=http://localhost:4317 java -javaagent:javaagent/build/libs/hypertrace-agent-<version>-all.jar -jar app.jar
+HT_REPORTING_ENDPOINT=http://localhost:5442 java -javaagent:javaagent/build/libs/hypertrace-agent-<version>-all.jar -jar app.jar
+```
+If TPA is configured with TLS server
+```bash
+HT_REPORTING_ENDPOINT=https://localhost:5443 java -javaagent:javaagent/build/libs/hypertrace-agent-<version>-all.jar -jar app.jar
 ```
 
 By default the agent uses Otlp exporter.

example-config.yaml

@@ -1,3 +1,3 @@
 service_name: service_name
 reporting:
-  endpoint: http://localhost:4317
+  endpoint: http://localhost:5442

otel-extensions/src/main/java/org/hypertrace/agent/otel/extensions/config/HypertraceConfig.java

@@ -78,7 +78,7 @@ private static List<StringValue> initDefaultContentTypes() {
 
   static final String DEFAULT_SERVICE_NAME = "unknown";
   // Default reporting endpoint for traces and metrics
-  static final String DEFAULT_REPORTING_ENDPOINT = "http://localhost:4317";
+  static final String DEFAULT_REPORTING_ENDPOINT = "http://localhost:5442";
   // 128 KiB
   static final int DEFAULT_BODY_MAX_SIZE_BYTES = 128 * 1024;
 

otel-extensions/src/main/proto

@@ -137,9 +137,9 @@ private void assertConfig(AgentConfig agentConfig) {
     Assertions.assertEquals(
         "/foo/bar/example.pem", agentConfig.getReporting().getCertFile().getValue());
     Assertions.assertEquals(
-        "http://localhost:4317", agentConfig.getReporting().getEndpoint().getValue());
+        "http://localhost:5442", agentConfig.getReporting().getEndpoint().getValue());
     Assertions.assertEquals(
-        "http://localhost:4317", agentConfig.getReporting().getMetricEndpoint().getValue());
+        "http://localhost:5442", agentConfig.getReporting().getMetricEndpoint().getValue());
     Assertions.assertEquals(true, agentConfig.getReporting().getSecure().getValue());
     Assertions.assertEquals(16, agentConfig.getDataCapture().getBodyMaxSizeBytes().getValue());
     Assertions.assertEquals(
@@ -172,15 +172,15 @@ public void configWithSystemProps() throws IOException {
   }
 
   @Test
-  @SetEnvironmentVariable(key = "HT_REPORTING_ENDPOINT", value = "http://oltp.hypertrace.org:4317")
+  @SetEnvironmentVariable(key = "HT_REPORTING_ENDPOINT", value = "http://oltp.hypertrace.org:5442")
   public void complexConfig() throws IOException {
     // GIVEN a config file with a non-default reporting endpoint and an env-var with a different
     // non-default otlp reporting endpoint
     URL resource = getClass().getClassLoader().getResource("config.yaml");
     // WHEN we load the config
     AgentConfig agentConfig = HypertraceConfig.load(resource.getPath());
     // VERIFY the trace and metric endpoints are the both the value of the env var
-    String expectedEndpoint = "http://oltp.hypertrace.org:4317";
+    String expectedEndpoint = "http://oltp.hypertrace.org:5442";
     Assertions.assertEquals(expectedEndpoint, agentConfig.getReporting().getEndpoint().getValue());
     Assertions.assertEquals(
         expectedEndpoint, agentConfig.getReporting().getMetricEndpoint().getValue());
@@ -238,7 +238,7 @@ void nonDefaultReportingEndpoint() throws IOException {
     // VERIFY the trace reporting type is OTLP
     Assertions.assertEquals(
         TraceReporterType.OTLP, agentConfig.getReporting().getTraceReporterType());
-    String expectedReportingEndpoint = "http://example.com:4317";
+    String expectedReportingEndpoint = "http://example.com:5442";
     // VERIFY the trace reporting and metric reporting endpoints are both the specified value
     Assertions.assertEquals(
         expectedReportingEndpoint, agentConfig.getReporting().getEndpoint().getValue());
@@ -257,10 +257,10 @@ void nonDefaultMetricsEndpoint() throws IOException {
         TraceReporterType.OTLP, agentConfig.getReporting().getTraceReporterType());
     // VERIFY the trace reporting endpoint is still the default value
     Assertions.assertEquals(
-        "http://localhost:4317", agentConfig.getReporting().getEndpoint().getValue());
+        "http://localhost:5442", agentConfig.getReporting().getEndpoint().getValue());
     // VERIFY the metric reporting endpoint is the specified value
     Assertions.assertEquals(
-        "http://example.com:4317", agentConfig.getReporting().getMetricEndpoint().getValue());
+        "http://example.com:5442", agentConfig.getReporting().getMetricEndpoint().getValue());
   }
 
   @Test

otel-extensions/src/test/java/org/hypertrace/agent/otel/extensions/config/HypertraceConfigTest.java

@@ -4,8 +4,8 @@ enabled: false
 propagationFormats:
   - B3
 reporting:
-  endpoint: http://localhost:4317
-  metric_endpoint: http://localhost:4317
+  endpoint: http://localhost:5442
+  metric_endpoint: http://localhost:5442
   secure: true
   trace_reporter_type: OTLP
   metric_reporter_type: METRIC_REPORTER_TYPE_OTLP

otel-extensions/src/test/resources/config.yaml

@@ -4,8 +4,8 @@ enabled: false
 propagationFormats:
   - B3
 reporting:
-  endpoint: http://localhost:4317
-  metric_endpoint: http://localhost:4317
+  endpoint: http://localhost:5442
+  metric_endpoint: http://localhost:5442
   secure: true
   trace_reporter_type: OTLP
   metric_reporter_type: METRIC_REPORTER_TYPE_OTLP

otel-extensions/src/test/resources/nonDefaultDataCaptureConfig.yaml

@@ -1,3 +1,3 @@
 service_name: service
 reporting:
-  metric_endpoint: http://example.com:4317
+  metric_endpoint: http://example.com:5442

otel-extensions/src/test/resources/nonDefaultMetricsEndpoint.yaml

@@ -1,3 +1,3 @@
 service_name: service
 reporting:
-  endpoint: http://example.com:4317
+  endpoint: http://example.com:5442

otel-extensions/src/test/resources/nonDefaultReportingEndpoint.yaml

@@ -1,4 +1,4 @@
 service_name: service
 reporting:
   trace_reporter_type: ZIPKIN
-  endpoint: http://example.com:9411/api/v2/spans
+  endpoint: http://example.com:5442/api/v2/spans