Also get rid of the previous (unused) 'swh db init' command.
Details
- Reviewers
ardumont - Group Reviewers
Reviewers - Commits
- rDCORE56d505f34a96: Split db-init cli in 2 (init and create) and move them under 'swh db'
Diff Detail
- Repository
- rDCORE Foundations and core functionalities
- Branch
- master
- Lint
Lint Skipped - Unit
Unit Tests Skipped - Build Status
Buildable 15375 Build 23678: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 23677: arc lint + arc unit
Event Timeline
Build is green
Patch application report for D4000 (id=14105)
Rebasing onto e5312bdaa8...
Current branch diff-target is up to date.
Changes applied before test
commit b7593ab57ab4507b4b05d643ed6e4a68b83e7913 Author: David Douard <david.douard@sdfa3.org> Date: Mon Sep 21 16:25:52 2020 +0200 Split db-init cli in 2 (init and create) and move them under 'swh db' Also get rid of the previous (unused) 'swh db init' command.
See https://jenkins.softwareheritage.org/job/DCORE/job/tests-on-diff/67/ for more details.
Build is green
Patch application report for D4000 (id=14106)
Rebasing onto e5312bdaa8...
Current branch diff-target is up to date.
Changes applied before test
commit 2a2c8f1bde96e25fba9bf69179dc65a2115ee507 Author: David Douard <david.douard@sdfa3.org> Date: Mon Sep 21 16:25:52 2020 +0200 Split db-init cli in 2 (init and create) and move them under 'swh db' The idea it to have the 'swh db create' command to create the database and prepare it by executing SQL -superuser- files, i.e. SQL files which execution requires superuser permissions -- typically install languages or extensions. Then the 'swh db init' execute the initialization SQL files that do not require superuser permissions. Also get rid of the previous (unused) 'swh db init' command.
See https://jenkins.softwareheritage.org/job/DCORE/job/tests-on-diff/68/ for more details.
lgtm
one comment/remark inline about a missed docstring update to drop some return value.
swh/core/cli/db.py | ||
---|---|---|
188 | Looks like this needs to be removed from the docstring. The last call is execute_sql_files (which seem to no longer return anything, --quiet is passed to psql). |
Build is green
Patch application report for D4000 (id=14107)
Rebasing onto e5312bdaa8...
Current branch diff-target is up to date.
Changes applied before test
commit 8ca045e0df8e07fb2cc6ad3f521c931001dba57f Author: David Douard <david.douard@sdfa3.org> Date: Mon Sep 21 16:25:52 2020 +0200 Split db-init cli in 2 (init and create) and move them under 'swh db' The idea it to have the 'swh db create' command to create the database and prepare it by executing SQL -superuser- files, i.e. SQL files which execution requires superuser permissions -- typically install languages or extensions. Then the 'swh db init' execute the initialization SQL files that do not require superuser permissions. Also get rid of the previous (unused) 'swh db init' command.
See https://jenkins.softwareheritage.org/job/DCORE/job/tests-on-diff/69/ for more details.
swh/core/cli/db.py | ||
---|---|---|
188 | I thought I did remove this Returns: field but I did not apparently, thx |
Build is green
Patch application report for D4000 (id=14108)
Rebasing onto e5312bdaa8...
Current branch diff-target is up to date.
Changes applied before test
commit 56d505f34a96e2ee76457a22c866cb3a204ae77b Author: David Douard <david.douard@sdfa3.org> Date: Mon Sep 21 16:25:52 2020 +0200 Split db-init cli in 2 (init and create) and move them under 'swh db' The idea it to have the 'swh db create' command to create the database and prepare it by executing SQL -superuser- files, i.e. SQL files which execution requires superuser permissions -- typically install languages or extensions. Then the 'swh db init' execute the initialization SQL files that do not require superuser permissions. Also get rid of the previous (unused) 'swh db init' command.
See https://jenkins.softwareheritage.org/job/DCORE/job/tests-on-diff/70/ for more details.