]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
common: make y2c.py choke on duplicate keys 40891/head
authorIlya Dryomov <idryomov@gmail.com>
Fri, 16 Apr 2021 11:55:56 +0000 (13:55 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Fri, 16 Apr 2021 14:53:49 +0000 (16:53 +0200)
commit7bef7e9b3b19cbbc3f4f185fe0e0caa2faae544a
treed3abeab628c29dc23a3c581dd59de6fc477d6c61
parent32b4e2e34945594b6a6ebaca10e9e30a02f868c1
common: make y2c.py choke on duplicate keys

Commit 5505fc0051a3 ("common: generate legacy_config_opts.h from
.yaml.in files") inadvertently reverted a change of a default value by
adding a second "default" key with the old value.  This was corrected
in commit 75e07f8638ef ("common/options/global: correct default of
auth_mon_ticket_ttl"), but highlights that mis-merging a yaml file
is rather easy.

To prevent this happening again, fail the build if duplicate keys
exist in any of src/common/options/*.yaml.in files.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
src/common/options/y2c.py