refactor: remove emojis and implement structured logging in e2e tests

- Remove all emoji characters from logging messages across the codebase
- Replace emoji-based logging with structured tracing fields
- Eliminate decorative headers for cleaner, machine-readable output
- Implement consistent field naming: operation, stage, action, status, component
- Update e2e tests, command execution, SSH connectivity, and validation modules
- Add clippy allow annotation for long function in docker_compose validation

The logging output is now suitable for JSON formatting and automated processing.

Author: josecelano

src/actions/cloud_init.rs

@@ -30,7 +30,10 @@ impl RemoteAction for CloudInitValidator {
     }
 
     async fn execute(&self, _server_ip: &IpAddr) -> Result<(), RemoteActionError> {
-        info!("🔍 Validating cloud-init completion...");
+        info!(
+            action = "cloud_init_validation",
+            "Validating cloud-init completion"
+        );
 
         // Check cloud-init status
         let status_output = self
@@ -64,9 +67,21 @@ impl RemoteAction for CloudInitValidator {
             });
         }
 
-        info!("✅ Cloud-init validation passed");
-        info!("   ✓ Cloud-init status is 'done'");
-        info!("   ✓ Completion marker file exists");
+        info!(
+            action = "cloud_init_validation",
+            status = "success",
+            "Cloud-init validation passed"
+        );
+        info!(
+            action = "cloud_init_validation",
+            check = "status_done",
+            "Cloud-init status is 'done'"
+        );
+        info!(
+            action = "cloud_init_validation",
+            check = "completion_marker",
+            "Completion marker file exists"
+        );
 
         Ok(())
     }

src/actions/docker.rs

@@ -30,19 +30,43 @@ impl RemoteAction for DockerValidator {
     }
 
     async fn execute(&self, _server_ip: &IpAddr) -> Result<(), RemoteActionError> {
-        info!("🔍 Validating Docker installation...");
+        info!(
+            action = "docker_validation",
+            "Validating Docker installation"
+        );
 
         // Check Docker version
         let Ok(docker_version) = self.ssh_client.execute("docker --version") else {
-            warn!("⚠️  Docker installation validation skipped");
-            warn!("   ℹ️  This is expected in CI environments with network limitations");
-            warn!("   ℹ️  The playbook ran successfully but Docker installation was skipped");
+            warn!(
+                action = "docker_validation",
+                status = "skipped",
+                reason = "ci_network_limitations",
+                "Docker installation validation skipped"
+            );
+            warn!(
+                action = "docker_validation",
+                note = "expected_in_ci",
+                "This is expected in CI environments with network limitations"
+            );
+            warn!(
+                action = "docker_validation",
+                note = "playbook_success",
+                "The playbook ran successfully but Docker installation was skipped"
+            );
             return Ok(()); // Don't fail the test, just skip validation
         };
 
         let docker_version = docker_version.trim();
-        info!("✅ Docker installation validated");
-        info!("   ✓ Docker version: {docker_version}");
+        info!(
+            action = "docker_validation",
+            status = "success",
+            "Docker installation validated"
+        );
+        info!(
+            action = "docker_validation",
+            version = docker_version,
+            "Docker version detected"
+        );
 
         // Check Docker daemon status (only if Docker is installed)
         let daemon_active = self
@@ -54,9 +78,18 @@ impl RemoteAction for DockerValidator {
             })?;
 
         if daemon_active {
-            info!("   ✓ Docker daemon is active");
+            info!(
+                action = "docker_validation",
+                check = "daemon_active",
+                "Docker daemon is active"
+            );
         } else {
-            warn!("   ⚠️  Docker daemon check skipped (service may not be running)");
+            warn!(
+                action = "docker_validation",
+                check = "daemon_skipped",
+                reason = "service_not_running",
+                "Docker daemon check skipped"
+            );
         }
 
         Ok(())

src/actions/docker_compose.rs

@@ -29,8 +29,12 @@ impl RemoteAction for DockerComposeValidator {
         "docker-compose-validation"
     }
 
