Refactoring/305 update dependencies (#306)

ArBridgeman · web-flow · commit 883320e36caf · 2025-07-09T12:44:58.000+02:00
* Update dependencies: bucketfs &amp; PTB

* Fix poetry objects to pass poetry check

* Update .gitignore to include .html-documentation

* Update changelog to reflect what happened

* Set GitHub token permissions &amp; switch GPU tests to updated PTB

* Format with project:fix

* Fix warning in github which indicates version is not being set for test-exaslct-alias-script job

* Prepare release 3.4.0

* Add changelog.md &amp; version.py to .gitattributes
diff --git a/.gitattributes b/.gitattributes
@@ -1,2 +1,4 @@
 poetry.lock linguist-generated=true
 setup.py linguist-generated=true
+doc/changes/changelog.md linguist-generated=true
+exasol/slc/version.py linguist-generated=true
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -193,6 +193,8 @@ jobs:
 
   test-exaslct-alias-script:
     needs: build-matrix
+    permissions:
+      contents: read
     strategy:
       fail-fast: false
       matrix: ${{ fromJson(needs.build-matrix.outputs.matrix) }}
@@ -205,7 +207,7 @@ jobs:
       - name: Setup Python & Poetry Environment
         uses: exasol/python-toolbox/.github/actions/python-environment@1.6.0
         with:
-          python-version: '${{ matrix.python_version }}'
+          python-version: ${{ matrix.python-version }}
 
       - name: Test exaslct
         run: poetry run exaslct --help
diff --git a/.github/workflows/gpu-tests.yml b/.github/workflows/gpu-tests.yml
@@ -7,6 +7,8 @@ jobs:
 
   prep-testbed-gpu:
     runs-on: ubuntu-24.04
+    permissions:
+      contents: read
 
     steps:
       - uses: actions/checkout@v4
@@ -23,6 +25,8 @@ jobs:
 
   test-gpu:
     needs: prep-testbed-gpu
+    permissions:
+      contents: read
     strategy:
       fail-fast: false
       matrix:
@@ -35,23 +39,11 @@ jobs:
     steps:
       - uses: actions/checkout@v4
 
-        #Cannot use exasol/python-toolbox/.github/actions/python-environment here
-        #because pipx is not installed on int-linux-x64-4core-gpu-t4-ubuntu24.04-1
-      - name: Setup Python ('${{ matrix.python_version }}')
-        uses: actions/setup-python@v5
+      - name: Setup Python & Poetry Environment
+        uses: exasol/python-toolbox/.github/actions/python-environment@1.6.0
         with:
           python-version: '${{ matrix.python_version }}'
-
-      - name: Install pipx and poetry
-        run: |
-          pip install pipx
-          pipx install poetry=="2.1.2"
-          echo "$HOME/.local/bin" >> $GITHUB_PATH
-
-      - name: Setup Poetry
-        run: |
-          poetry env use python3
-          poetry install
+          use-cache: 'false'
 
       - name: Allow unprivileged user namespaces
         run: sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0
diff --git a/.github/workflows/integration-tests.yml b/.github/workflows/integration-tests.yml
@@ -11,6 +11,8 @@ jobs:
 
   prep-testbed:
     runs-on: ubuntu-24.04
+    permissions:
+      contents: read
 
     steps:
       - uses: actions/checkout@v4
@@ -27,6 +29,8 @@ jobs:
 
   test-all:
     needs: prep-testbed
+    permissions:
+      contents: read
     strategy:
       fail-fast: false
       matrix:
diff --git a/.github/workflows/slow-checks.yml b/.github/workflows/slow-checks.yml
@@ -12,6 +12,8 @@ jobs:
   slow-tests-approval:
     name: Approve Slow Tests
     runs-on: ubuntu-24.04
+    permissions:
+      contents: read
 
     # Even though the environment "manual-approval" will be created automatically,
     # it still needs to be configured to require interactive review.
@@ -33,9 +35,13 @@ jobs:
     name: Integrations
     uses: ./.github/workflows/integration-tests.yml
     secrets: inherit
+    permissions:
+      contents: read
 
   gpu-tests:
     needs: slow-tests-approval
     name: Integrations
     uses: ./.github/workflows/gpu-tests.yml
     secrets: inherit
+    permissions:
+      contents: read
diff --git a/.gitignore b/.gitignore
@@ -136,6 +136,8 @@ dmypy.json
 # Jetbrains
 .idea/
 
+# documentation
+.html-documentation
 
 # Project
 .build_output/
diff --git a/doc/changes/changelog.md b/doc/changes/changelog.md
diff --git a/doc/changes/changes_3.4.0.md b/doc/changes/changes_3.4.0.md
@@ -0,0 +1,14 @@
+# 3.4.0 - 2025-07-04
+
+## Features
+
+ - #292: Created a new nox task to clean up images on dockerhub
+
+## Refactorings
+
+ - #301: Update PTB and workflows and activate generation of GH pages
+ - #305: Updated exasol-bucketfs to 2.0.0
+
+## Security
+
+ - #303: Update dependencies on top of 3.3.0, including exasol-toolbox to 1.6.0
diff --git a/doc/changes/unreleased.md b/doc/changes/unreleased.md
@@ -1,13 +1 @@
 # Unreleased
-
-## Features
-
- - #292: Created a new nox task to clean up images on dockerhub
-
-## Refactorings
-
- - #301: Update PTB and workflows and activate generation of GH pages
-
-## Security
-
- - #303: Update dependencies on top of 3.3.0
diff --git a/exasol/slc/internal/tasks/export/export_container_task.py b/exasol/slc/internal/tasks/export/export_container_task.py
@@ -35,7 +35,7 @@ def get_release_task(self) -> BaseTask:
         )  # pylint: disable=no-member
         return self.create_child_task(
             task_class=class_,
-            **self.required_task_info.params  # pylint: disable=no-member
+            **self.required_task_info.params,  # pylint: disable=no-member
         )
 
     def get_release_goal(self) -> str:
