Skip to content

Commit

Permalink
Docker/setups: Fix multiple merge keys in yaml
Browse files Browse the repository at this point in the history
> YAML parsing was improved in [docker-compose] v2.17 - the old behavior of
> allowing multiple merge keys was a quirk of the parser.

Reference: docker/compose#10394 (comment)
  • Loading branch information
awelzel committed Mar 15, 2024
1 parent 3982e24 commit 2e9afe2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 10 deletions.
9 changes: 3 additions & 6 deletions Docker/setups/default/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,16 +23,13 @@ x-zeek-controller: &ZEEK_CONTROLLER

services:
controller:
<<: *ZEEK_BASE
<<: *ZEEK_CONTROLLER
<<: [*ZEEK_BASE, *ZEEK_CONTROLLER]

inst1:
<<: *ZEEK_BASE
<<: *ZEEK_AGENT
<<: [*ZEEK_BASE, *ZEEK_AGENT]

inst2:
<<: *ZEEK_BASE
<<: *ZEEK_AGENT
<<: [*ZEEK_BASE, *ZEEK_AGENT]

client:
image: zeektest:latest
Expand Down
6 changes: 2 additions & 4 deletions Docker/setups/singlehost/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,7 @@ x-zeek-client: &ZEEK_CLIENT

services:
controller:
<<: *ZEEK_BASE
<<: *ZEEK_CONTROLLER
<<: [*ZEEK_BASE, *ZEEK_CONTROLLER]

client:
<<: *ZEEK_BASE
<<: *ZEEK_CLIENT
<<: [*ZEEK_BASE, *ZEEK_CLIENT]

0 comments on commit 2e9afe2

Please sign in to comment.