Deprecate load_providers_configuration,

restore_core_default_configuration instead of directly remove

Author: jason810496

shared/configuration/src/airflow_shared/configuration/parser.py

@@ -1271,6 +1271,43 @@ def _resolve_deprecated_lookup(
 
         return section, key, deprecated_section, deprecated_key, warning_emitted
 
+    def load_providers_configuration(self) -> None:
+        """
+        Load configuration for providers.
+
+        .. deprecated:: 3.2.0
+            Provider configuration is now loaded lazily via the ``configuration_description``
+            cached property.  This method is kept for backwards compatibility and will be
+            removed in a future version.
+        """
+        warnings.warn(
+            "load_providers_configuration() is deprecated. "
+            "Provider configuration is now loaded lazily via the "
+            "`configuration_description` cached property.",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        self._use_providers_configuration = True
+        self._invalidate_provider_flag_caches()
+
+    def restore_core_default_configuration(self) -> None:
+        """
+        Restore the parser state before provider-contributed sections were loaded.
+
+        .. deprecated:: 3.2.0
+            Use ``make_sure_configuration_loaded(with_providers=False)`` context manager
+            instead.  This method is kept for backwards compatibility and will be removed
+            in a future version.
+        """
+        warnings.warn(
+            "restore_core_default_configuration() is deprecated. "
+            "Use `make_sure_configuration_loaded(with_providers=False)` instead.",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        self._use_providers_configuration = False
+        self._invalidate_provider_flag_caches()
+
     @overload  # type: ignore[override]
     def get(self, section: str, key: str, fallback: str = ..., **kwargs) -> str: ...
 

shared/configuration/tests/configuration/test_parser.py

@@ -1159,3 +1159,32 @@ def test_invalidate_provider_flag_caches_allows_recomputation(self):
         sensitive_after = test_conf.sensitive_config_values
         assert desc_after == desc_before
         assert sensitive_after == sensitive_before
+
+    def test_load_providers_configuration_emits_deprecation_warning(self):
+        """Test that load_providers_configuration emits a DeprecationWarning."""
+        test_conf = AirflowConfigParser()
+        with pytest.warns(DeprecationWarning, match="load_providers_configuration.*deprecated"):
+            test_conf.load_providers_configuration()
+        assert test_conf._use_providers_configuration is True
+
+    def test_restore_core_default_configuration_emits_deprecation_warning(self):
+        """Test that restore_core_default_configuration emits a DeprecationWarning."""
+        test_conf = AirflowConfigParser()
+        with pytest.warns(DeprecationWarning, match="restore_core_default_configuration.*deprecated"):
+            test_conf.restore_core_default_configuration()
+        assert test_conf._use_providers_configuration is False
+
+    def test_deprecated_load_restore_round_trip(self):
+        """Test that the deprecated methods toggle _use_providers_configuration correctly."""
+        test_conf = AirflowConfigParser()
+        assert test_conf._use_providers_configuration is True
+
+        with pytest.warns(DeprecationWarning, match="restore_core_default_configuration"):
+            test_conf.restore_core_default_configuration()
+        assert test_conf._use_providers_configuration is False
+        assert "configuration_description" not in test_conf.__dict__
+
+        with pytest.warns(DeprecationWarning, match="load_providers_configuration"):
+            test_conf.load_providers_configuration()
+        assert test_conf._use_providers_configuration is True
+        assert "configuration_description" not in test_conf.__dict__