feat: add ready_ts to servers endpoint (#1006)

This will close RVTEE-316
rivet-gg · Aug 8, 2024 · 8b44a7c · 8b44a7c
1 parent a23073b
commit 8b44a7c
Show file tree

Hide file tree

Showing 13 changed files with 29 additions and 7 deletions.
diff --git a/fern/definition/servers/common.yml b/fern/definition/servers/common.yml
@@ -20,6 +20,7 @@ types:
           DELETE request if needed.
         type: optional<long>
       create_ts: long
+      start_ts: optional<long>
       destroy_ts: optional<long>
 
   Resources:

diff --git a/lib/convert/src/impls/ds.rs b/lib/convert/src/impls/ds.rs
@@ -13,6 +13,7 @@ impl ApiTryFrom<backend::ds::Server> for models::ServersServer {
 		Ok(models::ServersServer {
 			cluster_id: unwrap!(value.cluster_id).as_uuid(),
 			create_ts: value.create_ts,
+			start_ts: value.start_ts,
 			datacenter_id: unwrap!(value.datacenter_id).as_uuid(),
 			destroy_ts: value.destroy_ts,
 			game_id: unwrap!(value.game_id).as_uuid(),

diff --git a/proto/backend/ds.proto b/proto/backend/ds.proto
@@ -16,13 +16,13 @@ message Server {
     int64 kill_timeout_ms = 7;
     optional string webhook_url = 8;
     int64 create_ts = 9;
-    optional int64 destroy_ts = 10;
-    rivet.common.Uuid image_id = 11;
-    repeated string args = 12;
-    NetworkMode network_mode = 13;
-    map<string, string> environment = 14;
-    map<string, Port> network_ports = 15;
-
+    optional int64 start_ts = 10;
+    optional int64 destroy_ts = 11;
+    rivet.common.Uuid image_id = 12;
+    repeated string args = 13;
+    NetworkMode network_mode = 14;
+    map<string, string> environment = 15;
+    map<string, Port> network_ports = 16;
 }
 
 message ServerResources {

diff --git a/sdks/full/go/servers/types.go b/sdks/full/go/servers/types.go
diff --git a/sdks/full/openapi/openapi.yml b/sdks/full/openapi/openapi.yml
diff --git a/sdks/full/openapi_compat/openapi.yml b/sdks/full/openapi_compat/openapi.yml
diff --git a/sdks/full/rust-cli/docs/ServersServer.md b/sdks/full/rust-cli/docs/ServersServer.md
diff --git a/sdks/full/rust-cli/src/models/servers_server.rs b/sdks/full/rust-cli/src/models/servers_server.rs
diff --git a/sdks/full/rust/docs/ServersServer.md b/sdks/full/rust/docs/ServersServer.md
diff --git a/sdks/full/rust/src/models/servers_server.rs b/sdks/full/rust/src/models/servers_server.rs
diff --git a/svc/pkg/ds/db/servers/migrations/20240501133910_init.up.sql b/svc/pkg/ds/db/servers/migrations/20240501133910_init.up.sql
@@ -13,6 +13,7 @@ CREATE TABLE servers (
 	webhook_url STRING,
 
 	create_ts INT NOT NULL,
+	start_ts INT,
 	stop_ts INT,
 	finish_ts INT,
 	cleanup_ts INT,

diff --git a/svc/pkg/ds/ops/server-create/src/lib.rs b/svc/pkg/ds/ops/server-create/src/lib.rs
@@ -1402,6 +1402,7 @@ pub async fn handle(
 			kill_timeout_ms: ctx.kill_timeout_ms,
 			webhook_url: ctx.webhook_url.clone(),
 			create_ts,
+			start_ts: None,
 			destroy_ts: None,
 			args: ctx.args.clone(),
 			environment: ctx.environment.clone(),

diff --git a/svc/pkg/ds/ops/server-get/src/lib.rs b/svc/pkg/ds/ops/server-get/src/lib.rs
@@ -15,6 +15,7 @@ struct Server {
 	kill_timeout_ms: i64,
 	webhook_url: Option<String>,
 	create_ts: i64,
+	start_ts: Option<i64>,
 	destroy_ts: Option<i64>,
 	image_id: Uuid,
 	args: Vec<String>,
@@ -84,6 +85,7 @@ pub async fn handle(
 				kill_timeout_ms,
 				webhook_url,
 				create_ts,
+				start_ts,
 				destroy_ts,
 				image_id,
 				args,
@@ -223,6 +225,7 @@ pub async fn handle(
 				network_mode: server.network_mode.try_into()?,
 				network_ports: ports,
 				create_ts: server.create_ts,
+				start_ts: server.start_ts,
 				destroy_ts: server.destroy_ts,
 			};