Page MenuHomePhabricator

[services][backup] Move C++ impl to 'old' directory
ClosedPublic

Authored by bartek on Dec 8 2022, 4:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 16 2024, 5:34 PM
Unknown Object (File)
Nov 16 2024, 3:52 PM
Unknown Object (File)
Nov 16 2024, 3:35 PM
Unknown Object (File)
Nov 16 2024, 1:23 PM
Unknown Object (File)
Nov 10 2024, 10:05 PM
Unknown Object (File)
Nov 9 2024, 3:20 PM
Unknown Object (File)
Nov 9 2024, 2:19 PM
Unknown Object (File)
Nov 9 2024, 12:22 PM
Subscribers

Details

Summary

For the needs of the Rust service development, I moved the C++ implementation to the old dir.

  • Modified Dockerfile accordingly
  • Modified CMakeLists accordingly
  • Left blob_client untouched as it is convenient for me to keep it where it is.
Test Plan
  • Docker container builds
  • Service integration tests are passing

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Dec 8 2022, 4:13 AM
bartek edited the summary of this revision. (Show Details)
jon requested changes to this revision.Dec 8 2022, 12:21 PM
jon added inline comments.
services/backup/Dockerfile
26 ↗(On Diff #19246)

would like to avoid adding conditional cmake logic around our particular usage of docker, and would prefer the docker context to reflect the shape of the repository.

or you could do COPY services/backup/old/ /transferred/services/backup/old

This revision now requires changes to proceed.Dec 8 2022, 12:21 PM
services/backup/Dockerfile
26 ↗(On Diff #19246)

Yeah, me neither like this solution, however there are just two reasons I did this:

  • The same technique was used for blob service CMake (see snippet) before I ever touched it, so I just copy-pasted this solution
  • This is temporary solution, this CMake is going to be removed anyway

But I like your suggestion with COPY into the [...]/old dir in docker image, so I'll go with that.

Moved the blob_client to the old directory, reverted the CMake logic.

I decided to do this as it turned out I won't be re-using this code anyway.

This revision is now accepted and ready to land.Jan 4 2023, 9:15 AM

If we're not going to use it, should we just delete it?

If we're not going to use it, should we just delete it?

Yes, the blob_client Rust library is going to be deleted along with the C++ codebase in ENG-2403.