fix: memory leak when assigning the same key with ttl multiply times #228
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: kv_in_memory | |
| on: | |
| push: | |
| branches: | |
| - master | |
| - stable | |
| pull_request: | |
| branches: | |
| - master | |
| - stable | |
| jobs: | |
| memory_kv_test: | |
| name: Memory KV plugin (Go ${{ matrix.go }}, PHP ${{ matrix.php }}, OS ${{matrix.os}}) | |
| runs-on: ${{ matrix.os }} | |
| timeout-minutes: 60 | |
| strategy: | |
| matrix: | |
| php: ["8.3"] | |
| go: [stable] | |
| os: ["ubuntu-latest"] | |
| steps: | |
| - name: Set up Go ${{ matrix.go }} | |
| uses: actions/setup-go@v5 # action page: <https://github.com/actions/setup-go> | |
| with: | |
| go-version: ${{ matrix.go }} | |
| - name: Set up PHP ${{ matrix.php }} | |
| uses: shivammathur/setup-php@v2 # action page: <https://github.com/shivammathur/setup-php> | |
| with: | |
| php-version: ${{ matrix.php }} | |
| extensions: sockets | |
| - name: Check out code | |
| uses: actions/checkout@v4 | |
| - name: Get Composer Cache Directory | |
| id: composer-cache | |
| run: | | |
| cd tests/php_test_files | |
| echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT | |
| - name: Init Composer Cache # Docs: <https://git.io/JfAKn#php---composer> | |
| uses: actions/cache@v4 | |
| with: | |
| path: ${{ steps.composer-cache.outputs.dir }} | |
| key: ${{ runner.os }}-composer-${{ matrix.php }}-${{ hashFiles('**/composer.json') }} | |
| restore-keys: ${{ runner.os }}-composer- | |
| - name: Install Composer dependencies | |
| run: cd tests/php_test_files && composer update --prefer-dist --no-progress --ansi | |
| - name: Init Go modules Cache # Docs: <https://git.io/JfAKn#go---modules> | |
| uses: actions/cache@v4 | |
| with: | |
| path: ~/go/pkg/mod | |
| key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }} | |
| restore-keys: ${{ runner.os }}-go- | |
| - name: Install Go dependencies | |
| run: go mod download | |
| - name: Run golang tests with coverage | |
| run: | | |
| cd tests | |
| mkdir ./coverage-ci | |
| docker compose -f env/docker-compose-otel.yaml up -d | |
| sleep 30 | |
| if [ "$(docker ps -q -f status=running -f name=otel_collector)" ]; then | |
| echo "Memory Docker container started successfully." | |
| else | |
| echo "Failed to start Memory Docker container." | |
| exit 1 | |
| fi | |
| go test -timeout 20m -v -race -cover -tags=debug -failfast -coverpkg=$(cat pkgs.txt) -coverprofile=./coverage-ci/memory_kv.out -covermode=atomic kv_memory_test.go | |
| docker compose -f env/docker-compose-otel.yaml down | |
| - name: Archive code coverage results | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: coverage | |
| path: ./tests/coverage-ci/memory_kv.out | |
| codecov: | |
| name: Upload codecov | |
| runs-on: ubuntu-latest | |
| needs: | |
| - memory_kv_test | |
| timeout-minutes: 60 | |
| steps: | |
| - name: Download code coverage results | |
| uses: actions/download-artifact@v4 | |
| - run: | | |
| cd coverage | |
| echo 'mode: atomic' > summary.txt | |
| tail -q -n +2 *.out >> summary.txt | |
| sed -i '2,${/roadrunner/!d}' summary.txt | |
| - name: upload to codecov | |
| uses: codecov/codecov-action@v5 # Docs: <https://github.com/codecov/codecov-action> | |
| with: | |
| file: ./coverage/summary.txt | |
| fail_ci_if_error: false |