Build.py improvement (#16976)

* Add instance type info to output

* Add repro cmd to success branch of build.py

ci/build.py

@@ -268,9 +268,9 @@ def load_docker_cache(tag, docker_registry) -> None:
 
 
 def log_environment():
-    instance_id = ec2_instance_id_hostname()
-    if instance_id:
-        logging.info("EC2 Instance id: %s", instance_id)
+    instance_info = ec2_instance_info()
+    if instance_info:
+        logging.info("EC2: %s", instance_info)
     pp = pprint.PrettyPrinter(indent=4)
     logging.debug("Build environment: %s", pp.pformat(dict(os.environ)))
 

ci/safe_docker_run.py

@@ -143,6 +143,7 @@ def run(self, *args, **kwargs) -> int:
                     logging.info("Executed command for reproduction:\n\n%s\n", " ".join(sys.argv))
                 else:
                     logging.info("Container exited with success 👍")
+                    logging.info("Executed command for reproduction:\n\n%s\n", " ".join(sys.argv))
             except Exception as err:
                 logging.exception(err)
                 return 150

ci/util.py

@@ -94,11 +94,14 @@ def under_ci() -> bool:
     return 'JOB_NAME' in os.environ
 
 
-def ec2_instance_id_hostname() -> str:
+def ec2_instance_info() -> str:
     import requests
     if under_ci():
         result = []
         try:
+            r = requests.get("http://instance-data/latest/meta-data/instance-type")
+            if r.status_code == 200:
+                result.append(r.content.decode())
             r = requests.get("http://instance-data/latest/meta-data/instance-id")
             if r.status_code == 200:
                 result.append(r.content.decode())