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

[chore] Update setup.sh, with removed replacement script & added project-level ENVs #2115

Merged
merged 19 commits into from
Sep 13, 2023

Conversation

henit-chobisa
Copy link
Collaborator

@henit-chobisa henit-chobisa commented Sep 7, 2023

Tasks

  • Updated setup.sh file to split off frontend, backend and space environment variables.
  • Removed Replacement Script in Start.sh, Dockerfile.space and Dockerfile.web
  • Removed Environment Variables from docker-compose.yml, file based env variables are given
  • Added .env.example for apiserver as well

@vercel
Copy link

vercel bot commented Sep 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

6 Ignored Deployments
Name Status Preview Comments Updated (UTC)
plane ⬜️ Ignored (Inspect) Visit Preview Sep 13, 2023 2:50pm
plane-dev ⬜️ Ignored (Inspect) Visit Preview Sep 13, 2023 2:50pm
plane-sh ⬜️ Ignored (Inspect) Visit Preview Sep 13, 2023 2:50pm
plane-sh-dev ⬜️ Ignored (Inspect) Visit Preview Sep 13, 2023 2:50pm
plane-sh-stage ⬜️ Ignored (Inspect) Visit Preview Sep 13, 2023 2:50pm
plane-staging ⬜️ Ignored (Inspect) Visit Preview Sep 13, 2023 2:50pm

setup.sh Outdated Show resolved Hide resolved
setup.sh Outdated Show resolved Hide resolved
setup.sh Outdated Show resolved Hide resolved
setup.sh Outdated Show resolved Hide resolved
docker-compose.yml Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
@sriramveeraghanta sriramveeraghanta marked this pull request as draft September 7, 2023 09:42
@henit-chobisa henit-chobisa marked this pull request as ready for review September 10, 2023 14:35
@henit-chobisa
Copy link
Collaborator Author

The above PR is coupled with the following PR #2135

Copy link

@TheZoker TheZoker left a comment

Choose a reason for hiding this comment

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

The docker-compose-hub.yml file also needs to be updated

…y & decoupled Plane Deploy URL generation for web (#2135)

* chore: removed api url build arg from compose

* chore: set public api default argument to black string for self hosted

* chore: updated web services to accept blank string as API URL

* chore: added env variables for pg compose service

* chore: modified space app services to use accept empty string as api base

* chore: conditionally trigger web url value based on argument

* fix: made web to use identical host with spaces suffix on absense of Deploy URL for deploy

* chore: added example env for PUBLIC_DEPLOY Env

* chore: updated web dockerfile with addition as PLANE_DEPLOY Argument

* API BASE URL global update

* API BASE URL replace with api server

* api base url fixes

* typo  fixes

---------

Co-authored-by: sriram veeraghanta <[email protected]>
@sriramveeraghanta sriramveeraghanta merged commit d0f6ca3 into develop Sep 13, 2023
1 of 2 checks passed
@sriramveeraghanta sriramveeraghanta deleted the chore/update-setup-script branch September 13, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants