Skip to content

ci(ud): BASELINE — run against private snowflake-eng/universal-driver@main (App token)#4276

Draft
sfc-gh-fpawlowski wants to merge 1 commit into
ud-ci-workflow-auto-triggered-full-improvementsfrom
ud-ci-eng-main-baseline
Draft

ci(ud): BASELINE — run against private snowflake-eng/universal-driver@main (App token)#4276
sfc-gh-fpawlowski wants to merge 1 commit into
ud-ci-workflow-auto-triggered-full-improvementsfrom
ud-ci-eng-main-baseline

Conversation

@sfc-gh-fpawlowski

Copy link
Copy Markdown

Stacked on ud-ci-workflow-auto-triggered-full-improvements (PR #4275) to compare test coverage of a different UD source.

What differs from #4275

  • ci(ud): expand UD compatibility workflow scope + fix bugs + reproducible UD build #4275 builds UD from the public snowflakedb/universal-driver@snowpark-compatibility (carries the compat shims).
  • This branch builds UD from the private, canonical snowflake-eng/universal-driver@main (pinned d9e7cd7), authenticating with a GitHub App token (actions/create-github-app-token, owner: snowflake-eng, repositories: universal-driver) — the same pattern odbc-reports uses.

Purpose

  1. Validate we can auth to the private eng repo from snowpark-python CI (do the SNOWFLAKE_GH_ACTIONS_APP_ID/_TOKEN App secrets exist here?).
  2. Capture the bare eng/main baseline. eng/main does not carry the snowpark-compat shims (options.py, _describe_internal, etc. are absent on main), so expect heavy import-time failures.

Follow-up

The apples-to-apples comparison (eng/main + shims) requires replaying the ~20 snowpark-compatibility commits onto eng/main (non-trivial merge — connection.py modify/delete + cursor/pandas_tools conflicts). This draft is the first step so we can compare runs.

🤖 Generated with Claude Code

…n via App token

Stacked on the improvements branch. Points build-ud at the new canonical PRIVATE
snowflake-eng/universal-driver, authenticating with a GitHub App token via
actions/create-github-app-token (the same pattern the odbc-reports early-adopter
pipeline uses), pinned to eng/main HEAD (d9e7cd7).

Purpose: (1) validate we can auth to the private eng repo from snowpark-python CI,
and (2) capture the bare eng/main baseline. eng/main lacks the snowpark-compat
shims, so expect heavy failures; the apples-to-apples comparison (eng/main + shims)
is a follow-up once those commits are replayed onto eng/main.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.33%. Comparing base (b8b2a87) to head (8610eb9).

Additional details and impacted files
@@                                 Coverage Diff                                 @@
##           ud-ci-workflow-auto-triggered-full-improvements    #4276      +/-   ##
===================================================================================
+ Coverage                                            94.29%   94.33%   +0.04%     
===================================================================================
  Files                                                  171      171              
  Lines                                                44299    44299              
  Branches                                              7564     7564              
===================================================================================
+ Hits                                                 41772    41791      +19     
+ Misses                                                1665     1650      -15     
+ Partials                                               862      858       -4     

☔ 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants