Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore(ui): Upgrade UI dependencies #384

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

chore(ui): Upgrade UI dependencies #384

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
42ea8ba
Upgrade UI deps
ariefrahmansyah Jun 28, 2024
1a8348d
Update node version
deadlycoconuts Jun 28, 2024
fe97de5
Update js packages
deadlycoconuts Jul 5, 2024
a5497ba
Update turing github cicd file
deadlycoconuts Jul 5, 2024
d7eced4
Remove helm-chart release workflow
deadlycoconuts Jul 5, 2024
fca5044
Update ensembler job and service workflows
deadlycoconuts Jul 5, 2024
e801745
Update setup-python action in turing workflow
deadlycoconuts Jul 5, 2024
85cf241
Make setup-go optional for certain test-e2e test runs
deadlycoconuts Jul 5, 2024
1a8724a
Update python caching in go actions
deadlycoconuts Jul 5, 2024
cd3d4b6
Update cluster setup and publishing actions
deadlycoconuts Jul 5, 2024
98eeb36
Update artifact uploading action
deadlycoconuts Jul 5, 2024
6bb99dc
Remove unsupported conda cache type from ensembler job and service wo…
deadlycoconuts Jul 5, 2024
f5e57ce
Update docker login actions
deadlycoconuts Jul 5, 2024
7788fbd
Update k8s await workloads action
deadlycoconuts Jul 5, 2024
82bbb21
Add engines field in package.json
deadlycoconuts Jul 5, 2024
1e76ba0
Update github workflow publish step to use local github file definition
deadlycoconuts Jul 10, 2024
28b12a5
Temporarily remove tests before publishing
deadlycoconuts Jul 11, 2024
9bd9150
Deprecate ReactDOM.render
deadlycoconuts Jul 11, 2024
debf3c1
Replace EuiLoadingContent with EuiSkeletonText
deadlycoconuts Jul 11, 2024
9a11b87
Import queryString directly from package instead of individual functions
deadlycoconuts Jul 11, 2024
22b08c5
Update imports from react-collapsed
deadlycoconuts Jul 11, 2024
0530476
Fix all tables
deadlycoconuts Jul 12, 2024
51a67f5
Remove deprecated table props
deadlycoconuts Jul 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/actions/build-cluster-init/action.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ runs:
cluster-init:${{ steps.build-image.outputs.cluster-init-version }}

- name: Publish Artifact
uses: actions/upload-artifact@v2
uses: actions/upload-artifact@v4
with:
name: cluster-init.${{ steps.build-image.outputs.cluster-init-version }}.tar
path: cluster-init.${{ steps.build-image.outputs.cluster-init-version }}.tar
Expand Down
21 changes: 7 additions & 14 deletions .github/actions/setup-test-cluster/action.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,27 +59,28 @@ runs:
using: composite
steps:
- name: Set up Go
uses: actions/setup-go@v4
uses: actions/setup-go@v5
with:
go-version: ${{ inputs.go-version }}
cache-dependency-path: api/go.sum
Copy link
Contributor

@deadlycoconuts deadlycoconuts Jul 5, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can now use the cache-dependency-path input to indicate files that should be cached instead of having to manually caching them using the actions/actions/cache action. The actions/cache action has also been removed since it's no longer needed. This applies for all the updated actions/setup-[go/node/python] actions.

This is similar done in the MLP PR https://github.com/caraml-dev/mlp/pull/100/files.

Example of the cache being restored successfully: https://github.com/caraml-dev/turing/actions/runs/9804032065/job/27071184098#step:3:68


- name: Download Turing API Docker tar archive
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: ${{ inputs.turing_api_tar_archive_name }}

- name: Download Turing Router Docker tar archive
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: ${{ inputs.turing_router_tar_archive_name }}

- name: Download Experiment Engine Plugin Docker tar archive
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: ${{ inputs.experiment_engine_plugin_archive_name }}

