glothriel / wormhole

Exposes kubernetes services between clusters by setting up wireguard tunnels
MIT License
8 stars 1 forks source link

Nginx exposer is possibly called three times #63

Open glothriel opened 5 months ago

glothriel commented 5 months ago

Suspicious logs when exposing a single service:

│ time="2024-05-28T14:34:33Z" level=info msg="App local.nginx-nginx added"                                         │
│ time="2024-05-28T14:34:33Z" level=info msg="Created NGINX config file local-nginx-nginx.conf"                    │
│ time="2024-05-28T14:34:33Z" level=debug msg="Received sync from dev1 with 1 apps"                                │
│ time="2024-05-28T14:34:33Z" level=info msg="App dev1.nginx-nginx added"                                          │
│ time="2024-05-28T14:34:33Z" level=info msg="App local.nginx-nginx added"                                         │
│ time="2024-05-28T14:34:33Z" level=info msg="Created NGINX config file remote-dev1-nginx-nginx.conf"              │
│ time="2024-05-28T14:34:33Z" level=debug msg="Updating service dev1-nginx-nginx"                                  │
│ time="2024-05-28T14:34:33Z" level=debug msg="Received sync from dev2 with 1 apps"                                │
│ time="2024-05-28T14:34:33Z" level=info msg="App dev2.nginx-nginx added"                                          │
│ time="2024-05-28T14:34:33Z" level=info msg="App local.nginx-nginx added"                                         │
│ time="2024-05-28T14:34:33Z" level=info msg="Created NGINX config file remote-dev2-nginx-nginx.conf"   
jzyinq commented 5 months ago

It looks quite ok for me

glothriel commented 5 months ago
 msg="App local.nginx-nginx added"                                         │

is the offending part

jzyinq commented 5 months ago

Yeah, but in the devenv scenario (single cluster) every pod establishes local configuration as well.

In a regular scenario you would have either local or remote based on client/server nature right?