Pass appdef metadata to ServerSpec in monarch commands.info() (#1671)

Summary:
Pull Request resolved: #1671

This change ensures that metadata from the AppDef is propagated to the ServerSpec object returned by the info() function.

Reviewed By: moonli

Differential Revision: D85579216

fbshipit-source-id: 208205c32f16d60b8afb97dd045478cd3d373756

Author: Ritesh1905

python/monarch/tools/commands.py

@@ -211,6 +211,7 @@ def info(server_handle: str) -> Optional[ServerSpec]:
         scheduler=scheduler,
         namespace=namespace,
         ui_url=status.ui_url,
+        metadata=appdef.metadata,
     )
 
 

python/tests/tools/test_commands.py

@@ -175,6 +175,10 @@ def replica_status(idx: int, hostname: str) -> ReplicaStatus:
                         image="__unused__",
                     )
                 ],
+                metadata={
+                    "monarch/meshes/trainer/host_type": "gpu.medium",
+                    "monarch/meshes/trainer/gpus": "2",
+                },
             ),
             server_info,
         )
@@ -183,6 +187,14 @@ def replica_status(idx: int, hostname: str) -> ReplicaStatus:
         assert server_info
         self.assertEqual("node_d", server_info.host0("trainer"))
 
+        # verify metadata is correctly passed through
+        self.assertIn("monarch/meshes/trainer/host_type", server_info.metadata)
+        self.assertEqual(
+            server_info.metadata["monarch/meshes/trainer/host_type"], "gpu.medium"
+        )
+        self.assertIn("monarch/meshes/trainer/gpus", server_info.metadata)
+        self.assertEqual(server_info.metadata["monarch/meshes/trainer/gpus"], "2")
+
 
 UNUSED = "__UNUSED__"
 _5_MS = timedelta(milliseconds=5)