diff --git a/simulator/fault_injection/include/fault_injection/fault_injection_diag_updater.hpp b/simulator/fault_injection/include/fault_injection/fault_injection_diag_updater.hpp
index 4c05858cebff8..1aceb3266c5ae 100644
--- a/simulator/fault_injection/include/fault_injection/fault_injection_diag_updater.hpp
+++ b/simulator/fault_injection/include/fault_injection/fault_injection_diag_updater.hpp
@@ -155,11 +155,11 @@ class FaultInjectionDiagUpdater : public diagnostic_updater::DiagnosticTaskVecto
     if (vsnprintf(buff, kBufferSize, format, va) >= kBufferSize) {
       RCLCPP_DEBUG(logger_, "Really long string in diagnostic_updater::setHardwareIDf.");
     }
-    hwid_ = std::string(buff);
+    hardware_id_ = std::string(buff);
     va_end(va);
   }
 
-  void setHardwareID(const std::string & hwid) { hwid_ = hwid; }
+  void setHardwareID(const std::string & hardware_id) { hardware_id_ = hardware_id; }
 
 private:
   void reset_timer()
@@ -188,7 +188,7 @@ class FaultInjectionDiagUpdater : public diagnostic_updater::DiagnosticTaskVecto
         status.name = iter->getName();
         status.level = 2;
         status.message = "No message was set";
-        status.hardware_id = hwid_;
+        status.hardware_id = hardware_id_;
 
         iter->run(status);
 
@@ -240,7 +240,7 @@ class FaultInjectionDiagUpdater : public diagnostic_updater::DiagnosticTaskVecto
   rclcpp::Publisher<diagnostic_msgs::msg::DiagnosticArray>::SharedPtr publisher_;
   rclcpp::Logger logger_;
 
-  std::string hwid_;
+  std::string hardware_id_;
   std::string node_name_;
 };
 }  // namespace fault_injection