You only execute those statements every WaitTime milliseconds which is 10000 so each time through the loop, currentMillis - previousMillis will always be greater than your on/off time. That code needs to be moved outside your initial if() statement so it runs every time through loop()