-
Notifications
You must be signed in to change notification settings - Fork 520
Seq memory limits, API key etc. #1608
Comments
hey you should be able to access the SEQ connection string for your config using the Question: Why do you need an API key for a local development when using SEQ? |
API key is used to change logging level at runtime and it can be used in production as well |
Are you attempting to use tye to deploy SEQ aswell? |
No, we use docker compose for deployment, but reuse tye environment variables at runtime. |
Cool I'll happily take a look, but I'm struggling to understand why you need a SEQ api key for local dev environment - I don't use SEQ in my lifecycle so any more information you can give me will help. We can easily extend the SEQ extension to include the other variables, but I'm uncertain whether including the APIkey value is worthwhile considering all config for SEQ may be removed when the container restarts, or when another dev in your project pulls and runs it. |
I have not investigated more why Seq connectionString is not available at other service start, but that is what I have seen when degugging a service. |
I think the reason why SEQ as an extension works is because it set's itself as a dependency for all other services
There are some suggestions in #486 and #652 to control the start-up order. As there is an API available to us I could look into supporting a depend_on set up that may allow your custom SEQ container to properly inject the variables. |
Yes, I thought it was a dependency problem as well, but are not the connection strings setup for all services before any one of them is run? |
Yeah I use SQL server with connection strings and it works. Where is your SEQ container in your tye file, is it at the bottom? |
Seq container yaml is included in app.yaml before other services. |
Ahh okay, I'll take a look and submit a PR for your use case as it seems very interesting! I'll also look into the depends on feature |
Hey I've had a look, and I'll write some more E2E tests to validate the approach, but I'm using already existing API's so hopefully it works as expected :) |
I have tried using Seq logging extension and it launches Seq docker container and our services log to it when properly configured.
But I have not found a way how to specify other parameters like:
To overcome it, I have tried this approach instead of the extension:
But while it launches Seq container, it does not propagate seq connectionString to our services where it should be used to configure Serilog with Seq.
Any ideas?
Thank You
The text was updated successfully, but these errors were encountered: