Skip to content
Snippets Groups Projects

rework the cli db tooling for winery

Open David Douard requested to merge douardda/swh-core:fix-db-for-winery into master

These changes are necessary to make the objstorage winery backend fit in here.

The issue was in winery, the db settings are not at the top level of the
'winery' backend configuration, but under the "database" subsection,
which was an unsupported case.

These changes make it work, but for simplification purpose, we drop the
support for 'full module path' specification of the target backend to
initialize. Namely, one cannot use the pattern:

  # this will not work any more
  swh db init -d DBCNX swh.storage.backend.masking

but stick to the entry-point pattern:

  swh db init -d DBCNX storage:masking

Or use the config file based approach.

Also comes with a few cleanup revisions:

  • remove --module-config-key option
  • remove support for xxx_db configuration entries
  • Move import_swhmodule helper function in swh.core.config
  • db: Drop BW compat for non-entrypoint aware swh packages

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading