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

libsubprocess: do not spin on large lines #6281

Merged
merged 5 commits into from
Sep 25, 2024

libsubprocess/test: cover line buffer overflow

3302805
Select commit
Loading
Failed to load commit list.
Merged

libsubprocess: do not spin on large lines #6281

libsubprocess/test: cover line buffer overflow
3302805
Select commit
Loading
Failed to load commit list.
Mergify / Queue: Embarked in merge queue succeeded Sep 25, 2024 in 0s

The pull request embarked with master (67d7f80) will be merged soon

Required conditions for merge:

  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • all of [📃 From pull request rule rebase and merge when passing all checks]:
    • #approved-reviews-by>0
    • #changes-requested-reviews-by=0
    • -closed [📌 queue requirement]
    • -conflict [📌 queue requirement]
    • -draft [📌 queue requirement]
    • -title~=^\[*[Ww][Ii][Pp]
    • approved-reviews-by=@flux-framework/core
    • base=master
    • label!="work-in-progress"
    • label="merge-when-passing"
    • any of [📌 queue -> configuration change requirements]:
      • -mergify-configuration-changed
      • check-success = Configuration changed
    • any of [🔀 queue conditions]:
      • all of [📌 queue conditions of queue default]:
        • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
        • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
        • any of [🛡 GitHub branch protection]:
          • check-success = validate commits
          • check-neutral = validate commits
          • check-skipped = validate commits
        • any of [🛡 GitHub branch protection]:
          • check-success = address-sanitizer check
          • check-neutral = address-sanitizer check
          • check-skipped = address-sanitizer check
        • any of [🛡 GitHub branch protection]:
          • check-success = coverage
          • check-neutral = coverage
          • check-skipped = coverage
        • any of [🛡 GitHub branch protection]:
          • check-success = focal - py3.8
          • check-neutral = focal - py3.8
          • check-skipped = focal - py3.8
        • any of [🛡 GitHub branch protection]:
          • check-success = docs/readthedocs.org:flux-core
          • check-neutral = docs/readthedocs.org:flux-core
          • check-skipped = docs/readthedocs.org:flux-core
        • any of [🛡 GitHub branch protection]:
          • check-success = inception
          • check-neutral = inception
          • check-skipped = inception
        • any of [🛡 GitHub branch protection]:
          • check-success = flux-sched check
          • check-neutral = flux-sched check
          • check-skipped = flux-sched check
        • any of [🛡 GitHub branch protection]:
          • check-success = el8 - system,coverage
          • check-neutral = el8 - system,coverage
          • check-skipped = el8 - system,coverage
        • any of [🛡 GitHub branch protection]:
          • check-success = spelling
          • check-neutral = spelling
          • check-skipped = spelling
        • any of [🛡 GitHub branch protection]:
          • check-success = el8 - ascii
          • check-neutral = el8 - ascii
          • check-skipped = el8 - ascii
        • any of [🛡 GitHub branch protection]:
          • check-success = bookworm - 32 bit
          • check-neutral = bookworm - 32 bit
          • check-skipped = bookworm - 32 bit
        • any of [🛡 GitHub branch protection]:
          • check-success = bookworm - gcc-12,content-s3,distcheck
          • check-neutral = bookworm - gcc-12,content-s3,distcheck
          • check-skipped = bookworm - gcc-12,content-s3,distcheck
        • any of [🛡 GitHub branch protection]:
          • check-success = flux-accounting check
          • check-neutral = flux-accounting check
          • check-skipped = flux-accounting check
        • any of [🛡 GitHub branch protection]:
          • check-success = python linting
          • check-neutral = python linting
          • check-skipped = python linting
        • any of [🛡 GitHub branch protection]:
          • check-success = el9 - test-install
          • check-neutral = el9 - test-install
          • check-skipped = el9 - test-install
        • any of [🛡 GitHub branch protection]:
          • check-success = fedora40 - clang-18
          • check-neutral = fedora40 - clang-18
          • check-skipped = fedora40 - clang-18
        • any of [🛡 GitHub branch protection]:
          • check-success = noble - test-install
          • check-neutral = noble - test-install
          • check-skipped = noble - test-install
        • any of [🛡 GitHub branch protection]:
          • check-success = alpine - test-install
          • check-neutral = alpine - test-install
          • check-skipped = alpine - test-install
        • any of [🛡 GitHub branch protection]:
          • check-success = fedora40 - test-install
          • check-neutral = fedora40 - test-install
          • check-skipped = fedora40 - test-install
        • any of [🛡 GitHub branch protection]:
          • check-success = bookworm - test-install
          • check-neutral = bookworm - test-install
          • check-skipped = bookworm - test-install
        • any of [🛡 GitHub branch protection]:
          • check-success = jammy - test-install
          • check-neutral = jammy - test-install
          • check-skipped = jammy - test-install
        • any of [🛡 GitHub branch protection]:
          • check-success = flux-pam check
          • check-neutral = flux-pam check
          • check-skipped = flux-pam check
        • any of [🛡 GitHub branch protection]:
          • check-success = flux-pmix check
          • check-neutral = flux-pmix check
          • check-skipped = flux-pmix check
  • any of [🛡 GitHub branch protection]:
    • check-success = validate commits
    • check-neutral = validate commits
    • check-skipped = validate commits
  • any of [🛡 GitHub branch protection]:
    • check-success = address-sanitizer check
    • check-neutral = address-sanitizer check
    • check-skipped = address-sanitizer check
  • any of [🛡 GitHub branch protection]:
    • check-success = coverage
    • check-neutral = coverage
    • check-skipped = coverage
  • any of [🛡 GitHub branch protection]:
    • check-success = focal - py3.8
    • check-neutral = focal - py3.8
    • check-skipped = focal - py3.8
  • any of [🛡 GitHub branch protection]:
    • check-success = docs/readthedocs.org:flux-core
    • check-neutral = docs/readthedocs.org:flux-core
    • check-skipped = docs/readthedocs.org:flux-core
  • any of [🛡 GitHub branch protection]:
    • check-success = inception
    • check-neutral = inception
    • check-skipped = inception
  • any of [🛡 GitHub branch protection]:
    • check-success = flux-sched check
    • check-neutral = flux-sched check
    • check-skipped = flux-sched check
  • any of [🛡 GitHub branch protection]:
    • check-success = el8 - system,coverage
    • check-neutral = el8 - system,coverage
    • check-skipped = el8 - system,coverage
  • any of [🛡 GitHub branch protection]:
    • check-success = spelling
    • check-neutral = spelling
    • check-skipped = spelling
  • any of [🛡 GitHub branch protection]:
    • check-success = el8 - ascii
    • check-neutral = el8 - ascii
    • check-skipped = el8 - ascii
  • any of [🛡 GitHub branch protection]:
    • check-success = bookworm - 32 bit
    • check-neutral = bookworm - 32 bit
    • check-skipped = bookworm - 32 bit
  • any of [🛡 GitHub branch protection]:
    • check-success = bookworm - gcc-12,content-s3,distcheck
    • check-neutral = bookworm - gcc-12,content-s3,distcheck
    • check-skipped = bookworm - gcc-12,content-s3,distcheck
  • any of [🛡 GitHub branch protection]:
    • check-success = flux-accounting check
    • check-neutral = flux-accounting check
    • check-skipped = flux-accounting check
  • any of [🛡 GitHub branch protection]:
    • check-success = python linting
    • check-neutral = python linting
    • check-skipped = python linting
  • any of [🛡 GitHub branch protection]:
    • check-success = el9 - test-install
    • check-neutral = el9 - test-install
    • check-skipped = el9 - test-install
  • any of [🛡 GitHub branch protection]:
    • check-success = fedora40 - clang-18
    • check-neutral = fedora40 - clang-18
    • check-skipped = fedora40 - clang-18
  • any of [🛡 GitHub branch protection]:
    • check-success = noble - test-install
    • check-neutral = noble - test-install
    • check-skipped = noble - test-install
  • any of [🛡 GitHub branch protection]:
    • check-success = alpine - test-install
    • check-neutral = alpine - test-install
    • check-skipped = alpine - test-install
  • any of [🛡 GitHub branch protection]:
    • check-success = fedora40 - test-install
    • check-neutral = fedora40 - test-install
    • check-skipped = fedora40 - test-install
  • any of [🛡 GitHub branch protection]:
    • check-success = bookworm - test-install
    • check-neutral = bookworm - test-install
    • check-skipped = bookworm - test-install
  • any of [🛡 GitHub branch protection]:
    • check-success = jammy - test-install
    • check-neutral = jammy - test-install
    • check-skipped = jammy - test-install
  • any of [🛡 GitHub branch protection]:
    • check-success = flux-pam check
    • check-neutral = flux-pam check
    • check-skipped = flux-pam check
  • any of [🛡 GitHub branch protection]:
    • check-success = flux-pmix check
    • check-neutral = flux-pmix check
    • check-skipped = flux-pmix check

Check-runs and statuses of the embarked pull request #6281:

Analyze (cpp) details
Analyze (python) details
CodeQL — No new alerts in code changed by this pull requestdetails
Generate build matrix details
Generate docker manifest details
address-sanitizer check details
alpine - test-install details
bookworm - 32 bit details
bookworm - gcc-12,content-s3,distcheck details
bookworm - test-install details
codecov/patch — 90.90% of diff hit (target 83.32%)details
codecov/project — 83.31% (-0.02%) compared to 67d7f80details
coverage details
docs/readthedocs.org:flux-core Read the Docs build succeeded!details
el8 - ascii details
el8 - system,coverage details
el9 - test-install details
fedora40 - clang-18 details
fedora40 - test-install details
flux-accounting check details
flux-pam check details
flux-pmix check details
flux-sched check details
focal - py3.8 details
inception details
jammy - test-install details
noble - test-install details
python linting details
spelling details
validate commits details