Ignore overprovisioning in metadata protocol (#1391)

* Add method to indicate supports overprovisioning

* Ignore overprovisioning in metadata protocol

azurelinuxagent/common/protocol/metadata.py

@@ -123,6 +123,10 @@ def _get_trans_cert(self):
         content = fileutil.read_file(trans_crt_file)
         return textutil.get_bytes_from_pem(content)
 
+    def supports_overprovisioning(self):
+        # Metadata protocol does not support overprovisioning
+        return False
+
     def detect(self):
         self.get_vminfo()
         trans_prv_file = os.path.join(conf.get_lib_dir(),

azurelinuxagent/common/protocol/restapi.py

@@ -358,3 +358,6 @@ def report_ext_status(self, ext_handler_name, ext_name, ext_status):
 
     def report_event(self, event):
         raise NotImplementedError()
+
+    def supports_overprovisioning(self):
+        return True

azurelinuxagent/ga/exthandlers.py

@@ -316,10 +316,13 @@ def handle_ext_handlers(self, etag=None):
             return
 
         if conf.get_enable_overprovisioning():
-            artifacts_profile = self.protocol.get_artifacts_profile()
-            if artifacts_profile and artifacts_profile.is_on_hold():
-                logger.info("Extension handling is on hold")
-                return
+            if not self.protocol.supports_overprovisioning():
+                logger.verbose("Overprovisioning is enabled but protocol does not support it.")
+            else:
+                artifacts_profile = self.protocol.get_artifacts_profile()
+                if artifacts_profile and artifacts_profile.is_on_hold():
+                    logger.info("Extension handling is on hold")
+                    return
 
         wait_until = datetime.datetime.utcnow() + datetime.timedelta(minutes=DEFAULT_EXT_TIMEOUT_MINUTES)
         max_dep_level = max([handler.sort_key() for handler in self.ext_handlers.extHandlers])