Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test/networkd: add wireguard setup #348

Merged
merged 2 commits into from
Aug 1, 2022
Merged

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Jul 29, 2022

simple setup to test IP is assigned as expected.

Signed-off-by: Mathieu Tortuyaux mtortuyaux@microsoft.com


To be tested with: flatcar-archive/coreos-overlay#2061 - currently it's failing but it works when adding the following section:

storage:
  files:
    - path: /etc/systemd/network/zz-default.network.d/wg.conf
      contents:
        inline: |
          [Match]
          Type=!wireguard

as suggested in the ::coreos-overlay PR.

@tormath1 tormath1 self-assigned this Jul 29, 2022
@tormath1 tormath1 marked this pull request as ready for review July 29, 2022 14:19
@tormath1 tormath1 requested a review from a team July 29, 2022 14:20
@jepio
Copy link
Member

jepio commented Jul 29, 2022

We could also make the test slightly more realistic, and use 2 vms, check that they can communicate etc, but this should be good enough for now.

@tormath1
Copy link
Contributor Author

@jepio yeah I also wanted to have an actual provisioning to generate wireguard keys during the boot but it can also be done in a follow-up PR.

simple setup to test IP is assigned as expected.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
@tormath1 tormath1 force-pushed the tormath1/wireguard branch from 21843a6 to 99b8206 Compare August 1, 2022 12:53
@tormath1 tormath1 merged commit 8b23bf6 into flatcar-master Aug 1, 2022
@tormath1 tormath1 deleted the tormath1/wireguard branch August 1, 2022 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants