Fix namespace logos on s3

Issue: AAH-2575

.github/workflows/ci_oci-env-integration.yml

@@ -27,6 +27,7 @@ jobs:
           - TEST_PROFILE: insights
           - TEST_PROFILE: iqe_rbac
           - TEST_PROFILE: x_repo_search
+          - TEST_PROFILE: minio
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2

CHANGES/2575.bugfix

@@ -0,0 +1 @@
+Fix bug where namespace logos would fail to download with S3 backends.

Makefile

@@ -142,6 +142,10 @@ gh-action/standalone:
 gh-action/certified-sync:
 	python3 dev/oci_env_integration/actions/certified-sync.py
 
+.PHONY: gh-action/minio
+gh-action/minio:
+	python3 dev/oci_env_integration/actions/minio.py
+
 .PHONY: docker/loaddata
 docker/loaddata:  ## Load initial data from python script
 	$(call exec_or_run, api, "/bin/bash", "-c", "/entrypoint.sh manage shell < app/dev/common/setup_test_data.py")

dev/oci_env_integration/actions/minio.py

@@ -0,0 +1,14 @@
+import action_lib
+
+env = action_lib.OCIEnvIntegrationTest(
+    envs=[
+        {
+            "env_file": "minio.compose.env",
+            "run_tests": True,
+            "db_restore": None,
+
+            # the minio client can take a long time to download
+            "wait_before_tests": 120
+        }
+    ]
+)

dev/oci_env_integration/oci_env_configs/minio.compose.env

@@ -0,0 +1,19 @@
+COMPOSE_PROFILE=galaxy_ng/base:pminio
+COMPOSE_PROJECT_NAME=ci-standalone
+
+DEV_SOURCE_PATH=galaxy_ng
+COMPOSE_BINARY=docker
+SETUP_TEST_DATA=1
+UPDATE_UI=0
+ENABLE_SIGNING=1
+
+DJANGO_SUPERUSER_USERNAME=admin
+DJANGO_SUPERUSER_PASSWORD=admin
+
+PULP_GALAXY_API_PATH_PREFIX=/api/galaxy/
+
+API_PORT=53001
+
+S3_ENDPOINT_URL=http://localhost:11651
+MINIO_CONSOLE_PORT=7812
+MINIO_PORT=11651

galaxy_ng/app/tasks/namespaces.py

@@ -55,7 +55,12 @@ def _download_avatar(url):
             print("file is not an image")
             return
 
-    return Artifact.init_and_validate(tf)
+        # the artifact has to be saved before the file is closed, or s3transfer
+        # will throw an error.
+        artifact = Artifact.init_and_validate(tf)
+        artifact.save()
+
+        return artifact
 
 
 def _create_pulp_namespace(galaxy_ns_pk, download_logo):
@@ -99,8 +104,6 @@ def _create_pulp_namespace(galaxy_ns_pk, download_logo):
     else:
         with transaction.atomic():
             metadata.save()
-            if avatar_artifact:
-                avatar_artifact.save()
             ContentArtifact.objects.create(
                 artifact=avatar_artifact,
                 content=metadata,