fix: prevent multiple channel close in Telemetry Stop method #1949
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Tests DNStap | |
| on: | |
| push: | |
| paths-ignore: | |
| - 'docs/**' | |
| - 'README.md' | |
| branches: | |
| - main | |
| pull_request: | |
| paths-ignore: | |
| - 'docs/**' | |
| - 'README.md' | |
| branches: | |
| - main | |
| permissions: | |
| contents: read | |
| env: | |
| Q_VERSION: "0.19.2" | |
| PYTHON_VERSION: "3.13" | |
| GO_VERSION: "1.23" | |
| COLLECTOR_USER: "pdns" | |
| jobs: | |
| build_bin: | |
| runs-on: ubuntu-latest | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-go@v5 | |
| with: | |
| go-version: "${{ env.GO_VERSION }}" | |
| - name: Build Binary | |
| run: | | |
| CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -ldflags="-s -w" -o go-dnscollector *.go | |
| - name: Upload Binary Artifact | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: go-dnscollector | |
| path: go-dnscollector | |
| tests_logging: | |
| needs: build_bin | |
| runs-on: ubuntu-latest | |
| strategy: | |
| matrix: | |
| config: | |
| - { service: "unbound", version: "1.22.0", mode: "tcp" } | |
| - { service: "unbound", version: "1.21.0", mode: "tcp" } | |
| - { service: "coredns", version: "1.11.1", mode: "tcp" } | |
| - { service: "coredns", version: "1.11.1", mode: "tls" } | |
| - { service: "dnsdist", version: "19", mode: "tcp" } | |
| - { service: "dnsdist", version: "19", mode: "unix" } | |
| - { service: "dnsdist", version: "19", mode: "doq" } | |
| steps: | |
| - uses: actions/checkout@v4 | |
| - uses: actions/setup-python@v5 | |
| with: | |
| python-version: "${{ env.PYTHON_VERSION }}" | |
| - name: Download Binary Artifact | |
| uses: actions/download-artifact@v4 | |
| with: | |
| name: go-dnscollector | |
| - name: Make Binary Executable | |
| run: chmod +x go-dnscollector | |
| - name: Download q test doq test | |
| run: | | |
| wget https://github.com/natesales/q/releases/download/v0.19.2/q_${{ env.Q_VERSION }}_linux_amd64.tar.gz | |
| tar xvf q_${{ env.Q_VERSION }}_linux_amd64.tar.gz | |
| - name: Generate certificate | |
| run: | | |
| openssl genrsa 2048 > ca.key | |
| openssl req -days 365 -new -x509 -nodes -key ca.key -out ca.crt -config <(echo -e "[ req ]\nprompt = no\ndistinguished_name = req_distinguished_name\n[ req_distinguished_name ]\ncountryName = LU\nstateOrProvinceName = Space\nlocalityName = Moon\norganizationName = Github\norganizationalUnitName = Lab\ncommonName = dnscollector.dev\nemailAddress = [email protected]") | |
| echo -e "[ req ]\nprompt = no\ndistinguished_name = req_distinguished_name\nreq_extensions = req_ext\n[ req_distinguished_name ]\ncountryName = LU\nstateOrProvinceName = Space\nlocalityName = Moon\norganizationName = Github\norganizationalUnitName = DNScollector\ncommonName = dnscollector.dev\nemailAddress = [email protected]\n[ req_ext ]\nsubjectAltName = DNS: dnscollector.dev, IP: 127.0.0.1" > server.conf | |
| openssl req -newkey rsa:2048 -nodes -keyout dnscollector.key -out dnscollector.csr --config server.conf | |
| openssl x509 -req -days 365 -in dnscollector.csr -out dnscollector.crt -CA ca.crt -CAkey ca.key -extensions req_ext -extfile server.conf | |
| sudo chmod 644 dnscollector.key | |
| mv dnscollector.key ./tests/testsdata/ | |
| mv dnscollector.crt ./tests/testsdata/ | |
| - name: Add pdns user for unix socket test | |
| run: | | |
| sudo addgroup --system --gid 953 pdns | |
| sudo adduser --system --disabled-password --no-create-home -uid 953 --gid 953 pdns | |
| - name: Deploy docker image for "${{ matrix.config.service }}" | |
| run: | | |
| case ${{ matrix.config.service }} in | |
| "unbound") | |
| sudo docker run -d --network="host" --name=unbound --volume=$PWD/tests/testsdata/unbound/unbound_${{ matrix.config.mode }}.conf:/opt/unbound/etc/unbound/unbound.conf:z -v /tmp/:/opt/unbound/etc/unbound/tmp/:z mvance/unbound:${{ matrix.config.version }} | |
| ;; | |
| "coredns") | |
| sudo docker run -d --network="host" --name=coredns -v $PWD/tests/testsdata/:$PWD/tests/testsdata/ -v /tmp/:/tmp/ coredns/coredns:${{ matrix.config.version }} -conf $PWD/tests/testsdata/coredns/coredns_${{ matrix.config.mode }}.conf | |
| ;; | |
| "dnsdist") | |
| sudo docker run -d --network="host" --name=dnsdist --volume=$PWD/tests/testsdata/powerdns/dnsdist_${{ matrix.config.mode }}.conf:/etc/dnsdist/conf.d/dnsdist.conf:z --volume=$PWD/tests/testsdata/dnscollector.key:/etc/dnsdist/conf.d/server.key:z --volume=$PWD/tests/testsdata/dnscollector.crt:/etc/dnsdist/conf.d/server.crt:z -v /tmp/:/tmp/ powerdns/dnsdist-${{ matrix.config.version }} | |
| ;; | |
| esac | |
| until (dig -p 5553 www.github.com @127.0.0.1 | grep NOERROR); do sleep 5.0; done | |
| - name: Run Tests for ${{ matrix.config.service }} - ${{ matrix.config.mode }} | |
| run: | | |
| python3 -m venv venv | |
| source venv/bin/activate | |
| python3 -m pip install dnstap_pb fstrm dnspython protobuf | |
| python3 -m unittest tests.dnsquery_dnstap${{ matrix.config.mode }} -v |