Differential D7944

Add support for running the server with 'postgresql' storage cls
ClosedPublic
Actions

Authored by vlorentz on Jun 2 2022, 3:10 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDCIDXca4e91e5b75e: Add support for running the server with 'postgresql' storage cls

Summary

'local' is a deprecated alias

Diff Detail

Repository

rDCIDX Metadata indexer

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 29698
Build 46410: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 46409: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	3 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.indexer.tests.storage.test_server::test_load_and_check_config_remote_config_local_type_raise[memory] class_storage = 'memory' tmpdir = local('/tmp/pytest-of-jenkins/pytest-0/test_load_and_check_config_rem1')
	3 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.indexer.tests.storage.test_server::test_load_and_check_config_remote_config_local_type_raise[remote] class_storage = 'remote' tmpdir = local('/tmp/pytest-of-jenkins/pytest-0/test_load_and_check_config_rem0')
	4 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.indexer.codemeta::swh.indexer.codemeta.merge_values
	3 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.indexer.metadata_dictionary.maven::swh.indexer.metadata_dictionary.maven.MavenMapping.normalize_groupId
	8 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.indexer.metadata_dictionary.maven::swh.indexer.metadata_dictionary.maven.MavenMapping.parse_licenses
		View Full Test Results (2 Failed · 336 Passed · 15 Skipped)

Event Timeline

vlorentz created this revision.Jun 2 2022, 3:10 PM

vlorentz added a child revision: D7945: indexer_storage.yml: Replace deprecated alias 'local' with 'postgresql'.

Build has FAILED

Patch application report for D7944 (id=28613)

Rebasing onto 1aa8bdaca1...

Current branch diff-target is up to date.

Changes applied before test

commit e579cf48b6338a7fdc4f7f6815f5580268719d98
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Jun 2 15:10:22 2022 +0200

    Add support for running the server with 'postgresql' storage cls
    
    'local' is a deprecated alias

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/245/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/245/console

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 2 2022, 3:17 PM

Harbormaster failed remote builds in B29698: Diff 28613!

update tests

Build is green

Patch application report for D7944 (id=28618)

Rebasing onto 1aa8bdaca1...

Current branch diff-target is up to date.

Changes applied before test

commit ca4e91e5b75e524308f2131b55cb22d5570ea057
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Jun 2 15:10:22 2022 +0200

    Add support for running the server with 'postgresql' storage cls
    
    'local' is a deprecated alias

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

Harbormaster completed remote builds in B29703: Diff 28618.Jun 2 2022, 3:35 PM

vlorentz requested review of this revision.Jun 2 2022, 3:35 PM

ardumont accepted this revision.Jun 2 2022, 3:44 PM

This revision is now accepted and ready to land.Jun 2 2022, 3:44 PM

Closed by commit rDCIDXca4e91e5b75e: Add support for running the server with 'postgresql' storage cls (authored by vlorentz). · Explain WhyJun 3 2022, 10:30 AM

This revision was automatically updated to reflect the committed changes.

vlorentz added a commit: rDCIDXca4e91e5b75e: Add support for running the server with 'postgresql' storage cls.