Skip to content

fix(deps): pin kubernetes<36 due to broken bearer-token auth#2730

Merged
myakove merged 3 commits into
mainfrom
fix/pin-kubernetes-below-36
May 26, 2026
Merged

fix(deps): pin kubernetes<36 due to broken bearer-token auth#2730
myakove merged 3 commits into
mainfrom
fix/pin-kubernetes-below-36

Conversation

@myakove
Copy link
Copy Markdown
Collaborator

@myakove myakove commented May 26, 2026

What

Pin kubernetes dependency to >=31.0.0,<36 to avoid a broken bearer-token auth regression in v36.0.0.

Why

kubernetes v36.0.0 has a regression where Configuration.auth_settings() no longer recognizes the 'authorization' key in api_key, silently dropping the Authorization header from all API requests. This causes 401/403 errors for any authenticated API call.

The upstream fix (kubernetes-client/python#2585) is merged but not yet released. Pin to <36 until a patched v36.x ships.

Ref: kubernetes-client/python#2582

Done

  • Pin kubernetes<36 in pyproject.toml
  • Add inline comment explaining the pin with issue URL

Summary by CodeRabbit

  • Chores
    • Updated Kubernetes library dependency version constraints for improved compatibility

Review Change Stack

kubernetes v36.0.0 has a regression where auth_settings() no longer
recognizes the 'authorization' key in api_key, silently dropping the
Authorization header from all API requests.

The upstream fix (PR #2585) is merged but not yet released.
Pin to <36 until a patched v36.x ships.

Ref: kubernetes-client/python#2582
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 26, 2026

Warning

Review limit reached

@myakove, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 54 minutes and 35 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6d24c367-3eda-4a9d-ba84-0607b306aebe

📥 Commits

Reviewing files that changed from the base of the PR and between 01a2584 and e1aaa43.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pyproject.toml

Walkthrough

The kubernetes dependency constraint in pyproject.toml is narrowed from >=31.0.0 to >=31.0.0,<36 with an inline comment explaining the bearer-token authentication compatibility reason.

Changes

Kubernetes version constraint

Layer / File(s) Summary
Kubernetes version constraint
pyproject.toml
Kubernetes dependency upper bound is constrained to version <36 with a bearer-token authentication compatibility note.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description covers the key information: what was changed, why it was needed, and confirmation of completed tasks. It provides upstream references and context about the regression.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The pull request title accurately describes the main change: pinning the kubernetes dependency below version 36 due to a bearer-token authentication regression.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/pin-kubernetes-below-36

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rh-bot-1
Copy link
Copy Markdown

Report bugs in Issues

Welcome! 🎉

This pull request will be automatically processed with the following features:

🔄 Automatic Actions

  • Reviewer Assignment: Reviewers are automatically assigned based on the OWNERS file in the repository root
  • Size Labeling: PR size labels (XS, S, M, L, XL, XXL) are automatically applied based on changes
  • Issue Creation: Disabled for this repository
  • Branch Labeling: Branch-specific labels are applied to track the target branch
  • Auto-verification: Auto-verified users have their PRs automatically marked as verified
  • Labels: All label categories are enabled (default configuration)

📋 Available Commands

PR Status Management

  • /wip - Mark PR as work in progress (adds WIP: prefix to title)
  • /wip cancel - Remove work in progress status
  • /hold - Block PR merging (approvers only)
  • /hold cancel - Unblock PR merging
  • /verified - Mark PR as verified
  • /verified cancel - Remove verification status
  • /reprocess - Trigger complete PR workflow reprocessing (useful if webhook failed or configuration changed)
  • /regenerate-welcome - Regenerate this welcome message

Review & Approval

  • /lgtm - Approve changes (looks good to me)
  • /approve - Approve PR (approvers only)
  • /automerge - Enable automatic merging when all requirements are met (maintainers and approvers only)
  • /assign-reviewers - Assign reviewers based on OWNERS file
  • /assign-reviewer @username - Assign specific reviewer
  • /check-can-merge - Check if PR meets merge requirements

Testing & Validation

  • /retest tox - Run Python test suite with tox
  • /retest python-module-install - Test Python package installation
  • /retest conventional-title - Validate commit message format
  • /retest all - Run all available tests

Cherry-pick Operations

  • /cherry-pick <branch> - Schedule cherry-pick to target branch when PR is merged
    • Multiple branches: /cherry-pick branch1 branch2 branch3

Label Management

  • /<label-name> - Add a label to the PR
  • /<label-name> cancel - Remove a label from the PR

✅ Merge Requirements

This PR will be automatically approved when the following conditions are met:

  1. Approval: /approve from at least one approver
  2. Status Checks: All required status checks must pass
  3. No Blockers: No wip, hold, has-conflicts labels and PR must be mergeable (no conflicts)
  4. Verified: PR must be marked as verified

📊 Review Process

Approvers and Reviewers

Approvers:

  • myakove
  • rnetser

Reviewers:

  • myakove
  • rnetser
Available Labels
  • hold
  • verified
  • wip
  • lgtm
  • approve
  • automerge
AI Features
  • Conventional Title: Mode: fix (claude/claude-opus-4-6[1m])
  • Cherry-Pick Conflict Resolution: Enabled (claude/claude-opus-4-6[1m])

💡 Tips

  • WIP Status: Use /wip when your PR is not ready for review
  • Verification: The verified label is removed on new commits unless the push is detected as a clean rebase
  • Cherry-picking: Cherry-pick labels are processed when the PR is merged
  • Permission Levels: Some commands require approver permissions
  • Auto-verified Users: Certain users have automatic verification and merge privileges

For more information, please refer to the project documentation or contact the maintainers.

@rh-bot-1 rh-bot-1 requested a review from rnetser May 26, 2026 15:33
@rh-bot-1 rh-bot-1 changed the title Pin kubernetes<36 due to broken bearer-token auth fix(deps): pin kubernetes<36 due to broken bearer-token auth May 26, 2026
Remove aiohttp, frozenlist, multidict, propcache, yarl and async-timeout
dependencies. Pin kubernetes to <36. Update cloup to 3.1.0 and cyclopts
to 4.16.1.
@myakove myakove force-pushed the fix/pin-kubernetes-below-36 branch from c78ed07 to 2a59ee1 Compare May 26, 2026 15:35
The kubernetes dependency constraint is changed from `>=31.0.0,<36` to `>=31.0.0,!=36.0.0` to allow versions 36.0.1 and above while excluding the broken 36.0.0 release that has bearer-token authentication issues.
@myakove myakove merged commit 3b9e633 into main May 26, 2026
5 of 7 checks passed
@myakove myakove deleted the fix/pin-kubernetes-below-36 branch May 26, 2026 15:42
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