- name: Download Cluster Init Docker tar archive
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: ${{ inputs.cluster_init_tar_archive_name }}

Expand Down Expand Up @@ -209,7 +210,7 @@ runs:
kubectl apply -f infra/e2e/turing.mockserver.yaml

- name: Run action await k8 workloads
uses: jupyterhub/action-k8s-await-workloads@v1
uses: jupyterhub/action-k8s-await-workloads@v3
id: wait-for-deployment
with:
workloads: >-
Expand Down Expand Up @@ -240,11 +241,3 @@ runs:
--request POST \
--data @payload.json \
http://turing-gateway.127.0.0.1.nip.io/api/v1/projects

- name: Cache Test Dependencies
uses: actions/cache@v2
with:
path: api/.go/pkg/mod/
key: |
gomod-${{ hashFiles('api/go.mod') }}
restore-keys: gomod-
14 changes: 7 additions & 7 deletions .github/workflows/cluster-init.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:
outputs:
release-type: ${{ steps.release-rules.outputs.release-type }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- id: release-rules
uses: ./.github/actions/release-rules
with:
Expand All @@ -41,7 +41,7 @@ jobs:
cluster-init-version: ${{ steps.build-cluster-init.outputs.cluster-init-version }}
steps:
- name: Check out code
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
fetch-depth: 0

Expand All @@ -62,10 +62,10 @@ jobs:
- build-cluster-init
steps:
- name: Check out code
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Download Cluster Init Docker tar archieve
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: cluster-init.${{ env.CLUSTER_INIT_VERSION }}.tar

Expand Down Expand Up @@ -130,19 +130,19 @@ jobs:
- build-cluster-init
- test-e2e
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Log in to the Container registry
uses: docker/login-action@v1
uses: docker/login-action@v3
with:
registry: ghcr.io
username: ${{ github.actor }}
password: ${{ secrets.GITHUB_TOKEN }}

- name: Download Cluster Init Docker tar archieve
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: cluster-init.${{ needs.build-cluster-init.outputs.cluster-init-version }}.tar

Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/fluentd-plugin.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
outputs:
release-type: ${{ steps.release-rules.outputs.release-type }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- id: release-rules
uses: ./.github/actions/release-rules
with:
Expand All @@ -46,12 +46,12 @@ jobs:
needs:
- release-rules
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Log in to the Container registry
uses: docker/login-action@v1
uses: docker/login-action@v3
with:
registry: ghcr.io
username: ${{ github.actor }}
Expand Down
38 changes: 0 additions & 38 deletions .github/workflows/helm-chart.yaml
View file
Open in desktop

This file was deleted.

22 changes: 9 additions & 13 deletions .github/workflows/pyfunc-ensembler-job.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,16 @@ jobs:
python-version: ["3.8", "3.9", "3.10"]

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache-dependency-path: |
engines/pyfunc-ensembler-job/env-${{ matrix.python-version }}.yaml
engines/pyfunc-ensembler-job/requirements.txt
engines/pyfunc-ensembler-job/requirements.dev.txt

- name: Setup Java
uses: actions/setup-java@v2
Expand All @@ -51,14 +55,6 @@ jobs:
with:
auto-update-conda: true

- name: Cache Conda environment
uses: actions/cache@v2
with:
path: engines/pyfunc-ensembler-job/env
key: |
conda-${{ hashFiles('engines/pyfunc-ensembler-job/env-${{ matrix.python-version }}.yaml') }}-${{ hashFiles('engines/pyfunc-ensembler-job/requirements.txt') }}-${{ hashFiles('engines/pyfunc-ensembler-job/requirements.dev.txt') }}
restore-keys: conda-

- name: Run Tests
working-directory: engines/pyfunc-ensembler-job
run: |
Expand All @@ -70,7 +66,7 @@ jobs:
outputs:
release-type: ${{ steps.release-rules.outputs.release-type }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- id: release-rules
uses: ./.github/actions/release-rules
with:
Expand All @@ -97,12 +93,12 @@ jobs:
- release-rules
- test
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Log in to the Container registry
uses: docker/login-action@v1
uses: docker/login-action@v3
with:
registry: ghcr.io
username: ${{ github.actor }}
Expand Down
22 changes: 9 additions & 13 deletions .github/workflows/pyfunc-ensembler-service.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,26 +33,22 @@ jobs:
python-version: ["3.8", "3.9", "3.10"]

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache-dependency-path: |
engines/pyfunc-ensembler-service/env-${{ matrix.python-version }}.yaml
engines/pyfunc-ensembler-service/requirements.txt
engines/pyfunc-ensembler-service/requirements.dev.txt

- name: Setup Conda
uses: conda-incubator/setup-miniconda@v2
with:
auto-update-conda: true

- name: Cache Conda environment
uses: actions/cache@v2
with:
path: engines/pyfunc-ensembler-service/env
key: |
conda-${{ hashFiles('engines/pyfunc-ensembler-service/env-${{ matrix.python-version }}.yaml') }}-${{ hashFiles('engines/pyfunc-ensembler-service/requirements.txt') }}-${{ hashFiles('engines/pyfunc-ensembler-service/requirements.dev.txt') }}
restore-keys: conda-

- name: Run Tests
working-directory: engines/pyfunc-ensembler-service
run: |
Expand All @@ -64,7 +60,7 @@ jobs:
outputs:
release-type: ${{ steps.release-rules.outputs.release-type }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- id: release-rules
uses: ./.github/actions/release-rules
with:
Expand All @@ -91,12 +87,12 @@ jobs:
- release-rules
- test
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Log in to the Container registry
uses: docker/login-action@v1
uses: docker/login-action@v3
with:
registry: ghcr.io
username: ${{ github.actor }}
Expand Down
36 changes: 15 additions & 21 deletions .github/workflows/sdk.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,20 +32,17 @@ jobs:
working-directory: ./sdk
steps:
- name: Check out code
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}

- name: Cache pip dependencies
uses: actions/cache@v2
with:
path: ~/.cache/pip
key: |
pip-${{ hashFiles('sdk/requirements.txt') }}-${{ hashFiles('sdk/requirements.dev.txt') }}-${{ matrix.python-version }}
restore-keys: pip-
cache: pip
cache-dependency-path: |
sdk/requirements.txt
sdk/requirements.dev.txt
engines/pyfunc-ensembler-service/requirements.dev.txt
Comment on lines +41 to +45
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Example of the cache being restored successfully: https://github.com/caraml-dev/turing/actions/runs/9804032057/job/27071120655#step:3:22


- name: Install dependencies
run: make setup
Expand All @@ -65,7 +62,7 @@ jobs:
outputs:
release-type: ${{ steps.release-rules.outputs.release-type }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- id: release-rules
uses: ./.github/actions/release-rules
with:
Expand All @@ -87,23 +84,20 @@ jobs:
environment: ${{ needs.release-rules.outputs.release-type == 'dev' && 'manual' || '' }}
steps:
- name: Check out code
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
# fetch full history for a proper version number assignment
fetch-depth: 0

- name: Setup Python
uses: actions/setup-python@v2
uses: actions/setup-python@v5
with:
python-version: 3.8

- name: Cache pip dependencies
uses: actions/cache@v2
with:
path: ~/.cache/pip
key: |
pip-${{ hashFiles('sdk/requirements.txt') }}-${{ hashFiles('sdk/requirements.dev.txt') }}
restore-keys: pip-
cache: pip
cache-dependency-path: |
sdk/requirements.txt
sdk/requirements.dev.txt
engines/pyfunc-ensembler-service/requirements.dev.txt

- name: Build
working-directory: ./sdk
Expand Down
Loading
Loading