Bug 221208 - [build.webkit.org] is_test_mode_enabled should default to True
Summary: [build.webkit.org] is_test_mode_enabled should default to True
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Aakash Jain
URL:
Keywords: InRadar
Depends on:
Blocks: 175056
  Show dependency tree / graph
 
Reported: 2021-02-01 07:40 PST by Aakash Jain
Modified: 2021-02-08 07:37 PST (History)
4 users (show)

See Also:

Attachments
Patch (1.06 KB, patch)
2021-02-01 07:41 PST, Aakash Jain 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2021-02-01 07:40:53 PST 
Similar to https://bugs.webkit.org/show_bug.cgi?id=196248, is_test_mode_enabled variable should default to True in order to avoid accidentally running production settings on a local/test instance. Currently we need to explicitly add an environment variable BUILDBOT_TESTING in order to indicate test instance. If someone forgets to set this environment variable on a local instance, he/she would run with production settings. Currently it's very easy to make this mistake. We should reverse the logic to make it harder to do such mistake. 

We should instead require to set an environment variable BUILDBOT_PRODUCTION on production server. It will be a one time configuration on the production server (by configuring environment variable in a manner that persist across reboots and various shell sessions).
Comment 1 Aakash Jain 2021-02-01 07:41:43 PST 
Created attachment 418872 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2021-02-01 08:05:34 PST 
<rdar://problem/73829462>
Comment 3 EWS 2021-02-01 09:07:51 PST 
Committed r272144: <https://trac.webkit.org/changeset/272144>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 418872 [details].