Skip to content

fix: set MaxKeyLength=0 on principal cache to fix shard concentration#70

Open
fsul7o wants to merge 5 commits intomasterfrom
fix-bug-principal-cache
Open

fix: set MaxKeyLength=0 on principal cache to fix shard concentration#70
fsul7o wants to merge 5 commits intomasterfrom
fix-bug-principal-cache

Conversation

@fsul7o
Copy link
Contributor

@fsul7o fsul7o commented Feb 25, 2026

Description

AthenZ/athenz-authorizer#28

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes, no api changes)
  • Non-code changes (update documentation, pipeline, etc.)

Flags

  • Breaks backward compatibility
  • Requires a documentation update
  • Has untestable code

Related issue/PR

Delete this section if there are no issues or pull requests that relate to this pull request.

  • Fixes #issue
  • Closes #PR

Checklist

  • Followed the guidelines in the CONTRIBUTING document
  • Added prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip] in the PR title if necessary
  • Tested and linted the code
  • Commented the code
  • Made corresponding changes to the documentation
  • Passed all pipeline checking

Checklist for maintainer

  • Use Squash and merge
  • Double-confirm the merge message has prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip]
  • Delete the branch after merge

fsul7o added 3 commits March 2, 2026 10:45
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
@fsul7o fsul7o force-pushed the fix-bug-principal-cache branch from 28070a3 to 33f99be Compare March 2, 2026 01:45
Copy link
Contributor

@t4niwa t4niwa left a comment

Choose a reason for hiding this comment

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

PRタイトルとDescriptionの修正おねがいします

cloud.google.com/go => cloud.google.com/go v0.123.0
github.com/AthenZ/athenz-authorizer/v5 => github.com/AthenZ/athenz-authorizer/v5 v5.8.0
github.com/kpango/gache/v2 => github.com/kpango/gache/v2 v2.1.1
github.com/AthenZ/athenz-authorizer/v5 => github.com/AthenZ/athenz-authorizer/v5 v5.8.1-0.20260302014201-5254e56865ed
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO: athenz-authorizerリリース後に修正

@fsul7o fsul7o changed the title [wip] bugfix fix: set MaxKeyLength=0 on principal cache to fix shard concentration Mar 6, 2026
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
@fsul7o fsul7o self-assigned this Mar 6, 2026
@fsul7o fsul7o marked this pull request as ready for review March 6, 2026 06:56
Signed-off-by: fsul7o <75571344+fsul7o@users.noreply.github.com>
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