Skip to content

Conversation

@fozzie15
Copy link
Contributor

Add plumbing for the recent support for kerberos files to reside in cloud locations to the python use case. This will allow for python users to take advantage of the feature introduced in java.


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
  • Update CHANGES.md with noteworthy changes.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests
Go tests

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

@codecov
Copy link

codecov bot commented Sep 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.21%. Comparing base (fe07fe7) to head (470a751).
⚠️ Report is 12 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #36211      +/-   ##
============================================
+ Coverage     36.25%   40.21%   +3.96%     
- Complexity     1666     3395    +1729     
============================================
  Files          1059     1221     +162     
  Lines        165727   186896   +21169     
  Branches       1195     3545    +2350     
============================================
+ Hits          60077    75161   +15084     
- Misses       103478   108368    +4890     
- Partials       2172     3367    +1195     
Flag Coverage Δ
java 70.34% <ø> (+2.17%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch 4 times, most recently from 63aa1e2 to 8aa0b7c Compare September 29, 2025 21:43
@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch from 8aa0b7c to e4e4bcf Compare October 30, 2025 18:20
@fozzie15 fozzie15 marked this pull request as ready for review October 30, 2025 18:21
@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch from e4e4bcf to ad26a9c Compare October 30, 2025 19:41
@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch 2 times, most recently from 63daa6d to 01a6bf5 Compare October 30, 2025 21:15
@github-actions
Copy link
Contributor

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

…to support consumer factories which pull files from GCS.
…classes to support consumer factories which pull files from GCS."

This reverts commit f8f69d9.
… sure to remove these later as they cannot stay.
…ll contain the factory functions to be utilized by users and the cross lang expansion service.
…his is a more lightweight dependency for the expansion service.
…ll contain the factory functions to be utilized by users and the cross lang expansion service.
@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch 2 times, most recently from 6442d56 to 7582d0d Compare October 31, 2025 16:51
@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch from 7582d0d to 470a751 Compare October 31, 2025 17:30
@fozzie15
Copy link
Contributor Author

fozzie15 commented Nov 4, 2025

assign set of reviewers

@github-actions
Copy link
Contributor

github-actions bot commented Nov 4, 2025

Assigning reviewers:

R: @jrmccluskey for label python.
R: @m-trieu for label java.
R: @Dippatel98 for label kafka.

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch 6 times, most recently from 0225a23 to 8c389bc Compare November 6, 2025 23:16
@fozzie15 fozzie15 force-pushed the kerberos_auth_python branch from 8c389bc to c7c583d Compare November 10, 2025 20:47
@github-actions
Copy link
Contributor

Reminder, please take a look at this pr: @jrmccluskey @m-trieu @Dippatel98

@jrmccluskey jrmccluskey merged commit eadbc6e into apache:master Nov 18, 2025
122 of 126 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants