When generating sectionLoads config, dbctl knows to rewrite 's3' as 'DEFAULT': https://gerrit.wikimedia.org/r/plugins/gitiles/operations/software/conftool/+/refs/heads/master/conftool/extensions/dbconfig/config.py#125
However, it does not do the same for readOnlyBySection because we don't tell it to:
https://gerrit.wikimedia.org/r/plugins/gitiles/operations/software/conftool/+/refs/heads/master/conftool/extensions/dbconfig/config.py#99
We should perhaps also modify the schema to prevent s3 from occurring where we know it shouldn't?
you mean in the readOnlyBySection part of the schema?
Maybe, but then we need to remember to change it if we ever move away from the current model.
Change 538732 had a related patch set uploaded (by Giuseppe Lavagetto; owner: Giuseppe Lavagetto):
[operations/software/conftool@master] Translate the default section name to DEFAULT in ReadOnlyBySection too
Change 538732 merged by jenkins-bot:
[operations/software/conftool@master] Translate the default section name to DEFAULT in ReadOnlyBySection too
Mentioned in SAL (#wikimedia-operations) [2019-09-24T13:40:14Z] <_joe_> uploaded conftool 1.1.4-3 to stretch-wikimedia, T233679
Change 539859 had a related patch set uploaded (by CDanis; owner: CDanis):
[operations/software/conftool@master] dbctl schema: disallow 's3' value when appropriate
Change 539859 merged by jenkins-bot:
[operations/software/conftool@master] dbctl schema: disallow 's3' value when appropriate
Change 540119 had a related patch set uploaded (by CDanis; owner: CDanis):
[operations/puppet@production] dbctl schema: apply Ic60e40d2 in production
Change 540119 merged by CDanis:
[operations/puppet@production] dbctl schema: apply Ic60e40d2 in production