Update resource-count-azure.py

jpummill commented 1 year ago

Added initial code to count AKS nodes

Description

As part of the main program loop which iterates through Azure account id's, added a new "try" block to retrieve AKS clusters using "az aks list".

Report the number of nodes for each cluster found and then add the nodes to an account total variable (az_account_aks_node_count) and print the result.

After processing each account, add the number of nodes per account (az_account_aks_node_count) to a global total (global_az_aks_node_count) and print it.

Motivation and Context

In order to count VM's for VM based pricing models it is necessary to include AKS (Kubernetes) nodes to existing Azure Virtual Machine counts.

How Has This Been Tested?

Tested by running script in an Azure tenant with 4 subscriptions.
One subscription had multiple clusters.
Individual cluster node counts were correct.
Account cluster node counts were correct.
Global cluster node counts were correct.

Screenshots (if appropriate)

pcs-sizing-script-for-azure-screenshot

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[ ] All new and existing tests passed.

welcome-to-palo-alto-networks[bot] commented 1 year ago

:tada: Thanks for opening this pull request! We really appreciate contributors like you! :raised_hands:

lsmithpanw commented 1 year ago

LGTM

welcome-to-palo-alto-networks[bot] commented 12 months ago

:tada: Congrats on getting your first pull request merged! We here at Palo Alto Networks are so grateful! :heart:

PaloAltoNetworks / pcs-sizing-scripts