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

list_images: Missing fields in extra_volumes #636

Closed
gg3nx opened this issue Aug 29, 2024 · 5 comments
Closed

list_images: Missing fields in extra_volumes #636

gg3nx opened this issue Aug 29, 2024 · 5 comments
Assignees
Labels
bug Something isn't working instance Instance issues, bugs and feature requests

Comments

@gg3nx
Copy link

gg3nx commented Aug 29, 2024

Versions used:

scaleway               2.6.0           Scaleway SDK for Python
scaleway-core          2.6.0           Scaleway SDK for Python

When using list_images from instance.v1, unmarshall_Volume fails with the following error:

TypeError: Volume.__init__() missing 5 required positional arguments: 'organization', 'project', 'tags', 'state', and 'zone'

According to https://www.scaleway.com/en/developers/api/instance/#path-images-list-instance-images , these fields should be present in the response from the API.

When testing with curl on https://api.scaleway.com/instance/v1/zones/fr-par-2/images?public=no, it looks like these fields are really missing:

      "extra_volumes": {
        "1": {
          "id": "1f034b3b-XXXX-XXXX-XXXX-81b4554d8090",
          "name": "XXX",
          "volume_type": "unified",
          "size": 300000000000
        }
      },

Is it a bug in the SDK or in the API?

@Laure-di Laure-di self-assigned this Sep 23, 2024
@Laure-di Laure-di added bug Something isn't working instance Instance issues, bugs and feature requests labels Sep 23, 2024
@gg3nx
Copy link
Author

gg3nx commented Nov 12, 2024

Any news or plans about this issue?
I need both Instance and Cockpit API but can't find any version having Cockpit API where Instance API is not buggy (total_count issue or this one) :-/

@Laure-di
Copy link
Contributor

Hello,

The issue has been raised to the product team, and a fix should be released at the beginning of January. We apologize for the inconvenience and the delay.

Thank you for reporting this and for your patience.

Laure

@Laure-di
Copy link
Contributor

Hello @gg3nx,

Could you please provide more details about the total_count issue?

@gg3nx
Copy link
Author

gg3nx commented Dec 18, 2024

Could you please provide more details about the total_count issue?

It was #555. It has been solved but after the current issue has been introduced.

@Laure-di
Copy link
Contributor

You can use the master branch to fix the issue; it should be definitively fixed with the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working instance Instance issues, bugs and feature requests
Projects
None yet
Development

No branches or pull requests

2 participants