pinMode(relayPin[i], INPUT_PULLUP); //Sets the relay pins as pullups. This is to avoid floating state for the pins.
pinMode(relayPin[i], OUTPUT); // sets the relay pins as outputs, defaults state HIGH, relay OFF
Using the pinMode command, you can't have both. INPUT_PULLUP is overwritten by OUTPUT.
How are the relays wired? From Arduino Pin to Ground? How many relays are there? How much current does each on require?