Re: [PR] [azure arm] delete VM OS disk if it is a managed disk [libcloud]

2023-10-03 Thread via GitHub
vicwicker commented on code in PR #1957: URL: https://github.com/apache/libcloud/pull/1957#discussion_r1343874253 ## libcloud/compute/drivers/azure_arm.py: ## @@ -912,6 +917,19 @@ def destroy_node( raise time.sleep(10) +# Optio

Re: [PR] [azure arm] delete VM OS disk if it is a managed disk [libcloud]

2023-10-03 Thread via GitHub
vicwicker commented on code in PR #1957: URL: https://github.com/apache/libcloud/pull/1957#discussion_r1343880775 ## libcloud/compute/drivers/azure_arm.py: ## @@ -808,6 +808,7 @@ def destroy_node( ex_destroy_vhd=True, ex_poll_qty=10, ex_poll_wait=10, +

Re: [PR] [azure arm] delete VM OS disk if it is a managed disk [libcloud]

2023-10-03 Thread via GitHub
vicwicker commented on code in PR #1957: URL: https://github.com/apache/libcloud/pull/1957#discussion_r1343880775 ## libcloud/compute/drivers/azure_arm.py: ## @@ -808,6 +808,7 @@ def destroy_node( ex_destroy_vhd=True, ex_poll_qty=10, ex_poll_wait=10, +

[GitHub] [libcloud]: Workflow run "Publish pricing.json to S3 bucket" failed!

2023-10-03 Thread GitBox
The GitHub Actions job "Publish pricing.json to S3 bucket" on libcloud.git has failed. Run started by GitHub user asfgit (triggered by asfgit). Head commit for run: 966ba6d6f133f6668d5e1dbfec8490a1763c0367 / Tomaz Muraus Merge pull request #1956 from apache/dependabot/pip/cryptography-41.0.4

[PR] remove executable bit for ssl key fixtures [libcloud]

2023-10-03 Thread via GitHub
kbrock opened a new pull request, #1960: URL: https://github.com/apache/libcloud/pull/1960 ### Description We have a security scanner that noticed that these fixtures are chmod x+a Since these are data files, best to make them non-executable ### Status - done, ready for

[GitHub] [libcloud]: Workflow run "Publish pricing.json to S3 bucket" is working again!

2023-10-03 Thread GitBox
The GitHub Actions job "Publish pricing.json to S3 bucket" on libcloud.git has succeeded. Run started by GitHub user Kami (triggered by Kami). Head commit for run: 966ba6d6f133f6668d5e1dbfec8490a1763c0367 / Tomaz Muraus Merge pull request #1956 from apache/dependabot/pip/cryptography-41.0.4 B