Bug 1383690

Summary:	[DOCS] Documentation on "oc patch" is missing
Product:	OpenShift Container Platform	Reporter:	Jaspreet Kaur <jkaur>
Component:	Documentation	Assignee:	brice <bfallonf>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Xiaowei Li <xiaoli>
Severity:	medium	Docs Contact:	Vikram Goyal <vigoyal>
Priority:	medium
Version:	3.3.0	CC:	aos-bugs, jkaur, jokerman, mmccomas
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-11-22 00:48:55 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

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

Comment 6 brice 2016-11-22 00:48:55 UTC

Link to released docs:

https://docs.openshift.com/container-platform/3.3/cli_reference/basic_cli_operations.html#patch