Differential D317

Make the vault's scheduler configuration adjustable
ClosedPublic
Actions

Authored by ardumont on May 3 2018, 3:54 PM.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Commits

rDVAU3688b7d627c2: api/server: Use remote service for default configuration storage
rDVAUc3a07c6b73cb: Make the vault's scheduler configuration adjustable

Summary

Use the scheduler's api instantiation

Not urgent

This can be merged at some point, when you are done with your current tasks for example ;)

Related T1035

Diff Detail

Repository

rDVAU Software Heritage Vault

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.May 3 2018, 3:54 PM

Harbormaster completed remote builds in B1213: Diff 1038.May 3 2018, 3:54 PM

LGTM, I just have one nitpick on the default configuration that we might want to fix before merging this.

swh/vault/api/server.py
44–49	Aren't the default usually the remote services, and not the local ones?

This revision is now accepted and ready to land.May 4 2018, 2:22 PM

ardumont added inline comments.May 4 2018, 2:28 PM

swh/vault/api/server.py
44–49	I kept your default because i did not want to impose anything. Running this by default, you would need to either run a scheduler service (or edit/add a ~/.config/swh/vault/server.yml with the local setup). But sure, we can use the remote one. That's what i did for the deposit [1]. [1] rDDEP7e535ab1c196fdeab8722ef610ca1d02ceadbc58

ardumont added inline comments.May 4 2018, 2:45 PM

swh/vault/api/server.py
44–49	Also, i'm matching the other default configuration of that very same file. No other entry in the default configuration is using any remote service ;)

ardumont mentioned this in T1138: Migrate remaining modules to remote scheduler api use.Jul 18 2018, 11:42 AM

Amend the prior commit to fix and use the remote scheduler configuration

Herald added a reviewer: Reviewers. · View Herald TranscriptJul 18 2018, 2:50 PM

This revision now requires review to proceed.Jul 18 2018, 2:50 PM

Harbormaster completed remote builds in B1352: Diff 1229.Jul 18 2018, 2:51 PM

ardumont added inline comments.Jul 18 2018, 2:51 PM

swh/vault/api/server.py
44–49	Ok, so i had a remark about this on D372#inline-1828 that goes your way. So fixing this. Cheers,

ardumont marked 3 inline comments as done.Jul 18 2018, 2:51 PM

api/server: Use remote service for default configuration storage

Harbormaster completed remote builds in B1353: Diff 1230.Jul 18 2018, 2:55 PM

LGTM, I just have one nitpick on the default configuration that we might want to fix before merging this.

Fixed for the scheduler in the amended commit.
Fixed in the second commit for the storage part.

ardumont marked an inline comment as done.Jul 18 2018, 2:58 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 1 2018, 3:23 PM

Closed by commit rDVAUc3a07c6b73cb: Make the vault's scheduler configuration adjustable (authored by ardumont). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

debian/

4 lines

requirements-swh.txt

2 lines

swh/

vault/

api/

9 lines

9 lines

tests/

vault_testing.py

2 lines

Diff 1255

debian/control

Loading...

requirements-swh.txt

Loading...

swh/vault/api/server.py

Loading...

swh/vault/backend.py

Loading...

swh/vault/tests/vault_testing.py

Loading...