Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2533)

Unified Diff: appengine/swarming/swarming_bot/bot_code/bot_main_test.py

Issue 2958853002: Propagate name of system service account to the bot. (Closed)
Patch Set: Created 3 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « appengine/swarming/swarming_bot/bot_code/bot_auth_test.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/swarming/swarming_bot/bot_code/bot_main_test.py
diff --git a/appengine/swarming/swarming_bot/bot_code/bot_main_test.py b/appengine/swarming/swarming_bot/bot_code/bot_main_test.py
index cbaa5d6dcb5c347242a3319300c666d3b51a9abb..f38909679ae367466cc9575bb103c5f75aa79049 100755
--- a/appengine/swarming/swarming_bot/bot_code/bot_main_test.py
+++ b/appengine/swarming/swarming_bot/bot_code/bot_main_test.py
@@ -754,7 +754,8 @@ class TestBotMain(TestBotBase):
expected_auth_params_json={
'swarming_http_headers': {'A': 'a'},
'swarming_http_headers_exp': int(time.time() + 3600),
- 'task_service_account': 'bot', # as in task manifest
+ 'system_service_account': 'robot@example.com', # as in task manifest
+ 'task_service_account': 'bot',
})
manifest = {
@@ -763,7 +764,10 @@ class TestBotMain(TestBotBase):
'grace_period': 30,
'hard_timeout': 60,
'host': 'https://localhost:3',
- 'service_account': 'bot',
+ 'service_accounts': {
+ 'system': {'service_account': 'robot@example.com'},
+ 'task': {'service_account': 'bot'},
+ },
'task_id': '24',
}
self.assertEqual(self.root_dir, self.bot.base_dir)
« no previous file with comments | « appengine/swarming/swarming_bot/bot_code/bot_auth_test.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698