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

[ops] meta: Add "version" graph #6206

Merged
merged 1 commit into from
Oct 18, 2021
Merged

[ops] meta: Add "version" graph #6206

merged 1 commit into from
Oct 18, 2021

Conversation

geropl
Copy link
Member

@geropl geropl commented Oct 14, 2021

Description

Adds a simple graph for "version" as reported from server:
image

Note that cluster is not set in devstaging, but is in prod/staging, thus the label is a tad off. Cmp. prod:
image

Query: sum (gitpod_version_info{}) by (cluster, gitpod_version)

Related Issue(s)

Fixes gitpod-io/observability#270.

How to test

Release Notes

NONE

Documentation

  • /werft with-observability
  • /werft with-clean-slate-deployment

@geropl
Copy link
Member Author

geropl commented Oct 14, 2021

/werft run

👍 started the job as gitpod-build-gpl-ops-270-version-graph.7

Make the build green (would work before, but timed out on ingress 🙄 )

@JanKoehnlein
Copy link
Contributor

/lgtm

@roboquat
Copy link
Contributor

LGTM label has been added.

Git tree hash: 70e6256a847e5bb46cd149434c50a9e2961c4280

@ArthurSens
Copy link
Contributor

ArthurSens commented Oct 14, 2021

/approve
/werft run

👍 started the job as gitpod-build-gpl-ops-270-version-graph.8

@ArthurSens
Copy link
Contributor

ArthurSens commented Oct 14, 2021

Just noticed the merge conflict 😬

/approve cancel

Does it make sense to add this panel to the Server dashboard? I know that this is exposed by server, but maybe making a new dashboard "Meta Overview" would make more sense?

Not sure what to put in there besides the running version. Maybe also add each components version as well? Maybe some database metrics? 🤷 Just food for thought

@roboquat
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JanKoehnlein

Associated issue: #270

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

@roboquat
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@geropl
Copy link
Member Author

geropl commented Oct 18, 2021

/werft run

👍 started the job as gitpod-build-gpl-ops-270-version-graph.11

@geropl geropl added the lgtm label Oct 18, 2021
@roboquat roboquat merged commit e1741df into main Oct 18, 2021
@roboquat roboquat deleted the gpl/ops-270-version-graph branch October 18, 2021 08:39
@laushinka
Copy link
Contributor

maybe making a new dashboard "Meta Overview" would make more sense?

@ArthurSens Yes! That is planned here #6126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants