[Mark McLeod]

The loop inside Teleop is not good.
It will never end, because the value of controller Button 2 is only checked once when the loop begins, then is never checked again.

In general, Teleop should execute and leave as soon as possible. No loops waiting for other things to happen.
That's because Teleop is expected to process a command packet sent from the Driver Station every 20ms.

Your code will do what you want if you leave everything, but remove just the loop from around your code.
Teleop is actually inside a loop itself, so your code will get called over and over.