Add individual failure states for before/after lease hooks, switch to status polling

Author: kirkbrauer

proto/jumpstarter/v1/common.proto

@@ -8,13 +8,14 @@ package jumpstarter.v1;
 
 // Exporter status information
 enum ExporterStatus {
-  EXPORTER_STATUS_UNSPECIFIED = 0;       // Unspecified exporter status
-  EXPORTER_STATUS_OFFLINE = 1;           // Exporter is offline
-  EXPORTER_STATUS_AVAILABLE = 2;         // Exporter is available to be leased
-  EXPORTER_STATUS_BEFORE_LEASE_HOOK = 3; // Exporter is executing before lease hook(s)
-  EXPORTER_STATUS_LEASE_READY = 4;       // Exporter is leased and ready to accept commands
-  EXPORTER_STATUS_AFTER_LEASE_HOOK = 5;  // Exporter is executing after lease hook(s)
-  EXPORTER_STATUS_ERROR = 6;             // Exporter is in an error state and cannot accept leases
+  EXPORTER_STATUS_UNSPECIFIED = 0;              // Unspecified exporter status
+  EXPORTER_STATUS_OFFLINE = 1;                  // Exporter is offline
+  EXPORTER_STATUS_AVAILABLE = 2;                // Exporter is available to be leased
+  EXPORTER_STATUS_BEFORE_LEASE_HOOK = 3;        // Exporter is executing before lease hook(s)
+  EXPORTER_STATUS_LEASE_READY = 4;              // Exporter is leased and ready to accept commands
+  EXPORTER_STATUS_AFTER_LEASE_HOOK = 5;         // Exporter is executing after lease hook(s)
+  EXPORTER_STATUS_BEFORE_LEASE_HOOK_FAILED = 6; // Exporter before lease hook failed
+  EXPORTER_STATUS_AFTER_LEASE_HOOK_FAILED = 7;  // Exporter after lease hook failed
 }
 
 // Source of log stream messages

proto/jumpstarter/v1/jumpstarter.proto

@@ -23,7 +23,7 @@ service ControllerService {
   rpc Unregister(UnregisterRequest) returns (UnregisterResponse);
 
   // Exporter status report
-  // Allows exporters to report their own status
+  // Allows exporters to report their own status to the controller
   rpc ReportStatus(ReportStatusRequest) returns (ReportStatusResponse);
 
   // Exporter listening
@@ -116,7 +116,7 @@ message AuditStreamRequest {
 
 message ReportStatusRequest {
   ExporterStatus status = 1;
-  optional string status_message = 2;  // Optional human-readable status detail
+  optional string message = 2;  // Optional human-readable status message
 }
 
 message ReportStatusResponse {}
@@ -130,12 +130,7 @@ service ExporterService {
   rpc StreamingDriverCall(StreamingDriverCallRequest) returns (stream StreamingDriverCallResponse);
   rpc LogStream(google.protobuf.Empty) returns (stream LogStreamResponse);
   rpc Reset(ResetRequest) returns (ResetResponse);
-
-  // Get current exporter status
   rpc GetStatus(GetStatusRequest) returns (GetStatusResponse);
-
-  // Stream status updates
-  rpc StatusStream(StatusStreamRequest) returns (stream StatusStreamResponse);
 }
 
 message GetReportResponse {
@@ -225,13 +220,5 @@ message GetStatusRequest {}
 
 message GetStatusResponse {
   ExporterStatus status = 1;
-  optional string status_message = 2;
-}
-
-message StatusStreamRequest {}
-
-message StatusStreamResponse {
-  ExporterStatus status = 1;
-  optional string status_message = 2;
-  optional google.protobuf.Timestamp timestamp = 3;
-}
+  optional string message = 2;
+}