Skip to content

LOG-1975: return back metrics service for CLO #1272

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 3, 2021
Merged

LOG-1975: return back metrics service for CLO #1272

merged 1 commit into from
Dec 3, 2021

Conversation

vparfonov
Copy link
Contributor

@vparfonov vparfonov commented Nov 30, 2021 •
edited
Loading

Signed-off-by: Vitalii Parfonov vparfono@redhat.com

Description

/cc
/assign

/cherry-pick

Links

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 30, 2021

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 30, 2021
@openshift-ci openshift-ci bot added the midstream/Dockerfile A Dockerfile.in sync is needed with midstream label Nov 30, 2021
@vparfonov vparfonov force-pushed the log1975 branch 2 times, most recently from 71bd97c to 932fc5c Compare November 30, 2021 21:58
@jcantrill
Copy link
Contributor

/refresh

@vparfonov vparfonov force-pushed the log1975 branch 2 times, most recently from d6910bd to 1d37b83 Compare December 2, 2021 18:16
@vparfonov vparfonov marked this pull request as ready for review December 2, 2021 18:18
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 2, 2021
@vparfonov vparfonov force-pushed the log1975 branch 2 times, most recently from 911a16e to 5d8b13e Compare December 2, 2021 18:27
jcantrill
jcantrill reviewed Dec 2, 2021
View reviewed changes
@vparfonov
Copy link
Contributor Author

/test functional

@vparfonov
LOG-1975: Return back Service Metrics
a0f4e73 
Signed-off-by: Vitalii Parfonov <vparfono@redhat.com>
@vparfonov vparfonov requested a review from jcantrill December 3, 2021 16:47
@jcantrill
Copy link
Contributor

/approve
/lgtm
/hold cancel

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 3, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jcantrill, vparfonov

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

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Dec 3, 2021
@openshift-merge-robot openshift-merge-robot merged commit 18c005d into openshift:master Dec 3, 2021
@vparfonov vparfonov deleted the log1975 branch December 3, 2021 23:44
@vparfonov
Copy link
Contributor Author

/cherry-pick release-5.3

@openshift-cherrypick-robot
Copy link

@vparfonov: new pull request created: #1276

In response to this:

/cherry-pick release-5.3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Dec 3, 2021
Closed
@vparfonov vparfonov mentioned this pull request Dec 6, 2021
Closed
pmoogi-redhat pushed a commit to pmoogi-redhat/cluster-logging-operator that referenced this pull request Apr 26, 2022
@openshift-merge-robot
Merge pull request openshift#1272 from vparfonov/log1975
6132628 
LOG-1975: return back metrics service for CLO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pmoogi-redhat pmoogi-redhat Awaiting requested review from pmoogi-redhat

@syedriko syedriko Awaiting requested review from syedriko

@jcantrill jcantrill Awaiting requested review from jcantrill

Assignees

@jcantrill jcantrill

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. midstream/Dockerfile A Dockerfile.in sync is needed with midstream
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vparfonov @jcantrill @openshift-cherrypick-robot @openshift-merge-robot