particle-os/debian-forge-composer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

composer: add sanity checks

Browse source 
Running composer without workers or APIs enabled is pretty much a no-op.
Let's forbid that.

Signed-off-by: Ondřej Budai <ondrej@budai.cz>
This commit is contained in:
Ondřej Budai 2020-11-16 12:03:29 +01:00 • committed by Tom Gundersen
parent 71428d3131
commit 117da5aa8a
1 changed files with 9 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

9
cmd/osbuild-composer/composer.go
View file

@ -156,6 +156,15 @@ func (c *Composer) InitRemoteWorkers(cert, key string, l net.Listener) error {
// //
// Running without the weldr API is currently not supported. // Running without the weldr API is currently not supported.
func (c *Composer) Start() error { func (c *Composer) Start() error {
// sanity checks
if c.localWorkerListener == nil && c.workerListener == nil {
log.Fatal("neither the local worker socket nor the remote worker socket is enabled, osbuild-composer is useless without workers")
}
if c.apiListener == nil && c.weldrListener == nil {
log.Fatal("neither the weldr API socket nor the composer API socket is enabled, osbuild-composer is useless without one of these APIs enabled")
}
if c.localWorkerListener != nil { if c.localWorkerListener != nil {
go func() { go func() {
err := c.workers.Serve(c.localWorkerListener) err := c.workers.Serve(c.localWorkerListener)