Wednesday, January 24, 2024
apps@conference.yunohost.org
January
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
       
             

[00:21:59] <Yunohost Git/Infra notifications> App guacamole failed all tests in job [#22497](https://ci-apps.yunohost.org/ci/job/22497) :(
[02:07:26] <Yunohost Git/Infra notifications> App redmine failed all tests in job [#22501](https://ci-apps.yunohost.org/ci/job/22501) :(
[05:08:27] <Yunohost Git/Infra notifications> App qbittorrent rises from level 7 to 8 in job [#22505](https://ci-apps.yunohost.org/ci/job/22505) !
[06:01:33] <Yunohost Git/Infra notifications> Job [#22496](https://ci-apps.yunohost.org/ci/job/22496) for framaforms failed miserably :(
[06:52:56] <mrflos> > <@Alekswag:matrix.org> anyway seems to be running just fine

wahouw, i didn't know this feature exists, is this real? i want it for yeswiki #liveDangerously
[10:59:05] <Yunohost Git/Infra notifications> App question2answer stays at level 2 in job [#22512](https://ci-apps.yunohost.org/ci/job/22512)
[12:15:08] <Aleks (he/him/il/lui)> > <@mrfloss:matrix.org> wahouw, i didn't know this feature exists, is this real? i want it for yeswiki #liveDangerously

it's already there: https://github.com/YunoHost-Apps/yeswiki_ynh/blob/master/manifest.toml#L65
[12:16:54] <Aleks (he/him/il/lui)> NB: it means "create a PR when there's a new tag automatically", not "existing install will autoudate when there's a new release" 😬
[12:19:18] <mrflos> > <@Alekswag:matrix.org> NB: it means "create a PR when there's a new tag automatically", not "existing install will autoudate when there's a new release" 😬

ok, i misunderstood this.. it's still a good feature but i was imagining something like the browser's background upgrades, but surely it's a bit risky in yunohost context
[12:20:06] <Aleks (he/him/il/lui)> well uh yeah things need to be tested at least on the CI otherwise there may be regressions etc
[12:21:40] <mrflos> sure.. and CI may have blind spots...
In any case, it's nice to see the ecosystem getting so mature
[12:37:49] <Yunohost Git/Infra notifications> App zap failed all tests in job [#22515](https://ci-apps.yunohost.org/ci/job/22515) :(
[12:38:03] <Salamandar> oh lol
[12:38:06] <Salamandar> my bad
[12:38:27] <Salamandar> Thanks, CI, for giving me the correct shasums
[12:39:46] <Aleks (he/him/il/lui)> xD
[12:40:55] <Salamandar> fixed
[13:04:55] <Yunohost Git/Infra notifications> App nodered rises from level 6 to 8 in job [#22514](https://ci-apps.yunohost.org/ci/job/22514) !
[13:11:51] <Yunohost Git/Infra notifications> [nextcloud_ynh] @supermamie opened [issue #654](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/654): Add a link to incompatible apps on big upgrades
[15:13:49] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1973](https://github.com/YunoHost/apps/pull/1973): Add Osclass to wishlist
[15:13:49] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to add-to-wishlist-osclass: Add Osclass to wishlist ([cf1dc1d8](https://github.com/YunoHost/apps/commit/cf1dc1d8a698dbe77c35e97a677b5a951a66b5cb))
[15:33:01] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/654#issuecomment-1908371079) on [issue #654](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/654) Add a link to incompatible apps on big upgrades: cf https://github.com/YunoHost-Apps/nextcloud_ynh/pull/647 which is "almost ready", just need to debug one weird failing...
[18:38:33] <Aleks (he/him/il/lui)> https://ci-apps-dev.yunohost.org/ci/job/12952 -> upgade failed, Info: This log is now available via https://paste.yunohost.org/raw/uyigecetox
[18:38:33] <Aleks (he/him/il/lui)> ooooh i realize https://github.com/YunoHost/package_check/pull/144 is merged and in prod and it wurks :o
[18:38:34] <Aleks (he/him/il/lui)> Failure was successful ✅
[18:38:35] <Aleks (he/him/il/lui)> thiswasatriumph.mp3
[18:38:36] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin pushed 12 commits to check-app-compatibility-before-upgrade ([d7e05b002216...eed5f66f01f9](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d7e05b002216...eed5f66f01f9))
[18:38:36] <Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/647#issuecomment-1908657302) on [issue #647](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/647) Upgrade script: check that installed apps are compatible with the future version before actually starting the upgrade: testme
[18:38:37] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/647#issuecomment-1908657385) on [issue #647](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/647) Upgrade script: check that installed apps are compatible with the future version before actually starting the upgrade: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/12952/ba...
[18:38:37] <Yunohost Git/Infra notifications> [nextcloud_ynh/check-app-compatibility-before-upgrade] Merge branch testing into filter-boring-occ-warnings - Alexandre Aubin
[18:38:38] <Yunohost Git/Infra notifications> [apps] @orhtej2 opened [pull request #1974](https://github.com/YunoHost/apps/pull/1974): Support for GitLab upstream repos part 2.
[19:10:34] <kayou> That sounds nice, but seems to be redundant with the "traceback" on error and the link at the end of the tests
The full log is available at https://ci-apps-dev.yunohost.org/ci/logs/12951.log

[19:10:34] <kayou> Yeah, I see
[19:10:34] <kayou> That sounds nice, but seems to be redundant with the "traceback" on error and the link at the end of the tests

> The full log is available at https://ci-apps-dev.yunohost.org/ci/logs/12951.log
[19:10:35] <kayou> That sounds nice, but seems to be redundant with the "traceback" and the link at the end of the tests

> The full log is available at https://ci-apps-dev.yunohost.org/ci/logs/12951.log
[19:10:36] <Aleks (he/him/il/lui)> yeah but sometimes the traceback is not enough, and the full log is "way too much" (it's not filtered to remove useless bash debug lines) and is only available at the very end of the test
[19:11:22] <orhtej2> Github not working for anyone else?
[19:13:24] <orhtej2> https://media.giphy.com/media/yx400dIdkwWdsCgWYp/giphy.gif
[19:21:19] <Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1974](https://github.com/YunoHost/apps/pull/1974): Support for GitLab upstream repos part 2.
[19:21:19] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([0fe7c0d743a6...fd5469dd11a4](https://github.com/YunoHost/apps/compare/0fe7c0d743a6...fd5469dd11a4))
[19:21:19] <Aleks (he/him/il/lui)> works for me i think
[19:21:27] <Yunohost Git/Infra notifications> [apps/master] Support for GitLab upstream repos part 2. - orhtej2
[19:53:29] *tituspijean manually installs an app in a python venv and runs it: ✅
[19:54:07] *tituspijean takes the same commands into a ynh package and tries to install it: 💥
[19:56:51] <tituspijean> If anyone knows anything about Flask...:
```
Warning: Failed to create app
Warning: Traceback (most recent call last):
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/app.py", line 40, in create_app
Warning: app_initializer.init_app()
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/initialization/__init__.py", line 512, in init_app
Warning: self.init_app_in_ctx()
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/initialization/__init__.py", line 438, in init_app_in_ctx
Warning: self.configure_fab()
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/initialization/__init__.py", line 562, in configure_fab
Warning: appbuilder.init_app(self.superset_app, db.session)
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 218, in init_app
Warning: self._add_admin_views()
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 325, in _add_admin_views
Warning: self.sm.register_views()
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/security/sqla/manager.py", line 95, in register_views
Warning: super(SecurityManager, self).register_views()
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/security/manager.py", line 743, in register_views
Warning: self.user_view = self.appbuilder.add_view(
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 440, in add_view
Warning: baseview = self._check_and_init(baseview)
Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 355, in _check_and_init
Warning: if getattr(baseview, "datamodel").session is None:
Warning: AttributeError: 'NoneType' object has no attribute 'session'
```
The env has a FLASK_APP variable
[20:00:41] <lapineige> > <@titus:pijean.ovh> If anyone knows anything about Flask...:
> ```
> Warning: Failed to create app
> Warning: Traceback (most recent call last):
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/app.py", line 40, in create_app
> Warning: app_initializer.init_app()
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/initialization/__init__.py", line 512, in init_app
> Warning: self.init_app_in_ctx()
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/initialization/__init__.py", line 438, in init_app_in_ctx
> Warning: self.configure_fab()
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/superset/initialization/__init__.py", line 562, in configure_fab
> Warning: appbuilder.init_app(self.superset_app, db.session)
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 218, in init_app
> Warning: self._add_admin_views()
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 325, in _add_admin_views
> Warning: self.sm.register_views()
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/security/sqla/manager.py", line 95, in register_views
> Warning: super(SecurityManager, self).register_views()
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/security/manager.py", line 743, in register_views
> Warning: self.user_view = self.appbuilder.add_view(
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 440, in add_view
> Warning: baseview = self._check_and_init(baseview)
> Warning: File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 355, in _check_and_init
> Warning: if getattr(baseview, "datamodel").session is None:
> Warning: AttributeError: 'NoneType' object has no attribute 'session'
> ```
> The env has a FLASK_APP variable

Look at the app V2 PR ?
[20:00:41] <tituspijean> and the code: https://github.com/YunoHost-Apps/superset\_ynh/tree/testing
[20:00:45] <tituspijean> Which one?
[20:07:30] <tituspijean> thing is, Superset depends on Flask, but all I run is the `superset` command and set a settings file. But the error above is displayed in any case whenever I call the command. I don't see any specifics in flask_ynh that would apply here. 😕
[20:15:21] <tituspijean> that's what I'm doing now, what should I try?
[20:15:22] <lapineige> Flask ynh
[20:15:23] <tituspijean> `yunohost app shell` ftw
[20:15:23] <Aleks (he/him/il/lui)> could be an issue like "it doesn't find the stuff it's supposed to build" idk
[20:15:24] <Aleks (he/him/il/lui)> if you are able to reproduce the issue locally, can't you install the app with --no-remove-on-failure and enter the venv ?
[20:15:24] <Aleks (he/him/il/lui)> yeah the issue is related specifically to flask_appbuilder
[20:15:24] <Aleks (he/him/il/lui)> "baseview" (whatever that is) doesn't have a "basemodel"
[20:15:25] <Aleks (he/him/il/lui)> but i don't know enough about this stuff
[20:22:32] <tituspijean> ok I'll try thanks !
[20:22:36] <Aleks (he/him/il/lui)> re-running the same command that crashed i guess, and then uuuh idk personalyl i would open `File "/var/www/superset/venv/lib/python3.9/site-packages/flask_appbuilder/base.py", line 355` and add an `import pdb; pdb.set_trace()` before that line to enter an interactive python shell after retriggerign the issue, and trying to understand what `baseview` is and why it doesn't have a "datamodel" but ugh that assumes you're comfortable with pdb 😬
[20:22:42] <tituspijean> (dinner first 😅)
[20:22:54] <tituspijean> and Pdb doc reading second
[22:52:25] <Salamandar> yeah I admit that for some apps I did not go through testing… mostly because I would've just ran another CI that would have given the exactly same results :D
[22:52:25] <Salamandar> :D
[22:52:25] <lapineige> > <@Salamandar:matrix.org> yeah I admit that for some apps I did not go through testing… mostly because I would've just ran another CI that would have given the exactly same results :D

Just merge without launching CI ?
[22:52:25] <Aleks (he/him/il/lui)> but that doesn't guarantee that the autoupdate script find the appropriate version though, maybe there *is* an issue with version parsing
[22:52:25] <Émy - OniriCorpe> do you know how to convert a sqlite DB to a postgresql one?
[22:52:28] <Aleks (he/him/il/lui)> howdareyou.jpg
[22:52:38] <Émy - OniriCorpe> > <@oniricorpe:im.emelyne.eu> do you know how to convert a sqlite DB to a postgresql one?

for https://github.com/YunoHost-Apps/pufferpanel_ynh/pull/13
[23:21:34] <orhtej2> > <@oniricorpe:im.emelyne.eu> do you know how to convert a sqlite DB to a postgresql one?

If schema is compatible just dump it to sql and run that on pgsql one? Syntax for inserts is ANSI
[23:23:07] <Émy - OniriCorpe> i don't know, i'm really not used to databases