Absolutely nothing wrong... just inconsistent... maybe?
It's a one time decision. If he hadn't, he'd have been obligated to serve in the IDF, and couldn't renounce his citizenship for decades, until his IDF obligation had expired. Lots of countries have policies like that, including the US, though informally.