Skip to content

Commit

Permalink
fix Dockerfile
Browse files Browse the repository at this point in the history
  • Loading branch information
ksugden committed Nov 22, 2024
1 parent 59b9c31 commit 0d9bfbe
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
2 changes: 1 addition & 1 deletion images/tools/database-copy/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM public.ecr.aws/docker/library/postgres:16

RUN apt-get update && apt upgrade
RUN apt-get update && apt upgrade -y
RUN apt-get install -y curl zip jq
RUN curl "https://awscli.amazonaws.com/awscli-exe-linux-aarch64.zip" -o "awscliv2.zip"
RUN unzip awscliv2.zip
Expand Down
8 changes: 3 additions & 5 deletions integration-tests/integration_tests/test_database_copy.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,9 +69,7 @@ def test_can_speak_to_s3(s3_client):
def test_data_is_dumped_into_s3(s3_client):
response = s3_client.list_objects_v2(Bucket="test-dump-bucket")
object = s3_client.get_object(Bucket="test-dump-bucket", Key="data_dump.sql")
print(f"FILE CONTENTS: {object["Body"].read().decode("utf-8", errors='ignore')}")
# assert response["Contents"][0]["Key"] == "data_dump.sql"
assert False

# Trying to set up the source DB with a table so that there's something to migrate
assert response["Contents"][0]["Key"] == "data_dump.sql"


# Trying to set up the source DB with a table so that there's something to migrate

0 comments on commit 0d9bfbe

Please sign in to comment.