famedly / ansible-collection-network

`famedly.network` ansible collection for network-related services (vpn, reverse proxies, ...)
GNU Affero General Public License v3.0
0 stars 1 forks source link

feat(wireguard): add role for wireguard - [merged] #15

Closed famedly-bot closed 1 year ago

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jun 28, 2021, 08:41

Merges transcaffeine/wireguard -> main

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 14, 2021, 13:00

added 6 commits

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 15, 2021, 06:03

marked this merge request as ready

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 15, 2021, 06:59

added 1 commit

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 15, 2021, 07:05

added 1 commit

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 15, 2021, 07:06

requested review from @jadyn.dev

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 15, 2021, 07:08

added 1 commit

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @lrsksr on Jul 16, 2021, 10:43

Commented on roles/wireguard/tasks/key-generation.yml line 47

Why do you slurp the privkey again at this point? Isn't that the same as the task in line 20 even with the same condition?

famedly-bot commented 3 years ago

In GitLab by @lrsksr on Jul 16, 2021, 10:43

Commented on roles/wireguard/README.md line 12

typo doine

famedly-bot commented 3 years ago

In GitLab by @lrsksr on Jul 16, 2021, 10:43

Commented on roles/wireguard/defaults/main.yml line 17

Can you update this to the example in the readme or remove it?

famedly-bot commented 3 years ago

In GitLab by @lrsksr on Jul 16, 2021, 10:46

requested review from @lrsksr and removed review request for @jadyn.dev

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:35

Commented on roles/wireguard/tasks/key-generation.yml line 47

changed this line in version 8 of the diff

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:35

Commented on roles/wireguard/README.md line 12

changed this line in version 8 of the diff

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:35

added 1 commit

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:35

Commented on roles/wireguard/tasks/key-generation.yml line 47

Yes, idempotent handling of partially generated keys is a bit messy-ish, so i accidentally duplicated that

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:51

added 1 commit

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:53

Commented on roles/wireguard/defaults/main.yml line 17

Updated to no longer contain invalid keys

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 16, 2021, 11:53

resolved all threads

famedly-bot commented 3 years ago

In GitLab by @jadyn.dev on Jul 19, 2021, 08:54

requested review from @jadyn.dev

famedly-bot commented 3 years ago

In GitLab by @jdreichmann on Jul 19, 2021, 09:46

added 1 commit

Compare with previous version

famedly-bot commented 3 years ago

In GitLab by @lrsksr on Jul 19, 2021, 10:05

approved this merge request