Skip to content

Conversation

lacroixthomas
Copy link
Collaborator

@lacroixthomas lacroixthomas commented Aug 13, 2025

What type of PR is this?
/kind feature

What this PR does / Why we need it:

  • Add unit test on the rust sdk (code + Makefile)
  • Integrate the beta for the rust sdk
  • Add conformance test for counter and list (beta)

Which issue(s) this PR fixes:

Closes #3644

Special notes for your reviewer:
I'm new to rust, this will need a proper review 😄

@github-actions github-actions bot added kind/feature New features for Agones size/L labels Aug 13, 2025
@lacroixthomas lacroixthomas marked this pull request as ready for review August 13, 2025 23:19
Comment on lines +187 to +189
if feature_gates.contains("CountsAndLists=true") {
run_counts_and_lists_features(sdk.beta().clone())?;
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It was like that for the profile part, but I wonder if we can retrieve that in a different way, it feels hacky to have an hardcoded string for the feature gates (same for the async)

@lacroixthomas
Copy link
Collaborator Author

@Jake-Shadle @MiniaczQ
Sorry about that, I've seen that you were tagged on the ticket and that you have rust experiences ?
If you have some time for review, I'm new to rust, so don't hesitate if you see anything wrong 😄

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: 22c397bf-174e-441e-8a13-c04c1bf43021

Status: TIMEOUT

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

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: a6f71218-b5cc-4b89-9130-fdd23b6c44aa

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-2447197

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: a6b6abc8-f2e4-42e8-85a0-840d7b4b61e3

Status: FAILURE

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

@lacroixthomas
Copy link
Collaborator Author

/gcbrun

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 1a429709-cec1-48c4-b8f8-b63c20b85c5b

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-ac1da39

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 8620f7b4-b29a-4871-aaf2-42c7a2497ea3

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-47905c7

@lacroixthomas
Copy link
Collaborator Author

/gcbrun

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: b77f2050-fce7-4757-82d5-1f1783dba06b

Status: FAILURE

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

@lacroixthomas
Copy link
Collaborator Author

Gonna double check the logs in GCP, but might be the issue fixed by this PR: #4224

/gcbrun

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: fefdd5ad-177b-4c5e-ac06-15a86ffe2410

Status: FAILURE

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

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: ecf81ab0-ed5b-476d-92f8-e1b01482e12a

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-71a1568

@lacroixthomas
Copy link
Collaborator Author

/gcbrun

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: 61f96e87-14d9-4932-b6c7-055b0f1627a8

Status: FAILURE

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

@lacroixthomas
Copy link
Collaborator Author

And now the unit tests are failing on metrics randomely 😭

/gcbrun

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 4a16bd5f-da93-47fd-a555-51ca7dd672e6

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-71a1568

@lacroixthomas
Copy link
Collaborator Author

/gcbrun

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 8b3a4eff-e98c-4104-afb3-cf6631f588cc

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-71a1568

@lacroixthomas
Copy link
Collaborator Author

/gcbrun

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: befe5480-f030-46d1-a261-cb9012171359

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-71a1568

@lacroixthomas
Copy link
Collaborator Author

/gcbrun

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 42d5f4c2-13d1-4694-809f-1c362c5b37c3

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/4247/head:pr_4247 && git checkout pr_4247
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.52.0-dev-71a1568

@lacroixthomas
Copy link
Collaborator Author

I'll create another issue + PR with the fix of the flaky test around the Step 25: submit-upgrade-test-cloud-build, it seems to be working well !
Otherwise it's ready for review, as I was saying on the description, I'm still new to rust (only theory learning so far, it's my first rust contribution ever), don't hesitate if there is anything that doesn't seems correct 😄
I'm more than happy to have feedbacks

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: 1403ae8d-b6fc-4c9d-b201-e7498e4720a1

Status: FAILURE

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New features for Agones size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rust CountsAndLists SDK implementation and conformance tests
2 participants