[upload] Ignore metadata when passing empty string

conan/internal/upload_metadata.py

@@ -18,7 +18,21 @@ def _metadata_files(folder, metadata):
     return result
 
 
-def gather_metadata(package_list, cache, metadata):
+def gather_metadata(package_list, cache, metadata: list):
+    """
+    List and configure the metadata files to be uploaded.
+    The metadata supports patterns, so it can be used to upload only a subset of the metadata files.
+    When metadata is not specified, all metadata files are uploaded.
+    But, when metadata is empty string (""), it means that no metadata files should be uploaded.
+
+    :param package_list: Package to be uploaded
+    :param cache: Conan client cache
+    :param metadata: A list of patterns of metadata that should be uploaded. Default None
+    """
+
+    if metadata == ['']:
+        return
+
     for rref, recipe_bundle in package_list.refs().items():
         if metadata or recipe_bundle["upload"]:
             metadata_folder = cache.recipe_layout(rref).metadata()

conans/test/integration/metadata/test_metadata_commands.py

@@ -198,3 +198,25 @@ def test_no_download_cached(self):
         assert "mylogs2!!!!" in mylogs
         mybuildlogs = load(os.path.join(c2_pkg_metadata_path, "logs", "mybuildlogs.txt"))
         assert "mybuildlogs2!!!!" in mybuildlogs
+
+    def test_upload_ignored_metadata(self):
+        """
+        Upload command should ignore metadata files when passing --metadata=""
+        """
+        client = TestClient(default_server_user=True)
+        client.save({"conanfile.py": GenConanfile("pkg", "0.1")})
+        client.run("create .")
+        pid = client.created_package_id("pkg/0.1")
+
+        def _save_metadata(client, ref):
+            client.run(f"cache path {ref} --folder=metadata")
+            metadata_path = str(client.stdout).strip()
+            myfile = os.path.join(metadata_path, "logs", "somefile.log")
+            save(myfile, ref)
+
+        _save_metadata(client, "pkg/0.1")
+        _save_metadata(client, f"pkg/0.1:{pid}")
+
+        client.run('upload * --confirm --remote=default --metadata=""')
+        assert "Recipe metadata" not in client.out
+        assert "Package metadata" not in client.out