Hello Simon and everyone,
Municipalities are already tagged with boundary=administrative and admin_level=8. There is no need to tag them a second time with a node with place=municipality, that would be redundant. If anything, a place=label node could be added in the existing relations.
In regard to ele tag, this should only be applied to nodes where we know for sure where the data elevation is from. In the past, when undoing Donovalny's changes, I unfortunately copied the "ele" tag from his fake villages to the boundary, as I did not understand what this tag was and it is good practice on OSM to not remove tags we don't understand. However now that I understood it is elevation, and that this is simply copied from Wikipedia, I do not think it should stand on municipality relations, as it does not make any sense to have elevation of more than one node.
Even if the data such as elevation or population from Wikipedia would be legal to use (I don't think that's the case) it would need to be auto-updated or machine-assisted updated. It does not make sense to have people regularly copying data from Wikipedia as the population increase. Using directly the source Wikipedia uses to know the population and elevation makes much more sense.
Best regards,