ci: unify memfault versioning across build and release pipelines

Trond-F-Christiansen · bjda · commit e5f6b14d6fd1 · 2026-03-27T12:11:36.000+01:00
unify the way the Memfault firmware version and type are set between
manual builds and the CI pipelines.

Signed-off-by: Trond F. Christiansen &lt;trond.christiansen@nordicsemi.no&gt;
diff --git a/.github/actions/build-step/action.yml b/.github/actions/build-step/action.yml
@@ -5,14 +5,10 @@ inputs:
     type: string
     required: false
     default: "fookey"
-  memfault_fw_type:
-    type: string
-    required: false
-    default: "att-dev"
-  memfault_fw_version_prefix:
-    type: string
+  memfault_release:
+    type: boolean
     required: false
-    default: "0.0.0-dev"
+    default: false
   board:
     type: string
     required: true
@@ -68,9 +64,9 @@ runs:
           cp overlay-memfault.conf overlay-debug-att.conf
           cat overlay-upload-modem-traces-to-memfault.conf >> overlay-debug-att.conf
           echo CONFIG_MEMFAULT_NCS_PROJECT_KEY=\"${{ inputs.memfault_project_key }}\" >> overlay-debug-att.conf
-          echo CONFIG_MEMFAULT_NCS_FW_VERSION_AUTO=y >> overlay-debug-att.conf
-          echo CONFIG_MEMFAULT_NCS_FW_VERSION_PREFIX=\"${{ inputs.memfault_fw_version_prefix }}+\" >> overlay-debug-att.conf
-          echo CONFIG_MEMFAULT_NCS_FW_TYPE=\"${{ inputs.memfault_fw_type }}\" >> overlay-debug-att.conf
+          if [[ "${{ inputs.memfault_release }}" == "true" ]]; then
+            echo CONFIG_MEMFAULT_NCS_FW_VERSION_STATIC=y >> overlay-debug-att.conf
+          fi
           params+=("-DEXTRA_CONF_FILE=overlay-debug-att.conf")
         fi
         if [[ "${{ inputs.mqtt }}" == "true" ]]; then
diff --git a/.github/workflows/attach_release_assets.yml b/.github/workflows/attach_release_assets.yml
@@ -17,7 +17,6 @@ jobs:
       pull-requests: write
     secrets: inherit
     with:
-      memfault_fw_type: "att"
       build_type: release
 
   attach-assets:
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -3,11 +3,6 @@ name: Build
 on:
   workflow_dispatch:
     inputs:
-      memfault_fw_type:
-        description: Memfault Firmware Type
-        type: string
-        required: false
-        default: "att-dev"
       build_type:
         description: "Build scope: minimal (thingy91x only), release (all boards, no test builds), all (everything)"
         type: choice
@@ -24,10 +19,6 @@ on:
         default: false
   workflow_call:
     inputs:
-      memfault_fw_type:
-        type: string
-        required: false
-        default: "att-dev"
       nrfsdk_sha_update:
         type: boolean
         required: false
@@ -144,15 +135,6 @@ jobs:
       - name: Set build type
         run: echo "build_type=${{ inputs.build_type || 'minimal' }}" >> $GITHUB_ENV
 
-      - name: Set MEMFAULT_FW_TYPE and MEMFAULT_FW_VERSION_PREFIX
-        shell: bash
-        run: |
-          if [[ -z "${{ inputs.memfault_fw_type }}" ]]; then
-            echo "MEMFAULT_FW_TYPE=att-dev" >> $GITHUB_ENV
-          else
-            echo "MEMFAULT_FW_TYPE=${{ inputs.memfault_fw_type }}" >> $GITHUB_ENV
-          fi
-
       # Asset Tracker Template firmware build
       - name: Build thingy91x firmware
         if: ${{ env.build_type == 'minimal' || env.build_type == 'release' || env.build_type == 'all' }}
@@ -199,8 +181,7 @@ jobs:
         uses: ./asset-tracker-template/.github/actions/build-step
         with:
           memfault_project_key: ${{ secrets.MEMFAULT_PROJECT_KEY }}
-          memfault_fw_type: ${{ env.MEMFAULT_FW_TYPE }}
-          memfault_fw_version_prefix: ${{ env.MEMFAULT_FW_VERSION_PREFIX }}
+          memfault_release: ${{ env.build_type == 'release' }}
           board: thingy91x/nrf9151/ns
           short_board: thingy91x
           version: ${{ env.VERSION }}-debug
diff --git a/app/CMakeLists.txt b/app/CMakeLists.txt
@@ -11,7 +11,7 @@ project("Asset Tracker Template")
 
 # Include files that are common for all modules
 zephyr_include_directories(src/common)
-zephyr_include_directories(config)
+zephyr_include_directories(memfault/config)
 
 # Add main application source
 target_sources(app PRIVATE src/main.c)
diff --git a/app/Kconfig b/app/Kconfig
@@ -18,6 +18,7 @@ rsource "src/modules/button/Kconfig.button"
 rsource "src/modules/storage/Kconfig.storage"
 rsource "src/modules/sgp4/Kconfig.sgp4"
 rsource "src/inspect/Kconfig.inspect"
+rsource "memfault/Kconfig.memfault"
 endmenu
 
 menu "Zephyr Kernel"
diff --git a/app/memfault/Kconfig.memfault b/app/memfault/Kconfig.memfault
@@ -0,0 +1,13 @@
+#
+# Copyright (c) 2026 Nordic Semiconductor ASA
+#
+# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+#
+
+if MEMFAULT
+
+# Set Memfault software_version prefix from the VERSION file in the application
+config MEMFAULT_NCS_FW_VERSION_PREFIX
+	default "$(APPVERSION)+" if "$(APPVERSION)" != ""
+
+endif # MEMFAULT
diff --git a/app/memfault/config/memfault_nrf_platform_battery_model.h b/app/memfault/config/memfault_nrf_platform_battery_model.h
diff --git a/app/memfault/config/memfault_platform_config.h b/app/memfault/config/memfault_platform_config.h
diff --git a/app/overlay-memfault.conf b/app/overlay-memfault.conf
@@ -30,6 +30,7 @@ CONFIG_MEMFAULT_NCS_STACK_METRICS=y
 CONFIG_MEMFAULT_NCS_INTERNAL_FLASH_BACKED_COREDUMP=y
 CONFIG_MEMFAULT_NCS_POST_COREDUMP_ON_NETWORK_CONNECTED=y
 CONFIG_MEMFAULT_NCS_POST_COREDUMP_RETRIES_MAX=10
+CONFIG_MEMFAULT_NCS_FW_TYPE="att"
 
 # Memfault depends on POSIX, disable unneeded POSIX features
 CONFIG_POSIX_FILE_SYSTEM=n
diff --git a/app/src/modules/cloud/Kconfig.cloud b/app/src/modules/cloud/Kconfig.cloud
@@ -140,14 +140,6 @@ config APP_CLOUD_MSG_PROCESSING_TIMEOUT_SECONDS
 	  Maximum time allowed for processing a single message in the module's state machine.
 	  The value must be smaller than CONFIG_APP_CLOUD_WATCHDOG_TIMEOUT_SECONDS.
 
-if MEMFAULT
-
-# Set Memfault software_version prefix from the VERSION file in the application
-config MEMFAULT_NCS_FW_VERSION_PREFIX
-	default "$(APPVERSION)+" if "$(APPVERSION)" != ""
-
-endif # MEMFAULT
-
 module = APP_CLOUD
 module-str = Cloud
 source "subsys/logging/Kconfig.template.log_config"
