Skip to content

Commit

Permalink
no need to expose metrics port; no need to ensure config volume
Browse files Browse the repository at this point in the history
Signed-off-by: hfuss <[email protected]>
  • Loading branch information
onelapahead committed Nov 23, 2021
1 parent b138b29 commit 32faba1
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
4 changes: 0 additions & 4 deletions internal/docker/docker_config.go
Original file line number Diff line number Diff line change
Expand Up @@ -99,10 +99,6 @@ func CreateDockerCompose(s *types.Stack) *DockerComposeConfig {
Logging: StandardLogOptions,
}

if s.PrometheusEnabled {
compose.Services["firefly_core_"+member.ID].Ports = append(compose.Services["firefly_core_"+member.ID].Ports, fmt.Sprintf("%d:%d", member.ExposedFireflyMetricsPort, member.ExposedFireflyMetricsPort))
}

compose.Volumes[fmt.Sprintf("firefly_core_%s", member.ID)] = struct{}{}
}
if s.Database == "postgres" {
Expand Down
4 changes: 0 additions & 4 deletions internal/stacks/stack_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -264,10 +264,6 @@ func (s *StackManager) ensureDirectories() error {
}
}

if err := os.MkdirAll(filepath.Join(dataDir, "prometheus_config"), 0755); err != nil {
return err
}

return nil
}

Expand Down

0 comments on commit 32faba1

Please sign in to comment.