I see no problem with adding extra attribute to NodeLocation class. On Sun, Apr 21, 2019, 00:58 Dimitris Moraitis <d...@mist.io> wrote:
> Hi all, > > I've done some enhancements on the Packet driver, like adding support for > volumes, ip addresses and more. The pull request is ready for review: > https://github.com/apache/libcloud/pull/1282 > > Please let me know if you have any questions or concerns. > > One question/concern that I have is about the extra metadata on locations. > Packet provides rich metadata regarding the locations (aka facilities) like > available features & IP address ranges. We would like to use and expose > these in Mist.io but the NodeLocation class doesn't have an "extra" > property, which I'm tempted to add. Would anyone object if I were to do so? > > Thanks, > Dimitris Moraitis > > -- > > CTO & co-founder > > https://mist.io > > manage the clouds >
