-
-
Notifications
You must be signed in to change notification settings - Fork 998
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
Improve debugging experience #12961
base: main
Are you sure you want to change the base?
Improve debugging experience #12961
Conversation
✅ Deploy Preview for authentik-storybook canceled.
|
✅ Deploy Preview for authentik-docs canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #12961 +/- ##
==========================================
- Coverage 92.75% 92.74% -0.02%
==========================================
Files 785 785
Lines 39580 39580
==========================================
- Hits 36714 36710 -4
- Misses 2866 2870 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@BeryJu: For testing purpose I added some minor python changes. After that the breaking points around the changes didn't get hit anymore (which makes sense since the changes aren't visible in the container itself). volumes:
- ./authentik:/authentik to Maybe you could reevaluate your problem with this PR/comment |
Details
This PR is ment to improve the debugging experiance:
docker-compose.override.yml
is necessary due to website/docs: updated debugging docs #12809 it addsdocker-compose.override.yml
to gitignore to remove noice when comittingFurther thoughts:
Debug: Attach Worker
configuration since it only duplicatesDebug: Attach Server Core
in its curretns state. RenamingDebug: Attach Server Core
would be useful for clarification thenworker
andserver
should be able at the same time it would be necessary distinguish between both when setting up the debugging ports on startup or via port forwarding indocker-compose.override.yml
.Both needs to be documented somewhere
Dockerfile
(Sidemark: Image size will increase ~30%)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
)