Hallo,
was möchtest du mit den beiden ersten Zeilen deiner Loop bewirken?
void loop()
{
digitalWrite(relais1, HIGH);
digitalWrite(relais2, HIGH);
Hier werden in jedem Loop Durchlauf die Relais abgeschaltet, also auch wenn du einen Taster drückst, wird das Relais nur für einen Loop Durchlauf eingeschaltet.
Gruß Ardubu