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

Updated log levels in pkg #1359

Merged
merged 3 commits into from
Mar 5, 2020
Merged

Conversation

akremsa
Copy link
Contributor

@akremsa akremsa commented Feb 24, 2020

This PR can be considered as a part of #1223

Updated log levels in pkg package.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 2afcd540-bf92-4442-b2df-b1c124562a03

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-966676d

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 787d674b-d4aa-4d8e-9b6c-9977ecb68b43

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

@akremsa akremsa force-pushed the log-levels branch 2 times, most recently from 744fdd3 to 1b7a331 Compare February 25, 2020 09:55
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 93e7d229-ff62-42a3-9027-0f0f92e5e35c

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-744fdd3

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 16cc8cd4-191e-40ec-b1b6-0602b14ba262

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-1b7a331

pkg/sdkserver/localsdk.go Outdated Show resolved Hide resolved
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 800f6752-3751-4c1b-8b3a-934929d7afa9

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-76321a0

CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Member

@markmandel markmandel left a comment

Choose a reason for hiding this comment

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

Outside of moving the CONTRIBUTING.md change to a new PR, and @aLekSer 's nit, this looks good to go for me. Once those changes are in, I'm happy to approve.

@markmandel markmandel added area/operations Installation, updating, metrics etc feature-freeze-do-not-merge Only eligible to be merged once we are out of feature freeze (next full release) kind/cleanup Refactoring code, fixing up documentation, etc labels Feb 26, 2020
@akremsa akremsa changed the title Updated log levels in sdkserver Updated log levels in pkg Feb 27, 2020
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 4a77a221-bfe5-4a40-871e-34b01fb856b5

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-f656eaf

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 055ae952-47c6-4880-85c0-f56845930328

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

@akremsa
Copy link
Contributor Author

akremsa commented Feb 27, 2020

@markmandel thanks for the review. I've removed changes from the md file and added other log levels updates to this PR because all of them are related to the same package - pkg, and I think it's better to keep everything in a single PR.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: b192871b-b93c-423a-bfa6-589ad0cf3f5e

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

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 328bb4f6-b919-434b-8e9a-c3991a6acb1a

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

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: d0b59459-2f8e-4966-97f6-0698b811fc17

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

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 3a65be0d-fc0e-4f4d-887e-ea45013fd0f5

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-378f9a9

@akremsa
Copy link
Contributor Author

akremsa commented Mar 3, 2020

/assign @markmandel

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 90467466-57d5-47c1-8373-4d57a746824d

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.4.0-97d79f4

@aLekSer
Copy link
Collaborator

aLekSer commented Mar 4, 2020

@markmandel I think we could merge this PR before the release. It would significantly reduce logging in SDK server with Info level used by default.
There could be a plenty of GameServers and so number of logs would be reduced by Replicas count in all Fleets.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 526ce469-1338-49a9-aa23-f6248215a12a

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

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 4b3aad55-9344-4d93-b25e-5a4033834f15

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

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: ba29a176-537a-45d7-9f46-5adfb9eac803

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

@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 Failed 😱

Build Id: 2dc544ba-462c-4824-b31b-9196955f4389

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

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: akremsa, aLekSer
To complete the pull request process, please assign markmandel
You can assign the PR to them by writing /assign @markmandel in a comment when ready.

The full list of commands accepted by this bot can be found 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

@google-oss-robot
Copy link

New changes are detected. LGTM label has been removed.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 30e2b327-041b-46e5-b2ee-b65f2ceec2f2

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/1359/head:pr_1359 && git checkout pr_1359
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.5.0-2aae596

@markmandel markmandel merged commit cf3d21d into googleforgames:master Mar 5, 2020
@markmandel markmandel added this to the 1.5.0 milestone Mar 5, 2020
ilkercelikyilmaz pushed a commit to ilkercelikyilmaz/agones that referenced this pull request Oct 23, 2020
* updated log levels in sdkserver
* updated log levels in pkg

Co-authored-by: Mark Mandel <markmandel@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operations Installation, updating, metrics etc kind/cleanup Refactoring code, fixing up documentation, etc size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants