search

assimilation / assimilation-official

This is the official main repository for the Assimilation project
51 stars 9 forks source link

travis-ci builds fail due to sudoers command #9

Closed Alan-R closed 9 years ago

Alan-R commented 9 years ago

The symptoms point to sort giving different results on different platforms - without any errors being indicated. https://travis-ci.org/assimilation/assimilation-official/builds/81379935

The only thing I know of that could possibly cause that is the locale. Sort command man page suggests setting LC_ALL=C. So I'm trying that.

borgified commented 9 years ago

what's the expected discovery output of sudoers vs what is actually being output? i cant see it from the build log

Alan-R commented 9 years ago

Here's the chunk of build output. You can find it pretty quickly by searching for "sudoers"

 "Defaults": {
-          "ANY": {"syslog": "auth"},
-          "command:PAGERS": {"noexec": true},
           "user:FULLTIMERS": {"lecture": false},
           "user:millert": {"authenticate": false},
+          "command:PAGERS": {"noexec": true},
       "runas:root": {"set_logname": false},
-          "system:SERVERS": {"log_year": true, "logfile": "/var/log/sudo.log"}
+          "system:SERVERS": {"log_year": true, "logfile": "/var/log/sudo.log"},
+          "ANY": {"syslog": "auth"}
     },

I just pushed a change that sets LC_ALL='C' - which should make sort always behave the same way... We'll see if that fixed it...

As you can see, in the travis.ci output, "command:PAGERS" sorts down farther, as does "ANY":.

Alan-R commented 9 years ago

That appears to have fixed it. "Discovery test sudoers succeeded."

borgified commented 9 years ago

awesome. green is my favorite color too!