Skip to content

Commit 95f60ad

Browse files
Setting app secrets can only be done by the leader - ops 2.9+ enforces this like Juju. (#322)
1 parent ac3a314 commit 95f60ad

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

tests/unit/test_charm.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1732,7 +1732,12 @@ def test_migration_from_single_secret(self, scope, is_leader, _, __):
17321732
assert self.harness.charm.get_secret(scope, "operator-password") == "bla"
17331733

17341734
# Reset new secret
1735+
# Only the leader can set app secret content.
1736+
with self.harness.hooks_disabled():
1737+
self.harness.set_leader(True)
17351738
self.harness.charm.set_secret(scope, "operator-password", "blablabla")
1739+
with self.harness.hooks_disabled():
1740+
self.harness.set_leader(is_leader)
17361741
assert self.harness.charm.model.get_secret(label=f"postgresql.{scope}")
17371742
assert self.harness.charm.get_secret(scope, "operator-password") == "blablabla"
17381743
assert SECRET_INTERNAL_LABEL not in self.harness.get_relation_data(

0 commit comments

Comments
 (0)