Skip to content

Commit

Permalink
Merge pull request #10822 from Budibase/fix/single-image-top-level-path
Browse files Browse the repository at this point in the history
Fix single image top level path
  • Loading branch information
mike12345567 committed Jun 7, 2023
2 parents aaf5f6a + f7d8e79 commit 4a532ed
Show file tree
Hide file tree
Showing 11 changed files with 33 additions and 56 deletions.
13 changes: 1 addition & 12 deletions .github/workflows/release-develop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,18 +48,7 @@ jobs:

- run: yarn install --frozen-lockfile
- name: Update versions
run: |
version=$(cat lerna.json \
| grep version \
| head -1 \
| awk -F: '{gsub(/"/,"",$2);gsub(/[[:space:]]*/,"",$2); print $2}' \
| sed 's/[",]//g')
echo "Setting version $version"
yarn lerna exec "yarn version --no-git-tag-version --new-version=$version"
echo "Updating dependencies"
node scripts/syncLocalDependencies.js $version
echo "Syncing yarn workspace"
yarn
run: ./scripts/updateVersions.sh
- run: yarn build
- run: yarn build:sdk

Expand Down
13 changes: 1 addition & 12 deletions .github/workflows/release-master.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,18 +41,7 @@ jobs:

- run: yarn install --frozen-lockfile
- name: Update versions
run: |
version=$(cat lerna.json \
| grep version \
| head -1 \
| awk -F: '{gsub(/"/,"",$2);gsub(/[[:space:]]*/,"",$2); print $2}' \
| sed 's/[",]//g')
echo "Setting version $version"
yarn lerna exec "yarn version --no-git-tag-version --new-version=$version"
echo "Updating dependencies"
node scripts/syncLocalDependencies.js $version
echo "Syncing yarn workspace"
yarn
run: ./scripts/updateVersions.sh
- run: yarn lint
- run: yarn build
- run: yarn build:sdk
Expand Down
26 changes: 2 additions & 24 deletions .github/workflows/release-singleimage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,33 +47,11 @@ jobs:
- name: Run Yarn
run: yarn
- name: Update versions
run: |
version=$(cat lerna.json \
| grep version \
| head -1 \
| awk -F: '{gsub(/"/,"",$2);gsub(/[[:space:]]*/,"",$2); print $2}' \
| sed 's/[",]//g')
echo "Setting version $version"
yarn lerna exec "yarn version --no-git-tag-version --new-version=$version"
echo "Updating dependencies"
node scripts/syncLocalDependencies.js $version
echo "Syncing yarn workspace"
yarn
run: ./scripts/updateVersions.sh
- name: Runt Yarn Lint
run: yarn lint
- name: Update versions
run: |
version=$(cat lerna.json \
| grep version \
| head -1 \
| awk -F: '{gsub(/"/,"",$2);gsub(/[[:space:]]*/,"",$2); print $2}' \
| sed 's/[",]//g')
echo "Setting version $version"
yarn lerna exec "yarn version --no-git-tag-version --new-version=$version"
echo "Updating dependencies"
node scripts/syncLocalDependencies.js $version
echo "Syncing yarn workspace"
yarn
run: ./scripts/updateVersions.sh
- name: Run Yarn Build
run: yarn build:docker:pre
- name: Login to Docker Hub
Expand Down
1 change: 1 addition & 0 deletions hosting/single/runner.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ declare -a DOCKER_VARS=("APP_PORT" "APPS_URL" "ARCHITECTURE" "BUDIBASE_ENVIRONME
[[ -z "${WORKER_PORT}" ]] && export WORKER_PORT=4002
[[ -z "${WORKER_URL}" ]] && export WORKER_URL=http://localhost:4002
[[ -z "${APPS_URL}" ]] && export APPS_URL=http://localhost:4001
[[ -z "${SERVER_TOP_LEVEL_PATH}" ]] && export SERVER_TOP_LEVEL_PATH=/app
# export CUSTOM_DOMAIN=budi001.custom.com

# Azure App Service customisations
Expand Down
2 changes: 1 addition & 1 deletion packages/server/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ ENV SERVICE=app-service
ENV POSTHOG_TOKEN=phc_bIjZL7oh2GEUd2vqvTBH8WvrX0fWTFQMs6H5KQxiUxU
ENV TENANT_FEATURE_FLAGS=*:LICENSING,*:USER_GROUPS,*:ONBOARDING_TOUR
ENV ACCOUNT_PORTAL_URL=https://account.budibase.app

ENV TOP_LEVEL_PATH=/

# handle node-gyp
RUN apt-get update \
Expand Down
6 changes: 3 additions & 3 deletions packages/server/src/api/controllers/static/index.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
require("svelte/register")

import { resolve, join } from "../../../utilities/centralPath"
import { join } from "../../../utilities/centralPath"
const uuid = require("uuid")
import { ObjectStoreBuckets } from "../../../constants"
import { processString } from "@budibase/string-templates"
Expand Down Expand Up @@ -49,7 +49,7 @@ export const toggleBetaUiFeature = async function (ctx: any) {
return
}

let builderPath = resolve(TOP_LEVEL_PATH, "new_design_ui")
let builderPath = join(TOP_LEVEL_PATH, "new_design_ui")

// // download it from S3
if (!fs.existsSync(builderPath)) {
Expand All @@ -67,7 +67,7 @@ export const toggleBetaUiFeature = async function (ctx: any) {
}

export const serveBuilder = async function (ctx: any) {
const builderPath = resolve(TOP_LEVEL_PATH, "builder")
const builderPath = join(TOP_LEVEL_PATH, "builder")
await send(ctx, ctx.file, { root: builderPath })
}

Expand Down
3 changes: 2 additions & 1 deletion packages/server/src/environment.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,8 @@ const environment = {
isInThread: () => {
return process.env.FORKED_PROCESS
},
TOP_LEVEL_PATH: process.env.TOP_LEVEL_PATH,
TOP_LEVEL_PATH:
process.env.TOP_LEVEL_PATH || process.env.SERVER_TOP_LEVEL_PATH,
}

// threading can cause memory issues with node-ts in development
Expand Down
4 changes: 2 additions & 2 deletions packages/server/src/utilities/fileSystem/filesystem.ts
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
import { PathLike } from "fs"
import fs from "fs"
import { budibaseTempDir } from "../budibaseDir"
import { join } from "path"
import { resolve, join } from "path"
import env from "../../environment"
import tar from "tar"
import environment from "../../environment"
const uuid = require("uuid/v4")

export const TOP_LEVEL_PATH =
environment.TOP_LEVEL_PATH || join(__dirname, "..", "..", "..")
environment.TOP_LEVEL_PATH || resolve(join(__dirname, "..", "..", ".."))

/**
* Upon first startup of instance there may not be everything we need in tmp directory, set it up.
Expand Down
2 changes: 1 addition & 1 deletion packages/worker/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -104,4 +104,4 @@
"update-dotenv": "1.1.1"
},
"gitHead": "d1836a898cab3f8ab80ee6d8f42be1a9eed7dcdc"
}
}
7 changes: 7 additions & 0 deletions scripts/resetVersions.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/bin/bash
echo "Resetting package versions"
yarn lerna exec "yarn version --no-git-tag-version --new-version=0.0.0"
echo "Updating dependencies"
node scripts/syncLocalDependencies.js "0.0.0"
git checkout package.json
echo "Package versions reset!"
12 changes: 12 additions & 0 deletions scripts/updateVersions.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
#!/bin/bash
version=$(cat lerna.json \
| grep version \
| head -1 \
| awk -F: '{gsub(/"/,"",$2);gsub(/[[:space:]]*/,"",$2); print $2}' \
| sed 's/[",]//g')
echo "Setting version $version"
yarn lerna exec "yarn version --no-git-tag-version --new-version=$version"
echo "Updating dependencies"
node scripts/syncLocalDependencies.js $version
echo "Syncing yarn workspace"
yarn

0 comments on commit 4a532ed

Please sign in to comment.