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.

Sign up for GitHub

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

Jump to bottom

Update DB Variables to be ENV Variables #18

Merged
merged 3 commits into from Mar 10, 2021
Merged

Update DB Variables to be ENV Variables #18

merged 3 commits into from Mar 10, 2021

Conversation

jtdub
Copy link
Contributor

@jtdub jtdub commented Mar 9, 2021 •
edited

Fixes #16

This gives the power user the ability to utilize an external database for testing.

  • moved tzseeds.txt, supervisord.conf, and requirements.txt to the ./templates folder to reduce the number of COPY commands in the Dockerfile.
  • pb_nautobot_install.yml now sets up the supervisord.conf file.

itdependsnetworks
itdependsnetworks reviewed Mar 9, 2021
View reviewed changes
Dockerfile Outdated Show resolved Hide resolved
@jtdub
Copy link
Contributor Author

jtdub commented Mar 9, 2021

@itdependsnetworks Fixed!

@jtdub jtdub merged commit 0ed1147 into nautobot:main Mar 10, 2021
@jtdub jtdub deleted the env branch March 10, 2021 01:39
@jtdub jtdub mentioned this pull request Mar 10, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itdependsnetworks itdependsnetworks itdependsnetworks approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow for user defined database credentials
2 participants
@jtdub @itdependsnetworks