-
Notifications
You must be signed in to change notification settings - Fork 5
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
Docker all the things! #2
base: master
Are you sure you want to change the base?
Conversation
@@ -11,7 +11,7 @@ | |||
*/ | |||
"AzureAd": { | |||
"Instance": "https://login.microsoftonline.com/", | |||
"Domain": "commerce.zunes.me", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like I missed one.
Member member = ctx.Members.First(m => m.ZuneTag == zuneTag); | ||
if (member == null) | ||
return StatusCode(StatusCodes.Status400BadRequest, $"User {zuneTag} does not exist."); | ||
// using var ctx = new ZuneNetContext(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not compile right now. Do you have a branch where it does, that I should rebase against?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, I just haven't touched messaging in a long time. Don't worry about it
Zune.DB.Console/Program.cs
Outdated
@@ -15,7 +15,7 @@ class Program | |||
static async Task Main(string[] args) | |||
{ | |||
// Set up CLI options | |||
string connectionString = "mongodb://localhost:27017"; | |||
string connectionString = "mongodb://mongo:27017"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be pulled into IConfiguration
so that it can be one of many configuration types including environment variables.
@@ -1,6 +1,6 @@ | |||
{ | |||
"ZuneNetContext": { | |||
"ConnectionString": "mongodb://localhost:27017", | |||
"ConnectionString": "mongodb://mongo:27017", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In docker, you can expose things like connection strings to a running container as an environment variables. I'm gonna probably revert this to localhost and define the runtime connection strings in docker-compose.yml
WORKDIR /source | ||
|
||
# copy csproj and restore as distinct layers | ||
COPY ./ . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally this should be broken into layers of dependency. I need to research the right way to do this for a .net project like this.
version: '3.4' | ||
|
||
services: | ||
mongodb: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably need to see if this can start and finish booting before the rest of the containers start
…to recover your own SID from the POST and add it to your profile to use it though.
… availble in all build targets.
also Parallel.ForEachAsync is cool. also albumart sort of loads
…someday, ill add the restart policy
…same resizing routines for the v3.2 endpoints
…en the tracking gif is missing by adding one.
- actual image resizing - MIX should now be returning something sensible
I still need to hook up all the networking, but
docker compose build
(implementing your constants, obviously) builds, anddocker compose up
starts all the services.