fix: update Juju environment variable assignments #613
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gfouillet
force-pushed
the
ci/handle-ipv6-controller-ip-addresses
branch
5 times, most recently
from
298b71b to
0391513
Compare
Updated Juju environment variable assignments for consistency and correctness in various workflow files and documentation. - Before this commit, the environment variable assignments were inconsistent and used mixed formatting, including usage of `tr` command instead of relying on `raw` option for `yq` output - After this commit, the assignments are consistent across all files, and we rely more on `yq` option to get the expected result, improving the readability and reducing the likelihood of errors.
gfouillet
force-pushed
the
ci/handle-ipv6-controller-ip-addresses
branch
from
0391513 to
e76c16f
Compare
gfouillet
changed the title
hmlanigan
added
the
state/codefreeze
alesstimec
removed
the
state/codefreeze
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
<Please provide a description of this PR. What, why, when, how, where, who?>
Fixes:
Type of change
<What type of a change is this? Please keep one or more relevant lines from below and delete the others.>
Environment
Juju controller version:
Terraform version:
QA steps
Manual QA steps should be done to test this PR.
Additional notes
<Please add relevant notes & information, links to mattermost chats, other related issues/PRs, anything to help understand and QA the PR.>