Skip to content

Commit

Permalink
Remove UI
Browse files Browse the repository at this point in the history
  • Loading branch information
milenkovicm committed Oct 11, 2024
1 parent 2f223db commit 839c5fc
Show file tree
Hide file tree
Showing 40 changed files with 4 additions and 12,815 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,5 @@ jobs:
'{ballista,docs}/**/*.md' \
'!ballista/CHANGELOG.md' \
README.md \
CONTRIBUTING.md \
'ballista/**/*.{ts,tsx}'
CONTRIBUTING.md
git diff --exit-code
46 changes: 1 addition & 45 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,44 +72,6 @@ jobs:
/github/home/target/release/ballista-scheduler
/github/home/target/release/ballista-executor
react-build:
name: React build
runs-on: ubuntu-latest # proprietary github image, not ubuntu:latest
container:
image: ubuntu:latest # actual ubuntu:latest that ubuntu publishes
env:
DEBIAN_FRONTEND: "noninteractive"
steps:
- uses: actions/checkout@v3
- name: Install dependencies
run: |
cat /etc/*-release
apt-get -qq update
apt-get -qq upgrade
apt-get -qq install -y curl
curl -fsSL https://deb.nodesource.com/setup_18.x | bash
apt-get -qq update
apt-get -qq install -y nodejs
npm install -g yarn
which node
which npm
which yarn
node --version
npm --version
yarn --version
- name: Run yarn build
run: |
cd ballista/scheduler/ui
pwd
yarn install
yarn build
- name: Save artifacts
uses: actions/upload-artifact@v4
with:
name: react-artifacts
path: |
ballista/scheduler/ui/build
# test the crate
linux-test:
name: Test Workspace on AMD64 Rust ${{ matrix.rust }}
Expand Down Expand Up @@ -338,7 +300,7 @@ jobs:

docker:
name: Docker
needs: [linux-build-lib, react-build]
needs: [linux-build-lib]
runs-on: ubuntu-latest
permissions:
contents: read
Expand All @@ -350,11 +312,6 @@ jobs:
with:
name: rust-artifacts
path: target/release
- name: Restore react artifacts
uses: actions/download-artifact@v4
with:
name: react-artifacts
path: ballista/scheduler/ui/build
- name: Build and push Docker image
run: |
echo "github user is $DOCKER_USER"
Expand Down Expand Up @@ -468,7 +425,6 @@ jobs:
env:
CARGO_HOME: "/github/home/.cargo"
CARGO_TARGET_DIR: "/github/home/target"

# Coverage job was failing. https://github.com/apache/arrow-datafusion/issues/590 tracks re-instating it

# coverage:
Expand Down
4 changes: 0 additions & 4 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -95,10 +95,6 @@ for the executor becomes `BALLISTA_EXECUTOR_SCHEDULER_HOST`

Refer to the instructions in the Python Bindings [README](./python/README.md)

### Javascript Environment

Refer to the instructions in the Scheduler Web UI [README](./ballista/scheduler/ui/README.md)

## Integration Tests

The integration tests can be executed by running the following command from the root of the repository.
Expand Down
2 changes: 0 additions & 2 deletions ballista/scheduler/ui/.dockerignore

This file was deleted.

23 changes: 0 additions & 23 deletions ballista/scheduler/ui/.gitignore

This file was deleted.

61 changes: 0 additions & 61 deletions ballista/scheduler/ui/README.md

This file was deleted.

18 changes: 0 additions & 18 deletions ballista/scheduler/ui/index.d.ts

This file was deleted.

60 changes: 0 additions & 60 deletions ballista/scheduler/ui/package.json

This file was deleted.

Binary file removed ballista/scheduler/ui/public/favicon.ico
Binary file not shown.
62 changes: 0 additions & 62 deletions ballista/scheduler/ui/public/index.html

This file was deleted.

Binary file removed ballista/scheduler/ui/public/logo192.png
Binary file not shown.
Binary file removed ballista/scheduler/ui/public/logo512.png
Binary file not shown.
25 changes: 0 additions & 25 deletions ballista/scheduler/ui/public/manifest.json

This file was deleted.

20 changes: 0 additions & 20 deletions ballista/scheduler/ui/public/robots.txt

This file was deleted.

Loading

0 comments on commit 839c5fc

Please sign in to comment.