Friday, December 09, 2022
apps@conference.yunohost.org
December
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
31
 
             

[08:18:21] <Guillaume Bouzige> hello all,

how does the CI auto update works for app ?
What is the trigger in the upstream repo to start yunohost ci to test for next-release and how does it automagically updates the yunohost app ?
[12:31:58] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar commented on issue #517 Testing: testme https://github.com/YunoHost-Apps/nextcloud_ynh/pull/517#issuecomment-1344131087
[12:31:58] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot commented on issue #517 Testing: Alrighty
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/4515/badge)](ht... https://github.com/YunoHost-Apps/nextcloud_ynh/pull/517#issuecomment-1344131119
[15:26:55] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to testing: 25.0.2 https://github.com/YunoHost-Apps/nextcloud_ynh/commit/c5cf91ad30149e1924c23b19e93f483c3ed3edd8
[15:26:55] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README https://github.com/YunoHost-Apps/nextcloud_ynh/commit/46bd6d27060b1f5569a640231c2fc976184e22e0
[15:29:16] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar commented on issue #517 Testing: testme https://github.com/YunoHost-Apps/nextcloud_ynh/pull/517#issuecomment-1344446160
[15:29:16] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot commented on issue #517 Testing: :rocket:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/4517/badge)](htt... https://github.com/YunoHost-Apps/nextcloud_ynh/pull/517#issuecomment-1344446215
[15:34:37] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to Update-to-PHP8.1: Auto-update README https://github.com/YunoHost-Apps/nextcloud_ynh/commit/932d2961a1531f6dd9dc37c3f68810854400f2f2
[15:34:37] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to Update-to-PHP8.1: Update to PHP8.1 https://github.com/YunoHost-Apps/nextcloud_ynh/commit/c6f8c7947c2db8b82cce018cc7c9ab7dbf8a1a1b
[15:34:37] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar created new branch Update-to-PHP8.1
[15:36:54] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar opened pull request #530: Update to php8.1 https://github.com/YunoHost-Apps/nextcloud_ynh/pull/530
[15:37:10] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar commented on issue #530 Update to php8.1: testme https://github.com/YunoHost-Apps/nextcloud_ynh/pull/530#issuecomment-1344454323
[15:37:14] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot commented on issue #530 Update to php8.1: :v:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/4518/badge)](https://... https://github.com/YunoHost-Apps/nextcloud_ynh/pull/530#issuecomment-1344454364
[16:42:31] <tituspijean> Guillaume Bouzige: I would say RTFM, but it is not documented sorry :p Its current design (example with Flarum in parentheses) is:

1. Use a GitHub Action (https://github.com/YunoHost-Apps/flarum_ynh/blob/master/.github/workflows/updater.yml) to periodically run a bash script (by default, every day)
2. In the bash script (https://github.com/YunoHost-Apps/flarum_ynh/blob/master/.github/workflows/updater.sh), we pull information on the latest releases available on the upstream app (by any means necessary, usually we call Github's API) and determine if a newer version has been released.
3. If not, we stop the script and nothing more happens. If so, the script continues and we do whatever is needed for the app (for Flarum, we update its app.src by downloading the target archive and calculating its checksum, and we update a variable in _common.sh)
4. The script ends and the Github Action takes over, commits the changes in the repo, and opens a pull request.
5. There is a tweak in our Github hooks, if a PR is opened by `github-actions[bot]` then we automagically start a CI job.
[16:44:00] <tituspijean> Depending on the apps, the required steps have to be adapted, hence the reliance on a bash script.
There's a Python version too that's pretty neat, by Salamandar : https://github.com/YunoHost/example_ynh/pull/187
[16:44:36] <tituspijean> (sorry for the ping ^^")
[16:45:56] <Guillaume Bouzige> tituspijean merci !!
[16:47:05] <tituspijean> > tituspijean merci !!

don't hesitate asking for help if you need ;)
[16:49:57] <rodinux[m]> > <@yalh:matrix.yalh.net> Nope, I wasn't very available lately, I will look at that this week-end

Ok, nice, I know my code is sometimes dirty, but I let you see when you will have time, it is related with this ticket on the source project, closed now as I found how resolve it: https://framagit.org/framasoft/mobilizon/-/issues/1211
[16:53:38] <Guillaume Bouzige> > <@titus:pijean.ovh> don't hesitate asking for help if you need ;)

since you proposed...would you mind merging the testing PR cryptpad branch ? so peoples can test it more easily. i can do another PR to solve the issue with upgrade/restore ?
[16:53:39] <tituspijean> Aleks (he/him/il/lui): Memo sorry for the mess with corepack, in yarn's documentation that's supposed to be the official way to install it, and it seemed to be working fine in my initial tests... :/
[17:04:04] <tituspijean> Guillaume Bouzige: done! https://github.com/YunoHost-Apps/cryptpad_ynh/pull/145
[17:04:12] <Guillaume Bouzige> tituspijean thanks for merging the cryptpad testing, I shall solve soon the last standing issues and hopefully make it to master branch one day
[17:05:17] <Guillaume Bouzige> ah and also, how do we add an app to the catalogue ? I could not find it in the docs...
[17:05:49] <tituspijean> https://github.com/YunoHost/apps/#how-to-add-your-app-to-the-application-catalog
[17:08:57] <Guillaume Bouzige> g8 thanks
[17:35:54] <Guillaume Bouzige> oh and also regarding cryptpad I have send an email to the declared maintainer of the app. since he is maintaining quite a few apps maybe someone know him better around here...
[17:43:24] <tituspijean> We may ping him here too. frju365 😇
[17:48:38] <Guillaume Bouzige> oh yeah of course :)
[18:10:59] <Memo> tituspijean: I just wanted to inform you that due to some private challenges I won't be online in the near future and I won't be able to maintain the Misskey repository either.

Sorry for that.
[18:13:32] <tituspijean> I thought you already mentioned that a few weeks ago. It's duely noted. :)
[18:15:40] <Memo> I can't unfortunately inform Eric and Yahl here also 😕
[18:15:40] <tituspijean> no worries, no strings attached here, and you are not listed in the repo: https://github.com/YunoHost-Apps/misskey_ynh/blob/ed53059287317bb64a6282419d902e8a1ba9f72f/manifest.json#L18
[18:27:56] <tituspijean> I petition to rename `/etc/yunohost/apps` into `/etc/yunohost/touch_me_and_kittens_will_be_sad`.
[18:37:35] <Aleks (he/him/il/lui)> ah ?
[18:38:03] <tituspijean> https://forum.yunohost.org/t/ntfy-creation-de-compte-utilisateur/22560/2
[18:38:03] <Aleks (he/him/il/lui)> somebody again modified the damn stuff manually ?
[18:38:03] <tituspijean> yup :/
[18:38:59] <tituspijean> bon je grogne, mais on devrait vraiment documenter ça
[18:38:59] <Aleks (he/him/il/lui)> djeezus christ
[18:42:27] <Aleks (he/him/il/lui)> bold of you to assume people will read the documentation
[18:44:09] <Aleks (he/him/il/lui)> en vrai y'aurais le coup de faire `chattr +i` sur ces fichiers qui serait pas mal ... genre ça met vraiment le truc en read-only et souvent les gens captent pas
[18:44:13] <Aleks (he/him/il/lui)> (enfin tu vas me dire c'est un coup à ce qu'ils essayent de brutefocer à coup de `chmod 777 -R` ...)
[18:44:13] <Aleks (he/him/il/lui)> ou bien sinon y'aurais la possibilité de genre zipper le dossier ou un truc du genre...
[19:01:38] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to modify_level: Update apps level https://github.com/YunoHost/apps/commit/9645a4478aea26e0b015639f0b01c0aaf4497a02
[19:01:38] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch modify_level
[19:02:22] <Yunohost Git/Infra notifications> [apps] @ericgaspar opened pull request #1548: Update apps level https://github.com/YunoHost/apps/pull/1548