From 009c0a56cc94f0ade36deaaa9ae99c6a47cfe94f Mon Sep 17 00:00:00 2001 From: Rene Groeschke Date: Wed, 6 Nov 2024 12:23:27 +0100 Subject: [PATCH 1/2] Mute ArchiveTests on windows (#116314) see https://github.com/elastic/elasticsearch/issues/116299 From 604722c964369449bb183776c8a9b1a19485f100 Mon Sep 17 00:00:00 2001 From: Rene Groeschke Date: Wed, 6 Nov 2024 13:23:27 +0100 Subject: [PATCH 2/2] Mute ArchiveTests for now until see https://github.com/elastic/elasticsearch/issues/116299 --- .../elasticsearch/packaging/test/ArchiveTests.java | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/qa/packaging/src/test/java/org/elasticsearch/packaging/test/ArchiveTests.java b/qa/packaging/src/test/java/org/elasticsearch/packaging/test/ArchiveTests.java index 34dd3976dfa52..4e7577be8e5fc 100644 --- a/qa/packaging/src/test/java/org/elasticsearch/packaging/test/ArchiveTests.java +++ b/qa/packaging/src/test/java/org/elasticsearch/packaging/test/ArchiveTests.java @@ -19,7 +19,6 @@ import org.elasticsearch.packaging.util.ServerUtils; import org.elasticsearch.packaging.util.Shell; import org.elasticsearch.packaging.util.Shell.Result; -import org.junit.Assume; import org.junit.BeforeClass; import java.nio.file.Files; @@ -49,6 +48,7 @@ import static org.junit.Assume.assumeThat; import static org.junit.Assume.assumeTrue; +@PackagingTestCase.AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/116299") public class ArchiveTests extends PackagingTestCase { @BeforeClass @@ -113,10 +113,6 @@ public void test32SpecialCharactersInJdkPath() throws Exception { } public void test40AutoconfigurationNotTriggeredWhenNodeIsMeantToJoinExistingCluster() throws Exception { - Assume.assumeFalse( - "https://github.com/elastic/elasticsearch/issues/116299", - distribution.platform == Distribution.Platform.WINDOWS - ); // auto-config requires that the archive owner and the process user be the same, Platforms.onWindows(() -> sh.chown(installation.config, installation.getOwner())); FileUtils.assertPathsDoNotExist(installation.data); @@ -130,10 +126,6 @@ public void test40AutoconfigurationNotTriggeredWhenNodeIsMeantToJoinExistingClus } public void test41AutoconfigurationNotTriggeredWhenNodeCannotContainData() throws Exception { - Assume.assumeFalse( - "https://github.com/elastic/elasticsearch/issues/116299", - distribution.platform == Distribution.Platform.WINDOWS - ); // auto-config requires that the archive owner and the process user be the same Platforms.onWindows(() -> sh.chown(installation.config, installation.getOwner())); ServerUtils.addSettingToExistingConfiguration(installation, "node.roles", "[\"voting_only\", \"master\"]"); @@ -146,10 +138,6 @@ public void test41AutoconfigurationNotTriggeredWhenNodeCannotContainData() throw } public void test42AutoconfigurationNotTriggeredWhenNodeCannotBecomeMaster() throws Exception { - Assume.assumeFalse( - "https://github.com/elastic/elasticsearch/issues/116299", - distribution.platform == Distribution.Platform.WINDOWS - ); // auto-config requires that the archive owner and the process user be the same Platforms.onWindows(() -> sh.chown(installation.config, installation.getOwner())); ServerUtils.addSettingToExistingConfiguration(installation, "node.roles", "[\"ingest\"]");