Merge pull request #22 from peppy/redis-default-host

Change default redis host to `localhost`
ppy · Dec 2, 2022 · f8ee6a8 · f8ee6a8
2 parents 4b20611 + 35c3ab9
commit f8ee6a8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -32,4 +32,6 @@ jobs:
       - name: Build
         run: dotnet build --no-restore
       - name: Test
+        env:
+          REDIS_HOST: redis
         run: dotnet test --no-restore --verbosity normal
diff --git a/osu.Server.QueueProcessor/QueueProcessor.cs b/osu.Server.QueueProcessor/QueueProcessor.cs
@@ -49,7 +49,7 @@ public abstract class QueueProcessor<T> where T : QueueItem
         /// An option queue to push to when finished.
         /// </summary>
         private readonly ConnectionMultiplexer redis = ConnectionMultiplexer.Connect(
-            Environment.GetEnvironmentVariable("REDIS_HOST") ?? "redis");
+            Environment.GetEnvironmentVariable("REDIS_HOST") ?? "localhost");
 
         private long totalProcessed;