Remove bst-artifact-server#1526
Conversation
gtristan
requested review from
BenjaminSchubert,
cs-shadow and
juergbi
as code owners
|
At the very least for testing we need to keep the server around. It makes sense to remove it from the list of entry points to install to make it clear that we recommend better servers for production use. However, for testing, this trivial implementation is sensible, in my opinion. |
Perhaps we should move it to the tests directory, or better yet, to |
gtristan
force-pushed
the
tristan/remove-artifact-server
branch
from
20bb5a3 to
01f89a5
Compare
Scratch that, I think it makes better sense to keep it all together with it's family in the I'll finish this up by amending documentation and explaining how to use |
gtristan
force-pushed
the
tristan/remove-artifact-server
branch
2 times, most recently
from
cd8fe50 to
0bdba89
Compare
Since we are removing the artifact server entrypoint, we will only run the service tests using buildbarn.
This fixes #1468
gtristan
force-pushed
the
tristan/remove-artifact-server
branch
from
0bdba89 to
5138b50
Compare
This patch removes the documentation about the removed `bst-artifact-server` and documents BuildStream's neutral stance as requiring any implementation of the ContentAddressableStorage and RemoteAsset protocols. A list of known implementations is given with only Buildbarn listed for now, with a small simple example and links given.
2 participants
This fixes #1468