diff --git a/exasol/slc/internal/tasks/test/test_runner_db_test_base_task.py b/exasol/slc/internal/tasks/test/test_runner_db_test_base_task.py
@@ -260,9 +260,12 @@ def get_generic_language_tests(self, test_config):
         return generic_language_tests
 
     def read_test_config(self):
-        with pathlib.Path(self.flavor_path).joinpath("flavor_base").joinpath(
-            "testconfig"
-        ).open("r") as file:
+        with (
+            pathlib.Path(self.flavor_path)
+            .joinpath("flavor_base")
+            .joinpath("testconfig")
+            .open("r") as file
+        ):
             test_config_str = file.read()
             test_config = {}
             for line in test_config_str.splitlines():
diff --git a/exasol/slc/internal/utils/tar_safe_extract.py b/exasol/slc/internal/utils/tar_safe_extract.py
@@ -17,7 +17,7 @@ def safe_extract(
     path: str = ".",
     members: Optional[list[str]] = None,
     *,
-    numeric_owner: bool = False
+    numeric_owner: bool = False,
 ):
     """
     This function implements a patch for the CVE-2007-4559. The patch essentially checks
diff --git a/exasol/slc/version.py b/exasol/slc/version.py
diff --git a/poetry.lock b/poetry.lock
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,7 @@
 [project]
 name = "exasol-script-languages-container-tool"
-version = "3.3.0"
+version = "3.4.0"
+requires-python = ">=3.10,<4"
 description = "Script Languages Container Tool"
 keywords = ['exasol', 'udf', 'script-languages']
 
@@ -12,6 +13,8 @@ authors = [
     { name = "Thomas Uebensee", email = "Thomas Uebensee <ext.thomas.uebensee@exasol.com>"}
 ]
 
+dynamic = ["dependencies"]
+
 [tool.poetry]
 
 requires-poetry = ">=2.1.0"
@@ -32,12 +35,11 @@ issues = "https://github.com/exasol/script-languages-container-tool/issues"
 
 
 [tool.poetry.dependencies]
-python = ">=3.10,<4"
 importlib_metadata = ">=4.6.0"
 importlib-resources = ">=6.4.0"
 networkx = "^3.3.0"
 exasol-integration-test-docker-environment = "^4.1.0"
-exasol-bucketfs="^1.0.1"
+exasol-bucketfs="^2.0.0"
 pydantic="^2.10.2"
 jsonschema="^4.23.0"
 
@@ -48,7 +50,7 @@ build-backend = "poetry.core.masonry.api"
 [tool.poetry.group.dev.dependencies]
 toml = ">=0.10.2"
 configobj = "^5.0.8" # moved to dev dependencie so security alert does not propagate further up
-exasol-toolbox = "^1.0.1"
+exasol-toolbox = "^1.6.0"
 aiohttp = "^3.12.7"
 tqdm = "^4.67.1"
 

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ def get_release_task(self) -> BaseTask:`
`35`	`35`	`) # pylint: disable=no-member`
`36`	`36`	`return self.create_child_task(`
`37`	`37`	`task_class=class_,`
`38`		`- **self.required_task_info.params # pylint: disable=no-member`
	`38`	`+ **self.required_task_info.params, # pylint: disable=no-member`
`39`	`39`	`)`
`40`	`40`
`41`	`41`	`def get_release_goal(self) -> str:`