Skip to content

[release-4.10] Use new podinfo command to gather pod data #908

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
Closed
openshift-cherrypick-robot wants to merge 1 commit into from
Closed

[release-4.10] Use new podinfo command to gather pod data #908

openshift-cherrypick-robot wants to merge 1 commit into from

Conversation

@openshift-cherrypick-robot
Copy link

@openshift-cherrypick-robot openshift-cherrypick-robot commented Jun 21, 2022

This is an automated cherry-pick of #907

/assign jlojosnegros

@jlojosnegros
Use new podinfo command to gather pod data
c502080 
From v1.7 `debug-tools` has a new `podinfo` command to help us to gather
info from pods to complement PodResourcesAPI info.
The information gathered is the same that was gotten with the current
`oc` command.
@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Jun 21, 2022
Merged
@openshift-ci openshift-ci bot requested review from fedepaol and jlojosnegros June 21, 2022 10:06
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 21, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: openshift-cherrypick-robot
To complete the pull request process, please assign fromanirh after the PR has been reviewed.
You can assign the PR to them by writing /assign @fromanirh in a comment when ready.

The full list of commands accepted by this bot can be found 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

@coveralls
Copy link

coveralls commented Jun 21, 2022

Pull Request Test Coverage Report for Build 2518

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 60.703%
Totals Coverage Status
Change from base Build 2508: 0.0%
Covered Lines: 1588
Relevant Lines: 2616
💛 - Coveralls

@jlojosnegros
Copy link
Contributor

jlojosnegros commented Jun 22, 2022

/hold

@openshift-ci openshift-ci bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jun 22, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 24, 2022

@openshift-cherrypick-robot: PR needs rebase.

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.

@jlojosnegros
Copy link
Contributor

jlojosnegros commented Jun 28, 2022

/close
Already done by #914

@openshift-ci openshift-ci bot closed this Jun 28, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 28, 2022

@jlojosnegros: Closed this PR.

In response to this:

/close
Already done by #914

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.

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

Reviewers

@fedepaol fedepaol Awaiting requested review from fedepaol

@jlojosnegros jlojosnegros Awaiting requested review from jlojosnegros

Assignees

@jlojosnegros jlojosnegros

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@openshift-cherrypick-robot @coveralls @jlojosnegros