Skip to content

Commit 701cb94

Browse files
authored
Merge pull request #444 from SimonRichardson/handle-application-set-status
#444 Because pylibjuju has an internal cache, very similar to how juju model cache works, we need to correctly handle when the application status is set by the charm author, vs how the application is dervied via the unit status.
2 parents fd531e8 + 637b529 commit 701cb94

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

juju/application.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,10 +80,12 @@ def status(self):
8080
workload status and highlight the most relevant (severity).
8181
"""
8282
status = self.safe_data['status']['current']
83-
unit_status = [status]
84-
for unit in self.units:
85-
unit_status.append(unit.workload_status)
86-
return derive_status(unit_status)
83+
if status == "unset":
84+
unit_status = []
85+
for unit in self.units:
86+
unit_status.append(unit.workload_status)
87+
return derive_status(unit_status)
88+
return status
8789

8890
@property
8991
def status_message(self):

0 commit comments

Comments
 (0)