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

Removed reflector metric usage #4056

Merged
merged 7 commits into from
Dec 18, 2024

Conversation

vicentefb
Copy link
Collaborator

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:
Closes #3540

Special notes for your reviewer:

@github-actions github-actions bot added kind/cleanup Refactoring code, fixing up documentation, etc size/S labels Dec 4, 2024
@github-actions github-actions bot added the size/L label Dec 4, 2024
@markmandel markmandel added the kind/breaking Breaking change label Dec 4, 2024
@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 16a19f7e-22f1-4611-8795-2b26d98e49e5

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/4056/head:pr_4056 && git checkout pr_4056
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.46.0-dev-a9910c9

@0xaravindh
Copy link
Member

@vicentefb

We have to remove docs as well.

| agones_k8s_client_cache_list_total | The total number of list operations for client-go caches | counter |
| agones_k8s_client_cache_list_duration_seconds | Duration of a Kubernetes list API call in seconds | histogram |
| agones_k8s_client_cache_list_items | Count of items in a list from the Kubernetes API | histogram |
| agones_k8s_client_cache_watches_total | The total number of watch operations for client-go caches | counter |
| agones_k8s_client_cache_last_resource_version | Last resource version from the Kubernetes API | gauge |

- {{< ghlink href="/build/grafana/dashboard-goclient-caches.yaml" branch="main" >}}Agones Controller go-client caches{{< /ghlink >}} displays Agones Controller Kubernetes Watches/Lists operations used.

@gongmax gongmax requested a review from 0xaravindh December 4, 2024 17:09
@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: e5173ba4-85fb-4d78-ae8b-c3bb68f0db0d

Status: FAILURE

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

@0xaravindh 0xaravindh changed the title removed reflector metric usage Removed reflector metric usage Dec 5, 2024
@vicentefb vicentefb requested a review from 0xaravindh December 9, 2024 23:14
@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: efc8fc6f-0007-4780-a209-6ef6a2ac8e64

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: b4f8ac77-2b57-4d59-b429-f63d06c4ad10

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/4056/head:pr_4056 && git checkout pr_4056
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.46.0-dev-8e63db8

@vicentefb
Copy link
Collaborator Author

Missing to test locally

@0xaravindh
Copy link
Member

LGTM

@vicentefb vicentefb enabled auto-merge (squash) December 17, 2024 18:56
Copy link
Collaborator

@gongmax gongmax left a comment

Choose a reason for hiding this comment

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

Did you test locally to ensure nothing is broken?

@vicentefb vicentefb disabled auto-merge December 17, 2024 19:14
@vicentefb
Copy link
Collaborator Author

Oops not yet, i just meant to update the branch not merge it !

@agones-bot
Copy link
Collaborator

Build Failed 😭

Build Id: d172f73b-57d0-4dc7-9db1-c12548709808

Status: FAILURE

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

@vicentefb
Copy link
Collaborator Author

Yes these changes are working as expected and the dashboard looks good.

@igooch
Copy link
Collaborator

igooch commented Dec 18, 2024

/gcbrun

@igooch igooch enabled auto-merge (squash) December 18, 2024 00:54
@agones-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 04424d2a-2ea1-4136-84a0-8557dd0c5a21

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/4056/head:pr_4056 && git checkout pr_4056
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.46.0-dev-029f62f

@igooch igooch merged commit 8166704 into googleforgames:main Dec 18, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/breaking Breaking change kind/cleanup Refactoring code, fixing up documentation, etc size/L size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing agones_k8s_client_cache_* metrics
6 participants