search

cloud-bulldozer / benchmark-operator

The Chuck Norris of cloud benchmarks
Apache License 2.0
282 stars 128 forks source link

corrected indentation with `filter` #735

Closed mukrishn closed 2 years ago

mukrishn commented 2 years ago

Description

macro is not passing all current variables to the include'd template, so metadata dict was always undefined and job definition was missing initContainers section. Replaced that with regular include with filter to indent properly

I could see it includes metadata.yml on both server and client pod definition now,

...
  initContainers:
  - args:
    - |
      python3 stockpile-wrapper.py -s=https://search-perfscale-dev-chmf5l4sh66lvxbnadi4bznl3a.us-west-2.es.amazonaws.com:443 -u=13a31b07-4c90-535f-90cd-37c8bda42c8b -n=${my_node_name} -N=${my_pod_name} --redisip=10.129.40.107 --redisport=6379 --tags=common,k8s,openshift
    command:
    - /bin/sh
    - -c
...

Fixes

725

mukrishn commented 2 years ago

@dry923 PTAL, it works for me.