add note to re-consider root warning for containers

fabiolb · Feb 2, 2018 · d2b173f · d2b173f
1 parent 94973bd
commit d2b173f
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/rootwarn_unix.go b/rootwarn_unix.go
@@ -33,6 +33,8 @@ const warn17behavior = `
 var once sync.Once
 
 func WarnIfRunAsRoot(allowRoot bool) {
+	// todo(fs): should we emit the same warning when running inside a container?
+	// todo(fs): check for existence of `/.dockerenv` to determine Docker environment.
 	isRoot := os.Getuid() == 0
 	if !isRoot {
 		return