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

Introduce karton debug #220

Merged
merged 10 commits into from
Jul 20, 2023
Merged

Introduce karton debug #220

merged 10 commits into from
Jul 20, 2023

Conversation

nazywam
Copy link
Member

@nazywam nazywam commented Jul 13, 2023

The karton debug mode helps with developing karton services while connected to the production karton network. It does that by:

  • appending a random suffix to the karton service identity to avoid disrupting production services
  • permanently setting the persistent attribute to False - the bind will disappear after the service quits
  • not sending the (error) logs to karton listeners - this will help you to avoid spamming logging services in your deployment

The debug mode works as any other karton configuartion and can be set by:

  • setting the env value KARTON_KARTON_DEBUG=1
  • setting debug=1 in the karton section in karton.ini
  • passing --debug to karton services via cli

@msm-code
Copy link
Contributor

appending a random suffix to the karton service identity to avoid disrupting production services

Random suffix as well as -dev. It's nice to see immediately which services are the dev ones. But I see the code already does that.

setting the env value KARTON_KARTON_DEBUG=1

A bit uglier than just KARTON_DEBUG but I understand this is consistent with other config variables. So makes sense I guess.

@nazywam nazywam changed the title Draft: Introduce karton debug Introduce karton debug Jul 13, 2023
docs/advanced_concepts.rst Outdated Show resolved Hide resolved
docs/advanced_concepts.rst Outdated Show resolved Hide resolved
docs/advanced_concepts.rst Outdated Show resolved Hide resolved
Copy link
Member

@psrok1 psrok1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@psrok1 psrok1 merged commit fee4947 into master Jul 20, 2023
6 checks passed
@psrok1 psrok1 deleted the feature/karton-dev branch July 20, 2023 15:49
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.

3 participants