chore: release v1.3.1 (#2924)

Co-authored-by: Ralf Grubenmann <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mohammad Alisafaee <[email protected]>
Co-authored-by: Lorenzo Cavazzi <[email protected]>
Co-authored-by: Rok Roškar <[email protected]>

Author: 6 people

.github/workflows/test_deploy.yml

@@ -64,7 +64,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -106,7 +106,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -145,7 +145,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -190,7 +190,7 @@ jobs:
         if: steps.cache.outputs.cache-hit != 'true'
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           python -m pip install .[all]
       - name: Install renku into cache
@@ -250,7 +250,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           pip install .[all]
@@ -296,7 +296,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -354,7 +354,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -418,7 +418,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -482,7 +482,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -532,7 +532,7 @@ jobs:
           curl -L https://raw.githubusercontent.com/Homebrew/homebrew-core/43842898fd3ff43273466052722f5ba2789196cb/Formula/git-lfs.rb > git-lfs.rb && brew install git-lfs.rb && rm git-lfs.rb
           brew install shellcheck node || brew link --overwrite node
           python -m pip install --upgrade pip
-          python -m pip install wheel poetry-dynamic-versioning
+          python -m pip install wheel poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -568,7 +568,7 @@ jobs:
           curl -L https://raw.githubusercontent.com/Homebrew/homebrew-core/43842898fd3ff43273466052722f5ba2789196cb/Formula/git-lfs.rb > git-lfs.rb && brew install git-lfs.rb && rm git-lfs.rb
           brew install shellcheck node || brew link --overwrite node
           python -m pip install --upgrade pip
-          python -m pip install wheel poetry-dynamic-versioning
+          python -m pip install wheel poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -611,7 +611,7 @@ jobs:
           curl -L https://raw.githubusercontent.com/Homebrew/homebrew-core/43842898fd3ff43273466052722f5ba2789196cb/Formula/git-lfs.rb > git-lfs.rb && brew install git-lfs.rb && rm git-lfs.rb
           brew install shellcheck node || brew link --overwrite node
           python -m pip install --upgrade pip
-          python -m pip install wheel poetry-dynamic-versioning
+          python -m pip install wheel poetry-dynamic-versioning==0.17.1
           make download-templates
           poetry-dynamic-versioning
           python -m pip install .[all]
@@ -696,7 +696,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -775,7 +775,7 @@ jobs:
         if: steps.dependency-cache.outputs.cache-hit != 'true' || 'refs/heads/master' == github.ref || 'refs/heads/develop' == github.ref || startsWith(github.ref, 'refs/tags/')
         run: |
           python -m pip install --upgrade pip
-          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning
+          python -m pip install coveralls poetry wheel twine poetry-dynamic-versioning==0.17.1
           poetry-dynamic-versioning
           make download-templates
           python -m pip install .[all]
@@ -890,7 +890,7 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          python -m pip install poetry poetry-dynamic-versioning poetry-lock-package twine
+          python -m pip install poetry poetry-dynamic-versioning==0.17.1 poetry-lock-package twine
           python -m pip install .[all]
           git config --global --add user.name "Renku Bot"
           git config --global --add user.email "[email protected]"

CHANGES.rst

@@ -18,6 +18,23 @@
 Changes
 =======
 
+`1.3.1 <https://github.com/SwissDataScienceCenter/renku-python/compare/v1.3.0...v1.3.1>`__ (2022-05-23)
+-------------------------------------------------------------------------------------------------------
+
+Bug Fixes
+~~~~~~~~~
+
+-  **ci:** pin poetry-dynamic-versioning to 0.17.1
+   (`#2920 <https://github.com/SwissDataScienceCenter/renku-python/issues/2920>`__)
+   (`239ef76 <https://github.com/SwissDataScienceCenter/renku-python/commit/239ef766c441ee534a850a965337d9a8126a1588>`__)
+
+Features
+~~~~~~~~
+
+-  **cli:** show spinner when cloning repo for dataset import/update
+   (`#2914 <https://github.com/SwissDataScienceCenter/renku-python/issues/2914>`__)
+   (`46508af <https://github.com/SwissDataScienceCenter/renku-python/commit/46508af8573dba967e8ac91420a69c2180d0d7c5>`__)
+
 `1.3.0 <https://github.com/SwissDataScienceCenter/renku-python/compare/v1.2.4...v1.3.0>`__ (2022-05-20)
 -------------------------------------------------------------------------------------------------------
 

Dockerfile

@@ -15,7 +15,7 @@ COPY pyproject.toml poetry.lock README.rst CHANGES.rst Makefile /code/renku/
 WORKDIR /code/renku
 RUN poetry export --without-hashes -f requirements.txt --output /tmp/requirements.txt && \
     pip install -r /tmp/requirements.txt && \
-    pip install poetry-dynamic-versioning
+    pip install poetry-dynamic-versioning==0.17.1
 
 COPY .git /code/renku/.git
 COPY renku /code/renku/renku

helm-chart/renku-core/Chart.yaml

@@ -3,4 +3,4 @@ appVersion: "1.0"
 description: A Helm chart for Kubernetes
 name: renku-core
 icon: https://avatars0.githubusercontent.com/u/53332360?s=400&u=a4311d22842343604ef61a8c8a1e5793209a67e9&v=4
-version: 1.3.0
+version: 1.3.1

helm-chart/renku-core/values.yaml

@@ -109,7 +109,7 @@ versions:
     fullnameOverride: ""
     image:
       repository: renku/renku-core
-      tag: "v1.3.0"
+      tag: "v1.3.1"
       pullPolicy: IfNotPresent
   v8:
     name: v8

renku/core/dataset/dataset.py

@@ -1023,9 +1023,10 @@ def update_dataset_git_files(
             if url in visited_repos:
                 remote_repository, remote_client = visited_repos[url]
             else:
-                remote_repository = clone_repository(
-                    url=url, path=get_cache_directory_for_repository(client=client, url=url), checkout_revision=ref
-                )
+                with communication.busy(msg="Cloning remote repository..."):
+                    remote_repository = clone_repository(
+                        url=url, path=get_cache_directory_for_repository(client=client, url=url), checkout_revision=ref
+                    )
                 remote_client = LocalClient(path=remote_repository.path)
                 visited_repos[url] = remote_repository, remote_client
 

renku/core/dataset/providers/renku.py

@@ -359,22 +359,23 @@ def _fetch_dataset(self, client_dispatcher: IClientDispatcher, database_dispatch
 
         urls = (self._project_url_ssh, self._project_url_http)
         # Clone the project
-        for url in urls:
-            try:
-                repository = clone_renku_repository(
-                    url=url,
-                    path=get_cache_directory_for_repository(client=client, url=url),
-                    gitlab_token=self._gitlab_token,
-                    deployment_hostname=parsed_uri.netloc,
-                    depth=None,
-                    reuse_existing_repository=True,
-                    use_renku_credentials=True,
-                )
-            except errors.GitError:
-                pass
-            else:
-                self._project_url = url
-                break
+        with communication.busy(msg="Cloning remote repository..."):
+            for url in urls:
+                try:
+                    repository = clone_renku_repository(
+                        url=url,
+                        path=get_cache_directory_for_repository(client=client, url=url),
+                        gitlab_token=self._gitlab_token,
+                        deployment_hostname=parsed_uri.netloc,
+                        depth=None,
+                        reuse_existing_repository=True,
+                        use_renku_credentials=True,
+                    )
+                except errors.GitError:
+                    pass
+                else:
+                    self._project_url = url
+                    break
 
         if self._project_url is None or repository is None:
             raise errors.ParameterError("Cannot clone remote projects:\n\t" + "\n\t".join(urls), param_hint=self._uri)

renku/core/session/session.py

@@ -21,8 +21,6 @@
 from itertools import chain
 from typing import Optional
 
-from yaspin import yaspin
-
 from renku.command.command_builder import inject
 from renku.core import errors
 from renku.core.interface.client_dispatcher import IClientDispatcher
@@ -94,7 +92,7 @@ def session_start(
                 abort=True,
             )
 
-            with yaspin(text="Building image"):
+            with communication.busy(msg="Building image"):
                 _ = provider_api.build_image(client.docker_path.parent, image_name, config)
     else:
         if not provider_api.find_image(image_name, config):

renku/core/util/communication.py

@@ -17,6 +17,7 @@
 # limitations under the License.
 """Communicator classes for printing output."""
 
+from contextlib import ExitStack, contextmanager
 from functools import wraps
 from threading import RLock
 
@@ -56,6 +57,14 @@ def has_prompt(self):
     def prompt(self, msg, type=None, default=None, **kwargs):
         """Show a message prompt."""
 
+    @contextmanager
+    def busy(self, msg):
+        """Indicate a busy status.
+
+        For instance, show a spinner in the CLI.
+        """
+        yield
+
 
 def lock_communication(method):
     """Ensure communicator is locked."""
@@ -177,6 +186,14 @@ def finalize_progress(self, name):
         for listener in self._listeners:
             listener.finalize_progress(name)
 
+    @contextmanager
+    def busy(self, msg):
+        """Show busy indicators."""
+        with ExitStack() as stack:
+            for listener in self._listeners:
+                stack.enter_context(listener.busy(msg))
+            yield
+
     @lock_communication
     def disable(self):
         """Disable all outputs; by default everything is enabled."""
@@ -243,7 +260,7 @@ def warn(msg):
 
 @ensure_manager
 def error(msg):
-    """Write an info message to all listeners."""
+    """Write an error message to all listeners."""
     _thread_local.communication_manager.error(msg)
 
 
@@ -283,6 +300,14 @@ def finalize_progress(name):
     _thread_local.communication_manager.finalize_progress(name)
 
 
+@ensure_manager
+@contextmanager
+def busy(msg):
+    """Indicate busy status to all listeners."""
+    with _thread_local.communication_manager.busy(msg):
+        yield
+
+
 @ensure_manager
 def get_listeners():
     """Return a list of subscribed listeners."""

renku/ui/cli/utils/callback.py

@@ -18,9 +18,11 @@
 """Communicator class for printing click output."""
 
 import sys
+from contextlib import contextmanager
 
 import click
 from tqdm import tqdm
+from yaspin import yaspin
 
 import renku.ui.cli.utils.color as color
 from renku.core.util.communication import CommunicationCallback
@@ -93,6 +95,12 @@ def finalize_progress(self, name):
         self._progress_bars[name].close()
         del self._progress_bars[name]
 
+    @contextmanager
+    def busy(self, msg):
+        """Indicate busy status using a spinner."""
+        with yaspin(text=msg):
+            yield
+
 
 class ClickCallback(StandardOutput):
     """CommunicationCallback implementation for ``click`` messages."""