Skip to content

Commit 2da83fb

Browse files
committed
chore: minor renaming stuff
1 parent 2b477d1 commit 2da83fb

32 files changed

+93
-74
lines changed

.dockerignore

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,18 @@
1-
# cache and dot files
1+
# Cache and dot files
2+
.coverage/
23
.eggs/
3-
.venv/
4+
.github/
5+
.gitignore
46
.mypy_cache/
57
.pytest_cache/
6-
.coverage/
78
.ruff/
89
.ruff_cache/
9-
.eggs/
10-
.github/
11-
.gitignore
12-
# Docs and atifacts
10+
.venv/
11+
# Docs and artifacts
1312
./deployment
1413
./docs
15-
/images
16-
# build
14+
./images
15+
# Build
1716
./build
1817
./dist
1918
# Scripts and tools

.github/workflows/helm_test.yaml renamed to .github/workflows/build_and_publish_charts.yaml

Lines changed: 54 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,18 @@
1-
name: Helm Chart Testing
1+
name: Build and Publish Helm Charts
22

33
on:
4-
pull_request:
5-
branches: [main]
4+
push:
5+
branches:
6+
- main
7+
pull_request:
8+
branches:
9+
- main
610

711
jobs:
8-
helm-test:
12+
create-cluster:
13+
name: Create k3s Cluster
914
runs-on: ubuntu-latest
15+
needs: checkout
1016
steps:
1117
- name: Checkout
1218
uses: actions/[email protected]
@@ -20,31 +26,71 @@ jobs:
2026

