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.

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

V1.3.5 release #755

Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
931420d
Updated version number to 1.3.0 and added changelog file for 1.3.0, t…
downiec Nov 19, 2024
e6b309f
Staticbuild v1.3.0 update (#690)
downiec Jan 16, 2025
e9cc9ae
Fixed some linting errors related to unused imports
downiec Jan 16, 2025
f2fcf4d
Some minor modifications to the pre-commit files, including removing …
downiec Jan 16, 2025
3b6e1d1
Fix broken link in Globus transfer panel (#719)
bstrdsmkr Jan 22, 2025
031d80d
V1.3.0 additional fixes (#721)
downiec Jan 27, 2025
549f1f9
Dark mode (#718)
bstrdsmkr Jan 28, 2025
aa33ccc
Merge remote-tracking branch 'origin/master' into v1.3.0-release
downiec Jan 29, 2025
d581b83
Custom footer (#727)
downiec Jan 29, 2025
786785a
V1.3.1 release
downiec Jan 29, 2025
0b34d88
Traefik for prod (#734)
bstrdsmkr Feb 10, 2025
69825a3
Helm deployment fixes (#735)
downiec Feb 15, 2025
aa20375
Minor fix that was missed for changing duplicate Originals and Replic…
downiec Feb 15, 2025
d3d0d5d
Initial commit for version 1.3.2. Includes a refactoring of how messa…
downiec Feb 19, 2025
c644ef2
Fixed minor issues with helm and wget downloads.
downiec Feb 19, 2025
3a1abe6
Updated management script to properly quit if tests fail. Updated do_…
downiec Feb 20, 2025
ca83511
Initial v1.3.3 release version update, includes changes to News displ…
downiec Feb 20, 2025
782c34a
Added a dev action that will automatically generate necessary changes…
downiec Feb 20, 2025
de1ea6d
Initial v1.3.4 commit includes fix for path updating to proper value.
downiec Feb 21, 2025
60ddee7
Fixed linting error, tests regard changes to path and added some extr…
downiec Feb 21, 2025
8fb9968
Fixed linting error, tests regard changes to path and added some extr…
downiec Feb 21, 2025
bb1b569
Updated the way paths are set so that an auth step is included if no …
downiec Feb 21, 2025
930d05a
Updated change log to include mention of how path set functionality w…
downiec Feb 21, 2025
25fd8ff
Add hash annotation to Deployments to ensure restart when config chan…
bstrdsmkr Feb 24, 2025
6fe3157
Removed unnecessary settings from the config helper.
downiec Feb 24, 2025
8ebdbaf
Updated app version number for the Helm chart, added step in updateVe…
downiec Feb 25, 2025
a10e494
Fixes helm & containers (#744)
jasonb5 Feb 27, 2025
bbe8565
Feature/v1.3.4 minor updates (#747)
downiec Feb 27, 2025
58fe923
V1.3.4 patch (#749)
downiec Mar 6, 2025
f35ed7e
Update version used in README.md for helm
downiec Mar 6, 2025
ab05132
Initial update to version numbers.
downiec Mar 6, 2025
e1e75eb
Improve cart items and selections synchronization (#752)
downiec Mar 11, 2025
d7c0662
Fixes the JSON buttons by updating the clickableRoute function (#757)
downiec Mar 13, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
27 changes: 11 additions & 16 deletions .github/workflows/backend.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,22 +10,17 @@ defaults:
working-directory: backend

env:
DJANGO_SETTINGS_MODULE: metagrid.config.local
DOMAIN_NAME: http://localhost:8000
DJANGO_SECURE_SSL_REDIRECT: False
CORS_ORIGIN_WHITELIST: http://localhost:3000
KEYCLOAK_URL: https://esgf-login.ceda.ac.uk/
KEYCLOAK_REALM: esgf
KEYCLOAK_CLIENT_ID: metagrid-localhost
DATABASE_URL: pgsql://postgres:postgres@localhost:5432/postgres
REACT_APP_SEARCH_URL: https://esgf-node.llnl.gov/esg-search/search
REACT_APP_WGET_API_URL: https://esgf-node.llnl.gov/esg-search/wget
REACT_APP_ESGF_NODE_STATUS_URL: https://aims2.llnl.gov/metagrid-backend/proxy/status
REACT_APP_ESGF_SOLR_URL: https://esgf-fedtest.llnl.gov/solr
DJANGO_LOGIN_REDIRECT_URL: http://localhost:3000/search
DJANGO_LOGOUT_REDIRECT_URL: http://localhost:3000/search
GLOBUS_CLIENT_KEY: ${{ secrets.GLOBUS_KEY }}
GLOBUS_CLIENT_SECRET: ${{ secrets.GLOBUS_SECRET }}
DJANGO_SETTINGS_MODULE: config.settings
PGHOST: localhost
PGPASSWORD: postgres
PGUSER: postgres
METAGRID_SEARCH_URL: https://esgf-node.llnl.gov/esg-search/search
METAGRID_WGET_URL: https://esgf-node.llnl.gov/esg-search/wget
METAGRID_STATUS_URL: https://esgf-node.llnl.gov/proxy/status
METAGRID_SOLR_URL: https://esgf-node.llnl.gov/esg-search
METAGRID_SOCIAL_AUTH_GLOBUS_KEY: ${{ secrets.GLOBUS_KEY }}
METAGRID_SOCIAL_AUTH_GLOBUS_SECRET: ${{ secrets.GLOBUS_SECRET }}
METAGRID_GLOBUS_CLIENT_ID: 21982de0-2b7a-4ba8-bef5-5606ae098201

jobs:
build:
Expand Down
42 changes: 36 additions & 6 deletions .github/workflows/containers.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ on:
- 'v*'

pull_request:
branches:
- '*-release'

workflow_dispatch:

Expand All @@ -32,14 +34,15 @@ jobs:
with:
images: ghcr.io/esgf2-us/metagrid-frontend
tags: |
type=semver,pattern={{version}}
type=ref,event=pr
type=ref,event=tag
- uses: docker/build-push-action@v4
with:
cache-from: type=gha
cache-to: type=gha,mode=max
context: frontend/
file: frontend/docker/production/react/Dockerfile
push: ${{ github.event_name != 'pull_request' }}
file: frontend/Dockerfile
push: true
tags: ${{ steps.metadata.outputs.tags }}
labels: ${{ steps.metadata.outputs.labels }}
backend:
Expand All @@ -60,13 +63,40 @@ jobs:
with:
images: ghcr.io/esgf2-us/metagrid-backend
tags: |
type=semver,pattern={{version}}
type=ref,event=pr
type=ref,event=tag
- uses: docker/build-push-action@v4
with:
cache-from: type=gha
cache-to: type=gha,mode=max
context: backend/
file: backend/docker/production/django/Dockerfile
push: ${{ github.event_name != 'pull_request' }}
file: backend/Dockerfile
push: true
tags: ${{ steps.metadata.outputs.tags }}
labels: ${{ steps.metadata.outputs.labels }}
test:
name: Test Helm Chart
runs-on: ubuntu-latest
needs: [frontend, backend]
if: github.event_name == 'pull_request'
steps:
- uses: actions/checkout@v4
- name: Start minikube
uses: medyagh/setup-minikube@latest
- name: Set up Helm
uses: azure/[email protected]
- name: Install Helm Chart
working-directory: helm/
run: |
PR_NUMBER=$(echo ${GITHUB_REF} | awk -F'/' '{print $3}')

helm repo add bitnami https://charts.bitnami.com/bitnami

helm dependency build

helm install test . \
--set frontend.image.tag=pr-${PR_NUMBER} \
--set backend.image.tag=pr-${PR_NUMBER} \
--wait

kubectl get pods
16 changes: 3 additions & 13 deletions .github/workflows/frontend.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,21 +12,16 @@ defaults:
run:
working-directory: frontend

env:
REACT_APP_KEYCLOAK_REALM: esgf
REACT_APP_KEYCLOAK_URL: https://esgf-login.ceda.ac.uk/
REACT_APP_KEYCLOAK_CLIENT_ID: metagrid-localhost

jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Use Node.js 22.x
- name: Use Node.js 23.x
uses: actions/setup-node@v4
with:
node-version: "22.x"
node-version: "23.x"

- name: Cache node modules
uses: actions/cache@v4
Expand All @@ -46,13 +41,8 @@ jobs:
- name: Run Tests
env:
RELEASE: dev
ENV_FILE: .envs/.react
HTML_PATH: public
run: |
# Replaces react-scripts substitution during build for index.html and generates runtime_env.js
docker/production/react/entrypoint

yarn test:coverage
run: yarn test:coverage

- name: Upload Coverage Report
uses: codecov/codecov-action@v3
Expand Down
26 changes: 26 additions & 0 deletions .github/workflows/helm-chart.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
name: Helm chart

on:
push:
tags:
- 'v*'

jobs:
publish:
name: Publish
runs-on: ubuntu-latest
permissions:
packages: write
steps:
- uses: actions/checkout@v4
- name: Chart | Push
uses: appany/[email protected]
with:
name: metagrid
repository: esgf2-us
tag: ${{ github.ref_name }}
path: helm
registry: ghcr.io
registry_username: ${{ github.actor }}
registry_password: ${{ secrets.GITHUB_TOKEN }}
update_dependencies: 'true'
20 changes: 10 additions & 10 deletions .github/workflows/pre-commit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,17 +6,17 @@ on:
branches: [master]

jobs:
pre-commit-hooks:
pre-commit:
runs-on: ubuntu-latest
steps:
- name: Checkout Code Repository
- name: Checkout code
uses: actions/checkout@v4

# Required to run the local ESLint hook
- name: Use Node.js 22.x
uses: actions/setup-node@v4
with:
node-version: "22.x"
node-version: '22.x'

- name: Cache node modules
uses: actions/cache@v4
Expand All @@ -36,12 +36,12 @@ jobs:
yarn install --frozen-lockfile

- name: Set up Python
uses: actions/setup-python@v4
uses: actions/setup-python@v5
with:
python-version: 3.11
python-version: '3.11'

# Run all pre-commit hooks on all the files.
# Getting only staged files can be tricky in case a new PR is opened
# since the action is run on a branch in detached head state
- name: Install and Run Pre-commit
uses: pre-commit/[email protected]
- name: Install requirements
run: pip install -r backend/requirements/local.txt

- name: Run pre-commit
run: pre-commit run --all-files
13 changes: 7 additions & 6 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ metagrid_configs/backups
# .nfs files are created when an open file is removed but is still being accessed
.nfs*


### VisualStudioCode template
**/.vscode/*

Expand Down Expand Up @@ -80,8 +79,6 @@ crashlytics.properties
crashlytics-build.properties
fabric.properties



### Windows template
# Windows thumbnail cache files
Thumbs.db
Expand All @@ -106,7 +103,6 @@ $RECYCLE.BIN/
# Windows shortcuts
*.lnk


### macOS template
# General
*.DS_Store
Expand Down Expand Up @@ -135,7 +131,6 @@ Network Trash Folder
Temporary Items
.apdisk


### SublimeText template
# Cache files for Sublime Text
*.tmlanguage.cache
Expand Down Expand Up @@ -168,7 +163,6 @@ bh_unicode_properties.cache
# https://packagecontrol.io/packages/sublime-github
GitHub.sublime-settings


### Vim template
# Swap
[._]*.s[a-v][a-z]
Expand All @@ -189,4 +183,11 @@ tags
.env
.envs/*
!.envs/.local/
docker-compose*-overlay.yml

# Documentation artifacts
docs/site
frontend/public/runtime_env.js

#Helm chart deps
helm/charts/*
29 changes: 25 additions & 4 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
exclude: "docs|node_modules|migrations|.git|.tox"
exclude: 'docs|node_modules|migrations|.git|.tox'
default_stages: [commit]
fail_fast: true

Expand All @@ -9,27 +9,48 @@ repos:
- id: trailing-whitespace
- id: end-of-file-fixer
- id: check-yaml
exclude: traefik/*.yml|helm/templates/.*

# Back-end
# ------------------------------------------------------------------------------
- repo: https://github.com/pycqa/flake8
rev: 7.1.1
hooks:
- id: flake8
args: ["--config=backend/setup.cfg"]
args: ['--config=backend/setup.cfg']
additional_dependencies: [flake8-isort]

- repo: https://github.com/psf/black
rev: 24.10.0
hooks:
- id: black
args: ["--config=backend/pyproject.toml"]
args: ['--config=backend/pyproject.toml']

- repo: https://github.com/pre-commit/mirrors-mypy
rev: v1.12.0
hooks:
- id: mypy
additional_dependencies: ["types-requests"]
additional_dependencies: ['types-requests']

- repo: https://github.com/radeklat/settings-doc
rev: '4.3.1'
hooks:
- id: settings-doc-markdown
name: Generate backend settings markdown
language: system
entry: env PYTHONPATH=$PYTHONPATH:backend settings-doc generate --output-format markdown
args:
- '--class'
- 'config.settings.site_specific.MetagridBackendSettings'
- '--update'
- 'docs/docs/users/configurable_environment_variables.md'
- '--between'
- '<!-- start generated backend settings markdown -->'
- '<!-- end generated backend settings markdown -->'
- '--templates'
- 'docs/custom_templates'
- '--heading-offset'
- '3'

# Front-end
# ------------------------------------------------------------------------------
Expand Down
19 changes: 11 additions & 8 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -53,16 +53,18 @@
"typescript.updateImportsOnFileMove.enabled": "always",
// ESLint
// -----------------------------
"eslint.validate": [
"javascript",
"javascriptreact",
"typescript",
"typescriptreact"
],
"eslint.validate": ["javascript", "javascriptreact", "typescript", "typescriptreact"],
"eslint.workingDirectories": ["./frontend/"],
// Jest
// -----------------------------
"jest.pathToJest": "yarn test:watch",
"jest.rootPath": "./frontend/src",
"jest.jestCommandLine": "yarn jest",
"jest.monitorLongRun": "off",
"jest.enable": true,
"jest.runMode": {
"type": "on-demand",
"coverage": true
},
// HTML, CSS, JSON
// -----------------------------
"[html]": {
Expand All @@ -78,5 +80,6 @@
"editor.defaultFormatter": "esbenp.prettier-vscode"
},
"python.analysis.extraPaths": ["backend/venv/bin/python"],
"editor.formatOnSave": true
"python.testing.pytestEnabled": true,
"python.testing.pytestArgs": ["backend"]
}
1 change: 0 additions & 1 deletion backend/.dockerignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
.*
!.coveragerc
!.env
!.pylintrc
Loading