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

Adding Fleet Active GameServerSet Percentage Metrics #4021

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

0xaravindh
Copy link
Contributor

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug

/kind cleanup

/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

Which issue(s) this PR fixes: #3663

Closes #3663

Special notes for your reviewer:

@github-actions github-actions bot added kind/cleanup Refactoring code, fixing up documentation, etc size/M labels Oct 23, 2024
@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: cfa78591-f21a-4ef7-b788-64edd441876f

Status: FAILURE

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

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: a4f0265c-59e0-41a5-a4cd-ab29381d9b0c

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/googleforgames/agones.git pull/4021/head:pr_4021 && git checkout pr_4021
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.45.0-dev-000872c

@peterzhongyi
Copy link
Collaborator

peterzhongyi commented Oct 23, 2024

How should I verify that the newly added metric is working as intended? Is there some test that covers it, or can I perform some series of actions to verify the metric is working in the dashboard? If so, can you add to the description of the pr?

@0xaravindh
Copy link
Contributor Author

@peterzhongyi For additional guidance, refer to PR #3663, where Mark provided helpful suggestions that I have followed closely.

@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 126b88be-5d05-44c5-a174-bacf455d4d48

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/googleforgames/agones.git pull/4021/head:pr_4021 && git checkout pr_4021
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.45.0-dev-0a00781

@gongmax
Copy link
Collaborator

gongmax commented Nov 6, 2024

@0xaravindh would you like to test your change by following https://agones.dev/site/docs/guides/metrics/#installation to install prometheus and grafana locally and see whether the new added metrics shows up?

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: e9783c89-b541-41e5-be81-aca7e55b611c

Status: FAILURE

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

@0xaravindh
Copy link
Contributor Author

0xaravindh commented Nov 11, 2024

@0xaravindh would you like to test your change by following https://agones.dev/site/docs/guides/metrics/#installation to install prometheus and grafana locally and see whether the new added metrics shows up?

@gongmax I’ve tested it by following the guide to install Prometheus and Grafana locally, and the new metrics are showing up correctly in gs dashboard. do you think stats are looking enough? or should I include any additional data in that field?

72WA6kVaV8wk4D2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Refactoring code, fixing up documentation, etc size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding GameServerSet Metric
4 participants