Skip to content

Update SLO workflow paths#668

Open
KirillKurdyukov wants to merge 2 commits into
masterfrom
slo-workload-query-paths-master
Open

Update SLO workflow paths#668
KirillKurdyukov wants to merge 2 commits into
masterfrom
slo-workload-query-paths-master

Conversation

@KirillKurdyukov

Copy link
Copy Markdown
Contributor

Summary

  • Point the SLO image build script at ydb-java-examples/slo-workload/query/Dockerfile after the workload relocation.
  • Pass checkout paths to the helper script as workspace-relative paths instead of ${GITHUB_WORKSPACE}/....

Test plan

  • bash -n .github/scripts/build-slo-image.sh
  • YAML parsed locally with Ruby YAML.load_file

Depends on

Merge order

Merge after ydb-platform/ydb-java-examples#64 lands on master.

Made with Cursor

Point the Java SDK SLO image build at the relocated query workload and pass checkout paths relative to the workspace.
Extend build-slo-image.sh with --workload and optional --jdbc checkout;
JDBC tanks rebuild the driver against the SDK under test before packaging
the workload jar.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.64%. Comparing base (3d03644) to head (1c55bbf).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #668      +/-   ##
============================================
- Coverage     71.69%   71.64%   -0.05%     
  Complexity     3429     3429              
============================================
  Files           386      386              
  Lines         16070    16070              
  Branches       1688     1688              
============================================
- Hits          11521    11514       -7     
- Misses         3897     3901       +4     
- Partials        652      655       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants