openstack provider: ignore ec2 metadata if not present

Signed-off-by: Riccardo Piccoli <[email protected]>
coreos · Nov 15, 2024 · ebf930c · ebf930c
1 parent de95d78
commit ebf930c
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 11 deletions.
diff --git a/docs/release-notes.md b/docs/release-notes.md
@@ -10,6 +10,8 @@ Major changes:
 
 Minor changes:
 
+- Openstack: do not fail if ec2 metadata is not found
+
 Packaging changes:
 
 ## Afterburn 5.7.0

diff --git a/src/providers/openstack/configdrive.rs b/src/providers/openstack/configdrive.rs
@@ -101,20 +101,27 @@ impl OpenstackConfigDrive {
     }
 
     /// The metadata is stored as key:value pair in ec2/latest/meta-data.json file
-    fn read_metadata_ec2(&self) -> Result<MetadataEc2JSON> {
+    fn read_metadata_ec2(&self) -> Result<Option<MetadataEc2JSON>> {
+        use std::io::ErrorKind::NotFound;
+
         let filename = self.metadata_dir("ec2").join("meta-data.json");
-        let file =
-            File::open(&filename).with_context(|| format!("failed to open file '{filename:?}'"))?;
+        let file = match File::open(&filename) {
+            Ok(file) => file,
+            Err(e) if e.kind() == NotFound => return Ok(None),
+            Err(e) => return Err(anyhow::Error::new(e).context(format!("failed to open file '{filename:?}'"))),
+        };
         let bufrd = BufReader::new(file);
-        Self::parse_metadata_ec2(bufrd)
-            .with_context(|| format!("failed to parse file '{filename:?}'"))
+        match Self::parse_metadata_ec2(bufrd)
+            .with_context(|| format!("failed to parse file '{filename:?}'")) {
+                Ok(metadata) => Ok(Some(metadata)),
+                Err(e) => Err(e),
+            }
     }
 
     /// The metadata is stored as key:value pair in openstack/latest/meta_data.json file
     fn read_metadata_openstack(&self) -> Result<MetadataOpenstackJSON> {
         let filename = self.metadata_dir("openstack").join("meta_data.json");
-        let file =
-            File::open(&filename).with_context(|| format!("failed to open file '{filename:?}'"))?;
+        let file = File::open(&filename).with_context(|| format!("failed to open file '{filename:?}'"))?;
         let bufrd = BufReader::new(file);
         Self::parse_metadata_openstack(bufrd)
             .with_context(|| format!("failed to parse file '{filename:?}'"))
@@ -144,17 +151,22 @@ impl OpenstackConfigDrive {
 impl MetadataProvider for OpenstackConfigDrive {
     fn attributes(&self) -> Result<HashMap<String, String>> {
         let mut out = HashMap::with_capacity(6);
-        let metadata_ec2: MetadataEc2JSON = self.read_metadata_ec2()?;
         let metadata_openstack: MetadataOpenstackJSON = self.read_metadata_openstack()?;
         if let Some(hostname) = metadata_openstack.hostname {
             out.insert("OPENSTACK_HOSTNAME".to_string(), hostname);
         }
-        if let Some(instance_id) = metadata_ec2.instance_id {
-            out.insert("OPENSTACK_INSTANCE_ID".to_string(), instance_id);
-        }
         if let Some(uuid) = metadata_openstack.uuid {
             out.insert("OPENSTACK_INSTANCE_UUID".to_string(), uuid);
         }
+
+        let metadata_ec2 = match self.read_metadata_ec2() {
+            Ok(Some(metadata)) => metadata,
+            Ok(None) => return Ok(out),
+            Err(e) => return Err(e),
+        };
+        if let Some(instance_id) = metadata_ec2.instance_id {
+            out.insert("OPENSTACK_INSTANCE_ID".to_string(), instance_id);
+        }
         if let Some(instance_type) = metadata_ec2.instance_type {
             out.insert("OPENSTACK_INSTANCE_TYPE".to_string(), instance_type);
         }