Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: config-changed is triggered by Juju trust #1357

Merged

Conversation

tonyandrewmeyer
Copy link
Contributor

Note that juju trust changes will trigger a config-changed event.

I couldn't find any other cases that aren't already mentioned in the ConfigChangedEvent doc that would trigger config-changed, so I hope this is all.

Fixes #1211.

Copy link
Contributor

@james-garner-canonical james-garner-canonical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!

@tonyandrewmeyer tonyandrewmeyer merged commit e794460 into canonical:main Sep 5, 2024
32 checks passed
@tonyandrewmeyer tonyandrewmeyer deleted the trust-config-changed-1211 branch September 5, 2024 04:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve docs about what triggers config-changed
3 participants