Skip to content
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

Extra Debugging for TestGameServerReserve #1334

Merged

Conversation

markmandel
Copy link
Member

@markmandel markmandel commented Feb 11, 2020

Adds new WaitForGameServerStateWithLogger() function that makes it easier to track debug output for state checking for specific tests.

Also dumped out event data for the pod and gameserver on this test to see what else we can see.

@markmandel markmandel added kind/bug These are bugs. area/tests Unit tests, e2e tests, anything to make sure things don't break labels Feb 11, 2020
@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 14057ea8-eaae-42a7-879b-80d41d8c673a

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

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: b9d86cb6-3903-430f-a49f-1786f1a86801

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-3400ebf

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: a005efc0-a999-48e1-a7e4-c0c6bb6ee56a

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-05d433d

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 2f53a152-b9fb-4f00-b85e-9218d2da049b

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

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 67dcfe8f-ee39-400e-86d7-c66198a1703d

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-e15ce6a

@markmandel markmandel added the feature-freeze-do-not-merge Only eligible to be merged once we are out of feature freeze (next full release) label Feb 26, 2020
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 1ea95318-d7ec-45e6-9820-c49fc08315be

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-5555eac

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 74d4295f-55d7-49ac-94d0-fd690a18d797

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-a93680b

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 4e39623a-92e2-4a0d-b8f2-aac8abae6df3

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-a65bb72

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 40c20f0c-68b1-497b-8687-067733c7ecd6

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-a65bb72

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: b1bd92dc-8863-4d11-997a-c81655314047

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

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: a87a7e76-5360-4b7f-97bb-5023e3b1dfab

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-58a83a6

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 6653a3e8-f92e-4209-b91a-8f48e9a9fbe5

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-58a83a6

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: a45005c0-1658-4c7e-af69-447bc5b1ab4b

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-58a83a6

@markmandel markmandel removed the feature-freeze-do-not-merge Only eligible to be merged once we are out of feature freeze (next full release) label Mar 4, 2020
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: f73cc997-4c64-444a-b147-a5ec78bd38c6

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-28b3bb9

Adds new WaitForGameServerStateWithLogger() function that makes it
easier to track debug output for state checking for specific tests.

Also dumped out event data for the pod and gameserver on this test to
see what else we can see.
@markmandel markmandel changed the title Fix for Flaky TestGameServerReserve Extra Debugging for TestGameServerReserve Mar 5, 2020
@markmandel
Copy link
Member Author

markmandel commented Mar 5, 2020

Marking this as Ready for review (a) to try and get more data on why this is failing once it's been merged and (b) there are some useful tools in here that might be valuable to others.

@markmandel markmandel marked this pull request as ready for review March 5, 2020 18:58
@aLekSer aLekSer self-requested a review March 5, 2020 19:10
Copy link
Collaborator

@aLekSer aLekSer left a comment

Choose a reason for hiding this comment

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

Not sure why go.sum with klog is there. But overall we need to know why E2E tests crash.
go.sum was probably the main difference with k8s.io/apimachinery root.

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aLekSer, markmandel

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 44aedfd1-62c7-4106-beff-22cdb7059dc2

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/GoogleCloudPlatform/agones.git pull/1334/head:pr_1334 && git checkout pr_1334
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.5.0-cc29939

@markmandel markmandel added this to the 1.5.0 milestone Mar 5, 2020
@markmandel markmandel merged commit 53fafa9 into googleforgames:master Mar 5, 2020
@markmandel markmandel deleted the flaky/TestGameServerReserve branch March 5, 2020 19:23
ilkercelikyilmaz pushed a commit to ilkercelikyilmaz/agones that referenced this pull request Oct 23, 2020
Adds new WaitForGameServerStateWithLogger() function that makes it
easier to track debug output for state checking for specific tests.

Also dumped out event data for the pod and gameserver on this test to
see what else we can see.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved area/tests Unit tests, e2e tests, anything to make sure things don't break kind/bug These are bugs. lgtm size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants