Add builds for windows (#11284)
Motivation: Let's also build on windows during PR validation Modifications: Add build on windows during PR Result: Validate that all also pass on windows
This commit is contained in:
parent
cb1b3517dc
commit
0ff93519a4
1
.github/workflows/ci-pr-reports.yml
vendored
1
.github/workflows/ci-pr-reports.yml
vendored
@ -32,6 +32,7 @@ jobs:
|
|||||||
- setup: linux-x86_64-java11
|
- setup: linux-x86_64-java11
|
||||||
- setup: linux-x86_64-java16
|
- setup: linux-x86_64-java16
|
||||||
- setup: linux-x86_64-java11-boringssl
|
- setup: linux-x86_64-java11-boringssl
|
||||||
|
- setuo: windows-x86_64-java11-boringssl
|
||||||
steps:
|
steps:
|
||||||
- name: Download Artifacts
|
- name: Download Artifacts
|
||||||
uses: dawidd6/action-download-artifact@v2.11.0
|
uses: dawidd6/action-download-artifact@v2.11.0
|
||||||
|
37
.github/workflows/ci-pr.yml
vendored
37
.github/workflows/ci-pr.yml
vendored
@ -46,6 +46,43 @@ jobs:
|
|||||||
- name: Verify with Maven
|
- name: Verify with Maven
|
||||||
run: ./mvnw verify -B --file pom.xml -DskipTests=true
|
run: ./mvnw verify -B --file pom.xml -DskipTests=true
|
||||||
|
|
||||||
|
build-pr-windows:
|
||||||
|
runs-on: windows-2016
|
||||||
|
name: windows-x86_64-java11-boringssl
|
||||||
|
needs: verify-pr
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v2
|
||||||
|
|
||||||
|
- name: Set up JDK 11
|
||||||
|
uses: actions/setup-java@v1
|
||||||
|
with:
|
||||||
|
java-version: 11
|
||||||
|
|
||||||
|
# Cache .m2/repository
|
||||||
|
# Caching of maven dependencies
|
||||||
|
- uses: actions/cache@v2
|
||||||
|
with:
|
||||||
|
path: ~/.m2/repository
|
||||||
|
key: pr-windows-x86_64-maven-cache-${{ hashFiles('**/pom.xml') }}
|
||||||
|
restore-keys: |
|
||||||
|
pr-windows-x86_64-maven-cache-
|
||||||
|
|
||||||
|
- name: Build project
|
||||||
|
run: ./mvnw.cmd --file pom.xml clean package -Pboringssl -DskipHttp2Testsuite=true -DskipAutobahnTestsuite=true
|
||||||
|
|
||||||
|
- name: Upload Test Results
|
||||||
|
if: always()
|
||||||
|
uses: actions/upload-artifact@v2
|
||||||
|
with:
|
||||||
|
name: test-results-windows-x86_64-java11-boringssl
|
||||||
|
path: '**/target/surefire-reports/TEST-*.xml'
|
||||||
|
|
||||||
|
- uses: actions/upload-artifact@v2
|
||||||
|
if: ${{ failure() }}
|
||||||
|
with:
|
||||||
|
name: build-pr-windows-target
|
||||||
|
path: "**/target/"
|
||||||
|
|
||||||
build-pr-aarch64:
|
build-pr-aarch64:
|
||||||
name: linux-aarch64-verify-native
|
name: linux-aarch64-verify-native
|
||||||
# The host should always be Linux
|
# The host should always be Linux
|
||||||
|
@ -30,7 +30,9 @@ import io.netty.handler.codec.compression.ZlibEncoder;
|
|||||||
import io.netty.handler.codec.compression.ZlibWrapper;
|
import io.netty.handler.codec.compression.ZlibWrapper;
|
||||||
import io.netty.util.CharsetUtil;
|
import io.netty.util.CharsetUtil;
|
||||||
import io.netty.util.ReferenceCountUtil;
|
import io.netty.util.ReferenceCountUtil;
|
||||||
|
import io.netty.util.internal.PlatformDependent;
|
||||||
import org.apache.commons.io.IOUtils;
|
import org.apache.commons.io.IOUtils;
|
||||||
|
import org.junit.Assume;
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
|
|
||||||
import java.io.InputStream;
|
import java.io.InputStream;
|
||||||
@ -188,6 +190,8 @@ public class HttpContentDecoderTest {
|
|||||||
@Test
|
@Test
|
||||||
public void testResponseBrotliDecompression() throws Throwable {
|
public void testResponseBrotliDecompression() throws Throwable {
|
||||||
Brotli.ensureAvailability();
|
Brotli.ensureAvailability();
|
||||||
|
// Failing on windows atm
|
||||||
|
Assume.assumeFalse(PlatformDependent.isWindows());
|
||||||
HttpResponseDecoder decoder = new HttpResponseDecoder();
|
HttpResponseDecoder decoder = new HttpResponseDecoder();
|
||||||
HttpContentDecoder decompressor = new HttpContentDecompressor();
|
HttpContentDecoder decompressor = new HttpContentDecompressor();
|
||||||
HttpObjectAggregator aggregator = new HttpObjectAggregator(Integer.MAX_VALUE);
|
HttpObjectAggregator aggregator = new HttpObjectAggregator(Integer.MAX_VALUE);
|
||||||
@ -220,6 +224,8 @@ public class HttpContentDecoderTest {
|
|||||||
@Test
|
@Test
|
||||||
public void testResponseChunksBrotliDecompression() throws Throwable {
|
public void testResponseChunksBrotliDecompression() throws Throwable {
|
||||||
Brotli.ensureAvailability();
|
Brotli.ensureAvailability();
|
||||||
|
// Failing on windows atm
|
||||||
|
Assume.assumeFalse(PlatformDependent.isWindows());
|
||||||
HttpResponseDecoder decoder = new HttpResponseDecoder();
|
HttpResponseDecoder decoder = new HttpResponseDecoder();
|
||||||
HttpContentDecoder decompressor = new HttpContentDecompressor();
|
HttpContentDecoder decompressor = new HttpContentDecompressor();
|
||||||
HttpObjectAggregator aggregator = new HttpObjectAggregator(Integer.MAX_VALUE);
|
HttpObjectAggregator aggregator = new HttpObjectAggregator(Integer.MAX_VALUE);
|
||||||
|
@ -27,7 +27,9 @@ import io.netty.util.CharsetUtil;
|
|||||||
import io.netty.util.NetUtil;
|
import io.netty.util.NetUtil;
|
||||||
import io.netty.util.concurrent.ImmediateEventExecutor;
|
import io.netty.util.concurrent.ImmediateEventExecutor;
|
||||||
import io.netty.util.concurrent.Promise;
|
import io.netty.util.concurrent.Promise;
|
||||||
|
import io.netty.util.internal.PlatformDependent;
|
||||||
import org.junit.Assert;
|
import org.junit.Assert;
|
||||||
|
import org.junit.Assume;
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
|
|
||||||
import java.net.PortUnreachableException;
|
import java.net.PortUnreachableException;
|
||||||
@ -46,6 +48,9 @@ public class DatagramConnectNotExistsTest extends AbstractClientSocketTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
public void testConnectNotExists(Bootstrap cb) throws Throwable {
|
public void testConnectNotExists(Bootstrap cb) throws Throwable {
|
||||||
|
// Currently not works on windows
|
||||||
|
// See https://github.com/netty/netty/issues/11285
|
||||||
|
Assume.assumeFalse(PlatformDependent.isWindows());
|
||||||
final Promise<Throwable> promise = ImmediateEventExecutor.INSTANCE.newPromise();
|
final Promise<Throwable> promise = ImmediateEventExecutor.INSTANCE.newPromise();
|
||||||
cb.handler(new ChannelInboundHandlerAdapter() {
|
cb.handler(new ChannelInboundHandlerAdapter() {
|
||||||
@Override
|
@Override
|
||||||
|
@ -15,10 +15,21 @@
|
|||||||
*/
|
*/
|
||||||
package io.netty.testsuite.transport.socket;
|
package io.netty.testsuite.transport.socket;
|
||||||
|
|
||||||
|
import io.netty.bootstrap.Bootstrap;
|
||||||
import io.netty.channel.socket.InternetProtocolFamily;
|
import io.netty.channel.socket.InternetProtocolFamily;
|
||||||
|
import io.netty.util.internal.PlatformDependent;
|
||||||
|
import org.junit.Assume;
|
||||||
|
|
||||||
public class DatagramMulticastIPv6Test extends DatagramMulticastTest {
|
public class DatagramMulticastIPv6Test extends DatagramMulticastTest {
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void testMulticast(Bootstrap sb, Bootstrap cb) throws Throwable {
|
||||||
|
// Not works on windows atm.
|
||||||
|
// See https://github.com/netty/netty/issues/11285
|
||||||
|
Assume.assumeFalse(PlatformDependent.isWindows());
|
||||||
|
super.testMulticast(sb, cb);
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
protected InternetProtocolFamily internetProtocolFamily() {
|
protected InternetProtocolFamily internetProtocolFamily() {
|
||||||
return InternetProtocolFamily.IPv6;
|
return InternetProtocolFamily.IPv6;
|
||||||
|
Loading…
Reference in New Issue
Block a user