Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: import/export charm actions, config and lxd-profile #151

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

SimonRichardson
Copy link
Member

@SimonRichardson SimonRichardson commented Jul 29, 2024

Import/export charm actions, config, and lxd-profiles for a given charm. These represent the actual files that can be found in a charm. We require these configuration files, so that when we import a 3.6 or 4.0 model into a 4.0 controller, we can correctly infer all of the RI (referential integrity) when creating applications.

Charm actions can be used to ensure RI (referential integrity) when
constructing a charm/application in Juju 4.0.
@SimonRichardson SimonRichardson self-assigned this Jul 29, 2024
This adds charm configs to the description package. It should be
noted that there already exists a CharmConfig in the application
already, but that is the current values of the config. What this
is representing is the actual charm config. It should be noted
that CharmConfig was originally called settings, which doesn't
help either.
This also adds the LXD profile to the charm metadata. Adding this to
charm metadata as a blob allows us to pass it through without it
being incorrectly interpreted as a yaml.
@SimonRichardson SimonRichardson changed the title feat: import/export charm actions feat: import/export charm actions, config and lxd-profile Jul 30, 2024
@SimonRichardson
Copy link
Member Author

/merge

@jujubot jujubot merged commit 62f37b7 into juju:v7 Jul 31, 2024
1 of 2 checks passed
@SimonRichardson SimonRichardson deleted the charm-actions branch July 31, 2024 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants