Skip to content

Conversation

@bergerhoffer
Copy link
Contributor

@bergerhoffer bergerhoffer commented Aug 20, 2020

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 20, 2020
@bergerhoffer bergerhoffer added branch/enterprise-4.6 peer-review-needed Signifies that the peer review team needs to review this PR labels Aug 20, 2020
@bergerhoffer bergerhoffer added this to the Future Release milestone Aug 20, 2020
@bergerhoffer
Copy link
Contributor Author

Copy link
Contributor

Choose a reason for hiding this comment

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

@bergerhoffer, this is implemented by a CRD:

From oc api-resources:

podnetworkconnectivitychecks                           controlplane.operator.openshift.io    true         PodNetworkConnectivityCheck

So as of today at least, wouldn't this not use back ticks and s/resources/custom resources/?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jboxman technically yes, but I tend to just use "resource" because I don't think the distinction is important.

@mburke5678
Copy link
Contributor

One nit. LGTM

@bergerhoffer
Copy link
Contributor Author

@xingxingxia Are you the right person to review this? I wasn't sure since https://issues.redhat.com/browse/MSTR-1008 doesn't have a QE assignee set. Thanks!

@xingxingxia
Copy link
Contributor

/lgtm
Filed 1870922 . It is no-qe and I also agree it is considered no-qe, so didn't set QE assignee to this Alpha feature. Let me set to @wangke19 now, though.

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 21, 2020
@bergerhoffer bergerhoffer merged commit 9cdfaae into openshift:enterprise-4.6 Aug 24, 2020
@bergerhoffer bergerhoffer deleted the OSDOCS-1208 branch October 29, 2020 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.6 lgtm Indicates that a PR is ready to be merged. peer-review-needed Signifies that the peer review team needs to review this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants