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

fix unittest (fix dep)/integration test (remove short option) #188

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

sunya-ch
Copy link
Contributor

@sunya-ch sunya-ch commented Nov 8, 2023

This PR is to fix previous failed CI.
The integration test seems to be failed due to dependency of kubectl version.
The unit test got an error due to new release of Werkzeug. Need to fix the version in base image.

Traceback (most recent call last):
  File "src/server/model_server.py", line 1, in <module>
    from flask import Flask, request, json, make_response, send_file
  File "/usr/local/lib/python3.8/site-packages/flask/__init__.py", line 7, in <module>
    from .app import Flask as Flask
  File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 27, in <module>
    from . import cli
  File "/usr/local/lib/python3.8/site-packages/flask/cli.py", line 17, in <module>
    from .helpers import get_debug_flag
  File "/usr/local/lib/python3.8/site-packages/flask/helpers.py", line 14, in <module>
    from werkzeug.urls import url_quote
ImportError: cannot import name 'url_quote' from 'werkzeug.urls' (/usr/local/lib/python3.8/site-packages/werkzeug/urls.py)

Signed-off-by: Sunyanan Choochotkaew [email protected]

@sunya-ch sunya-ch changed the title fix unittest (add wait time)/integration test (remove short option) fix unittest (fix rep)/integration test (remove short option) Nov 8, 2023
@sunya-ch sunya-ch changed the title fix unittest (fix rep)/integration test (remove short option) fix unittest (fix dep)/integration test (remove short option) Nov 8, 2023
@sunya-ch sunya-ch merged commit b970239 into sustainable-computing-io:main Nov 8, 2023
3 checks passed
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