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

fix: splitting the Collecting Dev Workspace Operator metrics procedure into upstream and downstream versions #2599

Merged
merged 3 commits into from
May 15, 2023

Conversation

max-cx
Copy link
Contributor

@max-cx max-cx commented May 15, 2023

What does this pull request change?

This PR clones #2595, which had an issue with https://accounts.eclipse.org/user/login?destination=legal/eca/validation/152502.

Step 2 of the procedure is kept in the Che docs but removed from the Dev Spaces docs:
https://www.eclipse.org/che/docs/stable/administration-guide/monitoring-the-dev-workspace-operator/#proc_collecting-dev-workspace-operator-metrics-with-prometheus

What issues does this pull request fix or reference?

This is a follow-up to #2586 as requested by @dkwon17

Specify the version of the product this pull request applies to

Merge to main
Cherry-pick to 7.62.x, 7.63.x, and 7.64.x

Pull Request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

@max-cx max-cx requested review from rkratky, deerskindoll and a team as code owners May 15, 2023 14:43
@max-cx max-cx requested a review from amisevsk May 15, 2023 14:43
@github-actions
Copy link

github-actions bot commented May 15, 2023

Click here to review and test in web IDE: Contribute

@github-actions
Copy link

github-actions bot commented May 15, 2023

🎊 Navigate the preview: https://646246bd88be352ba758eab4--eclipse-che-docs-pr.netlify.app 🎊

Copy link
Contributor

@deerskindoll deerskindoll left a comment

Choose a reason for hiding this comment

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

a tiny suggestion

Copy link
Contributor

@deerskindoll deerskindoll left a comment

Choose a reason for hiding this comment

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

lgtm

@max-cx max-cx merged commit f7c9b77 into eclipse-che:main May 15, 2023
max-cx added a commit that referenced this pull request May 15, 2023
…e into upstream and downstream versions (#2599)

* typo (#2582)

* Revert "typo (#2582)"

This reverts commit 01a4cb0.

* fix: replicating changes from PR 2595 due to a committer's signature issue

---------

Co-authored-by: Jose Manuel <[email protected]>
max-cx added a commit that referenced this pull request May 15, 2023
…e into upstream and downstream versions (#2599)

* typo (#2582)

* Revert "typo (#2582)"

This reverts commit 01a4cb0.

* fix: replicating changes from PR 2595 due to a committer's signature issue

---------

Co-authored-by: Jose Manuel <[email protected]>
max-cx added a commit that referenced this pull request May 15, 2023
…e into upstream and downstream versions (#2599)

* typo (#2582)

* Revert "typo (#2582)"

This reverts commit 01a4cb0.

* fix: replicating changes from PR 2595 due to a committer's signature issue

---------

Co-authored-by: Jose Manuel <[email protected]>
@max-cx max-cx deleted the monitoringsplit2 branch May 15, 2023 15:44
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