Skip to content
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

gha: move the event-based test over from Zuul #8490

Closed
wants to merge 4 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
42 changes: 42 additions & 0 deletions .github/workflows/event-based.yml
@@ -0,0 +1,42 @@
name: event-based

on:
# Trigger the workflow on push or pull requests, but only for the
# master branch
push:
branches:
- master
- '*/ci'
pull_request:
branches:
- master

jobs:
autotools:
name: ${{ matrix.build.name }}
runs-on: 'ubuntu-latest'
timeout-minutes: 60
strategy:
fail-fast: false
matrix:
build:
- name: event tests
install:
configure: --enable-debug --disable-shared --disable-threaded-resolver --enable-werror --with-libssh --with-openssl
tflags: -n -e '!TLS-SRP'

steps:
- run: |
sudo apt-get install libtool autoconf automake pkg-config stunnel4 python3-pip libpsl-dev libbrotli-dev libzstd-dev libssh-dev
sudo pip3 install impacket
name: install prereqs and impacket

- uses: actions/checkout@v2

- run: autoreconf -fi && ./configure ${{ matrix.build.configure }} && make V=1
name: 'configure and build'

- run: make V=1 test-ci
name: 'test'
env:
TFLAGS: "${{ matrix.build.tflags }}"
3 changes: 3 additions & 0 deletions lib/vssh/libssh.c
Expand Up @@ -2063,6 +2063,9 @@ static int myssh_getsock(struct Curl_easy *data,
if(conn->waitfor & KEEP_SEND)
bitmap |= GETSOCK_WRITESOCK(FIRSTSOCKET);

if(!conn->waitfor)
bitmap |= GETSOCK_WRITESOCK(FIRSTSOCKET);

return bitmap;
}

Expand Down
7 changes: 0 additions & 7 deletions scripts/zuul/script.sh
Expand Up @@ -42,13 +42,6 @@ if [ "$T" = "torture" ]; then
make "TFLAGS=-n --shallow=20 -t $tests" test-nonflaky
fi

if [ "$T" = "events" ]; then
./configure --enable-debug --disable-shared --disable-threaded-resolver --enable-code-coverage --enable-werror --with-libssh2 --with-openssl
make
tests="!TLS-SRP"
make "TFLAGS=-n -e $tests" test-nonflaky
fi

if [ "$T" = "debug" ]; then
./configure --enable-debug --enable-werror $C
make
Expand Down
4 changes: 0 additions & 4 deletions tests/data/test1459
Expand Up @@ -37,11 +37,7 @@ CURL_HOME=%PWD/log/test%TESTNUMBER.dir
<verify>
# old libssh installs return the wrong thing
<errorcode>
%if oldlibssh
67
%else
60
%endif
</errorcode>
<valgrind>
disable
Expand Down
17 changes: 0 additions & 17 deletions zuul.d/jobs.yaml
Expand Up @@ -292,22 +292,6 @@
CXX: g++-8
T: torture

- job:
name: curl-events
parent: curl-base
vars:
curl_apt_packages:
- lcov
- libpsl-dev
- libssl-dev
- libbrotli-dev
- libzstd-dev
- libssh2-1-dev
curl_env:
CC: gcc-8
CXX: g++-8
T: events

- job:
name: curl-fuzzer
parent: curl-base
Expand Down Expand Up @@ -385,7 +369,6 @@
- curl-cmake-boringssl-quiche
- curl-cmake-ngtcp2
- curl-torture
- curl-events
- curl-fuzzer
- curl-tidy
- curl-debug-clang-with-openssl-dl-ubsan
Expand Down