Merge ~ines-almeida/txpkgupload:fix-txpkgupload-service-charm into txpkgupload:master

[Ines Almeida]

Create temp directory in reactive code and update ListenStream socket to use ftp port

Update the README to ensure principal charm is setup correctly and update default value for sftp_authentication_endpoint

[Colin Watson (cjwatson)]

review: Approve

f2aece3... by Ines Almeida on 2023-07-28

Update ListenStream socket port and sftp_authentication_endpoint default value

3dd85f1... by Ines Almeida on 2023-07-28

Create temp directory where files are placed before being sent to fsroot

Update README with details about the txpkgupload relation with the principal charm

diff --git a/charm/txpkgupload/README.md b/charm/txpkgupload/README.md
index 962ff38..8d9b491 100644
--- a/charm/txpkgupload/README.md
+++ b/charm/txpkgupload/README.md
@@ -15,9 +15,20 @@ provides:
 ```
 
 As this is a subordinate charm, it will be deployed to the same machine as its
-principal charm. The charm will not deploy before the relation to its principal is set.
-You will need the following relations:
+principal charm. The charm will not deploy before the relation to its principal
+is set. You will need the following relations:
 
 ```shell
 juju relate txpkgupload <name of principal application>
 ```
+
+The principal charm should create an `fsroot` directory and set up txpkgupload
+with something along the lines of:
+
+```python
+    txpkgupload = endpoint_from_flag("upload-queue-processor.available")
+    txpkgupload.set_config(fsroot=fsroot)
+```
+
+Note: Ensure that the `txpkgupload` user has `write` and `execute` access to
+the `fsroot` directory.
diff --git a/charm/txpkgupload/config.yaml b/charm/txpkgupload/config.yaml
index 8257478..34d4410 100644
--- a/charm/txpkgupload/config.yaml
+++ b/charm/txpkgupload/config.yaml
@@ -9,7 +9,7 @@ options:
     description: The prefix used when generating OOPS reports.
   sftp_authentication_endpoint:
     type: string
-    default: http://xmlrpc-private.launchpad.dev:8087/authserver
+    default: http://xmlrpc-private.launchpad.test:8087/authserver
     description: The URL of the XML-RPC endpoint for authentication handling of SSH users.
   sftp_host_key_private:
     type: string
diff --git a/charm/txpkgupload/reactive/txpkgupload.py b/charm/txpkgupload/reactive/txpkgupload.py
index 39d224c..2ec1e90 100644
--- a/charm/txpkgupload/reactive/txpkgupload.py
+++ b/charm/txpkgupload/reactive/txpkgupload.py
@@ -84,7 +84,9 @@ def get_config():
     )
 
     oopses_dir = os.path.join(base.base_dir(), "oopses")
-    host.mkdir(oopses_dir, group=base.user(), perms=0o775)
+    host.mkdir(oopses_dir, owner=base.user(), group=base.user(), perms=0o775)
+    temp_dir = os.path.join(base.base_dir(), "tmp-incoming")
+    host.mkdir(temp_dir, owner=base.user(), group=base.user(), perms=0o775)
     config["oopses_dir"] = oopses_dir
     # Fetch data received from the parent upload queue processor service
     uploader_relation = endpoint_from_flag("upload-queue-processor.configured")
diff --git a/charm/txpkgupload/templates/txpkgupload.socket.j2 b/charm/txpkgupload/templates/txpkgupload.socket.j2
index cd54306..2bdfdf9 100644
--- a/charm/txpkgupload/templates/txpkgupload.socket.j2
+++ b/charm/txpkgupload/templates/txpkgupload.socket.j2
@@ -3,6 +3,7 @@ Description=Twisted package uploader service
 
 [Socket]
 ListenStream={{ sftp_port }}
+ListenStream={{ ftp_port }}
 
 [Install]
 WantedBy=sockets.target