s_all should run test/format/config.sh

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Lint
    • None
    • Storage Engines, Storage Engines - Foundations
    • SE Foundations - Q3+ Backlog
    • None

      The auto-generated header and configuration files in the test/format directory sometimes get out of sync with the human-generated declarations in test/format/config.sh.

      One example is WT-15017, where we made a change to the config but forgot to run the script. That resulted in the aforementioned files being out of sync on develop for multiple weeks.

      We got really lucky here – the mismatched files didn't seem to cause any issues, but it could easily have been different.

      We already have s_all to check the WT API config, and it wouldn't be hard to extend that to test/format, so let's do it.

            Assignee:
            [DO NOT USE] Backlog - Storage Engines Team
            Reporter:
            Will Korteland
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: