[PR] fix to handle listing nodes with mem in TB's [libcloud]

2023-11-09 Thread via GitHub
luss opened a new pull request, #1970: URL: https://github.com/apache/libcloud/pull/1970 ## Changes Title (replace this with a logical title for your changes) ### Description Replace this with the PR description (mention the changes you have made, why you have made them, prov

Re: [PR] fix to handle listing nodes with mem in TB's [libcloud]

2023-11-09 Thread via GitHub
luss commented on PR #1970: URL: https://github.com/apache/libcloud/pull/1970#issuecomment-1803873220 I've disccused this small fix to EquinixMetal with my friends at Equinix and they are supportive. -- This is an automated message from the Apache Git Service. To respond to the message, p

[PR] Deprecate facility in the favor of metro [libcloud]

2023-11-09 Thread via GitHub
aayushrangwala opened a new pull request, #1971: URL: https://github.com/apache/libcloud/pull/1971 ## Changes Title (replace this with a logical title for your changes) ### Description Replace this with the PR description (mention the changes you have made, why you have made

Re: [PR] Deprecate facility in the favor of metro [libcloud]

2023-11-09 Thread via GitHub
displague commented on code in PR #1971: URL: https://github.com/apache/libcloud/pull/1971#discussion_r1388283437 ## libcloud/compute/drivers/equinixmetal.py: ## @@ -238,7 +235,7 @@ def ex_list_nodes_for_project(self, ex_project_id, include="plan", page=1, per_p return

Re: [PR] Deprecate facility in the favor of metro [libcloud]

2023-11-09 Thread via GitHub
displague commented on code in PR #1971: URL: https://github.com/apache/libcloud/pull/1971#discussion_r1388284517 ## libcloud/compute/drivers/equinixmetal.py: ## @@ -491,7 +493,7 @@ def _to_size(self, data): except KeyError: cpus = None regions = [

Re: [PR] Deprecate facility in the favor of metro [libcloud]

2023-11-09 Thread via GitHub
displague commented on code in PR #1971: URL: https://github.com/apache/libcloud/pull/1971#discussion_r1388307253 ## libcloud/compute/drivers/equinixmetal.py: ## @@ -701,246 +703,14 @@ def ex_disassociate_address(self, address_uuid, include=None): result = self.connect

Re: [PR] Deprecate facility in the favor of metro [libcloud]

2023-11-09 Thread via GitHub
displague commented on PR #1971: URL: https://github.com/apache/libcloud/pull/1971#issuecomment-1804221452 @aayushrangwala perhaps the removal of strorage should be done in a separate PR from the facility deprecation and introduction of metros. -- This is an automated message from the Apa

Re: [PR] Deprecate facility in the favor of metro [libcloud]

2023-11-09 Thread via GitHub
displague commented on code in PR #1971: URL: https://github.com/apache/libcloud/pull/1971#discussion_r1388308929 ## libcloud/compute/drivers/equinixmetal.py: ## @@ -701,246 +703,14 @@ def ex_disassociate_address(self, address_uuid, include=None): result = self.connect

[GH] (libcloud): Workflow run "CI" failed!

2023-11-09 Thread GitBox
The GitHub Actions job "CI" on libcloud.git has failed. Run started by GitHub user asfgit (triggered by asfgit). Head commit for run: dfbb5959739870313f4c47de499963a46a23010d / Tomaz Muraus Reformat code with black. Report URL: https://github.com/apache/libcloud/actions/runs/6819575661 With r