Copy the bootstrap.sh file to /tmp before running about agent HOT 6 CLOSED

Things to be wary of:

• on secure systems, /tmp may have execute rights disabled, meaning any scripts copied there can't be run.
• I would suggest that the location should be configurable, and/or pulled from TMPDIR env variable.
• Use safe tempfile creation to avoid security issues raised by clobbering symlinks.

from agent.

I was gonna use this: http://golang.org/pkg/os/#TempDir Which I think handles all that for you automatically.

from agent.

I've been thinking about this, and I don't think I want to do it. The only reason is, I like the idea of being about to do a: ps aux | grep bootstrap.sh to see if the script is running and it's location. What do you think @tarragon? Do you think this is worth changing?

from agent.

Using a ps like that won't be reliable, for instance if you are using a system and running multiple buildbox agents in docker or running multiple agents on the same box. That said to get the result you want you could use a temporary directory instead, perhaps under the buildbox path, and copy the bootstrap.sh file there instead using it's original name.

from agent.

Never "ps | grep" anyway, always "pgrep" - Your friendly neighbourhood neckbeard

from agent.

I'm going to close this. If it crops up and starts being an issue, perhaps I can add a config option for it 🍂

from agent.