Feat: implement the processor server#4263
Feat: implement the processor server#4263lacroixthomas wants to merge 23 commits intogoogleforgames:mainfrom
Conversation
|
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. |
1 similar comment
|
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. |
|
Build Failed 😭 Build Id: 0dfb8b31-e31f-4756-9245-fc6a5ef6c6fe Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: be7478b2-5c1c-498c-8498-6d3e35aaedb5 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: |
|
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. |
1 similar comment
|
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. |
|
Build Failed 😭 Build Id: 91746f34-8ce0-429a-b492-139ab1d0e898 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: e64c3f64-8657-4721-8904-83a4c840bd47 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: 1241bbbd-7bde-46ec-95ec-3b8712be910f 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: |
6351400 to
c239fdb
Compare
|
Build Failed 😭 Build Id: 97a4c8b6-9fdc-444b-8694-653b14e80797 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Build Failed 😭 Build Id: 04dff753-1c63-4882-bb56-0f2ec1158e07 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Build Failed 😭 Build Id: 4aa098c0-e599-4386-8bd8-7f51ecf4034f Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
56cf40e to
de22bdc
Compare
|
Build Failed 😭 Build Id: 0e0967ca-55e6-4012-9f36-c8c00cf9108c Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
I'll come back to this task in the next days / week, didn't forget about it ! |
de22bdc to
3078f81
Compare
|
Build Failed 😭 Build Id: af923d1a-67cc-48c6-9039-634cda0feb28 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
3078f81 to
ddd9a28
Compare
|
Build Failed 😭 Build Id: 1c732008-568e-45a8-9455-40c393278ecd Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
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. |
|
Build Failed 😭 Build Id: 792e8178-8cf5-4479-b4f8-1f6dbd845439 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
87c55c4 to
2518c80
Compare
|
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. |
2ac6bcd to
2c56805
Compare
|
Build Succeeded 🥳 Build Id: ae481ca3-0326-495d-96ef-c576b0b897b1 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: 251769c1-44a6-4da2-8412-8fb482ce4b35 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'm waiting for the e2e to finish and I'll rollback the feature gate from alpha to dev, it's just to confirm that the e2e still works with the new processor (which seems to be alright now !) It's still in draft because I want to write some unit tests around the handler.go from the processor server, only added you as reviewers, if you want to have a look before the unit tests are done 😄 I'm gonna create an other PR to add new metrics |
|
Build Failed 😭 Build Id: 27499b51-b73c-4d32-88b4-125e998dc67b Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Build Succeeded 🥳 Build Id: a47da26c-d7ac-49b4-b2b6-9d931adfffd7 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: |
|
Note: Ensure that this issue is also part of this new design: #4415 |
|
Build Failed 😭 Build Id: b641d47d-5fda-403b-b3be-7a82fb901854 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:
Implementation of the processor server behind dev feature flag
TODO (some might be on another MR):
Which issue(s) this PR fixes:
Part of #4190
Special notes for your reviewer:
Opening the draft for early feedbacks