2127
- name: Create namespace
2228
run: kubectl create ns tesk
23-
29+
30+
helm-deps:
31+
name: Helm Dependencies
32+
runs-on: ubuntu-latest
33+
needs: checkout
34+
steps:
35+
- name: Checkout
36+
uses: actions/[email protected]
37+
with:
38+
fetch-depth: 0
39+
2440
- name: Helm Deps
2541
run: |
2642
for dir in $(ls -d deployment/charts/*); do
2743
helm dependency update $dir;
2844
done
29-
45+
46+
helm-lint:
47+
name: Helm Lint
48+
runs-on: ubuntu-latest
49+
needs: [helm-deps]
50+
steps:
51+
- name: Checkout
52+
uses: actions/[email protected]
53+
with:
54+
fetch-depth: 0
55+
3056
- name: Helm Lint
3157
run: |
3258
for dir in $(ls -d deployment/charts/*); do
3359
helm lint $dir
3460
done
35-
61+
62+
helm-install:
63+
name: Helm Install
64+
runs-on: ubuntu-latest
65+
needs: [create-cluster, helm-deps, helm-lint]
66+
steps:
67+
- name: Checkout
68+
uses: actions/[email protected]
69+
with:
70+
fetch-depth: 0
71+
3672
- name: Apply Helm file
3773
run: helm install -n tesk tesk . -f values.yaml
3874
working-directory: deployment/charts/tesk
3975

4076
- name: Sleep for 30 seconds
4177
run: sleep 30
42-
78+
4379
- name: Get Helm and k8s
4480
run: helm list -n tesk && kubectl get all -n tesk
4581

4682
- name: curl URL
4783
run: curl localhost -vL
84+
85+
chart-releaser:
86+
name: Run chart-releaser
87+
runs-on: ubuntu-latest
88+
needs: helm-install
89+
steps:
90+
- name: Checkout
91+
uses: actions/[email protected]
92+
with:
93+
fetch-depth: 0
4894

4995
- name: Configure Git
5096
run: |
@@ -57,4 +103,3 @@ jobs:
57103
skip_existing: true
58104
env:
59105
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
60-

.github/workflows/docker_build_publish_filer.yaml renamed to .github/workflows/build_and_publish_filer.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: tesk-core-filer
1+
name: Build and Publish Filer
22

33
on:
44
push:
@@ -14,6 +14,7 @@ on:
1414

1515
jobs:
1616
build-from-source:
17+
name: Build and publish Docker image
1718
runs-on: ubuntu-latest
1819
steps:
1920
- name: Checkout elixir-cloud-aai/tesk-core

.github/workflows/docker_build_publish_taskmaster.yaml renamed to .github/workflows/build_and_publish_taskmaster.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: tesk-core-taskmaster
1+
name: Build and Publish Taskmaster
22

33
on:
44
push:

.github/workflows/code_quality.yaml

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ on:
1010

1111
jobs:
1212
lint:
13+
name: Lint
1314
runs-on: ubuntu-latest
1415
steps:
1516
- name: Check out repository
@@ -25,15 +26,14 @@ jobs:
2526
python-version: '3.11'
2627
cache: 'poetry'
2728

28-
# Install lint deps only
29-
- name: Install dependencies
30-
# if: steps.cq.outputs.cache-hit != 'true'
29+
- name: Install lint dependencies
3130
run: poetry install --only=lint --no-interaction --no-root
3231

3332
- name: Check code quality
3433
run: poetry run ruff check .
3534

3635
format:
36+
name: Format
3737
runs-on: ubuntu-latest
3838
steps:
3939
- name: Check out repository
@@ -49,15 +49,14 @@ jobs:
4949
python-version: '3.11'
5050
cache: 'poetry'
5151

52-
# Install lint deps only
53-
- name: Install dependencies
54-
# if: steps.cq.outputs.cache-hit != 'true'
52+
- name: Install lint dependencies
5553
run: poetry install --only=lint --no-interaction --no-root
5654

5755
- name: Check code style
5856
run: poetry run ruff format --check
5957

6058
type-check:
59+
name: Type Check
6160
runs-on: ubuntu-latest
6261
steps:
6362
- name: Check out repository
@@ -73,15 +72,14 @@ jobs:
7372
python-version: '3.11'
7473
cache: 'poetry'
7574

76-
# Install only type check dependency
77-
- name: Install dependencies
78-
# if: steps.cqtc.outputs.cache-hit != 'true'
75+
- name: Install types and mypy dependencies
7976
run: poetry install --only=types --no-interaction --no-root
8077

8178
- name: Check types
8279
run: poetry run mypy tesk/
8380

8481
spell-check:
82+
name: Spell Check
8583
runs-on: ubuntu-latest
8684
steps:
8785
- name: Check out repository
@@ -97,9 +95,7 @@ jobs:
9795
python-version: '3.11'
9896
cache: 'poetry'
9997

100-
# Install lint deps only
101-
- name: Install dependencies
102-
# if: steps.cq.outputs.cache-hit != 'true'
98+
- name: Install lint dependencies
10399
run: poetry install --only=lint --no-interaction --no-root
104100

105101
- name: Check spellings

.github/workflows/code_test.yaml renamed to .github/workflows/code_test_unit.yaml

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Code Test
1+
name: Code Test Unit
22

33
on:
44
push:
@@ -10,29 +10,23 @@ on:
1010

1111
jobs:
1212
unit-test:
13+
name: Unit Test
1314
runs-on: ubuntu-latest
14-
strategy:
15-
matrix:
16-
python-version: ['3.11', '3.12', '3.13']
17-
1815
steps:
1916
- name: Check out repository
2017
uses: actions/checkout@v4
2118

2219
- name: Install poetry
2320
run: pipx install poetry
2421

25-
# Set up python versions with matrix to test on multiple versions
26-
- name: Set up Python ${{ matrix.python-version }}
22+
- name: Set up Python
2723
uses: actions/setup-python@v5
28-
id: ct-${{ matrix.python-version}}
24+
id: ct
2925
with:
30-
python-version: ${{ matrix.python-version }}
26+
python-version: '3.11'
3127
cache: 'poetry'
3228

33-
# Install all test dependencies
34-
- name: Install dependencies
35-
# if: steps.ct-${{ matrix.python-version }}.outputs.cache-hit != 'true'
29+
- name: Install test dependencies
3630
run: poetry install --only=test --no-interaction --no-root
3731

3832
- name: Run ATS

.github/workflows/pr_evaulations.yaml renamed to .github/workflows/validation_pr.yaml

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ jobs:
1515
env:
1616
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
1717
with:
18-
# Configure which types are allowed (newline-delimited).
19-
# Default: https://github.com/commitizen/conventional-commit-types
2018
types: |
2119
fix
2220
feat
@@ -29,12 +27,7 @@ jobs:
2927
ci
3028
chore
3129
revert
32-
# Configure additional validation for the subject based on a regex.
33-
# This example ensures the subject doesn't start with an uppercase character & has more than 50 chars.
3430
subjectPattern: ^(?![A-Z])(?=.{1,50}$).+$
35-
# If `subjectPattern` is configured, you can use this property to override
36-
# the default error message that is shown when the pattern doesn't match.
37-
# The variables `subject` and `title` can be used within the message.
3831
subjectPatternError: |
3932
The subject "{subject}" found in the pull request title "{title}"
4033
didn't match the configured pattern. Please ensure that the subject
@@ -49,6 +42,4 @@ jobs:
4942
- name: List files with merge conflict markers
5043
run: git --no-pager grep "<<<<<<<" ":(exclude).github/" || true
5144
- name: Fail or succeed job if any files with merge conflict markers have been checked in
52-
# Find lines containing "<<<<<<<", then count the number of lines.
53-
# 0 matching lines results in exit code 0, i.e. success.
5445
run: exit $(git grep "<<<<<<<" ":(exclude).github/" | wc --lines)

.github/workflows/vulnerability_test.yaml renamed to .github/workflows/vulnerabilities.yaml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ on:
1010

1111
jobs:
1212
code-vulnerabilities:
13+
name: Code
1314
runs-on: ubuntu-latest
1415
steps:
1516
- name: Check out repository
@@ -25,15 +26,14 @@ jobs:
2526
python-version: '3.11'
2627
cache: 'poetry'
2728

28-
# Install only vulnerability check dependency
29-
- name: Install dependencies
30-
# if: steps.vt.outputs.cache-hit != 'true'
29+
- name: Install vulnerabilities check dependencies
3130
run: poetry install --only=security --no-interaction --no-root
3231

33-
- name: Check code vulnerabilities
32+
- name: Check code vulnerabilities with bandit
3433
run: poetry run bandit -c pyproject.toml -r tesk/
3534

3635
dependency-vulnerabilities:
36+
name: Dependencies
3737
runs-on: ubuntu-latest
3838
steps:
3939
- name: Check out repository
@@ -49,10 +49,8 @@ jobs:
4949
python-version: '3.11'
5050
cache: 'poetry'
5151

52-
# Install only vulnerabilities check dependency
53-
- name: Install dependencies
54-
# if: steps.vt.outputs.cache-hit != 'true'
52+
- name: Install vulnerabilities check dependencies
5553
run: poetry install --only=security --no-interaction --no-root
5654

57-
- name: Check dependency vulnerabilities
55+
- name: Check dependency vulnerabilities with safety
5856
run: poetry run safety check --full-report

Makefile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
# TODO: Add commands to be compatible for windows
2-
# TODO: Add commands for tesk_api
3-
41
# Define variables
52
PYTHON_CMD := $(shell command -v python3 2> /dev/null)
63
BUILDAH_CMD := $(shell command -v buildah 2> /dev/null)

pyproject.toml

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,19 @@
11
[tool.poetry]
22
name = "tesk"
33
version = "0.1.0"
4-
description = "An implementation of a task execution engine based on the TES standard running on Kubernetes."
5-
authors = ["TBA <[email protected]>"]
6-
maintainers = ["TBA <[email protected]>", "TBA <[email protected]>"]
4+
description = "An implementation of a task execution engine based on the GA4GH Task Execution Service (TES) API standard running on Kubernetes."
5+
authors = ["Elixir Cloud AAI <[email protected]>"]
6+
maintainers = ["Elixir Cloud AAI <[email protected]>"]
77
license = "Apache-2.0"
88
readme = "README.md"
99
repository = "https://github.com/elixir-cloud-aai/TESK"
1010
keywords = ["TES", "Kubernetes", "Task Execution", "ebi", "TESK", "workflow", "automation"]
1111
classifiers = [
12-
"Development Status :: 4 - Alpha",
12+
"Development Status :: 4 - Beta",
1313
"Intended Audience :: System Administrators",
1414
"License :: OSI Approved :: Apache Software License",
1515
"Programming Language :: Python :: 3",
16-
"Programming Language :: Python :: 3.11",
17-
"Programming Language :: Python :: 3.12",
18-
"Programming Language :: Python :: 3.13",
16+
"Programming Language :: Python :: 3.11"
1917
]
2018

2119
[tool.poetry.scripts]

tests/test_unit/test_service/resources/copyDirTest/src/a/2.txt

Whitespace-only changes.

tests/test_unit/service/test_filer.py renamed to tests/test_unit/test_service/test_filer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
)
2626
from tesk.service.filer_s3 import S3Transput
2727
from tesk.service.path import containerPath
28-
from tests.test_unit.service.assertThrows import AssertThrowsMixin
28+
from tests.test_unit.test_service.assertThrows import AssertThrowsMixin
2929

3030

3131
def getTree(rootDir):
@@ -139,7 +139,7 @@ def test_copyDir(self):
139139
def rmDir(d):
140140
os.system(f'rm -r {d}')
141141

142-
baseDir = 'tests/test_unit/service/resources/copyDirTest/'
142+
baseDir = 'tests/test_unit/test_service/resources/copyDirTest/'
143143
src = os.path.join(baseDir, 'src')
144144
dst1 = os.path.join(baseDir, 'dst1')
145145
dst2 = os.path.join(baseDir, 'dst2')

tests/test_unit/service/test_filer_http_pytest.py renamed to tests/test_unit/test_service/test_filer_http_pytest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from tesk.service.filer import HTTPTransput, Type
88

99
PATH_DOWN = 'test_download_file.txt'
10-
PATH_UP = 'tests/test_unit/service/test_filer_http_pytest.py'
10+
PATH_UP = 'tests/test_unit/test_service/test_filer_http_pytest.py'
1111
SUCCESS = 200
1212
FAIL = 300
1313
URL = 'http://www.foo.bar'

0 commit comments

Comments
 (0)