feat: changed container work dir from app to opt (#684)

Signed-off-by: Alex Kehayov <[email protected]>

Author: AlexKehayov

block-node/app/docker/Dockerfile

@@ -71,7 +71,7 @@ RUN groupadd --gid 2000 hedera && \
 ARG VERSION
 
 # Set the working directory inside the container
-WORKDIR /app
+WORKDIR /opt/hiero/block-node
 
 # Copy Distribution TAR file
 # Ignore Codacy, the `distributions` context is set on command line and
@@ -82,15 +82,15 @@ COPY --from=distributions block-node-app-${VERSION}.tar .
 RUN tar -xvf block-node-app-${VERSION}.tar
 
 # Create a log directory
-RUN mkdir -p /app/logs/config
+RUN mkdir -p /opt/hiero/block-node/logs/config
 
 # Copy the logging properties file
-COPY logging.properties /app/logs/config/logging.properties
+COPY logging.properties /opt/hiero/block-node/logs/config/logging.properties
 
 WORKDIR /
 
 # Ensure proper file permissions
-RUN chown -R 2000:2000 /app /opt
+RUN chown -R 2000:2000 /opt/hiero/block-node
 
 ########################################
 ####    Deterministic Build Hack    ####
@@ -138,12 +138,12 @@ COPY --from=java-builder ${JAVA_HOME}/ ${JAVA_HOME}/
 EXPOSE 8080/tcp
 
 USER hedera
-WORKDIR /app
+WORKDIR /opt/hiero/block-node
 
 # HEALTHCHECK for liveness and readiness
 HEALTHCHECK --interval=30s --timeout=10s --start-period=3s --retries=3 \
   CMD curl -f http://localhost:8080/healthz/livez || exit 1 && \
       curl -f http://localhost:8080/healthz/readyz || exit 1
 
-# RUN the bin script for starting the server app
-ENTRYPOINT ["/bin/bash", "-c", "/app/block-node-app-${VERSION}/bin/block-node-app"]
+# RUN the bin script for starting the server
+ENTRYPOINT ["/bin/bash", "-c", "/opt/hiero/block-node/block-node-app-${VERSION}/bin/block-node-app"]

block-node/app/docker/docker-compose.yml

@@ -3,7 +3,7 @@ services:
     container_name: block-node-server
     image: block-node-server:${VERSION}
     volumes:
-      - ./logging.properties:/app/logs/config/logging.properties
+      - ./logging.properties:/opt/hiero/block-node/logs/config/logging.properties
     env_file:
       - .env
     ports:

block-node/app/docker/logging.properties

@@ -63,7 +63,7 @@ java.util.logging.ConsoleHandler.formatter = java.util.logging.SimpleFormatter
 # FileHandler properties
 ################################################################################
 # The pattern for the output file name.
-java.util.logging.FileHandler.pattern = /app/logs/blocknode-%g.log
+java.util.logging.FileHandler.pattern = /opt/hiero/block-node/logs/blocknode-%g.log
 # Set append to true if you want to keep appending to existing files
 java.util.logging.FileHandler.append = true
 # The limit in bytes before a new file is started.

block-node/app/docker/update-env.sh

@@ -18,7 +18,7 @@ project_version=$1
 echo "VERSION=$project_version" > .env
 echo "REGISTRY_PREFIX=" >> .env
 # Storage root path, this is temporary until we have a proper .properties file for all configs
-echo "BLOCKNODE_STORAGE_ROOT_PATH=/app/storage" >> .env
+echo "BLOCKNODE_STORAGE_ROOT_PATH=/opt/hiero/block-node/storage" >> .env
 
 if [ true = "$is_smoke_test" ]; then
   # add smoke test variables
@@ -33,11 +33,11 @@ fi
 if [ true = "$is_debug" ]; then
   # The server will wait for the debugger to attach on port 5005
   # JProfiler can attach on port 8849
-  echo "JAVA_TOOL_OPTIONS='-Djava.util.logging.config.file=/app/logs/config/logging.properties -agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=*:5005' -agentpath:/path/to/libjprofilerti.so=port=8849 " >> .env
+  echo "JAVA_TOOL_OPTIONS='-Djava.util.logging.config.file=/opt/hiero/block-node/logs/config/logging.properties -agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=*:5005' -agentpath:/path/to/libjprofilerti.so=port=8849 " >> .env
 else
   # we set normally the JAVA_TOOL_OPTIONS
   # file is mounted in the docker-compose.yml, changes to the file will be reflected in the container by simply restarting it
-  echo "JAVA_TOOL_OPTIONS='-Djava.util.logging.config.file=/app/logs/config/logging.properties '" >> .env
+  echo "JAVA_TOOL_OPTIONS='-Djava.util.logging.config.file=/opt/hiero/block-node/logs/config/logging.properties '" >> .env
 fi
 # Output the values
 echo ".env properties:"

block-node/block-providers/files.historic/src/main/java/org/hiero/block/node/blocks/files/historic/FilesHistoricConfig.java

@@ -24,7 +24,7 @@
  */
 @ConfigData("files.historic")
 public record FilesHistoricConfig(
-        @Loggable @ConfigProperty(defaultValue = "/opt/hashgraph/blocknode/data/historic") Path rootPath,
+        @Loggable @ConfigProperty(defaultValue = "/opt/hiero/block-node/data/historic") Path rootPath,
         @Loggable @ConfigProperty(defaultValue = "ZSTD") CompressionType compression,
         @Loggable @ConfigProperty(defaultValue = "4") @Min(1) @Max(6) int powersOfTenPerZipFileContents) {
     /**

block-node/block-providers/files.historic/src/test/java/org/hiero/block/node/blocks/files/historic/FilesHistoricConfigTest.java

@@ -39,7 +39,7 @@ class FilesHistoricConfigTest {
     @BeforeEach
     void setup() {
         jimfs = Jimfs.newFileSystem(Configuration.unix());
-        defaultRootPath = jimfs.getPath("/opt/hashgraph/blocknode/data/historic");
+        defaultRootPath = jimfs.getPath("/opt/hiero/block-node/data/historic");
         defaultCompression = CompressionType.ZSTD;
         powersOfTenPerZipFileContents = 4;
     }

block-node/block-providers/files.recent/src/main/java/org/hiero/block/node/blocks/files/recent/FilesRecentConfig.java

@@ -20,7 +20,7 @@
  */
 @ConfigData("files.recent")
 public record FilesRecentConfig(
-        @Loggable @ConfigProperty(defaultValue = "/opt/hashgraph/blocknode/data/live") Path liveRootPath,
+        @Loggable @ConfigProperty(defaultValue = "/opt/hiero/block-node/data/live") Path liveRootPath,
         @Loggable @ConfigProperty(defaultValue = "ZSTD") CompressionType compression,
         @Loggable @ConfigProperty(defaultValue = "3") int maxFilesPerDir) {
     /**

charts/block-node-server/README.md

@@ -65,14 +65,15 @@ There are several ways to configure the Hiero Block Node. The following is the m
 1. Create an override values file, `values.yaml`:
 2. Add the necessary environment configuration variables to the following section:
 
-   ```yaml
-   blockNode:
-     config:
-       # Add any additional env configuration here
-       # key: value
-       BLOCKNODE_STORAGE_ROOT_PATH: "/app/storage"
+```yaml
+blockNode:
+  config:
+    # Add any additional env configuration here
+    # key: value
+    BLOCKNODE_STORAGE_ROOT_PATH: "/opt/hiero/block-node/storage"
+
+```
 
-   ```
 3. Secrets should be set at the following structure:
 
    ```yaml

charts/block-node-server/templates/deployment.yaml

@@ -33,6 +33,7 @@ spec:
         - name: logging-config
           configMap:
             name: {{ include "hiero-block-node.fullname" . }}-logging-config
+        - name: logging-storage
         - name: archive-storage
           persistentVolumeClaim:
             claimName: {{ if .Values.blockNode.persistence.archive.create }}{{ include "hiero-block-node.fullname" . }}-archive{{ else }}{{ .Values.blockNode.persistence.archive.existingClaim }}{{ end }}
@@ -59,6 +60,8 @@ spec:
               mountPath: /live-pvc
             - name: archive-storage
               mountPath: /archive-pvc
+            - name: logging-storage
+              mountPath: /opt/hiero/block-node/logs
       containers:
       - name: {{ .Chart.Name }}
         securityContext:
@@ -79,8 +82,10 @@ spec:
               name: {{ include "hiero-block-node.fullname" . }}-secret
         volumeMounts:
           - name: logging-config
-            mountPath: /app/logs/config
+            mountPath: /opt/hiero/block-node/logs/config
             readOnly: true
+          - name: logging-storage
+            mountPath: /opt/hiero/block-node/logs
           - name: archive-storage
             mountPath: {{ .Values.blockNode.persistence.archive.mountPath }}
             subPath: {{ .Values.blockNode.persistence.archive.subPath }}

charts/block-node-server/values-overrides/mini.yaml

@@ -11,7 +11,7 @@ resources:
 
 blockNode:
   config:
-    JAVA_TOOL_OPTIONS: "-Djava.util.logging.config.file=/app/logs/config/logging.properties"
+    JAVA_TOOL_OPTIONS: "-Djava.util.logging.config.file=/opt/hiero/block-node/logs/config/logging.properties"
     JAVA_OPTS: "-Xms5G -Xmx5G"
     MEDIATOR_RING_BUFFER_SIZE: "2048"
   persistence:

charts/block-node-server/values.yaml

@@ -73,7 +73,7 @@ blockNode:
   config:
     # Add any additional env configuration here
     # key: value
-    JAVA_TOOL_OPTIONS: "-Djava.util.logging.config.file=/app/logs/config/logging.properties"
+    JAVA_TOOL_OPTIONS: "-Djava.util.logging.config.file=/opt/hiero/block-node/logs/config/logging.properties"
     JAVA_OPTS: "-Xms16G -Xmx16G"
     # PRODUCER_TYPE: "NO_OP"
     # PERSISTENCE_STORAGE_TYPE: "NO_OP"
@@ -90,7 +90,7 @@ blockNode:
       subPath: "archive-data"
       # If create is true, the following values are used to create the PVC
       # should match PERSISTENCE_STORAGE_ARCHIVE_ROOT_PATH, leave as is for default.
-      mountPath: "/opt/hashgraph/blocknode/data/historic"
+      mountPath: "/opt/hiero/block-node/data/historic"
       size: 800Gi
       # Optionally add a storage class name if needed
       # storageClass: "your-storage-class"
@@ -103,15 +103,15 @@ blockNode:
       subPath: "live-data"
       # If create is true, the following values are used to create the PVC
       # should match PERSISTENCE_STORAGE_LIVE_ROOT_PATH, leave as is for default.
-      mountPath: "/opt/hashgraph/blocknode/data/live"
+      mountPath: "/opt/hiero/block-node/data/live"
       size: 20Gi
       # Optionally add a storage class name if needed
       # storageClass: "your-storage-class"
     unverified:
       # unverified does a hard-coded emptyDir and is ephemeral, however needs to be mounted
       # this does not create a PVC.
       # should match PERSISTENCE_STORAGE_UNVERIFIED_ROOT_PATH, leave as is for default.
-      mountPath: "/opt/hashgraph/blocknode/data/unverified"
+      mountPath: "/opt/hiero/block-node/data/unverified"
   secret:
     PRIVATE_KEY: "fake_private_key"
   health:
@@ -135,7 +135,7 @@ blockNode:
       handlers: "java.util.logging.ConsoleHandler, java.util.logging.FileHandler"
       java.util.logging.ConsoleHandler.level: "FINE"
       java.util.logging.ConsoleHandler.formatter: "java.util.logging.SimpleFormatter"
-      java.util.logging.FileHandler.pattern: "/app/logs/blocknode-%g.log"
+      java.util.logging.FileHandler.pattern: "/opt/hiero/block-node/logs/blocknode-%g.log"
       java.util.logging.FileHandler.append: "true"
       java.util.logging.FileHandler.limit: "5_000_000"
       java.util.logging.FileHandler.count: "5"
@@ -153,6 +153,7 @@ blockNode:
       #   %5$ - log message
       #   %6$ - throwable trace
       java.util.logging.SimpleFormatter.format: "%TF %<TT.%<TL%<Tz %4$-7s [%2$s] %5$s%6$s%n"
+    mountPath: "/opt/hiero/block-node/logs"
 
 kubepromstack:
   enabled: true

simulator/docker/Dockerfile

@@ -6,13 +6,13 @@ ARG UID=2000
 ARG GID=2000
 RUN groupadd --gid $GID $UNAME
 RUN useradd --no-user-group --create-home --uid $UID --gid $GID --shell /bin/bash hedera
-WORKDIR /app
+WORKDIR /opt/hiero/block-node
 
 # Copy the distribution and resources
 COPY simulator-*.tar ./simulator.tar
 
-RUN mkdir -p /app/logs/config
-COPY logging.properties /app/logs/config/logging.properties
+RUN mkdir -p /opt/hiero/block-node/logs/config
+COPY logging.properties /opt/hiero/block-node/logs/config/logging.properties
 
 # Create resources directory and copy block data
 RUN mkdir -p src/main/resources
@@ -24,17 +24,18 @@ RUN tar -xf simulator.tar && \
     cd src/main/resources && \
     tar -xzf block-0.0.3.tar.gz && \
     rm block-0.0.3.tar.gz && \
-    cd /app
+    cd /opt/hiero/block-node && \
+    chown -R $UNAME:$UNAME /opt/hiero/block-node
 
-RUN mkdir -p /opt/simulator/data
-RUN chown -R $UID:$GID /app /opt/simulator/data
+RUN mkdir -p /opt/hiero/block-node/simulator/data
+RUN chown -R $UID:$GID /opt/hiero/block-node/simulator/data
 
 # Switch to non-root user
 USER $UNAME
 
 # Run the simulator using the extracted directory name
 RUN SIMULATOR_DIR=$(ls -d simulator-*/) && \
-    echo "#!/bin/bash\n/app/${SIMULATOR_DIR}bin/simulator" > /app/start.sh && \
-    chmod +x /app/start.sh
+    echo "#!/bin/bash\n/opt/hiero/block-node/${SIMULATOR_DIR}bin/simulator" > /opt/hiero/block-node/start.sh && \
+    chmod +x /opt/hiero/block-node/start.sh
 
-ENTRYPOINT ["/app/start.sh"]
+ENTRYPOINT ["/opt/hiero/block-node/start.sh"]

simulator/docker/update-env.sh

@@ -19,7 +19,7 @@ CONSUMER_BLOCK_STREAM_SIMULATOR_MODE=CONSUMER
 CONSUMER_PROMETHEUS_ENDPOINT_PORT_NUMBER=9997
 EOL
 
-logging_config_file_arg="-Djava.util.logging.config.file=/app/logs/config/logging.properties"
+logging_config_file_arg="-Djava.util.logging.config.file=/opt/hiero/block-node/logs/config/logging.properties"
 debug_arg="-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=*:5006"
 # determine if we should include debug opts
 [ "$1" = true ] && is_debug=true || is_debug=false

simulator/src/main/resources/logging.properties

@@ -36,7 +36,7 @@ java.util.logging.ConsoleHandler.formatter = java.util.logging.SimpleFormatter
 # FileHandler properties
 ################################################################################
 # The pattern for the output file name.
-java.util.logging.FileHandler.pattern = /app/logs/blocknode-%g.log
+java.util.logging.FileHandler.pattern = /opt/hiero/block-node/logs/blocknode-%g.log
 # Set append to true if you want to keep appending to existing files
 java.util.logging.FileHandler.append = true
 # The limit in bytes before a new file is started.

suites/src/main/java/org/hiero/block/suites/persistence/positive/PositiveDataPersistenceTests.java

@@ -23,7 +23,7 @@
 @DisplayName("Positive Data Persistence Tests")
 public class PositiveDataPersistenceTests extends BaseSuite {
     private final String[] GET_BLOCKS_COMMAND =
-            new String[] {"find", "../opt/hashgraph/blocknode/data/live", "-mindepth", "5", "-maxdepth", "7"};
+            new String[] {"find", "/opt/hiero/block-node/data/live", "-mindepth", "5", "-maxdepth", "7"};
 
     private Future<?> simulatorThread;