-
Notifications
You must be signed in to change notification settings - Fork 228
Support ComponentConfig for configuring defaults #315
Comments
Some defaults and values that could be configurable: - runtime
- sandbox.image
- network-plugin
- CNI bin dir
- CNI conf dir
- CNI subnet
- CNI options
- CNI network name
- docker network name
- dns.nameservers[] (not implemented)
- dns.search[] (not implemented)
- ipv6 support (not implemented)
- vm.cpus
- vm.size
- vm.memory
- log-level
- kernel.image
- kernel.args
- gitops.repo-url
- gitops.branch / ref
- gitops.paths These can be overridden by flags or the appropriate field in an Ignite VM API object. |
Just brainstorming: ignited daemon --config production-config.yaml apiVersion: ignite.weave.works/v1alpha1
kind: Configuration
metadata:
name: production-config
spec:
logLevel: debug
runtime: docker
sandbox:
image: docker.io/weaveworks/ignite:0.6.4
networkPlugin: docker-bridge
dockerBridge:
network: testnet
cni:
binDir: /opt/cni/bin
confDir: /etc/cni/net.d
subnet: 10.39.0.0/18
networkName: prod-ignite-vm-bridge # should this be configurable?
confFile: 10-ignite.prod.conflist # should this be configurable?
# options:
dns:
# ipv6: true
nameservers:
- 1.1.1.1
- 9.9.9.9
search:
- deployment.example.com
- production.example.com
gitops:
repoURL: git@github.com/stealthybox/example
ref: master
paths:
- ignite/production-backend
- ignite/production-cache
kernel:
image: <something>
cmdLine:
- my
- default
- kernel
- args
vm:
cpus: 4
size: 20GB
memory: 4GB |
For the
|
Possibly this discussion is relevant to PR #428 I'd say this micro-VM data belongs in XDG_DATA_HOME: manifest:
paths:
- /etc/firecracker/manifests
- /dfs/db/firecracker/manifests
- /dfs/sec/firecracker/manifests
- /home/me/.local/share/ignite/manifests
- /usr/local/share/ignite/manifests
- /usr/share/ignite/manifests What configuration goes where?
|
As
ignited
grows in complexity, instead of registering tons of flags on it, we'd use a declarative configuration file, aka. ComponentConfig, for it. (Could be even backed by git?)The text was updated successfully, but these errors were encountered: