irongut / CodeCoverageSummary

A GitHub Action that reads Cobertura format code coverage files and outputs a text or markdown summary.
MIT License
192 stars 54 forks source link

System.OutOfMemoryException when generating a coverage summary #133

Closed maddiemort closed 1 month ago

maddiemort commented 1 year ago

Bug Report

I'm trying to generate a coverage summary from a coverage report in cobertura format, produced using grcov, and this action fails with the error message "Parsing Error: Exception of type 'System.OutOfMemoryException' was thrown."

Workflow File

name: Coverage Report

# ...

env:
  RUSTFLAGS: -Dwarnings -Cinstrument-coverage
  # ...

jobs:
  coverage-report:
    runs-on: ubuntu-latest-32-cores
    steps:
      - name: Clone
        uses: actions/checkout@v2
      # ...
      - name: Install grcov
        run: cargo install grcov
      # ...
      - name: Gather Coverage Data
        run: cargo test
      - name: Generate Coverage Report
        run: grcov . -s . --binary-path ./target/debug/ -t cobertura --branch --ignore-not-existing -o ./target/debug/coverage.cobertura.xml
      - name: Produce Coverage Summary
        uses: irongut/CodeCoverageSummary@v1.3.0
        with:
          filename: target/debug/coverage.cobertura.xml
          badge: true
          format: 'markdown'
          output: 'both'
      - name: Add Coverage PR Comment
        uses: marocchino/sticky-pull-request-comment@v2
        if: github.event_name == 'pull_request'
        with:
          recreate: true
          path: code-coverage-results.md
      - name: Write to Job Summary
        run: cat code-coverage-results.md >> $GITHUB_STEP_SUMMARY

Actions Log

From the "Produce Coverage Summary" step:

Coverage File: /github/workspace/target/debug/coverage.cobertura.xml
Parsing Error: Exception of type 'System.OutOfMemoryException' was thrown. - /github/workspace/target/debug/coverage.cobertura.xml
##[debug]Docker Action run completed with exit code 254

Expected Behavior

I expected the Action to generate a coverage summary without running out of memory.

Additional Context

The coverage report is for a large Rust project (~300k LoC).

Linked To

None.

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this issue will be closed in 30 days.

maddiemort commented 1 year ago

It's not stale, it just hasn't received a response

github-actions[bot] commented 1 year ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this issue will be closed in 30 days.

maddiemort commented 1 year ago

Still not stale

github-actions[bot] commented 11 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this issue will be closed in 30 days.

maddiemort commented 11 months ago

Still not stale!

github-actions[bot] commented 8 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this issue will be closed in 30 days.

github-actions[bot] commented 5 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this issue will be closed in 30 days.

github-actions[bot] commented 2 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this issue will be closed in 30 days.

github-actions[bot] commented 1 month ago

This issue was closed because it has been stale for 30 days with no activity.