azure-integrator crashes while missing tenant-id

Bug #1964067 reported by Yang Kelvin Liu
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Azure Integrator Charm
Fix Released
Medium
Adam Dyess

Bug Description

azure-integrator fails if the azure credential is `service-principal-secret`.

unit-azure-integrator-0: 16:50:23 ERROR unit.azure-integrator/0.juju-log clients:17: Hook error:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-azure-integrator-0/.venv/lib/python3.8/site-packages/charms/reactive/__init__.py", line 74, in main
    bus.dispatch(restricted=restricted_mode)
  File "/var/lib/juju/agents/unit-azure-integrator-0/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 390, in dispatch
    _invoke(other_handlers)
  File "/var/lib/juju/agents/unit-azure-integrator-0/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 359, in _invoke
    handler.invoke()
  File "/var/lib/juju/agents/unit-azure-integrator-0/.venv/lib/python3.8/site-packages/charms/reactive/bus.py", line 181, in invoke
    self._action(*args)
  File "/var/lib/juju/agents/unit-azure-integrator-0/charm/reactive/azure.py", line 66, in handle_requests
    layer.azure.send_additional_metadata(request)
  File "/var/lib/juju/agents/unit-azure-integrator-0/charm/lib/charms/layer/azure.py", line 202, in send_additional_metadata
    tenant_id=credentials["tenant-id"],
KeyError: 'tenant-id'

Revision history for this message
Adam Dyess (addyess) wrote :

I've found a work-around to this bug

juju ssh azure-integrator/0
sudo vim /var/lib/juju/agents/unit-azure-integrator-0/charm/lib/charms/layer/azure.py

add the following to login_cli(...)

```python
    tenant_id = _get_tenant_id(sub_id)
    if tenant_id:
        creds_data["tenant-id"] = tenant_id
```

summary: - azure-integrator
+ azure-integrator crashes while missing tenant-id
Changed in charm-azure-integrator:
status: New → Confirmed
milestone: none → 1.25
milestone: 1.25 → 1.24+ck1
assignee: nobody → Adam Dyess (addyess)
Revision history for this message
Kevin W Monroe (kwmonroe) wrote :
Changed in charm-azure-integrator:
status: Confirmed → Fix Committed
importance: Undecided → Medium
Adam Dyess (addyess)
tags: added: backport-needed
Adam Dyess (addyess)
tags: removed: backport-needed
Adam Dyess (addyess)
Changed in charm-azure-integrator:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.