Skip to content

Conversation

@nblock
Copy link
Collaborator

@nblock nblock commented May 10, 2025

These steps are easier to accomplish and require only Headscale 0.26. They also work when a user has already upgraded the database.

See: #2567

  • have read the CONTRIBUTING.md file
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

These steps are easier to accomplish and require only Headscale 0.26.
They also work when a user has already upgraded the database.

See: juanfont#2567
@ghost
Copy link

ghost commented May 10, 2025

Pull Request Revisions

RevisionDescription
r1
Policy migration process simplifiedUpdated policy migration steps for database-stored policies, introducing environment variable method to transition between policy versions

☑️ AI review skipped for r1
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

We'd love to hear from you—reach out anytime at [email protected].

@nblock nblock requested a review from kradalby May 10, 2025 05:00
@nblock nblock marked this pull request as ready for review May 10, 2025 05:01
@nblock nblock requested a review from ohdearaugustin as a code owner May 10, 2025 05:01
@kradalby kradalby merged commit d81b005 into juanfont:main May 10, 2025
145 checks passed
@nblock nblock deleted the dbmigrate2 branch May 10, 2025 07:30
EdGeraghty added a commit to privacyint/docker-headscale that referenced this pull request May 11, 2025
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