Consider switching to Nextcloud alpine images #31

New Issue

Open

opened 2022-11-13 00:00:53 +00:00 by 3wordchant · 5 comments

3wordchant commented 2022-11-13 00:00:53 +00:00

Owner

Copy Link

Copy Source

Quoth @javielico:

does it makes sense to use the alpine versions as well instead of the debian ones? Just purely due to the fact that they will use less resources.

& @decentral1se:

Some thoughts: that sounds like a nice idea. That might require some testing and changes in other files e.g. in the entrypoint or may have some unforseen consequences, so I would rather perhaps open up an issue to see if there is feedback on it. Several folks rely on the recipe as it is now. Opening up another PR with just those changes would be good too.

I'd be very interested in this, reducing image size and increasing performance is always nice!

Quoth @javielico: > does it makes sense to use the alpine versions as well instead of the debian ones? Just purely due to the fact that they will use less resources. & @decentral1se: > Some thoughts: that sounds like a nice idea. That might require some testing and changes in other files e.g. in the entrypoint or may have some unforseen consequences, so I would rather perhaps open up an issue to see if there is feedback on it. Several folks rely on the recipe as it is now. Opening up another PR with just those changes would be good too. I'd be very interested in this, reducing image size and increasing performance is always nice!

javielico commented 2022-11-13 00:57:51 +00:00

Owner

Copy Link

Copy Source

I'd be happy to perform some tests next week to see whether by just switching the image it will work on a dev enviroment, not sure how you folks usually perform these kind of tests?

I'd be happy to perform some tests next week to see whether by just switching the image it will work on a dev enviroment, not sure how you folks usually perform these kind of tests?

3wordchant commented 2022-11-13 04:07:52 +00:00

Author

Owner

Copy Link

Copy Source

@javielico we have some continuous integration tests (they were broken for Nextcloud, I just fixed them 😳) which test basic deployment, but the coverage isn't amazing -- we don't do any system tests, just making sure the app comes up and passes its own healthcheck. So the real answer is "a bunch of manual clicking around" -- for Nextcloud, we'd probably want to test things like occ, Nextcloud client apps, and OIDC login. Let me know if you'd like to cowork or otherwise collab on it! Thanks again for your efforts 🙏

