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

Only create websocket account locally #1121

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlanCoding
Copy link
Member

Formally, this is a problem for the dev environment right now, so don't panic. See:

Traceback (most recent call last):
  File "/app/src/src/aap_eda/services/activation/activation_manager.py", line 213, in _start_activation_instance
    container_request = self._get_container_request()
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/src/src/aap_eda/services/activation/activation_manager.py", line 1033, in _get_container_request
    return self.db_instance.get_container_request()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/src/src/aap_eda/services/activation/engine/common.py", line 168, in get_container_request
    cmdline=self._build_cmdline(),
            ^^^^^^^^^^^^^^^^^^^^^
  File "/app/src/src/aap_eda/services/activation/engine/common.py", line 189, in _build_cmdline
    access_token, refresh_token = create_jwt_token()
                                  ^^^^^^^^^^^^^^^^^^
  File "/app/src/src/aap_eda/services/auth.py", line 49, in create_jwt_token
    user, new = User.objects.get_or_create(
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^

But eventually, this will be a real problem, that comes from related issue AAP-33341.

I tested manually with the dev environment, and found it fix the problem.

Copy link

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.

1 participant