scripts/slave/recipe_modules/goma/example.expected/linux.json - Issue 2237403002: Add cloudtail start/stop in recipe_modules/goma

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: scripts/slave/recipe_modules/goma/example.expected/linux.json

Issue 2237403002: Add cloudtail start/stop in recipe_modules/goma (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master

Patch Set: remove --log-id, sleep 3 seconds Created 4 years, 4 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« scripts/slave/recipe_modules/goma/api.py ('K') | « scripts/slave/recipe_modules/goma/example.py ('k') | scripts/slave/recipe_modules/goma/example.expected/linux_upload_logs.json » ('j') | scripts/slave/recipe_modules/goma/resources/cloudtail_utils.py » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 [	1 [

2 {	2 {

3 "cmd": [],	3 "cmd": [],

4 "name": "ensure_goma"	4 "name": "ensure_goma"

5 },	5 },

6 {	6 {

7 "cmd": [	7 "cmd": [

8 "python",	8 "python",

9 "-u",	9 "-u",

10 "RECIPE_MODULE[depot_tools::cipd]/resources/bootstrap.py",	10 "RECIPE_MODULE[depot_tools::cipd]/resources/bootstrap.py",

(...skipping 51 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
62 ],	62 ],

63 "env": {	63 "env": {

64 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json"	64 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json"

65 },	65 },

66 "name": "start_goma"	66 "name": "start_goma"

67 },	67 },

68 {	68 {

69 "cmd": [	69 "cmd": [

70 "python",	70 "python",

71 "-u",	71 "-u",

	72 "RECIPE_MODULE[build::goma]/resources/cloudtail_utils.py",

	73 "--start-cloudtail",

	74 "--cloudtail-path",

	75 "/opt/infra-tools/cloudtail",

	76 "--log-id",
	Yoshisato Yanagisawa 2016/08/16 04:16:24 might need to update JSON file? might need to update JSON file? tikuta 2016/08/16 05:09:55 Done. Show quoted text On 2016/08/16 04:16:24, Yoshisato Yanagisawa wrote: > might need to update JSON file? Done.
	77 "goma_compiler_proxy"

	78 ],

	79 "env": {

	80 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json"

	81 },

	82 "name": "start cloudtail",

	83 "stdout": "/path/to/tmp/"

	84 },

	85 {

	86 "cmd": [

	87 "python",

	88 "-u",

72 "[CACHE]/cipd/goma/goma_ctl.py",	89 "[CACHE]/cipd/goma/goma_ctl.py",

73 "stop"	90 "stop"

74 ],	91 ],

75 "env": {	92 "env": {

76 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json"	93 "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account -goma-client.json"

77 },	94 },

78 "name": "stop_goma"	95 "name": "stop_goma"

79 },	96 },

80 {	97 {

81 "cmd": [	98 "cmd": [

82 "python",	99 "python",

83 "-u",	100 "-u",

	101 "RECIPE_MODULE[build::goma]/resources/cloudtail_utils.py",

	102 "--killed-pid",

	103 "12345"

	104 ],

	105 "name": "stop cloudtail"

	106 },

	107 {

	108 "cmd": [

	109 "python",

	110 "-u",

84 "RECIPE_PACKAGE_REPO[build]/scripts/slave/upload_goma_logs.py",	111 "RECIPE_PACKAGE_REPO[build]/scripts/slave/upload_goma_logs.py",

85 "--upload-compiler-proxy-info"	112 "--upload-compiler-proxy-info"

86 ],	113 ],

87 "name": "upload_log"	114 "name": "upload_log"

88 },	115 },

89 {	116 {

90 "name": "$result",	117 "name": "$result",

91 "recipe_result": null,	118 "recipe_result": null,

92 "status_code": 0	119 "status_code": 0

93 }	120 }

94 ]	121 ]

OLD	NEW