Skip to content

Commit

Permalink
improvement(sync): use native Mutagen daemon by default (#6227)
Browse files Browse the repository at this point in the history
* feat(sync): enable native Mutagen by default

* improvement(sync): update log warnings and hints
  • Loading branch information
vvagaytsev authored Jun 28, 2024
1 parent 1ecacc9 commit b33d9c8
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion core/src/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ export const gardenEnv = {
.required(false)
.default("https://get.garden.io/releases")
.asUrlString(),
GARDEN_ENABLE_NEW_SYNC: env.get("GARDEN_ENABLE_NEW_SYNC").required(false).default("false").asBool(),
GARDEN_ENABLE_NEW_SYNC: env.get("GARDEN_ENABLE_NEW_SYNC").required(false).default("true").asBool(),
// GARDEN_CLOUD_BUILDER will always override the config; That's why it doesn't have a default.
// FIXME: If the environment variable is not set, asBool returns undefined, unlike the type suggests. That's why we cast to `boolean | undefined`.
GARDEN_CLOUD_BUILDER: env.get("GARDEN_CLOUD_BUILDER").required(false).asBool() as boolean | undefined,
Expand Down
6 changes: 3 additions & 3 deletions core/src/mutagen.ts
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,7 @@ function logMutagenDaemonWarning(log: Log) {

log.warn(
deline`
It looks like you might have changed to a different version of the sync daemon.\n
It looks like the sync daemon might have been changed to a different version.\n
Therefore the sync daemon needs to be restarted, and the affected deploys must be redeployed.\n
Please, stop this command and follow the instructions below.\n
Expand Down Expand Up @@ -378,8 +378,8 @@ export class Mutagen {
deline`
Warning!\n
Garden will use the new sync daemon in the next release (0.13.34).
Thus, the default value of the \`GARDEN_ENABLE_NEW_SYNC\` environment variable will be switched to \`true\`.\n
Starting from 0.13.34, Garden uses the new sync daemon.
Thus, the default value of the \`GARDEN_ENABLE_NEW_SYNC\` environment variable is \`true\` now.\n
Please make sure you have tested the new sync daemon. See the troubleshooting docs for more details: ${makeDocsLinkStyled("guides/code-synchronization", "#restarting-sync-daemon")}\n`
)
Expand Down

0 comments on commit b33d9c8

Please sign in to comment.