Skip to content

Conversation

@bergerhoffer
Copy link
Contributor

@bergerhoffer bergerhoffer commented Apr 14, 2021

@bergerhoffer bergerhoffer added peer-review-needed Signifies that the peer review team needs to review this PR branch/enterprise-4.8 labels Apr 14, 2021
@bergerhoffer bergerhoffer added this to the Future Release milestone Apr 14, 2021
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 14, 2021
@netlify
Copy link

netlify bot commented Apr 14, 2021

Deploy preview for osdocs ready!

Built with commit e762125

https://deploy-preview-31583--osdocs.netlify.app

@jboxman
Copy link
Contributor

jboxman commented Apr 14, 2021

@bergerhoffer LGTM

@jboxman jboxman added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Apr 14, 2021
@bergerhoffer
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bergerhoffer not sure if the description for this is apt, as below is the info we see on prometheus for the descheduler_build_info, which does not contain anything about descheduler build information.

descheduler_build_info{GoVersion="go1.16.1", endpoint="https", instance="10.129.3.143:10258", job="metrics", namespace="openshift-kube-descheduler-operator", pod="cluster-6c89d56fdd-5qzrq", service="metrics"}

Copy link
Member

Choose a reason for hiding this comment

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

It depends on how the descheduler operand was actually built. Checking https://github.com/openshift/descheduler/blob/master/images/descheduler/Dockerfile.rhel7#L4 we don't set the right LDFLAGS. Lemme fix that. Though, the description is still valid.

Copy link
Member

Choose a reason for hiding this comment

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

openshift/descheduler#59

@kasturinarra thanks for noticing that.

@bergerhoffer bergerhoffer merged commit 2de1e63 into openshift:enterprise-4.8 Apr 21, 2021
@openshift-cherrypick-robot

@bergerhoffer: base branch (enterprise-4.8) needs to differ from target branch (enterprise-4.8)

In response to this:

/cherrypick enterprise-4.8

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.

@bergerhoffer bergerhoffer deleted the OSDOCS-2095 branch August 3, 2022 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.8 peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants