[07:47:05]
<florent> Hello there! (it's been a while)
[07:47:41]
<florent> For PeerTube, lots of data being backed using borg could be discarded if the admin decide so
[07:50:45]
<florent> There are videos which are encoded in different resolutions. The admin could decide whether they want either :
- backup all the resolution (so a restoration is quite simple, but the backup is larger) ;
- backup only the highest resolution (so the backup is lighter, but the restoration implies to reencode videos into their lower resolution) ;
[07:52:33]
<florent> Is there a way to somehow be clever when using ynh_backup? Like say use a callback to filter what file to actually backup?
[08:00:35]
<tituspijean[m]> I don't think it has ever been done, but you could implement a config panel to introduce a setting. Such setting would be read by the backup script and run the backup helper in a `if then else` bash statement.
[08:01:46]
<tituspijean[m]> I dunno if the backup helper accepts wildcards though, it depends on how the videos are stored (easiest: one directory per resolution; unknown: the filename changes depending on the resolution)
[08:06:11]
<florent> > <@titus:pijean.ovh> I dunno if the backup helper accepts wildcards though, it depends on how the videos are stored (easiest: one directory per resolution; unknown: the filename changes depending on the resolution)
Thanks for your answer! There is a directory per video, and underneath a video file with whose name contain the resolution.
[08:08:53]
<tituspijean[m]> If it doesn't, hum...
[08:09:28]
<tituspijean[m]> Maybe use `find` and run the backup helper for each file found?
[08:10:04]
<tituspijean[m]> I have no idea how it would work for the restore script though
[08:10:40]
<tituspijean[m]> You'd need to check if the backup helper works with a path like "$install_dir/<path_to_videos>/*/*<resolution>*.*" then 🤔
[08:11:15]
<florent> That would mean that the backup.csv file would be quite large, but I guess that would not be a real issue, right?
[08:11:44]
<tituspijean[m]> > That would mean that the backup.csv file would be quite large, but I guess that would not be a real issue, right?
Ah yeah indeed.
[08:14:15]
<florent> Or that could be another command, ynh_backup_ignore, that would add files to ignore while backuping?
[08:15:00]
<florent> Another column would be included in the backup.csv file in order to say whether to backup or not (the default value, if omitted, would be `Yes, backup/restore please`)
[08:18:57]
<tituspijean[m]> I don't think having a long csv file is an issue
[08:19:33]
<tituspijean[m]> As for the restore script a one-liner is possible with "$install_dir/<path to videos>"
[08:22:05]
<Yunohost Git/Infra notifications> App internetarchive failed all tests in job [#15581](https://ci-apps.yunohost.org/ci/job/15581) :(
[08:23:26]
<florent> Or another file `backup_ignore.txt` which would just contain files to discard while [listing files to be backed up](https://github.com/YunoHost/yunohost/blob/e59a4f849acae7c2c7a7d9bb87d7cf8a77b05798/src/backup.py#LL1885C17-L1885C23), probably a better fit, also it would not be backed up. Also probably less work.
[08:30:18]
<tituspijean[m]> Well, tweaking the backup helper would require quite some work, mainly to make sure we do not break all the other apps :p
[08:49:25]
<florent> Fair point!
[09:03:33]
<Yunohost Git/Infra notifications> App jackett stays at level 1 in job [#15583](https://ci-apps.yunohost.org/ci/job/15583)
[10:59:53]
<Yunohost Git/Infra notifications> [apps] @ericgaspar created new branch chatgpt-web
[10:59:53]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to chatgpt-web: Update apps.toml ([15227727](https://github.com/YunoHost/apps/commit/15227727e80ffc79ccaddc1155b006041846d509))
[11:00:10]
<Yunohost Git/Infra notifications> [apps] @ericgaspar opened [pull request #1675](https://github.com/YunoHost/apps/pull/1675): add chatgpt-web to catalog
[11:01:04]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to chatgpt-web: Create chatgpt-web.png ([c14f8cfa](https://github.com/YunoHost/apps/commit/c14f8cfa3c4b6f05bcd0c672b4943ceb53e6e3ae))
[11:04:43]
<Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1675#issuecomment-1546872764) on [issue #1675](https://github.com/YunoHost/apps/pull/1675) add chatgpt-web to catalog: If I recall correctly, ChatGPT is closed-source. Id add an anti-feature "relies on a proprietary platform", I don’t rem...
[11:09:08]
<Yunohost Git/Infra notifications> [apps] @ericgaspar [commented](https://github.com/YunoHost/apps/pull/1675#issuecomment-1546873705) on [issue #1675](https://github.com/YunoHost/apps/pull/1675) add chatgpt-web to catalog: correct. I am hesitating between non-free-networkand non-free-dependencies
[11:10:55]
<Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1675#issuecomment-1546874043) on [issue #1675](https://github.com/YunoHost/apps/pull/1675) add chatgpt-web to catalog: Id say network, since this app is a frontend for the closed-source backend, and ChatGPT is not actually installed on th...
[11:11:59]
<Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to chatgpt-web: Update apps.toml ([dd3ebcf5](https://github.com/YunoHost/apps/commit/dd3ebcf56f86b17e4d33441294f09448d31d5981))
[11:13:42]
<Yunohost Git/Infra notifications> [apps] @tituspijean approved [pull request #1675](https://github.com/YunoHost/apps/pull/1675#pullrequestreview-1425531201) add chatgpt-web to catalog
[11:20:54]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 4 commits to master ([c1579e5d2f4e...67105be7a494](https://github.com/YunoHost/apps/compare/c1579e5d2f4e...67105be7a494))
[11:20:55]
<Yunohost Git/Infra notifications> [apps] @tituspijean merged [pull request #1675](https://github.com/YunoHost/apps/pull/1675): add chatgpt-web to catalog
[11:20:59]
<Yunohost Git/Infra notifications> [apps/master] Create chatgpt-web.png - Éric Gaspar
[11:21:02]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[11:21:03]
<Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch chatgpt-web
[11:21:03]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1675 from YunoHost/chatgpt-web add chatgpt-web to catalog - tituspijean
[12:53:11]
<Yunohost Git/Infra notifications> App opensondage goes down from level 8 to 6 in job [#15586](https://ci-apps.yunohost.org/ci/job/15586)
[13:22:45]
<Yunohost Git/Infra notifications> App radicale failed all tests in job [#15588](https://ci-apps.yunohost.org/ci/job/15588) :(
[23:30:52]
<Yunohost Git/Infra notifications> App coin rises from level 6 to 7 in job [#15599](https://ci-apps.yunohost.org/ci/job/15599) !