Wednesday, April 02, 2025
apps@conference.yunohost.org
April
Mon Tue Wed Thu Fri Sat Sun
  1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
       
             

[22:16:59] <Yunohost Git/Infra notifications> [mastodon_ynh] y​unohost-bot opened [pull request #495](https://github.com/YunoHost-Apps/mastodon_ynh/pull/495): Upgrade to v4.3.7
[22:26:27] <Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:

- 10 pending update PRs
- 12 new apps PRs
- 14 failed apps updates: appflowy, jenkins, khatru-pyramid, kiwix, languagetool, lemmy, localai, misskey, ofbiz, opencloud, phpldapadmin, pixelfedglitch, snweb, stremio

See the full log here: https://paste.yunohost.org/raw/coseqatoxo
[23:19:36] <miro5001> > <@m606:matrix.org> also is that correct usage of the redis helper to create 2 distinct databases that way ?
> ```
> # Create Redis databases
> redis_db_backend=$(ynh_redis_get_free_db)
> ynh_app_setting_set --key=redis_db_backend --value=$redis_db_backend
>
> redis_db_broker=$(ynh_redis_get_free_db)
> ynh_app_setting_set --key=redis_db_broker --value=$redis_db_broker
> ```

No it won't work. I had a similar situation with indico. I created a small function to lock the first redis_db and asked for a second one then unlocked it.
https://github.com/YunoHost-Apps/indico_ynh/blob/1a4be95bcfa0aac73252e648d3d5d692cc0c4bd9/scripts/install#L19

```
redis_db=$(ynh_redis_get_free_db)
ynh_app_setting_set --app=$app --key=redis_db --value=$redis_db
redis_lock "redis_db"

celery_db=$(ynh_redis_get_free_db)
ynh_app_setting_set --app=$app --key=celery_db --value=$celery_db
redis_unlock "redis_db"
```

https://github.com/YunoHost-Apps/indico_ynh/blob/1a4be95bcfa0aac73252e648d3d5d692cc0c4bd9/scripts/_common.sh#L134


[23:25:22] <miro5001> OK. Rereading the script, I think I will skip storing redis_db as keys. This is not useful and can lead to conflicts if another app is using the same redis_db in case of restoring a backup
[23:44:23] <Aleks (he/him/il/lui)> zmlerf cant it be like that app id or something x_x
[23:44:42] <Aleks (he/him/il/lui)> i don't get why they would use a simple int indice as db name x_x
[23:58:08] <m606> > <@miro5001:matrix.org> OK. Rereading the script, I think I will skip storing redis_db as keys. This is not useful and can lead to conflicts if another app is using the same redis_db in case of restoring a backup

what do you mean "as keys" ? that you won't `ynh_app_setting_set` it ?