deephaven / deephaven-core

Deephaven Community Core
Other
254 stars 80 forks source link

Upload JVM Error Logs includes extraneous files #5903

Open devinrsmith opened 2 months ago

devinrsmith commented 2 months ago

The "Upload JVM Error Logs" tasks is picking up core.h files. Ideally, this task would be skipped when there are no relevant jvm error logs since we configure if-no-files-found: ignore.

- name: Upload JVM Error Logs
  uses: actions/upload-artifact@v4
  if: failure()
  with:
    name: check-ci-jvm-err
    path: |
      **/*_pid*.log
      **/core.*
    if-no-files-found: ignore

The core.* path is too general.

$ find nightly-check-java11-ci-jvm-err -type f
nightly-check-java11-ci-jvm-err/cpp-client/build/cppClient-docker/deephaven/dhcore/include/public/deephaven/third_party/fmt/core.h
nightly-check-java11-ci-jvm-err/cpp-client/build/cppClientPy-docker/deephaven/dhcore/include/public/deephaven/third_party/fmt/core.h
nightly-check-java11-ci-jvm-err/cpp-client/deephaven/dhcore/include/public/deephaven/third_party/fmt/core.h
devinrsmith commented 2 months ago

The change to include core.* was originally made in #2074