Skip to content
Snippets Groups Projects

Add a Storage.flavor property to the postgresql backend

1 unresolved thread

Add a Storage.flavor property to the postgresql backend

and add tests for 'mirror' and 'read_replica' flavors.

Depends on swh-core!283 (closed).

Test Plan

tox (failing since it needs that diff ^ landed and a bump from swh.core)


Migrated from D8067 (view on Phabricator)

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
24 24 yield db, cur
25 25
26 26
27 @pytest.mark.db
28 def test_pgstorage_flavor(swh_storage):
29 assert swh_storage.get_flavor() == "default"
  • lgtm, one suggestion inline.

  • Build is green

    Patch application report for D8067 (id=29108)

    Rebasing onto c19f53f1...

    Current branch diff-target is up to date.
    Changes applied before test
    commit 47caf04ea5e9d47c976200fff7d61164985a4390
    Author: David Douard <david.douard@sdfa3.org>
    Date:   Fri Jul 1 14:56:47 2022 +0200
    
        Add a Storage.flavor property to the postgresql backend
        
        and add tests for 'mirror' and 'read_replica' flavors.
    
    commit a00650ea0c34a98653d24693ce07555f55d816f4
    Author: David Douard <david.douard@sdfa3.org>
    Date:   Fri Jul 1 14:50:28 2022 +0200
    
        Update pytest_plugin for swh.core 2.10

    See https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1625/ for more details.

  • Antoine R. Dumont mentioned in merge request !796 (closed)

    mentioned in merge request !796 (closed)

  • mentioned in merge request swh-core!283 (closed)

  • lgtm (i've triggered back the build)

    one requirement change suggested inline.

  • Merge request was accepted

  • Antoine R. Dumont approved this merge request

    approved this merge request

  • Author Maintainer

    Merge request was merged

  • closed

  • Please register or sign in to reply
    Loading