Skip to content

Remove player tracking alpha feature - #4652

Open
indurireddy-TF wants to merge 6 commits into
agones-dev:mainfrom
indurireddy-TF:remove-player-tracking-alpha-feature
Open

Remove player tracking alpha feature#4652
indurireddy-TF wants to merge 6 commits into
agones-dev:mainfrom
indurireddy-TF:remove-player-tracking-alpha-feature

Conversation

@indurireddy-TF

Copy link
Copy Markdown
Collaborator

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug

/kind cleanup

/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

Which issue(s) this PR fixes:

Closes #4491

Did you use AI tools in preparing this PR?:

Y

Special notes for your reviewer:
The Go-related SDK changes have been pushed. I will push the remaining updates to the same PR shortly.

@indurireddy-TF indurireddy-TF self-assigned this Jul 21, 2026
@github-actions github-actions Bot added kind/cleanup Refactoring code, fixing up documentation, etc size/XL labels Jul 21, 2026
@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot

Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 276b0437-8da5-4991-80bc-4748dcb9bbac

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@indurireddy-TF

indurireddy-TF commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator Author

The Go-related SDK changes are now complete. The test failures are due to the fact that updates for the other SDKs (C#, Rust, etc.) have not yet been pushed. I will upload these remaining changes by the end of the day.

@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@indurireddy-TF
indurireddy-TF force-pushed the remove-player-tracking-alpha-feature branch from 06d32e1 to 297a558 Compare July 27, 2026 06:35
@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions

Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@markmandel

markmandel commented Jul 28, 2026

Copy link
Copy Markdown
Member

Is it possible to split this into a series of smaller PRs -- this is massssssive!?!

I'm seeing some potential issues, but this size makes it very hard to review, and I'm worried I'll miss other things.

@indurireddy-TF

indurireddy-TF commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator Author

Is it possible to split this into a series of smaller PRs -- this is massssssive!?!

I'm seeing some potential issues, but this size makes it very hard to review, and I'm worried I'll miss other things.

Sorry, I got a bit carried away and this became massssssive.Sure I’ll try to split this into smaller PRs and make sure each one is clean without build issues.

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

Labels

kind/cleanup Refactoring code, fixing up documentation, etc size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal: Remove PlayerTracking Alpha Feature

3 participants