@javielico we have [some continuous integration tests](https://build.coopcloud.tech/coop-cloud/nextcloud) (they were broken for Nextcloud, I just fixed them 😳) which test basic deployment, but the coverage isn't amazing -- we don't do any system tests, just making sure the app comes up and passes its own healthcheck. So the real answer is "a bunch of manual clicking around" -- for Nextcloud, we'd probably want to test things like `occ`, Nextcloud client apps, and OIDC login. Let me know if you'd like to cowork or otherwise collab on it! Thanks again for your efforts 🙏

javielico commented 2022-11-13 16:25:09 +00:00

Owner

Copy Link

Copy Source

Looks great, thank you for fixing the CI tests. I will test the new version with Alpine, occ and most popular client apps as well as the integration with OIDC. I'd love if we somehow we can collab or co-work on it, not sure how much time you folks have to dedicate to this per week so I don't want to distract you from something else, let me know what's the best way to move forward but also happy to crack on silently from my end until you've some time.

Looks great, thank you for fixing the CI tests. I will test the new version with Alpine, occ and most popular client apps as well as the integration with OIDC. I'd love if we somehow we can collab or co-work on it, not sure how much time you folks have to dedicate to this per week so I don't want to distract you from something else, let me know what's the best way to move forward but also happy to crack on silently from my end until you've some time.

❤️ 1

3wordchant commented 2022-11-13 17:53:32 +00:00

Author

Owner

Copy Link

Copy Source

@javielico I'll DM you on Matrix to try and finagle a link-up on this. Thanks for your thoughtfulness about time and priority; we haven't yet worked out exactly how much time we have to spend on recipe maintenance, but Nextcloud (and Keycloak, as per coop-cloud/keycloak#12 ) are definitely priorities given how many of them we're hosting.

@javielico I'll DM you on Matrix to try and finagle a link-up on this. Thanks for your thoughtfulness about time and priority; we haven't yet worked out exactly how much time we have to spend on recipe maintenance, but Nextcloud (and Keycloak, as per coop-cloud/keycloak#12 ) are definitely priorities given how many of them we're hosting.

javielico commented 2022-11-13 18:25:15 +00:00

Owner

Copy Link

Copy Source

Sounds good @3wordchant, also, given that we're benefiting from this work, I'm happy to dedicate free time to recipes maintenance and help out that way.

Sounds good @3wordchant, also, given that we're benefiting from this work, I'm happy to dedicate free time to recipes maintenance and help out that way.

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

main

renovate/docker.elastic.co-elasticsearch-elasticsearch-8.x

renovate/nextcloud-34.x

renovate/nextcloud-32.x

renovate/docker.elastic.co-elasticsearch-elasticsearch-9.x

renovate/nginx-1.x

renovate/pgautoupgrade-pgautoupgrade-18.x

upload-limit

renovate/mariadb-12.x

feature/imaginary

renovate/mariadb-11.x

kc_stable

nextcloud-v28

add-theming-v28.0.5

add-theming

update-nginx-conf

split-bbb-onlyoffice-compos

split-onlyoffice-bbb-config

authentik_sso

healthchecks

occ_cmds

auto_app_install

embed_nextcloud_in_iframe

auto_configure_sso

add-postgres-db

13.1.0+32.0.11-fpm

13.0.1+32.0.3-fpm

13.0.0+32.0.3-fpm

12.1.0+31.0.6-fpm

12.0.1+31.0.6-fpm

12.0.0+31.0.6-fpm

11.4.0+30.0.6-fpm

11.4.0+30.0.10-fpm

11.3.0+30.0.6-fpm

11.2.0+30.0.6-fpm

11.1.0+30.0.6-fpm

11.0.1+30.0.4-fpm

11.0.0+30.0.4-fpm

10.0.0+30.0.4-fpm

9.2.0+29.0.8-fpm

6.0.11+28.0.10-fpm

6.0.10+28.0.10-fpm

6.0.9+28.0.10-fpm

9.1.2+29.0.5-fpm

6.0.8+28.0.5-fpm

6.0.7+28.0.5-fpm

9.1.0+29.0.5-fpm

9.0.0+29.0.5-fpm

6.0.6+28.0.5-fpm

8.0.1+29.0.3-fpm

8.0.0+29.0.1-fpm

7.0.3+29.0.1-fpm

6.0.5+28.0.5-fpm

7.0.2+29.0.1-fpm

7.0.1+29.0.1-fpm

7.0.0+29.0.0-fpm

6.0.4+28.0.5-fpm

6.0.3+28.0.5-fpm

6.0.2+28.0.5-fpm

5.0.3+27.0.1-fpm

6.0.1+28.0.2-fpm

6.0.0+28.0.1-fpm

5.2.0+27.1.5-fpm

5.1.1+27.1.5-fpm

5.1.0+27.1.3-fpm

5.0.2+27.0.1-fpm

5.0.1+27.0.1-fpm

5.0.0+27.0.0-fpm

4.0.7+26.0.2-fpm

4.0.6+26.0.2-fpm

4.0.5+26.0.2-fpm

4.0.4+26.0.2-fpm

4.0.3+26.0.2-fpm

4.0.2+26.0.2-fpm

4.0.1+26.0.1-fpm

4.0.0+26.0.1-fpm

3.3.2+25.0.6-fpm

3.3.1+25.0.5-fpm

3.3.0+25.0.5-fpm

3.2.0+25.0.4-fpm

3.1.2+25.0.4-fpm

3.1.1+25.0.1-fpm

3.1.0+25.0.1-fpm

3.0.1+25.0.1-fpm

3.0.0+25.0.1-fpm

2.1.4+24.0.6-fpm

2.1.3+24.0.5-fpm

2.1.2+24.0.3-fpm

2.1.1+24.0.2-fpm

2.1.0+24.0.0-fpm

2.0.0+23.0.4-fpm

1.0.0+23.0.1-fpm

Labels

Clear labels

bug

Something is not working

documentation

For issues that are calling out documentation gap

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

question

More information is needed

wontfix

This won't be fixed

No Label

Milestone

No items

No Milestone

Assignees

Clear assignees

3wordchant aadil (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) Apfelwurm appletalk arjan basebuilder BornDeleuze Brooke carla cas (Cassowary) codegod100 coopcloud cyrnel decentral1se (decentral1se) dede devydave fauno (fauno) flancian Frando iexos jade (Jade Ambrose) javielico (Javielico) jjsfunhouse jmakdah2 joe-irving (Joe Irving) kawaiipunk (KawaiiPunk) knoflook kolaente lambdabundesverband linnealovespie marlon (marlon) mayel mirsal moosemower moritz nicksellen notplants oxaliq (sorrel) p4u1 pau pharaohgraphy (Andrew 🐦‍🔥❤️‍🔥✴️) PhiNatalie renovate-bot (Comrade Renovate Bot) ripclap rix rscmbbng sef (sef) simon sixsmith (Sixsmith) stevensting tobias trav val (val (he/him)) vaznasty virtualboys wolcen (Chris Thompson) wykwit xynosis yksflip

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/nextcloud#31