Skip to content

Conversation

@bergerhoffer
Copy link
Contributor

@bergerhoffer bergerhoffer commented Apr 13, 2020

@bergerhoffer bergerhoffer added this to the Future Release milestone Apr 13, 2020
@bergerhoffer bergerhoffer changed the title OSDOCS-876: Adding docs for bound service account tokens [WIP] OSDOCS-876: Adding docs for bound service account tokens Apr 13, 2020
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 13, 2020
@openshift-docs-preview-bot

The preview will be available shortly at:

Copy link

@marun marun left a comment

Choose a reason for hiding this comment

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

LGTM save a few suggested changes inline! Most comments are just responding to questions in the text.

@bergerhoffer bergerhoffer changed the title [WIP] OSDOCS-876: Adding docs for bound service account tokens OSDOCS-876: Adding docs for bound service account tokens Apr 15, 2020
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 15, 2020
@bergerhoffer bergerhoffer added the peer-review-needed Signifies that the peer review team needs to review this PR label Apr 15, 2020
@bergerhoffer
Copy link
Contributor Author

@openshift/team-documentation For peer review please.

Copy link

@marun marun left a comment

Choose a reason for hiding this comment

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

Just a couple more!

@bmcelvee bmcelvee 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 15, 2020
@bmcelvee
Copy link
Contributor

Just a couple of suggestions. Otherwise LGTM!

@bergerhoffer
Copy link
Contributor Author

@marun
Copy link

marun commented Apr 15, 2020

LGTM

@xingxingxia
Copy link
Contributor

xingxingxia commented Apr 16, 2020

LGTM. CC @wangke19 (just for knowing this doc :), no review required). Thanks.

@bergerhoffer bergerhoffer merged commit c0c35a3 into openshift:master Apr 16, 2020
@bergerhoffer
Copy link
Contributor Author

/cherrypick enterprise-4.4

@openshift-cherrypick-robot

@bergerhoffer: new pull request created: #21224

Details

In response to this:

/cherrypick enterprise-4.4

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

Labels

branch/enterprise-4.4 peer-review-done Signifies that the peer review team has reviewed this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants