apache-airflow-providers-git 0.0.6__tar.gz → 0.0.7__tar.gz

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: apache-airflow-providers-git
-Version: 0.0.6
+Version: 0.0.7
 Summary: Provider package apache-airflow-providers-git for Apache Airflow
 Keywords: airflow-provider,git,airflow,integration
 Author-email: Apache Software Foundation <dev@airflow.apache.org>
@@ -23,8 +23,8 @@ Classifier: Topic :: System :: Monitoring
 Requires-Dist: apache-airflow>=3.0.0
 Requires-Dist: GitPython>=3.1.44
 Project-URL: Bug Tracker, https://github.com/apache/airflow/issues
-Project-URL: Changelog, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6/changelog.html
-Project-URL: Documentation, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6
+Project-URL: Changelog, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7/changelog.html
+Project-URL: Documentation, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7
 Project-URL: Mastodon, https://fosstodon.org/@airflow
 Project-URL: Slack Chat, https://s.apache.org/airflow-slack
 Project-URL: Source Code, https://github.com/apache/airflow
@@ -55,9 +55,8 @@ Project-URL: YouTube, https://www.youtube.com/channel/UCSXwxpWZQ7XZ1WL3wqevChA/
 
 Package ``apache-airflow-providers-git``
 
-Release: ``0.0.6``
+Release: ``0.0.7``
 
-Release Date: ``|PypiReleaseDate|``
 
 `Distributed version control system (GIT) <https://git-scm.com/>`__
 
@@ -69,12 +68,12 @@ This is a provider package for ``git`` provider. All classes for this provider p
 are in ``airflow.providers.git`` python package.
 
 You can find package information and changelog for the provider
-in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6/>`_.
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7/>`_.
 
 Installation
 ------------
 
-You can install this package on top of an existing Airflow 2 installation (see ``Requirements`` below
+You can install this package on top of an existing Airflow installation (see ``Requirements`` below
 for the minimum Airflow version supported) via
 ``pip install apache-airflow-providers-git``
 
@@ -91,5 +90,5 @@ PIP package         Version required
 ==================  ==================
 
 The changelog for the provider package can be found in the
-`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6/changelog.html>`_.
+`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7/changelog.html>`_.
 

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/README.rst

@@ -23,9 +23,8 @@
 
 Package ``apache-airflow-providers-git``
 
-Release: ``0.0.6``
+Release: ``0.0.7``
 
-Release Date: ``|PypiReleaseDate|``
 
 `Distributed version control system (GIT) <https://git-scm.com/>`__
 
@@ -37,12 +36,12 @@ This is a provider package for ``git`` provider. All classes for this provider p
 are in ``airflow.providers.git`` python package.
 
 You can find package information and changelog for the provider
-in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6/>`_.
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7/>`_.
 
 Installation
 ------------
 
-You can install this package on top of an existing Airflow 2 installation (see ``Requirements`` below
+You can install this package on top of an existing Airflow installation (see ``Requirements`` below
 for the minimum Airflow version supported) via
 ``pip install apache-airflow-providers-git``
 
@@ -59,4 +58,4 @@ PIP package         Version required
 ==================  ==================
 
 The changelog for the provider package can be found in the
-`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6/changelog.html>`_.
+`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7/changelog.html>`_.

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/docs/changelog.rst

@@ -15,6 +15,36 @@
     specific language governing permissions and limitations
     under the License.
 
+
+Changelog
+---------
+
+0.0.7
+.....
+
+
+Release Date: ``|PypiReleaseDate|``
+
+Bug Fixes
+~~~~~~~~~
+
+* ``Catch Exception in git hook instantiation (#55079)``
+* ``Fix process leaks in 'GitDagBundle' repository management (#54997)``
+
+Doc-only
+~~~~~~~~
+
+* ``Add missing changelog provider for Git (#54496)``
+
+.. Below changes are excluded from the changelog. Move them to
+   appropriate section above if needed. Do not delete the lines(!):
+   * ``Fix git connection test by adding required conn_type field (#54681)``
+   * ``Replace API server's direct Connection access workaround in BaseHook (#54083)``
+   * ``Switch pre-commit to prek (#54258)``
+
+.. Review and move the new changes to one of the sections above:
+   * ``Fix Airflow 2 reference in README/index of providers (#55240)``
+
 0.0.6
 .....
 

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/docs/index.rst

@@ -72,9 +72,7 @@ apache-airflow-providers-git package
 `Distributed version control system (GIT) <https://git-scm.com/>`__
 
 
-Release: 0.0.6
-
-Release Date: ``|PypiReleaseDate|``
+Release: 0.0.7
 
 Provider package
 ----------------
@@ -85,7 +83,7 @@ All classes for this package are included in the ``airflow.providers.git`` pytho
 Installation
 ------------
 
-You can install this package on top of an existing Airflow 2 installation via
+You can install this package on top of an existing Airflow installation via
 ``pip install apache-airflow-providers-git``.
 For the minimum Airflow version supported, see ``Requirements`` below.
 

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/provider.yaml

@@ -22,12 +22,13 @@ description: |
     `Distributed version control system (GIT) <https://git-scm.com/>`__
 
 state: ready
-source-date-epoch: 1754503255
+source-date-epoch: 1756876853
 # Note that those versions are maintained by release manager - do not update them manually
 # with the exception of case where other provider in sources has >= new provider version.
 # In such case adding >= NEW_VERSION and bumping to NEW_VERSION in a provider have
 # to be done in the same PR
 versions:
+  - 0.0.7
   - 0.0.6
   - 0.0.5
   - 0.0.4

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/pyproject.toml

@@ -25,7 +25,7 @@ build-backend = "flit_core.buildapi"
 
 [project]
 name = "apache-airflow-providers-git"
-version = "0.0.6"
+version = "0.0.7"
 description = "Provider package apache-airflow-providers-git for Apache Airflow"
 readme = "README.rst"
 authors = [
@@ -54,7 +54,7 @@ requires-python = ">=3.10"
 
 # The dependencies should be modified in place in the generated file.
 # Any change in the dependencies is preserved when the file is regenerated
-# Make sure to run ``breeze static-checks --type update-providers-dependencies --all-files``
+# Make sure to run ``prek update-providers-dependencies --all-files``
 # After you modify the dependencies, and rebuild your Breeze CI image with ``breeze ci-image build``
 dependencies = [
     "apache-airflow>=3.0.0",
@@ -95,8 +95,8 @@ apache-airflow-providers-common-sql = {workspace = true}
 apache-airflow-providers-standard = {workspace = true}
 
 [project.urls]
-"Documentation" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6"
-"Changelog" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.6/changelog.html"
+"Documentation" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7"
+"Changelog" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.7/changelog.html"
 "Bug Tracker" = "https://github.com/apache/airflow/issues"
 "Source Code" = "https://github.com/apache/airflow"
 "Slack Chat" = "https://s.apache.org/airflow-slack"

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/src/airflow/providers/git/__init__.py

@@ -29,7 +29,7 @@ from airflow import __version__ as airflow_version
 
 __all__ = ["__version__"]
 
-__version__ = "0.0.6"
+__version__ = "0.0.7"
 
 if packaging.version.parse(packaging.version.parse(airflow_version).base_version) < packaging.version.parse(
     "3.0.0"

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/src/airflow/providers/git/bundles/git.py

@@ -80,7 +80,7 @@ class GitDagBundle(BaseDagBundle):
         self.hook: GitHook | None = None
         try:
             self.hook = GitHook(git_conn_id=git_conn_id or "git_default")
-        except AirflowException as e:
+        except Exception as e:
             self._log.warning("Could not create GitHook", conn_id=git_conn_id, exc=e)
 
         if not repo_url and self.hook and self.hook.repo_url:
@@ -93,6 +93,7 @@ class GitDagBundle(BaseDagBundle):
             with cm:
                 self._clone_bare_repo_if_required()
                 self._ensure_version_in_bare_repo()
+            self.bare_repo.close()
 
             self._clone_repo_if_required()
             self.repo.git.checkout(self.tracking_ref)
@@ -104,6 +105,7 @@ class GitDagBundle(BaseDagBundle):
                 self.repo.head.reset(index=True, working_tree=True)
             else:
                 self.refresh()
+            self.repo.close()
 
     def initialize(self) -> None:
         if not self.repo_url:
@@ -161,7 +163,8 @@ class GitDagBundle(BaseDagBundle):
         )
 
     def get_current_version(self) -> str:
-        return self.repo.head.commit.hexsha
+        with self.repo as repo:
+            return repo.head.commit.hexsha
 
     @property
     def path(self) -> Path:
@@ -184,6 +187,7 @@ class GitDagBundle(BaseDagBundle):
             cm = self.bare_repo.git.custom_environment(GIT_SSH_COMMAND=cmd)
         with cm:
             self.bare_repo.remotes.origin.fetch(refspecs)
+            self.bare_repo.close()
 
     def refresh(self) -> None:
         if self.version:
@@ -202,6 +206,7 @@ class GitDagBundle(BaseDagBundle):
                 else:
                     target = self.tracking_ref
                 self.repo.head.reset(target, index=True, working_tree=True)
+                self.repo.close()
 
     @staticmethod
     def _convert_git_ssh_url_to_https(url: str) -> str:

{apache_airflow_providers_git-0.0.6 → apache_airflow_providers_git-0.0.7}/tests/unit/git/bundles/test_git.py

@@ -33,6 +33,8 @@ from airflow.exceptions import AirflowException
 from airflow.models import Connection
 from airflow.providers.git.bundles.git import GitDagBundle
 from airflow.providers.git.hooks.git import GitHook
+from airflow.sdk.exceptions import ErrorType
+from airflow.sdk.execution_time.comms import ErrorResponse
 
 from tests_common.test_utils.config import conf_vars
 from tests_common.test_utils.version_compat import AIRFLOW_V_3_1_PLUS
@@ -66,6 +68,14 @@ def git_repo(tmp_path_factory):
     return (directory, repo)
 
 
+def assert_repo_is_closed(bundle: GitDagBundle):
+    # cat-file processes get left around if the repo is not closed, so check it was
+    assert bundle.repo.git.cat_file_all is None
+    assert bundle.bare_repo.git.cat_file_all is None
+    assert bundle.repo.git.cat_file_header is None
+    assert bundle.bare_repo.git.cat_file_header is None
+
+
 class TestGitDagBundle:
     @classmethod
     def teardown_class(cls) -> None:
@@ -135,6 +145,8 @@ class TestGitDagBundle:
 
         assert bundle.get_current_version() == repo.head.commit.hexsha
 
+        assert_repo_is_closed(bundle)
+
     @mock.patch("airflow.providers.git.bundles.git.GitHook")
     def test_get_specific_version(self, mock_githook, git_repo):
         repo_path, repo = git_repo
@@ -161,6 +173,8 @@ class TestGitDagBundle:
         files_in_repo = {f.name for f in bundle.path.iterdir() if f.is_file()}
         assert {"test_dag.py"} == files_in_repo
 
+        assert_repo_is_closed(bundle)
+
     @mock.patch("airflow.providers.git.bundles.git.GitHook")
     def test_get_tag_version(self, mock_githook, git_repo):
         repo_path, repo = git_repo
@@ -209,6 +223,8 @@ class TestGitDagBundle:
         files_in_repo = {f.name for f in bundle.path.iterdir() if f.is_file()}
         assert {"test_dag.py", "new_test.py"} == files_in_repo
 
+        assert_repo_is_closed(bundle)
+
     @pytest.mark.parametrize(
         "amend",
         [
@@ -248,6 +264,8 @@ class TestGitDagBundle:
         files_in_repo = {f.name for f in bundle.path.iterdir() if f.is_file()}
         assert {"test_dag.py", "new_test.py"} == files_in_repo
 
+        assert_repo_is_closed(bundle)
+
     @mock.patch("airflow.providers.git.bundles.git.GitHook")
     def test_refresh_tag(self, mock_githook, git_repo):
         """Ensure that the bundle refresh works when tracking a tag"""
@@ -653,13 +671,16 @@ class TestGitDagBundle:
 
                 assert "Repository path: %s not found" in str(exc_info.value)
 
-    @pytest.mark.db_test
-    @patch.dict(os.environ, {"AIRFLOW_CONN_MY_TEST_GIT": '{"host": "something"}'})
+    @patch.dict(os.environ, {"AIRFLOW_CONN_MY_TEST_GIT": '{"host": "something", "conn_type": "git"}'})
     @pytest.mark.parametrize(
         "conn_id, expected_hook_type",
         [("my_test_git", GitHook), ("something-else", type(None))],
     )
-    def test_repo_url_access_missing_connection_doesnt_error(self, conn_id, expected_hook_type):
+    def test_repo_url_access_missing_connection_doesnt_error(
+        self, conn_id, expected_hook_type, mock_supervisor_comms
+    ):
+        if expected_hook_type is type(None):
+            mock_supervisor_comms.send.return_value = ErrorResponse(error=ErrorType.CONNECTION_NOT_FOUND)
         bundle = GitDagBundle(
             name="testa",
             tracking_ref="main",