Skip to content

Temporary layer switch job -> event#316

Merged
knoepfel merged 1 commit intomainfrom
maintenance/job-event-temp-switch
Feb 12, 2026
Merged

Temporary layer switch job -> event#316
knoepfel merged 1 commit intomainfrom
maintenance/job-event-temp-switch

Conversation

@greenc-FNAL
Copy link
Copy Markdown
Contributor

  • Bypass caching issues

- Bypass caching issues
@greenc-FNAL
Copy link
Copy Markdown
Contributor Author

Review the full CodeQL report for details.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR switches the Python test data products and the Python plugin’s hard-wired product-query layer from "job" to "event" as a workaround to bypass caching-related issues.

Changes:

  • Update the C++ Python test source (cppsource4py) to publish all products on the "event" layer instead of "job".
  • Update the Python plugin wrapper to query inputs/outputs on the "event" layer by default.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
test/python/source.cpp Publishes the Python-test input products (i, j, k, etc.) on the event layer to match the new plugin behavior.
plugins/python/src/modulewrap.cpp Changes the Python plugin’s global hard-wired layer constant from job to event, affecting all Python module product queries.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #316      +/-   ##
==========================================
- Coverage   80.29%   80.13%   -0.16%     
==========================================
  Files         126      126              
  Lines        3065     3071       +6     
  Branches      545      545              
==========================================
  Hits         2461     2461              
+ Misses        383      381       -2     
- Partials      221      229       +8     
Flag Coverage Δ
unittests 80.13% <ø> (-0.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
plugins/python/src/modulewrap.cpp 75.04% <ø> (-0.45%) ⬇️

... and 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4db88e...d8d028a. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@knoepfel knoepfel merged commit 881866f into main Feb 12, 2026
53 checks passed
@knoepfel knoepfel deleted the maintenance/job-event-temp-switch branch February 12, 2026 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants