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

health check #306

Merged
merged 4 commits into from
Apr 24, 2023
Merged

health check #306

merged 4 commits into from
Apr 24, 2023

Conversation

fbuetler
Copy link

@fbuetler fbuetler commented Feb 7, 2023

Hi,

This is a PR in a serie of 6 PRs, as it was asked to split #294 up into multiple PRs.

Serie:

  • feature/bring-your-own-public-key
  • feature/remove-all-devices-for-user
  • feature/purge-inactive-devices
  • (THIS PR) feature/health-check
  • hygiene/put-common-auth-logic-into-func
  • feature/pre-shared-keys

Dependency

This PR depends on freifunkMUC/wg-embed#15 and its version has to be updated before this PR can be merged.

Feature:

This feature extends the wg-access-server with proper health checks. It pings the embedded wireguard and the storage backend.

closes: #100

@mergeable
Copy link

mergeable bot commented Feb 7, 2023

Thanks for creating a pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

@awlx awlx added the enhancement New feature or request label Feb 12, 2023
@GoliathLabs
Copy link
Member

@fbuetler can you update the version accordingly as the PR has been merged?

@fbuetler fbuetler requested a review from a team as a code owner March 13, 2023 08:09
@fbuetler fbuetler changed the title WIP: health check health check Mar 13, 2023
@GoliathLabs
Copy link
Member

@fbuetler can you rebase your changes again? After that we can merge your PR

@fbuetler
Copy link
Author

I think the linting errors have to be resolved on the master branch as they were not introduced by this branch. I guess these errors appreared after a linter update.

See:
4766783
#305

@fbuetler fbuetler merged commit 3b935da into freifunkMUC:master Apr 24, 2023
@fbuetler fbuetler deleted the feature/health-check branch April 24, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(enhancement) Implement a proper health check probe
4 participants