build: update golang dependencies across all go.mod files - #4659
build: update golang dependencies across all go.mod files#4659Ribhu-bhardwaj wants to merge 3 commits into
Conversation
Signed-off-by: Ribhu-bhardwaj <bhardwajribhu@gmail.com>
|
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. |
|
/gcbrun |
|
Looks like this needs a |
|
I should learn to check CI first before approving things :D |
|
mark i dont understand some parts of this . Basically it was a file mismatch right between what was expected and what was in the code. If this is the case I'm sorry and i will try to fix it! |
You got it! We can improve that error too in CI. |
Signed-off-by: ribs <143743519+Ribhu-bhardwaj@users.noreply.github.com>
|
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. |
|
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. |
|
/gcbrun |
|
Hey mark it appears to be failing still -I feel 13 minutes isnt that bad and i dont have access to the logs so cant figure out the tweaks needed. |
|
Our github bot is currently broken (secret needs a rotation), but to get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Ah this isn't your fault, this is the same CI issue we have that's solved in #4669 - it blocked CI. |
|
This should be better now, but we did just do a huge dependency update with the K8s upgrade - can you fix the conflicts, and we can try again? |
2 participants
What type of PR is this?
What this PR does / Why we need it: This is the PR2 for #4620
Which issue(s) this PR fixes: dependency upgrades.
Closes #4620
Did you use AI tools in preparing this PR?:
Y
Special notes for your reviewer: