Skip to content
This repository was archived by the owner on May 19, 2021. It is now read-only.

Commit 358747f

Browse files
author
Jonas Weismüller
authored
Merge pull request #17 from blue-yonder/bugfix/cast-subscription-id-to-string
cast subscription_id to str
2 parents 25aca34 + e06c2b3 commit 358747f

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

CHANGELOG.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,12 @@ Change Log
44
All notable changes to this project are noted in this file. This project adheres to [Semantic
55
Versioning](http://semver.org/).
66

7-
1.1.0
7+
1.1.1
88
-----
99

1010
- Added BILLING_SCRAPE_TIMEOUT to be able to set a different value then the default of 10 seconds
11+
- Bugfix: convert subscription_id to string, since azure management compute module raises an
12+
Exception if a unicode string is passed as an argument
1113

1214
1.0.0
1315
-----

azure_costs_exporter/allocated_vm_collector.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def _collect_allocated_vms(self, allocated_vms):
5252
rows = []
5353

5454
for subscription_id in self._subscription_ids:
55-
compute_client = ComputeManagementClient(self._credentials, subscription_id)
55+
compute_client = ComputeManagementClient(self._credentials, str(subscription_id))
5656
for vm in compute_client.virtual_machines.list_all():
5757
rows.append([
5858
subscription_id,

0 commit comments

Comments
 (0)