Skip to content

Commit

Permalink
Merge branch 'release/v0.14.4' into main
Browse files Browse the repository at this point in the history
  • Loading branch information
meiserloh authored and cesmarvin committed Nov 6, 2024
2 parents 6287498 + 8370495 commit d149212
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 4 deletions.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [Unreleased]

## [v0.14.4] - 2024-11-06
### Fixed
- [#48] Map nginx dependency to `nginx-ingress` and `nginx-static`.
Both Dogus are required in the installation process in the dogu-operator.
Only mapping `nginx-ingress` can result in following installation order `ingress`, `cas`, `static` because the `nginx-ingress` has no dependency and the new dependency sorting algorithm is not deterministic.

## [v0.14.3] - 2024-10-30
### Fixed
- [#47] map nginx dependencies to k8s equivalent dogus
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Set these to the desired values
ARTIFACT_ID=cesapp-lib
VERSION=0.14.3
VERSION=0.14.4

GOTAG?=1.18.6
MAKEFILES_VERSION=7.4.0
Expand Down
11 changes: 8 additions & 3 deletions core/dogu-sort.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,13 @@ var (
Type: DependencyTypeDogu,
Name: "nginx-ingress",
}
k8sDependencyMapping = map[string]Dependency{
"nginx": nginxIngressDependency,

nginxStaticDependency = Dependency{
Type: DependencyTypeDogu,
Name: "nginx-static",
}
k8sDependencyMapping = map[string][]Dependency{
"nginx": {nginxIngressDependency, nginxStaticDependency},
}
)

Expand Down Expand Up @@ -153,7 +158,7 @@ func (bd *sortByDependency) dependenciesToDogus(dependencies []Dependency) []*Do
func appendK8sMappedDependencies(dependencies []Dependency) []Dependency {
for _, dep := range dependencies {
if _, ok := k8sDependencyMapping[dep.Name]; ok {
dependencies = append(dependencies, k8sDependencyMapping[dep.Name])
dependencies = append(dependencies, k8sDependencyMapping[dep.Name]...)
}
}
return dependencies
Expand Down

0 comments on commit d149212

Please sign in to comment.