Skip to content

Conversation

@sergerdn
Copy link
Contributor

@sergerdn sergerdn commented Oct 16, 2020

digitalocean driver.list_nodes: size and image now is not null

Fixes #1507

Status

  • done, ready for review

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
  • Documentation
  • Tests
  • ICLA (required for bigger changes)

P.S.
DO return <nil>for some values instead of null, because Golang. I believe that it is a bug on their side. I'v created ticket in their support center.

Screenshot 2020-10-16 at 03 56 42

@sergerdn sergerdn marked this pull request as ready for review October 16, 2020 01:11
@Kami
Copy link
Member

Kami commented Nov 1, 2020

DO return for some values instead of null, because Golang. I believe that it is a bug on their side. I'v created ticket in their support center.

lovely, and people were complaining about JavaScript in the past... 😆

@sergerdn
Copy link
Contributor Author

sergerdn commented Nov 1, 2020

DO return for some values instead of null, because Golang. I believe that it is a bug on their side. I'v created ticket in their support center.

lovely, and people were complaining about JavaScript in the past... 😆

Support said that they don't care about this. We can make easy fix on Libcloud side special for gateway value:

if gateway == "<nil>":
    gateway = None

@asfgit asfgit merged commit b13e3d4 into apache:trunk Nov 5, 2020
@Kami
Copy link
Member

Kami commented Nov 5, 2020

Change has been merged into trunk - thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

digitalocean driver.list_nodes: size and image should be not null

3 participants