Skip to content

dev/ci: do *not* run e2e sourcegraph/server tests with DEPLOY_TYPE=dev

Warren Gifford requested to merge sg/no-deploy-type-override into main

Created by: slimsag

It appears that we introduced this in this PR and, although there is no description, the intent was to prevent this code from running.

However, it appears that code was never merged - at least blame says it has not existed since Sourcegraph became open source.

Also, the code is already protected by the user agent check - so I don't think we need this.

Most importantly: having DEPLOY_TYPE=dev is incredibly bad because it means we are not e2e testing the actual version of Sourcegraph we are releasing! conf.IsDeployTypeSingleDockerContainer() will return false when this is set, and various dev-only features will be turned on.

I only caught this because my PR to run some code in dev-only mode failed.

Helps #17218 (closed)

Signed-off-by: Stephen Gutekanst [email protected]

Merge request reports

Loading