freifunkh / ansible-configs

Here we store all of our actual ansible hosts in freifunk hanover
2 stars 1 forks source link

Human-readable MACs/IPs #12

Closed jue89 closed 7 years ago

jue89 commented 7 years ago

Actually I'm too silly for reading this hex-coded supernode numbers. Can we change it without breaking the Internet / Freifunk?

jue89 commented 7 years ago

Please double-check. I'm quite tiered right now ;)

lemoer commented 7 years ago

sn{02,03,04,05,09}: Human-readable MACs was applied as e21376c.

Further:

jue89 commented 7 years ago

You only changed the schema for the public IPs only for sn04 and sn05.. Why did you do this?

Well, the other IPs were human-readable. But yes, we should change them as well.

I will change the PR until tonight.

jue89 commented 7 years ago

Ready to go ... hopefully.

After reading your comment, I realised I didn't do it the way you wanted. I split the PR up pre super node. So we can deploy it node after node and don't bother all users at once.

jue89 commented 7 years ago

Push force again ^^

lemoer commented 7 years ago

sn{02,03,04,05,09}: Updated internal IPv6 addr is applied as 6d9a319b193fab0753860a8b6fadd4d805111ef5

Thanks for this, @jue89 I really like it :+1:

@eaxi @1977er What do you say to the last commit?

eaxi commented 7 years ago

I like it. At the time we installed sn02 the first time, i indicated that we should make it so. Nice that it's now done.

lemoer commented 7 years ago

@eaxi Yes. I thought that you also would like this... IIRC, you told me, that hezner binds the IPv6 addresses to MAC addresses? Is this a problem here?

eaxi commented 7 years ago

@lemoer No theres no problem. The function which you can bind your /64 to an MAC-Address is for routed setups. But if you bind the /64 to the Host, it will be like an L2-Network, like the other IP-Adresses.

lemoer commented 7 years ago

The last commit is also applied. (47812b5)

@jue89 is there a special reason, why you left out sn09?

I'll leave this PR open until this is also done.

jue89 commented 7 years ago

@lemoer: I'm blind.

lemoer commented 7 years ago

@1977er Do you also want to add the new address to the mesh_fastd_listen_on variable?

lemoer commented 7 years ago

I'll close this, since it's already in the master branch.