chore: Remove old code related to legacy datasets (#5274)

# Description
<!-- Please include a summary of the changes and the related issue.
Please also include relevant motivation and context. List any
dependencies that are required for this change. -->

This PR removes old code related to the legacy datasets

**Type of change**
<!-- Please delete options that are not relevant. Remember to title the
PR according to the type of change -->


**How Has This Been Tested**
<!-- Please add some reference about how your feature has been tested.
-->

**Checklist**
<!-- Please go over the list and make sure you've taken everything into
account -->

- I added relevant documentation
- I followed the style guidelines of this project
- I did a self-review of my code
- I made corresponding changes to the documentation
- I confirm My changes generate no new warnings
- I have added tests that prove my fix is effective or that my feature
works
- I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

Author: frascuchon

argilla-server/src/argilla_server/settings.py

@@ -102,8 +102,6 @@ class Settings(BaseSettings):
 
     docs_enabled: bool = True
 
-    namespace: str = Field(default=None, regex=r"^[a-z]+$")
-
     enable_migration: bool = Field(
         default=False,
         description="If enabled, try to migrate data from old rubrix installation",
@@ -210,36 +208,6 @@ def create_home_path(cls, values):
 
         return values
 
-    @property
-    def dataset_index_name(self) -> str:
-        ns = self.namespace
-        if ns:
-            return f"{self.namespace}.{self.__DATASETS_INDEX_NAME__}"
-        return self.__DATASETS_INDEX_NAME__
-
-    @property
-    def dataset_records_index_name(self) -> str:
-        ns = self.namespace
-        if ns:
-            return f"{self.namespace}.{self.__DATASETS_RECORDS_INDEX_NAME__}"
-        return self.__DATASETS_RECORDS_INDEX_NAME__
-
-    @property
-    def old_dataset_index_name(self) -> str:
-        index_name = ".rubrix<NAMESPACE>.datasets-v0"
-        ns = self.namespace
-        if ns is None:
-            return index_name.replace("<NAMESPACE>", "")
-        return index_name.replace("<NAMESPACE>", f".{ns}")
-
-    @property
-    def old_dataset_records_index_name(self) -> str:
-        index_name = ".rubrix<NAMESPACE>.dataset.{}.records-v0"
-        ns = self.namespace
-        if ns is None:
-            return index_name.replace("<NAMESPACE>", "")
-        return index_name.replace("<NAMESPACE>", f".{ns}")
-
     @property
     def database_engine_args(self) -> Dict:
         if self.database_is_sqlite:

argilla-server/tests/unit/commons/test_settings.py

@@ -13,26 +13,8 @@
 #  limitations under the License.
 
 import pytest
-from argilla_server.settings import Settings
-
-from tests.pydantic_v1 import ValidationError
-
-
-@pytest.mark.parametrize("bad_namespace", ["Badns", "bad-ns", "12-bad-ns", "@bad"])
-def test_wrong_settings_namespace(monkeypatch, bad_namespace):
-    monkeypatch.setenv("ARGILLA_NAMESPACE", bad_namespace)
-    with pytest.raises(ValidationError):
-        Settings()
-
 
-def test_settings_namespace(monkeypatch):
-    monkeypatch.setenv("ARGILLA_NAMESPACE", "namespace")
-
-    settings = Settings()
-
-    assert settings.namespace == "namespace"
-    assert settings.dataset_index_name == "namespace.ar.datasets"
-    assert settings.dataset_records_index_name == "namespace.ar.dataset.{}"
+from argilla_server.settings import Settings
 
 
 def test_settings_index_replicas_with_shards_defined(monkeypatch):