From 25a74c32341ac643a9d72773dd51f47017cef1a7 Mon Sep 17 00:00:00 2001 From: Dominion Date: Sun, 14 May 2023 10:49:43 -0400 Subject: [PATCH 01/12] Add missing events to DMAPI --- build/Version.props | 2 +- src/DMAPI/tgs.dm | 8 +++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/build/Version.props b/build/Version.props index 2f853ecd6c..cc6fbcfcf7 100644 --- a/build/Version.props +++ b/build/Version.props @@ -8,7 +8,7 @@ 9.10.2 10.4.1 11.4.2 - 6.4.2 + 6.4.3 5.6.0 1.2.2 1.2.1 diff --git a/src/DMAPI/tgs.dm b/src/DMAPI/tgs.dm index c8ef71112b..c562224c73 100644 --- a/src/DMAPI/tgs.dm +++ b/src/DMAPI/tgs.dm @@ -1,6 +1,6 @@ // tgstation-server DMAPI -#define TGS_DMAPI_VERSION "6.4.2" +#define TGS_DMAPI_VERSION "6.4.3" // All functions and datums outside this document are subject to change with any version and should not be relied on. @@ -104,6 +104,12 @@ #define TGS_EVENT_WORLD_PRIME 21 // DMAPI also doesnt implement this // #define TGS_EVENT_DREAM_DAEMON_LAUNCH 22 +/// After a single submodule update is performed. Parameters: Updated submodule name +#define TGS_EVENT_REPO_SUBMODULE_UPDATE 23 +/// After CodeModifications are applied, before DreamMaker is run. Parameters: Game directory path, origin commit sha, byond version +#define TGS_EVENT_PRE_DREAM_MAKER 24 +/// Whenever a deployment folder is deleted from disk. Parameters: Game directory path +#define TGS_EVENT_DEPLOYMENT_CLEANUP 25 // OTHER ENUMS From a0d596e0acfbe6dfe0a79e85dffc10171de38d6b Mon Sep 17 00:00:00 2001 From: Dominion Date: Sun, 14 May 2023 11:03:43 -0400 Subject: [PATCH 02/12] CI fixes and updates - Install whatever package GitHub is complaining about - Bump CI BYOND to 1606 --- .github/workflows/ci-suite.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci-suite.yml b/.github/workflows/ci-suite.yml index bb514afdd4..c5d154e8e2 100644 --- a/.github/workflows/ci-suite.yml +++ b/.github/workflows/ci-suite.yml @@ -103,14 +103,14 @@ jobs: if: "!(cancelled() || failure()) && needs.start-ci-run-gate.result == 'success'" env: BYOND_MAJOR: 515 - BYOND_MINOR: 1592 + BYOND_MINOR: 1606 runs-on: ubuntu-latest steps: - name: Install x86 libc Dependencies run: | sudo dpkg --add-architecture i386 sudo apt-get update - sudo apt-get install -y -o APT::Immediate-Configure=0 libc6-i386 libstdc++6:i386 + sudo apt-get install -y -o APT::Immediate-Configure=0 libc6-i386 libstdc++6:i386 libgcc-s1:i386 - name: Install BYOND if: steps.cache-byond.outputs.cache-hit != 'true' @@ -436,7 +436,7 @@ jobs: run: | sudo dpkg --add-architecture i386 sudo apt-get update - sudo apt-get install -y -o APT::Immediate-Configure=0 libc6-i386 libstdc++6:i386 gdb + sudo apt-get install -y -o APT::Immediate-Configure=0 libc6-i386 libstdc++6:i386 gdb libgcc-s1:i386 - name: Install Node 12.X uses: actions/setup-node@v3 From cc74e8393959747b90e8e2a67a7aa7c9e771d455 Mon Sep 17 00:00:00 2001 From: Dominion Date: Sun, 14 May 2023 18:28:48 -0400 Subject: [PATCH 03/12] Fix erroneous messages regarding unparsable release tags --- .../Core/ServerUpdater.cs | 44 ++++++++++--------- 1 file changed, 23 insertions(+), 21 deletions(-) diff --git a/src/Tgstation.Server.Host/Core/ServerUpdater.cs b/src/Tgstation.Server.Host/Core/ServerUpdater.cs index 34660174f0..bbada322dc 100644 --- a/src/Tgstation.Server.Host/Core/ServerUpdater.cs +++ b/src/Tgstation.Server.Host/Core/ServerUpdater.cs @@ -110,31 +110,33 @@ namespace Tgstation.Server.Host.Core if (Version.TryParse( release.TagName.Replace( updatesConfiguration.GitTagPrefix, String.Empty, StringComparison.Ordinal), - out var version) - && version == newVersion) + out var version)) { - var asset = release.Assets.Where(x => x.Name.Equals(updatesConfiguration.UpdatePackageAssetName, StringComparison.Ordinal)).FirstOrDefault(); - if (asset == default) - continue; - - serverUpdateOperation = new ServerUpdateOperation + if (version == newVersion) { - TargetVersion = version, - UpdateZipUrl = new Uri(asset.BrowserDownloadUrl), - SwarmService = swarmService, - }; + var asset = release.Assets.Where(x => x.Name.Equals(updatesConfiguration.UpdatePackageAssetName, StringComparison.Ordinal)).FirstOrDefault(); + if (asset == default) + continue; - try - { - if (!serverControl.TryStartUpdate(this, version)) - return ServerUpdateResult.UpdateInProgress; + serverUpdateOperation = new ServerUpdateOperation + { + TargetVersion = version, + UpdateZipUrl = new Uri(asset.BrowserDownloadUrl), + SwarmService = swarmService, + }; + + try + { + if (!serverControl.TryStartUpdate(this, version)) + return ServerUpdateResult.UpdateInProgress; + } + finally + { + serverUpdateOperation = null; + } + + return ServerUpdateResult.Started; } - finally - { - serverUpdateOperation = null; - } - - return ServerUpdateResult.Started; } else logger.LogDebug("Unparsable release tag: {releaseTag}", release.TagName); From 4ee05fcd0c4dba7df5a4f591c4569b01362b32eb Mon Sep 17 00:00:00 2001 From: Dominion Date: Sun, 14 May 2023 19:46:42 -0400 Subject: [PATCH 04/12] Dare to upgrade libgit2 and remove libssl dep --- README.md | 1 - build/Dockerfile | 7 ------- src/Tgstation.Server.Host/Tgstation.Server.Host.csproj | 3 +-- 3 files changed, 1 insertion(+), 10 deletions(-) diff --git a/README.md b/README.md index c06f4c72ed..6d0687038e 100644 --- a/README.md +++ b/README.md @@ -43,7 +43,6 @@ The following dependencies are required to run tgstation-server on Linux alongsi - libc6-i386 - libstdc++6:i386 -- libssl1.0.0 - gdb (for using gcore to create core dumps) - gcc-multilib (Only on 64-bit systems) diff --git a/build/Dockerfile b/build/Dockerfile index a48a345099..c5baa2f3ff 100644 --- a/build/Dockerfile +++ b/build/Dockerfile @@ -65,13 +65,6 @@ RUN apt-get update \ && apt-get install -y \ gcc-multilib \ gdb \ - curl \ - && curl http://archive.ubuntu.com/ubuntu/pool/main/g/glibc/multiarch-support_2.27-3ubuntu1_amd64.deb --output multiarch-support_2.27.deb \ - && curl http://security.debian.org/debian-security/pool/updates/main/o/openssl/libssl1.0.0_1.0.1t-1+deb8u12_amd64.deb --output libssl1.0.0.deb \ - && dpkg -i multiarch-support_2.27.deb \ - && dpkg -i libssl1.0.0.deb \ - && rm multiarch-support_2.27.deb \ - && rm libssl1.0.0.deb \ && rm -rf /var/lib/apt/lists/* EXPOSE 5000 diff --git a/src/Tgstation.Server.Host/Tgstation.Server.Host.csproj b/src/Tgstation.Server.Host/Tgstation.Server.Host.csproj index 0730a00c27..9f7129e7df 100644 --- a/src/Tgstation.Server.Host/Tgstation.Server.Host.csproj +++ b/src/Tgstation.Server.Host/Tgstation.Server.Host.csproj @@ -74,8 +74,7 @@ - - + From d8aa40011b9cb083411d00d3195baa361d71c50b Mon Sep 17 00:00:00 2001 From: Dominion Date: Sun, 14 May 2023 21:53:09 -0400 Subject: [PATCH 05/12] Create a new connection for each test attempt --- tests/Tgstation.Server.Tests/Live/TestLiveServer.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs b/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs index b03b721b91..410b8ad8a0 100644 --- a/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs +++ b/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs @@ -120,13 +120,13 @@ namespace Tgstation.Server.Tests.Live } var connectionFactory = new DatabaseConnectionFactory(); - using var connection = connectionFactory.CreateConnection(connectionString, databaseType); using var cts = new CancellationTokenSource(TimeSpan.FromMinutes(1)); var cancellationToken = cts.Token; while (true) { + using var connection = connectionFactory.CreateConnection(connectionString, databaseType); try { await connection.OpenAsync(cancellationToken); From d86b4c8e331f1042093f3bf4a94216f8caa6e431 Mon Sep 17 00:00:00 2001 From: Dominion Date: Wed, 17 May 2023 18:57:10 -0400 Subject: [PATCH 06/12] Use the GitHubAccessToken when downloading the ServerUpdatePackage Workaround for https://github.com/actions/runner-images/issues/7007 --- .../Components/Byond/ByondInstallerBase.cs | 2 +- src/Tgstation.Server.Host/Core/ServerUpdater.cs | 14 +++++++++++++- src/Tgstation.Server.Host/IO/FileDownloader.cs | 10 +++++++++- src/Tgstation.Server.Host/IO/IFileDownloader.cs | 3 ++- .../Components/Byond/TestPosixByondInstaller.cs | 2 +- tools/Tgstation.Server.Migrator/Program.cs | 2 +- 6 files changed, 27 insertions(+), 6 deletions(-) diff --git a/src/Tgstation.Server.Host/Components/Byond/ByondInstallerBase.cs b/src/Tgstation.Server.Host/Components/Byond/ByondInstallerBase.cs index 0b26625bc7..df32ce6e48 100644 --- a/src/Tgstation.Server.Host/Components/Byond/ByondInstallerBase.cs +++ b/src/Tgstation.Server.Host/Components/Byond/ByondInstallerBase.cs @@ -97,7 +97,7 @@ namespace Tgstation.Server.Host.Components.Byond Logger.LogTrace("Downloading BYOND version {major}.{minor}...", version.Major, version.Minor); var url = String.Format(CultureInfo.InvariantCulture, ByondRevisionsUrlTemplate, version.Major, version.Minor); - return fileDownloader.DownloadFile(new Uri(url), cancellationToken); + return fileDownloader.DownloadFile(new Uri(url), null, cancellationToken); } } } diff --git a/src/Tgstation.Server.Host/Core/ServerUpdater.cs b/src/Tgstation.Server.Host/Core/ServerUpdater.cs index bbada322dc..e552db745a 100644 --- a/src/Tgstation.Server.Host/Core/ServerUpdater.cs +++ b/src/Tgstation.Server.Host/Core/ServerUpdater.cs @@ -43,6 +43,11 @@ namespace Tgstation.Server.Host.Core /// readonly ILogger logger; + /// + /// The for the . + /// + readonly GeneralConfiguration generalConfiguration; + /// /// The for the . /// @@ -61,6 +66,7 @@ namespace Tgstation.Server.Host.Core /// The value of . /// The value of . /// The value of . + /// The containing the value of . /// The containing the value of . public ServerUpdater( IGitHubClientFactory gitHubClientFactory, @@ -68,6 +74,7 @@ namespace Tgstation.Server.Host.Core IFileDownloader fileDownloader, IServerControl serverControl, ILogger logger, + IOptions generalConfigurationOptions, IOptions updatesConfigurationOptions) { this.gitHubClientFactory = gitHubClientFactory ?? throw new ArgumentNullException(nameof(gitHubClientFactory)); @@ -75,6 +82,7 @@ namespace Tgstation.Server.Host.Core this.fileDownloader = fileDownloader ?? throw new ArgumentNullException(nameof(fileDownloader)); this.serverControl = serverControl ?? throw new ArgumentNullException(nameof(serverControl)); this.logger = logger ?? throw new ArgumentNullException(nameof(logger)); + generalConfiguration = generalConfigurationOptions?.Value ?? throw new ArgumentNullException(nameof(generalConfigurationOptions)); updatesConfiguration = updatesConfigurationOptions?.Value ?? throw new ArgumentNullException(nameof(updatesConfigurationOptions)); } @@ -183,7 +191,11 @@ namespace Tgstation.Server.Host.Core try { logger.LogTrace("Downloading zip package..."); - updateZipData = await fileDownloader.DownloadFile(serverUpdateOperation.UpdateZipUrl, cancellationToken); + var bearerToken = generalConfiguration.GitHubAccessToken; + if (String.IsNullOrWhiteSpace(bearerToken)) + bearerToken = null; + + updateZipData = await fileDownloader.DownloadFile(serverUpdateOperation.UpdateZipUrl, bearerToken, cancellationToken); } catch (Exception ex) { diff --git a/src/Tgstation.Server.Host/IO/FileDownloader.cs b/src/Tgstation.Server.Host/IO/FileDownloader.cs index 5cd08b7635..46405a2c65 100644 --- a/src/Tgstation.Server.Host/IO/FileDownloader.cs +++ b/src/Tgstation.Server.Host/IO/FileDownloader.cs @@ -1,11 +1,13 @@ using System; using System.IO; using System.Net.Http; +using System.Net.Http.Headers; using System.Threading; using System.Threading.Tasks; using Microsoft.Extensions.Logging; +using Tgstation.Server.Api; using Tgstation.Server.Common; namespace Tgstation.Server.Host.IO @@ -35,14 +37,20 @@ namespace Tgstation.Server.Host.IO } /// - public async Task DownloadFile(Uri url, CancellationToken cancellationToken) + public async Task DownloadFile(Uri url, string bearerToken, CancellationToken cancellationToken) { + if (url == null) + throw new ArgumentNullException(nameof(url)); + logger.LogDebug("Starting download of {url}...", url); using var httpClient = httpClientFactory.CreateClient(); using var request = new HttpRequestMessage( HttpMethod.Get, url); + if (bearerToken != null) + request.Headers.Authorization = new AuthenticationHeaderValue(ApiHeaders.BearerAuthenticationScheme, bearerToken); + var webRequestTask = httpClient.SendAsync(request, cancellationToken); using var response = await webRequestTask; response.EnsureSuccessStatusCode(); diff --git a/src/Tgstation.Server.Host/IO/IFileDownloader.cs b/src/Tgstation.Server.Host/IO/IFileDownloader.cs index 199d904f06..7a31775e1f 100644 --- a/src/Tgstation.Server.Host/IO/IFileDownloader.cs +++ b/src/Tgstation.Server.Host/IO/IFileDownloader.cs @@ -14,8 +14,9 @@ namespace Tgstation.Server.Host.IO /// Downloads a file from . /// /// The URL to download. + /// Optional to use as the "Bearer" value in the optional "Authorization" header for the request. /// A for the operation. /// A resulting in a of the downloaded file. - Task DownloadFile(Uri url, CancellationToken cancellationToken); + Task DownloadFile(Uri url, string bearerToken, CancellationToken cancellationToken); } } diff --git a/tests/Tgstation.Server.Host.Tests/Components/Byond/TestPosixByondInstaller.cs b/tests/Tgstation.Server.Host.Tests/Components/Byond/TestPosixByondInstaller.cs index a78b19a86e..a2da857479 100644 --- a/tests/Tgstation.Server.Host.Tests/Components/Byond/TestPosixByondInstaller.cs +++ b/tests/Tgstation.Server.Host.Tests/Components/Byond/TestPosixByondInstaller.cs @@ -51,7 +51,7 @@ namespace Tgstation.Server.Host.Components.Byond.Tests await Assert.ThrowsExceptionAsync(() => installer.DownloadVersion(null, default)); var ourArray = Array.Empty(); - mockFileDownloader.Setup(x => x.DownloadFile(It.Is(uri => uri == new Uri("https://secure.byond.com/download/build/511/511.1385_byond_linux.zip")), default)).Returns(Task.FromResult(new MemoryStream(ourArray))).Verifiable(); + mockFileDownloader.Setup(x => x.DownloadFile(It.Is(uri => uri == new Uri("https://secure.byond.com/download/build/511/511.1385_byond_linux.zip")), null, default)).Returns(Task.FromResult(new MemoryStream(ourArray))).Verifiable(); var result = await installer.DownloadVersion(new Version(511, 1385), default); diff --git a/tools/Tgstation.Server.Migrator/Program.cs b/tools/Tgstation.Server.Migrator/Program.cs index 6d6406500c..9742ee38ce 100644 --- a/tools/Tgstation.Server.Migrator/Program.cs +++ b/tools/Tgstation.Server.Migrator/Program.cs @@ -386,7 +386,7 @@ try using (var loggerFactory = LoggerFactory.Create(builder => { })) { var fileDownloader = new FileDownloader(httpClientFactory, loggerFactory.CreateLogger()); - using var tgsFiveZipMemoryStream = await fileDownloader.DownloadFile(new Uri(serverServiceAsset.BrowserDownloadUrl), default); + using var tgsFiveZipMemoryStream = await fileDownloader.DownloadFile(new Uri(serverServiceAsset.BrowserDownloadUrl), null, default); Console.WriteLine("Unzipping TGS5..."); await serverFactory.IOManager.ZipToDirectory(tgsInstallPath, tgsFiveZipMemoryStream, default); } From e4debac5bc38f99bbfea6848ebe3417f959a9814 Mon Sep 17 00:00:00 2001 From: Dominion Date: Wed, 17 May 2023 22:08:02 -0400 Subject: [PATCH 07/12] Switch tests from updating to 4.8.1 to 5.11.0 --- .../Live/TestLiveServer.cs | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs b/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs index 410b8ad8a0..096b4e8b12 100644 --- a/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs +++ b/tests/Tgstation.Server.Tests/Live/TestLiveServer.cs @@ -44,6 +44,8 @@ namespace Tgstation.Server.Tests.Live public static ushort DDPort { get; } = FreeTcpPort(); public static ushort DMPort { get; } = GetDMPort(); + readonly Version TestUpdateVersion = new Version(5, 11, 0); + readonly IServerClientFactory clientFactory = new ServerClientFactory(new ProductHeaderValue(Assembly.GetExecutingAssembly().GetName().Name, Assembly.GetExecutingAssembly().GetName().Version.ToString())); static void TerminateAllDDs() @@ -252,11 +254,10 @@ namespace Tgstation.Server.Tests.Live CheckInfo(controllerInfo); // test update - var testUpdateVersion = new Version(4, 8, 1); await controllerClient.Administration.Update( new ServerUpdateRequest { - NewVersion = testUpdateVersion + NewVersion = TestUpdateVersion }, cancellationToken); await Task.WhenAny(Task.Delay(TimeSpan.FromMinutes(2)), serverTask); @@ -270,7 +271,7 @@ namespace Tgstation.Server.Tests.Live Assert.IsTrue(File.Exists(updatedAssemblyPath), "Updated assembly missing!"); var updatedAssemblyVersion = FileVersionInfo.GetVersionInfo(updatedAssemblyPath); - Assert.AreEqual(testUpdateVersion, Version.Parse(updatedAssemblyVersion.FileVersion).Semver()); + Assert.AreEqual(TestUpdateVersion, Version.Parse(updatedAssemblyVersion.FileVersion).Semver()); Directory.Delete(server.UpdatePath, true); } @@ -435,11 +436,10 @@ namespace Tgstation.Server.Tests.Live await Assert.ThrowsExceptionAsync(() => node1Client.Instances.GetId(controllerInstance, cancellationToken)); // test update - var testUpdateVersion = new Version(4, 8, 1); await node1Client.Administration.Update( new ServerUpdateRequest { - NewVersion = testUpdateVersion + NewVersion = TestUpdateVersion }, cancellationToken); await Task.WhenAny(Task.Delay(TimeSpan.FromMinutes(2)), serverTask); @@ -453,7 +453,7 @@ namespace Tgstation.Server.Tests.Live Assert.IsTrue(File.Exists(updatedAssemblyPath), "Updated assembly missing!"); var updatedAssemblyVersion = FileVersionInfo.GetVersionInfo(updatedAssemblyPath); - Assert.AreEqual(testUpdateVersion, Version.Parse(updatedAssemblyVersion.FileVersion).Semver()); + Assert.AreEqual(TestUpdateVersion, Version.Parse(updatedAssemblyVersion.FileVersion).Semver()); Directory.Delete(server.UpdatePath, true); } @@ -479,7 +479,7 @@ namespace Tgstation.Server.Tests.Live await ApiAssert.ThrowsException(() => controllerClient2.Administration.Update( new ServerUpdateRequest { - NewVersion = testUpdateVersion + NewVersion = TestUpdateVersion }, cancellationToken), ErrorCode.SwarmIntegrityCheckFailed); @@ -493,7 +493,7 @@ namespace Tgstation.Server.Tests.Live await controllerClient2.Administration.Update( new ServerUpdateRequest { - NewVersion = testUpdateVersion + NewVersion = TestUpdateVersion }, cancellationToken); From 39692bec2c210292dae08c0d0faaebf121c56cf1 Mon Sep 17 00:00:00 2001 From: Dominion Date: Wed, 17 May 2023 23:10:38 -0400 Subject: [PATCH 08/12] Actually enable node request logging in swarm mode --- src/Tgstation.Server.Host/Core/Application.cs | 5 ++++- tests/Tgstation.Server.Tests/Live/LiveTestingServer.cs | 5 ++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/src/Tgstation.Server.Host/Core/Application.cs b/src/Tgstation.Server.Host/Core/Application.cs index 9ce061cd03..0daf42748d 100644 --- a/src/Tgstation.Server.Host/Core/Application.cs +++ b/src/Tgstation.Server.Host/Core/Application.cs @@ -435,7 +435,10 @@ namespace Tgstation.Server.Host.Core }); // setup the HTTP request pipeline - // Final point where we wrap exceptions in a 500 (ErrorMessage) response + // Add additional logging context to the request + applicationBuilder.UseAdditionalRequestLoggingContext(swarmConfiguration); + + // Wrap exceptions in a 500 (ErrorMessage) response applicationBuilder.UseServerErrorHandling(); // Add the X-Powered-By response header diff --git a/tests/Tgstation.Server.Tests/Live/LiveTestingServer.cs b/tests/Tgstation.Server.Tests/Live/LiveTestingServer.cs index ccc564461e..4508ee9fc1 100644 --- a/tests/Tgstation.Server.Tests/Live/LiveTestingServer.cs +++ b/tests/Tgstation.Server.Tests/Live/LiveTestingServer.cs @@ -14,6 +14,7 @@ using Tgstation.Server.Api.Models; using Tgstation.Server.Host; using Tgstation.Server.Host.Configuration; using Tgstation.Server.Host.Core; +using Tgstation.Server.Host.Extensions; using Tgstation.Server.Host.Setup; using Tgstation.Server.Host.Utils; @@ -189,7 +190,9 @@ namespace Tgstation.Server.Tests.Live if (firstRun) args[0] = string.Format(CultureInfo.InvariantCulture, "Database:DropDatabase={0}", false); - using (swarmNodeId != null + var swarmMode = swarmNodeId != null; + ApplicationBuilderExtensions.LogSwarmIdentifier = swarmMode; + using (swarmMode ? LogContext.PushProperty(SerilogContextHelper.SwarmIdentifierContextProperty, swarmNodeId) : null) await RealServer.Run(cancellationToken); From bb695b0308eef598e07969c488242ff3f4c515c6 Mon Sep 17 00:00:00 2001 From: Dominion Date: Thu, 18 May 2023 07:51:35 -0400 Subject: [PATCH 09/12] Temporary log Windows system and app event logs Diagnostics for https://github.com/actions/runner-images/issues/7007 --- .github/workflows/ci-suite.yml | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/.github/workflows/ci-suite.yml b/.github/workflows/ci-suite.yml index c5d154e8e2..e4ce51f68d 100644 --- a/.github/workflows/ci-suite.yml +++ b/.github/workflows/ci-suite.yml @@ -354,6 +354,18 @@ jobs: Start-Sleep -Seconds 10 dotnet test -c ${{ matrix.configuration }} --no-build --filter FullyQualifiedName~TestLiveServer --logger GitHubActions --collect:"XPlat Code Coverage" --settings ../../build/coverlet.runsettings --results-directory ../../TestResults + - name: get logs system + if: always() + run: | + Get-EventLog -LogName System | Format-List + shell: powershell + + - name: get logs application + if: always() + run: | + Get-EventLog -LogName Application | Format-List + shell: powershell + - name: Store Code Coverage uses: actions/upload-artifact@v3 with: From ae634b2e7a9189251811a965de74d1f8ff04bb11 Mon Sep 17 00:00:00 2001 From: Dominion Date: Fri, 19 May 2023 22:45:09 -0400 Subject: [PATCH 10/12] Downgrade this message to a warning --- src/DMAPI/tgs/v3210/api.dm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/DMAPI/tgs/v3210/api.dm b/src/DMAPI/tgs/v3210/api.dm index b881662d71..666201a322 100644 --- a/src/DMAPI/tgs/v3210/api.dm +++ b/src/DMAPI/tgs/v3210/api.dm @@ -179,7 +179,7 @@ /datum/tgs_api/v3210/Revision() if(!warned_revison) var/datum/tgs_version/api_version = ApiVersion() - TGS_ERROR_LOG("Use of TgsRevision on [api_version.deprefixed_parameter] origin_commit only points to master!") + TGS_WARNING_LOG("Use of TgsRevision on [api_version.deprefixed_parameter] origin_commit only points to master!") warned_revison = TRUE var/datum/tgs_revision_information/ri = new ri.commit = commit From ed180f0b92e3fd04058f8008a178b09e3bce7833 Mon Sep 17 00:00:00 2001 From: Dominion Date: Sat, 20 May 2023 11:52:36 -0400 Subject: [PATCH 11/12] Remove debug logging --- .github/workflows/ci-suite.yml | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/.github/workflows/ci-suite.yml b/.github/workflows/ci-suite.yml index e4ce51f68d..c5d154e8e2 100644 --- a/.github/workflows/ci-suite.yml +++ b/.github/workflows/ci-suite.yml @@ -354,18 +354,6 @@ jobs: Start-Sleep -Seconds 10 dotnet test -c ${{ matrix.configuration }} --no-build --filter FullyQualifiedName~TestLiveServer --logger GitHubActions --collect:"XPlat Code Coverage" --settings ../../build/coverlet.runsettings --results-directory ../../TestResults - - name: get logs system - if: always() - run: | - Get-EventLog -LogName System | Format-List - shell: powershell - - - name: get logs application - if: always() - run: | - Get-EventLog -LogName Application | Format-List - shell: powershell - - name: Store Code Coverage uses: actions/upload-artifact@v3 with: From 9d8c7fdf8ec0a1df1b0f70585b775cb3a0c82cfa Mon Sep 17 00:00:00 2001 From: Dominion Date: Sat, 20 May 2023 11:53:21 -0400 Subject: [PATCH 12/12] You fucking win GitHub. Windows Live Tests on 2019 --- .github/workflows/ci-suite.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci-suite.yml b/.github/workflows/ci-suite.yml index c5d154e8e2..0d46a56e45 100644 --- a/.github/workflows/ci-suite.yml +++ b/.github/workflows/ci-suite.yml @@ -315,7 +315,7 @@ jobs: matrix: watchdog-type: [ 'Basic', 'System' ] configuration: [ 'Debug', 'Release' ] - runs-on: windows-latest + runs-on: windows-2019 steps: - name: Setup dotnet uses: actions/setup-dotnet@v2