configparser.NoOptionError: No option 'storage-backend' in section: 'mirror'

[epifeny]

Hello,

I'm using v6.5.0 and when I attempt to use bandersnatch verify --delete I get a key error.
I read the docs, and from what I understand, storage-backend defaults to filesystem, so I didn't think it would be required to add it into the configuration file.
https://bandersnatch.readthedocs.io/en/latest/mirror_configuration.html#storage-backend

The mirror section looks like this:

[mirror]
directory = /opt/pip/external
hash-index = false
json = true
master = https://pypi.org
stop-on-error = false
timeout = 10
verifiers = 5
workers = 1

---

/ # bandersnatch --version
bandersnatch 6.5.0

---

/ # bandersnatch --config /etc/bandersnatch.conf verify --delete
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/configparser.py", line 805, in get
    value = d[option]
            ~^^^^^^^^
  File "/usr/local/lib/python3.11/collections/__init__.py", line 1006, in __getitem__
    return self.__missing__(key)            # support subclasses that define __missing__
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/collections/__init__.py", line 998, in __missing__
    raise KeyError(key)
KeyError: 'storage-backend'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/bandersnatch", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/local/lib/python3.11/site-packages/bandersnatch/main.py", line 225, in main
    return asyncio.run(async_main(args, config))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 654, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/bandersnatch/main.py", line 161, in async_main
    return await bandersnatch.verify.metadata_verify(config, args)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/bandersnatch/verify.py", line 241, in metadata_verify
    backend=config.get("mirror", "storage-backend"),
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/configparser.py", line 808, in get
    raise NoOptionError(option, section)
configparser.NoOptionError: No option 'storage-backend' in section: 'mirror'

---

/ # pip freeze
aiohttp==3.9.3
aiohttp-socks==0.8.4
aiohttp-xmlrpc==1.5.0
aiosignal==1.3.1
async-timeout==4.0.3
attrs==23.2.0
bandersnatch==6.5.0
filelock==3.13.3
frozenlist==1.4.1
humanfriendly==10.0
idna==3.6
lxml==5.2.1
multidict==6.0.5
packaging==24.0
python-socks==2.4.4
yarl==1.9.4

[cooperlees]

Thanks for the report. @flyinghyrax is working on making our defaults better as there are some gaps. This is one.

#1694 is part of it.

[flyinghyrax]

Indeed! I'm hoping to work more on it this weekend.