Page MenuHomePhabricator
Differential D8962

[backup] Update `BlobItem`
ClosedPublic
Actions

Authored by michal on Aug 28 2023, 12:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 12, 6:08 PM
Unknown Object (File)
Wed, Apr 2, 4:22 AM
Unknown Object (File)
Mon, Mar 24, 2:14 PM
Unknown Object (File)
Mon, Mar 24, 2:14 PM
Unknown Object (File)
Mon, Mar 24, 2:14 PM
Unknown Object (File)
Sun, Mar 23, 1:41 PM
Unknown Object (File)
Mar 16 2025, 11:58 PM
Unknown Object (File)
Mar 7 2025, 2:30 PM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
• patryk
kamil
• jon
Commits
rCOMM3c1ce0afaf0f: [backup] Update `BlobItem`
Summary

Modernize BlobItem and update it to match new database design.

Depends on D8954

Test Plan

cargo check, tested with the rest of the stack

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

michal created this revision.Aug 28 2023, 12:57 AM
Herald added a reviewer: • jon. · View Herald TranscriptAug 28 2023, 12:57 AM
Herald added subscribers: tomek, ashoat. · View Herald Transcript
Harbormaster completed remote builds in B22154: Diff 30419.Aug 28 2023, 1:14 AM
michal requested review of this revision.Aug 28 2023, 1:14 AM
michal added a child revision: D8963: [backup] Introduce upload endpoint.Aug 28 2023, 1:33 AM
bartek accepted this revision.Aug 28 2023, 5:00 AM
bartek added inline comments.
services/backup/src/constants.rs
17–24 ↗(On Diff #30419)

Nit/followup: You're right that it would be nice to add a submodule for table constants: constants::backup_table::FIELD_XYZ

This revision is now accepted and ready to land.Aug 28 2023, 5:00 AM
michal added inline comments.Aug 29 2023, 10:55 AM
services/backup/src/constants.rs
17–24 ↗(On Diff #30419)

Probably gonna do a separate diff with both backup and log constants

Closed by commit rCOMM3c1ce0afaf0f: [backup] Update `BlobItem` (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhyAug 30 2023, 1:28 AM
This revision was automatically updated to reflect the committed changes.
Michal Gniadek <michal.gniadek@swmansion.com> added a commit: rCOMM3c1ce0afaf0f: [backup] Update `BlobItem`.

Revision Contents
Changeset List

PathSize
services/
backup/
src/
4 lines
database/
147 lines
37 lines
comm-services-lib/
src/
22 lines

Diff 30529

View Options

services/backup/src/constants.rs

FROM commapp/services-base:1.1 FROM commapp/services-base:1.1
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y uuid-dev && \ apt-get install -y uuid-dev && \
rm -rf /var/lib/apt/lists/* rm -rf /var/lib/apt/lists/*
ARG COMM_TEST_SERVICES ARG COMM_TEST_SERVICES
ARG COMM_SERVICES_DEV_MODE ARG COMM_SERVICES_DEV_MODE
ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES} ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES}
ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE} ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE}
WORKDIR /transferred WORKDIR /transferred
COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/ COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/
COPY services/backup/docker/ scripts/
COPY services/lib/ . COPY services/lib/ .
COPY services/backup/ . COPY services/backup/ .
RUN scripts/build_server.sh RUN scripts/build_server.sh
CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi
View Options

services/backup/src/database/backup_item.rs

FROM commapp/services-base:1.1 FROM commapp/services-base:1.1
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y uuid-dev && \ apt-get install -y uuid-dev && \
rm -rf /var/lib/apt/lists/* rm -rf /var/lib/apt/lists/*
ARG COMM_TEST_SERVICES ARG COMM_TEST_SERVICES
ARG COMM_SERVICES_DEV_MODE ARG COMM_SERVICES_DEV_MODE
ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES} ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES}
ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE} ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE}
WORKDIR /transferred WORKDIR /transferred
COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/ COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/
COPY services/backup/docker/ scripts/
COPY services/lib/ . COPY services/lib/ .
COPY services/backup/ . COPY services/backup/ .
RUN scripts/build_server.sh RUN scripts/build_server.sh
CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi
View Options

services/backup/src/database/mod.rs

FROM commapp/services-base:1.1 FROM commapp/services-base:1.1
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y uuid-dev && \ apt-get install -y uuid-dev && \
rm -rf /var/lib/apt/lists/* rm -rf /var/lib/apt/lists/*
ARG COMM_TEST_SERVICES ARG COMM_TEST_SERVICES
ARG COMM_SERVICES_DEV_MODE ARG COMM_SERVICES_DEV_MODE
ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES} ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES}
ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE} ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE}
WORKDIR /transferred WORKDIR /transferred
COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/ COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/
COPY services/backup/docker/ scripts/
COPY services/lib/ . COPY services/lib/ .
COPY services/backup/ . COPY services/backup/ .
RUN scripts/build_server.sh RUN scripts/build_server.sh
CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi
View Options

services/comm-services-lib/src/database.rs

FROM commapp/services-base:1.1 FROM commapp/services-base:1.1
RUN apt-get update && \ RUN apt-get update && \
apt-get install -y uuid-dev && \ apt-get install -y uuid-dev && \
rm -rf /var/lib/apt/lists/* rm -rf /var/lib/apt/lists/*
ARG COMM_TEST_SERVICES ARG COMM_TEST_SERVICES
ARG COMM_SERVICES_DEV_MODE ARG COMM_SERVICES_DEV_MODE
ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES} ENV COMM_TEST_SERVICES=${COMM_TEST_SERVICES}
ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE} ENV COMM_SERVICES_DEV_MODE=${COMM_SERVICES_DEV_MODE}
WORKDIR /transferred WORKDIR /transferred
COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/ COPY native/cpp/CommonCpp/grpc/protos/backup.proto native/cpp/CommonCpp/grpc/protos/blob.proto protos/
COPY services/backup/docker/ scripts/
COPY services/lib/ . COPY services/lib/ .
COPY services/backup/ . COPY services/backup/ .
RUN scripts/build_server.sh RUN scripts/build_server.sh
CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi CMD if [ "$COMM_TEST_SERVICES" -eq 1 ]; then scripts/run_tests.sh; else scripts/run_server.sh; fi