[DavidGitz]

Couple problems:
-It is clicking because you have the same reference and are issuing 2 different commands at the same time. With a double acting solenoid this means that it should be operating on 2 channels. If you are using the solenoid module on the cRIO this means that you should use Channels 1 and 2 (or whatever you are using). The code that I originally posted should satisfy this exactly.
-Also you definitely want to avoid having while loops in your Teleop, this is a bad practice and will more than likely make your drive sluggish at best. With your code the while loop only executes once, so why even have it?