Merge pull request #290 from overmindtech/improve-hcp-info

Added more info to the HCP config

changes.proto

@@ -717,7 +717,7 @@ message ChangeProperties {
 
   // Repo information; can be an empty string. CLI attempts auto-population, but users can override. Not necessarily a URL. The UI will be responsible for any formatting/shortnening/sprucing up should it be required.
   string repo = 15;
-  
+
   // User-defined tags associated with this change, will be populated via the CLI and not the UI.
   map<string, string> tags = 16;
 }

config.proto

@@ -95,7 +95,12 @@ message CreateHcpConfigRequest {
 }
 
 message CreateHcpConfigResponse {
+    // The configuration of the HCP Run Task that was created
     HcpConfig config = 1;
+
+    // The API Key response for the API key that backs this integration. This
+    // API will have been created but not yet authorised, the user must still be
+    // redirected to the authorizeURL to complete the process.
     apikeys.CreateAPIKeyResponse apiKey = 2;
 }
 
@@ -110,6 +115,17 @@ message GetHcpConfigRequest {}
 
 message GetHcpConfigResponse {
     HcpConfig config = 1;
+
+    enum Status {
+        // The HCP Run Task configuration is active and can be used
+        CONFIGURED = 0;
+        // The HCP Run Task configuration is not fully configured and needs to
+        // be recreated, this is usually due to the API key being revoked or the
+        // user not completing the authorisation process
+        ERROR = 1;
+    }
+
+    Status status = 2;
 }
 
 message DeleteHcpConfigRequest {}