Bug 1383690 - [DOCS] Documentation on "oc patch" is missing
Summary: [DOCS] Documentation on "oc patch" is missing
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Documentation
Version: 3.3.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: brice
QA Contact: Xiaowei Li
Vikram Goyal
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-11 12:56 UTC by Jaspreet Kaur
Modified: 2020-01-17 16:00 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-22 00:48:55 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Jaspreet Kaur 2016-10-11 12:56:06 UTC
Document URL:  There is no mention in the documentation regarding oc patch command :


Section Number and Name:  https://access.redhat.com/documentation/en/openshift-container-platform/3.3/single/cli-reference/

Describe the issue: Please have some description regarding how this command can be used with an example and a brief description.

Suggestions for improvement: 

Additional information:

Comment 1 Jaspreet Kaur 2016-10-14 05:14:13 UTC
For the oc patch command, rather than being dropped into an editor, you can provide a JSON object definition for just the setting you want to add or update. This needs to include the full object path down to the setting, which could be a literal value, or a structured object in itself.

To update the serviceAccount setting using oc patch, we can use the command:

$ oc patch bc/wombat --patch '{"spec":{"serviceAccount": "secret:spy"}}'
"wombat" patched

Comment 2 brice 2016-11-01 23:56:29 UTC
Jaspreet,

Thanks for the information. I also checked out the cli files in the origin repo, then made this PR:

https://github.com/openshift/openshift-docs/pull/3146

Can I get an ack this is everything to fulfill this BZ? Thanks!

Comment 3 Jaspreet Kaur 2016-11-03 05:26:50 UTC
Yes it looks good.

Comment 4 brice 2016-11-03 05:32:56 UTC
Great, thanks! I'll put this onto review.

Comment 5 openshift-github-bot 2016-11-04 03:29:24 UTC
Commit pushed to master at https://github.com/openshift/openshift-docs

https://github.com/openshift/openshift-docs/commit/a3862d0306ad90b6d48d7a96e0ed3c394599c3d9
Merge pull request #3146 from bfallonf/patch-1383690

Bug 1383690 added oc patch to cli reference


Note You need to log in before you can comment on or make changes to this bug.