alan-turing-institute / data-safe-haven

https://data-safe-haven.readthedocs.io
BSD 3-Clause "New" or "Revised" License
61 stars 15 forks source link

No access to VMs/workspaces for user #2195

Closed craddm closed 1 month ago

craddm commented 1 month ago

:white_check_mark: Checklist

:computer: System information

Note that this branch is up-to-date with develop and should not differ from it on any relevant code

:package: Packages

List of packages ```none Paste list of packages here ```

:no_entry_sign: Describe the problem

On a freshly deployed SRE, my user cannot see any workspaces from the guacamole dashboard. The user is in the correct group on Entra ID.

:deciduous_tree: Log messages

Relevant log messages The `guacamole_user_sync` container is reporting an out-of-memory error during update of the user database ```none INSERT 0 0 psql:/app/resources/update_users.sql:47: ERROR: out of memory INSERT 0 0 INSERT 0 0 INSERT 0 0 2024-09-23T15:54:46+00:00 Finished database synchronisation Waiting 180 seconds before re-attempting synchronisation ```

:recycle: To reproduce

JimMadge commented 1 month ago

Possibly related problem,

Guacamole interface

Screenshot 2024-09-27 at 09 49 33
jemrobinson commented 1 month ago

@craddm: Can you check whether using guacamole-user-sync:main fixes this problem? If it does, I'll tag it and make a PR to DSH.

JimMadge commented 1 month ago

@jemrobinson What changes were made in that container?

I'm deploying with a larger memory allocation to guacamole-user-sync to see if that makes a difference 🤞.

jemrobinson commented 1 month ago

What changes were made in that container?

craddm commented 1 month ago

testing now

JimMadge commented 1 month ago

Increasing the memory available to the guacamole-user-sync container to 1Gi still gives the OOM error. No workspaces appear.

craddm commented 1 month ago

Not working at all: image

JimMadge commented 1 month ago

I'm getting the same, lots of restarts. Unfortunately you can't see the logs 😶.

jemrobinson commented 1 month ago

ldap library isn't properly included in the current Docker image.

JimMadge commented 1 month ago

Something @craddm or I can work on in https://github.com/alan-turing-institute/guacamole-user-sync ?

jemrobinson commented 1 month ago

Here's the issue https://github.com/alan-turing-institute/guacamole-user-sync/issues/17