-
-
Notifications
You must be signed in to change notification settings - Fork 874
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
website/docs: next release notes: add upgrade instructions for postgres to 16 #9727
website/docs: next release notes: add upgrade instructions for postgres to 16 #9727
Conversation
✅ Deploy Preview for authentik-storybook canceled.
|
❌ Deploy Preview for authentik-docs failed.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9727 +/- ##
=======================================
Coverage 92.64% 92.64%
=======================================
Files 710 710
Lines 34706 34745 +39
=======================================
+ Hits 32152 32189 +37
- Misses 2554 2556 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Great PR! Just several hours ago I independently upgraded to postgres 16, there were no problems and it seems like authentik is working much faster. |
Thanks! Out of curiosity, did you update with an existing database or did you recreate it? I'm currently facing the error |
We'll probably include this in the release notes, so |
Yeah we've been testing 16 in CI pretty much since in came out, no issue. Interesting that it seems to be faster though! |
Ok. thanks for the quick response. Regarding my issue, do you have any other suggestions or would you do it differently? |
You shouldn't need to re-create a database, restoring your dump file should be enough. |
I'll try that soon. Thanks! |
Regarding K8s, someone would have to write the documentation as I don't really have any experience with it |
We distribute postgres 15 with the helm chart, so we won't be upgrading that one soon. |
@rissson I think this is ready for first review |
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: 4d62 <157558804+4d62ext@users.noreply.github.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: 4d62 <157558804+4d62ext@users.noreply.github.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: 4d62 <157558804+4d62ext@users.noreply.github.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: 4d62 <157558804+4d62ext@users.noreply.github.com>
Co-authored-by: Tana M Berry <tanamarieberry@yahoo.com> Signed-off-by: 4d62 <157558804+4d62ext@users.noreply.github.com>
No problem and thanks for the quick review. |
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
|
I'm so sorry, I misscliked.
Yes! |
my bad, tried renaming branch on my fork |
Signed-off-by: 4d62 <ext@4d62.me>
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks again @4d62ext !
no problem! |
Details
Provides documentation to upgrade authentik-postgresql to PG v16
closes #9628
Checklist
ak test authentik/
)make lint-fix
)If an API change has been made
make gen-build
)If changes to the frontend have been made
make web
)If applicable
make website
)