-
Notifications
You must be signed in to change notification settings - Fork 858
Feat: implement rust sdk (counter and list) #4247
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Feat: implement rust sdk (counter and list) #4247
Conversation
if feature_gates.contains("CountsAndLists=true") { | ||
run_counts_and_lists_features(sdk.beta().clone())?; | ||
} |
There was a problem hiding this comment.
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)
@Jake-Shadle @MiniaczQ |
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. |
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:
|
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. |
/gcbrun |
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:
|
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:
|
/gcbrun |
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. |
Gonna double check the logs in GCP, but might be the issue fixed by this PR: #4224 /gcbrun |
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. |
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:
|
/gcbrun |
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. |
And now the unit tests are failing on metrics randomely 😭 /gcbrun |
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:
|
/gcbrun |
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:
|
/gcbrun |
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:
|
/gcbrun |
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:
|
I'll create another issue + PR with the fix of the flaky test around the |
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. |
What type of PR is this?
/kind feature
What this PR does / Why we need it:
Which issue(s) this PR fixes:
Closes #3644
Special notes for your reviewer:
I'm new to rust, this will need a proper review 😄