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

GET ​/python​/package​/versions gives 404 Error not found #1784

Closed
shreekarSS opened this issue Jul 26, 2022 · 2 comments
Closed

GET ​/python​/package​/versions gives 404 Error not found #1784

shreekarSS opened this issue Jul 26, 2022 · 2 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. sig/devsecops Categorizes an issue or PR as relevant to SIG DevSecOps.

Comments

@shreekarSS
Copy link
Member

Describe the bug

Get API gives 404 Error not found with default value , expecting to list available python versions

"error": "Package 'tensorflow' not found",
"parameters": {
"name": "tensorflow",
"order_by": null,
"os_name": "ubi",
"os_version": "9",
"python_version": "3.9"
}
}

To Reproduce
Steps to reproduce the behavior:

  1. Try https://khemenu.thoth-station.ninja/api/v1/ui/#/PythonPackages/list_python_package_versions

Expected behavior
Expecting to list available python versions

@shreekarSS shreekarSS added the kind/bug Categorizes issue or PR as related to a bug. label Jul 26, 2022
@goern
Copy link
Member

goern commented Aug 9, 2022

/sig devsecops
/priority critical-urgent

@sesheta sesheta added sig/devsecops Categorizes an issue or PR as relevant to SIG DevSecOps. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. labels Aug 9, 2022
@harshad16
Copy link
Member

The response seems to be correct, as if we don't find the package, it should be a 404

However, if we use the correct params, we would get 200
for example:
https://test.thoth-station.ninja/api/v1/python/package/versions?name=tensorflow&os_name=ubi&os_version=8&python_version=3.6

maybe we should keep the default example pointing to ubi8 and python 3.8
till we get ingestion done for ubi9 and python3.9

changing these respectively should give default results.
https://khemenu.thoth-station.ninja/api/v1/python/package/versions?name=tensorflow&os_name=ubi&os_version=8&python_version=3.8

example: "9"

example: "3.9"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. sig/devsecops Categorizes an issue or PR as relevant to SIG DevSecOps.
Projects
None yet
Development

No branches or pull requests

4 participants