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

Add Information about env vars into documentation #758

Closed
wants to merge 1 commit into from

Conversation

kaystrobach
Copy link

@kaystrobach kaystrobach commented Mar 28, 2018

The Problem/Issue/Bug:

Not easy to find how to change / add env vars manually.

How this PR Solves The Problem:

add documentation

Manual Testing Instructions:

read documentation

Automated Testing Overview:

no

Related Issue Link(s):

Release/Deployment notes:

@CLAassistant
Copy link

CLAassistant commented Mar 28, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

My initial reading suggests that this needs to wait for a more mature approach by ddev to environment variables, as in #757. However, I'd love to find a home for this approach, probably not the main docs though.

@@ -55,3 +55,43 @@ To load the new configuration, run `ddev restart`.

## Overriding default container images
The default container images provided by ddev are defined in the `config.yaml` file in the `.ddev` folder of your project. This means that _defining_ an alternative image for default services is as simple as changing the image definition in `config.yaml`. In practice, however, ddev currently has certain expectations and assumptions for what the web and database containers provide. At this time, it is recommended that the default container projects be referenced or used as a starting point for developing an alternative image. If you encounter difficulties integrating alternative images, please [file an issue and let us know](https://github.com/drud/ddev/issues/new).

## Adding environment variables
Copy link
Member

Choose a reason for hiding this comment

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

ddev is used for many different CMSs, dotenv-connector is only for TYPO3, right? That implies that we have to find a better place for this. Also, we're hoping for a much more general solution in Make an easy way to populate environment variables in web container.

Copy link
Author

Choose a reason for hiding this comment

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

dotenv is for any composer based project - not TYPO3 related

@rfay
Copy link
Member

rfay commented Mar 28, 2018

Oh and thanks so much for your contribution and reaching out with another perspective!

Please do update the title and fill in the PR to make the case for this, even though IMO it should land elsewhere.

@kaystrobach kaystrobach changed the title Update customization-extendibility.md Add Information about env vars into documentation Mar 28, 2018
@rickmanelius
Copy link
Contributor

Agreed. Let's leave this PR open until we find a proper home in #659

@rfay
Copy link
Member

rfay commented Apr 19, 2018

We're about to make the main docker-compose.yaml ddev-managed, meaning that users will be making a mistake by editing it. We plan to allow it, but it's not going to be mainline. We also plan to allow explicit setting of environment variables in .ddev/config.yaml, #757 .

With your permission I'd like to close this in favor of those future paths.

@rfay rfay closed this Apr 19, 2018
@kaystrobach
Copy link
Author

sounds good.

rfay added a commit to rfay/ddev that referenced this pull request Apr 30, 2018
@rfay
Copy link
Member

rfay commented Apr 30, 2018

@kaystrobach I hope you can take a look at #816 and see what you think. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants