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

Command fails, but shows as succeeded #255

Closed
babyhuey opened this issue Feb 13, 2024 · 0 comments · Fixed by #258
Closed

Command fails, but shows as succeeded #255

babyhuey opened this issue Feb 13, 2024 · 0 comments · Fixed by #258

Comments

@babyhuey
Copy link

During my pipeline runs, I have noticed that wsgi manage commands will run, and if the command fails, it still returns that it worked.

This should definitely fail the pipeline so that we're aware that there are issues.

sls wsgi manage --command "migrate -v2" --stage dev
Running "serverless" from node_modules
Warning: Invalid configuration encountered
  at root: unrecognized property 'constructs'
Learn more about configuration validation here: http://slss.io/configuration-validation
[
    1,
    Traceback (most recent call last):
  File \/var/task/wsgi_handler.py\, line 85, in handler
    management.call_command(*shlex.split(meta.get(\data\, \\)))
  File \/tmp/sls-py-req/django/core/management/__init__.py\, line 194, in call_command
    return command.execute(*args, **defaults)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/core/management/base.py\, line 458, in execute
    output = self.handle(*args, **options)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/core/management/base.py\, line 106, in wrapper
    res = handle_func(*args, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/core/management/commands/migrate.py\, line 356, in handle
    post_migrate_state = executor.migrate(
                         ^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/db/migrations/executor.py\, line 135, in migrate
    state = self._migrate_all_forwards(
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/db/migrations/executor.py\, line 167, in _migrate_all_forwards
    state = self.apply_migration(
            ^^^^^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/db/migrations/executor.py\, line 252, in apply_migration
    state = migration.apply(state, schema_editor)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File \/tmp/sls-py-req/django/db/migrations/migration.py\, line 127, in apply
    operation.database_forwards(
  File \/tmp/sls-py-req/django/db/migrations/operations/special.py\, line 193, in database_forwards
    self.code(from_state.apps, schema_editor)
  File \/var/task/api/migrations/0016_populate_asyncjob_inputs.py\, line 10, in update_inputs_for_response_and_prompt
    dict(student_response=job.student_response, prompt_text=job.prompt_text)
                          ^^^^^^^^^^^^^^^^^^^^
AttributeError: 'AsyncJob' object has no attribute 'student_response'
]
3 deprecations found: run 'serverless doctor' for more details
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 a pull request may close this issue.

2 participants
@babyhuey and others