+    #[allow(clippy::too_many_lines)]
     async fn execute(&self, _server_ip: &IpAddr) -> Result<(), RemoteActionError> {
-        info!("🔍 Validating Docker Compose installation...");
+        info!(
+            action = "docker_compose_validation",
+            "Validating Docker Compose installation"
+        );
 
         // First check if Docker is available (Docker Compose requires Docker)
         let docker_available =
@@ -42,23 +46,47 @@ impl RemoteAction for DockerComposeValidator {
                 })?;
 
         if !docker_available {
-            warn!("⚠️  Docker Compose validation skipped");
-            warn!("   ℹ️  Docker is not available, so Docker Compose cannot be validated");
-            warn!("   ℹ️  This is expected in CI environments with network limitations");
+            warn!(
+                action = "docker_compose_validation",
+                status = "skipped",
+                reason = "docker_unavailable",
+                "Docker Compose validation skipped"
+            );
+            warn!(
+                action = "docker_compose_validation",
+                note = "dependency_missing",
+                "Docker is not available, so Docker Compose cannot be validated"
+            );
+            warn!(
+                action = "docker_compose_validation",
+                note = "expected_in_ci",
+                "This is expected in CI environments with network limitations"
+            );
             return Ok(()); // Don't fail the test, just skip validation
         }
 
         // Check Docker Compose version
         let Ok(compose_version) = self.ssh_client.execute("docker-compose --version") else {
             warn!(
-                "⚠️  Docker Compose not found, this is expected if Docker installation was skipped"
+                action = "docker_compose_validation",
+                status = "not_found",
+                note = "expected_if_docker_skipped",
+                "Docker Compose not found, this is expected if Docker installation was skipped"
             );
             return Ok(()); // Don't fail, just note the situation
         };
 
         let compose_version = compose_version.trim();
-        info!("✅ Docker Compose installation validated");
-        info!("   ✓ Docker Compose version: {compose_version}");
+        info!(
+            action = "docker_compose_validation",
+            status = "success",
+            "Docker Compose installation validated"
+        );
+        info!(
+            action = "docker_compose_validation",
+            version = compose_version,
+            "Docker Compose version detected"
+        );
 
         // Test basic docker-compose functionality with a simple test file (only if Docker is working)
         let test_compose_content = r"services:
@@ -78,7 +106,12 @@ impl RemoteAction for DockerComposeValidator {
             })?;
 
         if !create_test_success {
-            warn!("   ⚠️  Could not create test docker-compose.yml file");
+            warn!(
+                action = "docker_compose_validation",
+                check = "test_file_creation",
+                status = "failed",
+                "Could not create test docker-compose.yml file"
+            );
             return Ok(()); // Don't fail, just skip the functional test
         }
 
@@ -92,9 +125,19 @@ impl RemoteAction for DockerComposeValidator {
             })?;
 
         if validate_success {
-            info!("   ✓ Docker Compose configuration validation passed");
+            info!(
+                action = "docker_compose_validation",
+                check = "configuration_validation",
+                status = "success",
+                "Docker Compose configuration validation passed"
+            );
         } else {
-            warn!("   ⚠️  Docker Compose configuration validation skipped");
+            warn!(
+                action = "docker_compose_validation",
+                check = "configuration_validation",
+                status = "skipped",
+                "Docker Compose configuration validation skipped"
+            );
         }
 
         // Clean up test file

src/ansible/template_renderer.rs

@@ -170,7 +170,11 @@ impl AnsibleTemplateRenderer {
         template_manager: &TemplateManager,
         inventory_context: &InventoryContext,
     ) -> Result<(), ConfigurationTemplateError> {
-        tracing::info!("🎭 Stage 3: Rendering configuration templates with variables...");
+        tracing::info!(
+            stage = "configuration_rendering",
+            template_type = "ansible",
+            "Rendering configuration templates with variables"
+        );
 
         // Create build directory structure
         let build_ansible_dir = self.create_build_directory().await?;
@@ -183,19 +187,30 @@ impl AnsibleTemplateRenderer {
             .await?;
 
         tracing::debug!(
-            "   ✅ Configuration templates rendered to: {}",
-            build_ansible_dir.display()
+            stage = "configuration_rendering",
+            template_type = "ansible",
+            output_dir = %build_ansible_dir.display(),
+            "Configuration templates rendered"
         );
         tracing::debug!(
-            "   ✅ Inventory rendered with IP: {}",
-            inventory_context.ansible_host()
+            stage = "configuration_rendering",
+            template_type = "ansible_inventory",
+            ansible_host = %inventory_context.ansible_host(),
+            "Inventory rendered with IP"
         );
         tracing::debug!(
-            "   ✅ Inventory rendered with SSH key: {}",
-            inventory_context.ansible_ssh_private_key_file()
+            stage = "configuration_rendering",
+            template_type = "ansible_inventory",
+            ssh_key = %inventory_context.ansible_ssh_private_key_file(),
+            "Inventory rendered with SSH key"
         );
 
-        tracing::info!("✅ Stage 3 complete: Configuration templates ready");
+        tracing::info!(
+            stage = "configuration_rendering",
+            template_type = "ansible",
+            status = "complete",
+            "Configuration templates ready"
+        );
         Ok(())
     }