jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ali Bazlamit <notificati...@github.com>
Subject Re: [jclouds/jclouds-labs] JCLOUDS-1386 1&1 Baremetal servers (#431)
Date Thu, 29 Mar 2018 14:31:13 GMT
alibazlamit commented on this pull request.



>     @Override
    public List<HardwareFlavour> listHardwareProfiles() {
-      return api.serverApi().listHardwareFlavours();
+      List<HardwareFlavour> result = new ArrayList<HardwareFlavour>();
+      List<HardwareFlavour> cloudModels = api.serverApi().listHardwareFlavours();
+      for (HardwareFlavour flavor : cloudModels) {
+         result.add(flavor);
+      }
+      List<BareMetalModel> baremetalModels = api.serverApi().listBaremetalModels();
+      for (BareMetalModel model : baremetalModels) {

I need to combine `BareMetalModel` with `HardwareFlavour` as one list, to mix the baremetal
flavors with the cloud flavors, do you think that could be achieved using the function you
suggested? 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/431#discussion_r178074709
Mime
View raw message