Skip to content
This repository has been archived by the owner on Aug 14, 2020. It is now read-only.

Commit

Permalink
ace validator: fix app annotations validation.
Browse files Browse the repository at this point in the history
The runtimeApp annotations were merged over themself and not over the image
manifest annotations.
  • Loading branch information
sgotti committed Aug 19, 2016
1 parent 737c544 commit d183e7b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ace/validator.go
Original file line number Diff line number Diff line change
Expand Up @@ -328,9 +328,9 @@ func validatePodMetadata(metadataURL string, pm *schema.PodManifest) results {
func validateAppAnnotations(metadataURL string, pm *schema.PodManifest, app *schema.RuntimeApp, img *schema.ImageManifest) results {
r := results{}

// build a map of expected annotations by merging app.Annotations
// build a map of expected annotations by merging img.Annotations
// with PodManifest overrides
expectedAnnots := app.Annotations
expectedAnnots := img.Annotations
a := pm.Apps.Get(app.Name)
if a == nil {
panic("could not find app in manifest!")
Expand Down

0 comments on commit d183e7b

Please sign in to comment.