coop-cloud/nextcloud
coop-cloud
/
nextcloud
5
1
Fork 3
Code Issues 6 Pull Requests Releases Wiki Activity

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

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
Member
Copy Link

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

@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
Member
Copy Link

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

@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
Member
Copy Link

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
authentik_sso
healthchecks
occ_cmds
auto_app_install
embed_nextcloud_in_iframe
auto_configure_sso
add-postgres-db
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
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/nextcloud#31
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?