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

Read application properties from env first and add Docker configuration. #32

Merged
merged 3 commits into from
Sep 11, 2024

Conversation

G8XSU
Copy link
Collaborator

@G8XSU G8XSU commented Sep 6, 2024

  • Read application properties from env first.
    Prioritize environment variables over configuration file properties.
    This allows the service to adapt dynamically to different deployment
    environments by overriding config file settings with environment-specific
    values when available.

  • Change default war filename to vss.

  • Add Docker configuration.
    Add docker-compose and Dockerfile, which spins up a postgres and
    vss instance. We override env variables such as vss.jdbc.url,
    to point to postgres container.

Prioritize environment variables over configuration file properties.
This allows the service to adapt dynamically to different deployment
environments by overriding config file settings with environment-specific
values when available.
@G8XSU G8XSU changed the title Add Docker configuration. Read application properties from env first. & Add Docker configuration. Sep 6, 2024
@G8XSU G8XSU changed the title Read application properties from env first. & Add Docker configuration. Read application properties from env first and add Docker configuration. Sep 6, 2024
@G8XSU G8XSU requested a review from jkczyz September 6, 2024 23:13
Add docker-compose and Dockerfile, which spins up a postgres and
vss instance. We override env variables such as vss.jdbc.url,
to point to postgres container.
@G8XSU G8XSU merged commit 5f917b0 into lightningdevkit:main Sep 11, 2024
1 check passed
This was referenced Sep 11, 2024
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.

2 participants