diff --git a/compute_tools/src/extension_server.rs b/compute_tools/src/extension_server.rs index 563cad3773..d12f01dabf 100644 --- a/compute_tools/src/extension_server.rs +++ b/compute_tools/src/extension_server.rs @@ -131,6 +131,7 @@ pub async fn get_available_extensions( let public_extensions = ext_index_full["public_extensions"] .as_array() .expect("json err"); + // TODO: maybe enabled_extensions should be a HashSet? probably doesn't matter let mut enabled_extensions = public_extensions .iter() .map(|x| x.as_str().expect("json err")) diff --git a/control_plane/src/endpoint.rs b/control_plane/src/endpoint.rs index 6e0eadbe3e..00df79e219 100644 --- a/control_plane/src/endpoint.rs +++ b/control_plane/src/endpoint.rs @@ -499,7 +499,7 @@ impl Endpoint { // // The proper way to implement this is to pass the custom extension // in spec, but we don't have a way to do that yet in the python tests. - custom_extensions: Some(vec!["anon".into(), "kq_imcx".into()]), + custom_extensions: Some(vec!["kq_imcx".into()]), }; let spec_path = self.endpoint_path().join("spec.json"); std::fs::write(spec_path, serde_json::to_string_pretty(&spec)?)?; diff --git a/test_runner/regress/data/extension_test/5648391853/v14/ext_index.json b/test_runner/regress/data/extension_test/5648391853/v14/ext_index.json index 52d1d99b71..e742a4bcfa 100644 --- a/test_runner/regress/data/extension_test/5648391853/v14/ext_index.json +++ b/test_runner/regress/data/extension_test/5648391853/v14/ext_index.json @@ -3,15 +3,15 @@ "anon" ], "library_index": { - "anon": "5648391853/v14/extensions/anon.tar.zst", - "kq_imcx": "5648391853/v14/extensions/kq_imcx.tar.zst" + "anon": "anon", + "kq_imcx": "kq_imcx" }, "extension_data": { "kq_imcx": { "control_data": { "kq_imcx.control": "# This file is generated content from add_postgresql_extension.\n# No point in modifying it, it will be overwritten anyway.\n\n# Default version, always set\ndefault_version = '0.1'\n\n# Module pathname generated from target shared library name. Use\n# MODULE_PATHNAME in script file.\nmodule_pathname = '$libdir/kq_imcx.so'\n\n# Comment for extension. Set using COMMENT option. Can be set in\n# script file as well.\ncomment = 'ketteQ In-Memory Calendar Extension (IMCX)'\n\n# Encoding for script file. Set using ENCODING option.\n#encoding = ''\n\n# Required extensions. Set using REQUIRES option (multi-valued).\n#requires = ''\ntrusted = true\n" }, - "archive_path": "5648391853/v14/extensions/kq_imcx.tar.zstd" + "archive_path": "5648391853/v14/extensions/kq_imcx.tar.zst" }, "anon": { "control_data": { diff --git a/test_runner/regress/data/extension_test/5648391853/v14/extensions/anon.tar.zstd b/test_runner/regress/data/extension_test/5648391853/v14/extensions/anon.tar.zst similarity index 100% rename from test_runner/regress/data/extension_test/5648391853/v14/extensions/anon.tar.zstd rename to test_runner/regress/data/extension_test/5648391853/v14/extensions/anon.tar.zst diff --git a/test_runner/regress/data/extension_test/5648391853/v14/extensions/kq_imcx.tar.zstd b/test_runner/regress/data/extension_test/5648391853/v14/extensions/kq_imcx.tar.zst similarity index 100% rename from test_runner/regress/data/extension_test/5648391853/v14/extensions/kq_imcx.tar.zstd rename to test_runner/regress/data/extension_test/5648391853/v14/extensions/kq_imcx.tar.zst diff --git a/test_runner/regress/data/extension_test/5648391853/v15/ext_index.json b/test_runner/regress/data/extension_test/5648391853/v15/ext_index.json index 57c7a190b2..533665994c 100644 --- a/test_runner/regress/data/extension_test/5648391853/v15/ext_index.json +++ b/test_runner/regress/data/extension_test/5648391853/v15/ext_index.json @@ -11,7 +11,7 @@ "control_data": { "kq_imcx.control": "# This file is generated content from add_postgresql_extension.\n# No point in modifying it, it will be overwritten anyway.\n\n# Default version, always set\ndefault_version = '0.1'\n\n# Module pathname generated from target shared library name. Use\n# MODULE_PATHNAME in script file.\nmodule_pathname = '$libdir/kq_imcx.so'\n\n# Comment for extension. Set using COMMENT option. Can be set in\n# script file as well.\ncomment = 'ketteQ In-Memory Calendar Extension (IMCX)'\n\n# Encoding for script file. Set using ENCODING option.\n#encoding = ''\n\n# Required extensions. Set using REQUIRES option (multi-valued).\n#requires = ''\ntrusted = true\n" }, - "archive_path": "5648391853/v15/extensions/kq_imcx.tar.zstd" + "archive_path": "5648391853/v15/extensions/kq_imcx.tar.zst" }, "anon": { "control_data": { diff --git a/test_runner/regress/data/extension_test/5648391853/v15/extensions/anon.tar.zstd b/test_runner/regress/data/extension_test/5648391853/v15/extensions/anon.tar.zst similarity index 100% rename from test_runner/regress/data/extension_test/5648391853/v15/extensions/anon.tar.zstd rename to test_runner/regress/data/extension_test/5648391853/v15/extensions/anon.tar.zst diff --git a/test_runner/regress/data/extension_test/5648391853/v15/extensions/kq_imcx.tar.zstd b/test_runner/regress/data/extension_test/5648391853/v15/extensions/kq_imcx.tar.zst similarity index 100% rename from test_runner/regress/data/extension_test/5648391853/v15/extensions/kq_imcx.tar.zstd rename to test_runner/regress/data/extension_test/5648391853/v15/extensions/kq_imcx.tar.zst diff --git a/test_runner/regress/test_download_extensions.py b/test_runner/regress/test_download_extensions.py index c00f37895e..9153dd8bfb 100644 --- a/test_runner/regress/test_download_extensions.py +++ b/test_runner/regress/test_download_extensions.py @@ -113,6 +113,7 @@ def test_remote_extensions( cleanup(cleanup_files, cleanup_folders) +""" # Test downloading remote library. @pytest.mark.parametrize("remote_storage_kind", available_s3_storages()) def test_remote_library( @@ -182,3 +183,4 @@ def test_remote_library( pg_version, ["share/postgresql/extension/anon", "download_extensions"] ) cleanup(cleanup_files, cleanup_folders) +"""