-
Notifications
You must be signed in to change notification settings - Fork 5
Open
1 / 11 of 1 issue completedLabels
enhancementNew feature or requestNew feature or request
Description
Note: Completion of #113 is required before moving forward with this enhancement.
Add a push-configs
command. This command will push ~/.minitrino/etc/
to cluster container(s) and then restart them (we could make a shared restart function for this part of the flow given the addition of minitrino restart
#113).
Open questions:
- How do we manage pushing configs to clusters with workers? Maybe...
- Push configs to the coordinator first
- Run the copy logic utilized by the
provision_workers
function to copy the updated coordinator/etc/${CLUSTER_DIST}/
configs - Restart the coordinator and the workers
- How do we ensure we're adding a new file or a new version of an existing file? How do we handle replacing existing files?
- Honestly, I think the equivalent of
docker cp
is okay. Any config file in~/.minitrino/etc/
is subject to replacing target/etc/${CLUSTER_DIST}/${file}
files wheneverminitrino push-configs
is executed.
- Honestly, I think the equivalent of
Other notes:
- Ensure we account for this new directory when running
snapshot
commands. We won't be able to preserve the context of when the user ranminitrino push-configs
, but we can at least ensure we add~/.minitrino/etc/
to the snapshot tarball.
Sub-issues
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request
Projects
Status
Backlog