Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Nextcloud App #31

Closed
Ornias1993 opened this issue Dec 28, 2020 · 18 comments
Closed

Add Nextcloud App #31

Ornias1993 opened this issue Dec 28, 2020 · 18 comments

Comments

@Ornias1993
Copy link
Member

Yes, this is already in upstream.
But Upstream is missing a lot of features we (might) want:

  • CustomDB urls
  • Redis Support
  • Seperating the different kinds of Nextcloud files (most importantly: data)
  • Ingress support

We can trickle some of those back into upstream.
But we do want to include ingress support and it's easier to fork it than to try and find a middleground on upstream. Because upstream hasn't decided on a ingress format yet and ours is mostly done.

@stavros-k
Copy link
Member

stavros-k commented Feb 10, 2021

@Ornias1993
Copy link
Member Author

There is a bug where we can't have two of the same (named) apps in different catalogs at the same time.
In this case official and truecharts.

Untill thats fixed, this is basically blocked

@stavros-k
Copy link
Member

Just curious, are you planning to use official NC docker? Or LinuxServer.io
I think official docker needs extra setup for cron or require cron jobs on the host machine.

@Ornias1993
Copy link
Member Author

@stavros-k Doesn't really mater actually... I know how to setup both :)
(what i'm actually saying is: Both docker containers are complete SHIT.)

Actually with Nextcloud 21, this gets pretty intersting with their new Rust based alternative backend. Really interesting to get that going! :)

@stavros-k
Copy link
Member

@stavros-k Doesn't really mater actually... I know how to setup both :)
(what i'm actually saying is: Both docker containers are complete SHIT.)

Hehe, i can't tell if the are SHIT, but i run into problems some times with LSIO atleast, never tried official one. Was a bit more involved to setup, and i was kinda bored :D

Actually with Nextcloud 21, this gets pretty intersting with their new Rust based alternative backend. Really interesting to get that going! :)

yea 21 looks very promising. LSIO for now don't have php8, but i thing they might do it before official.
On the other hand lsio is alpine which some packages are somewhat different and some times proves difficult to do things or things dont work as intented.
e.g. php-iconv had some problems, the correct package was on edge, so it wasn't preinstalled in docker (fixed now)
same with pdlib (still on edge repo) can only be installed with script on startup!

I might need a some quadruple backups trying to migrate from LSIO to official. :D

@Ornias1993
Copy link
Member Author

The problem with the official container (and I think even with linuxserver, but i would need to check) is their totally weird choice, to download the actual nextcloud installation instead of including it in the image and their updater more often breaks installs than it updates them if you have a lot of Apps installed.
ohh and forget figuring out your list of Apps it so nicely removed for you during update, it doesn't save the list.

Combine that with their desire to not maintain their "Enterprise" features like SAML (which is now literally broken vaporware) and Nextcloud has moved from "Enterprise Grade" to "Shilling PR stunt to proclaim other companies "Partner" for money".

Shame there is no decent alternative.

@stavros-k
Copy link
Member

You are correct with the update and apps. But i must say after NC 20 i didn't run into any problems in update (even betas) (I run 2 containers one production and one testing).
Except of course the app list ..

@Ornias1993
Copy link
Member Author

I've a special "ohh no, it's nextcloud update day"-day.

That being said, the App is comming... somewhere after 21.04 ALPHA that is :P

@Sagit-chu
Copy link
Contributor

I hope it's easy to configure HTTPS

1 similar comment
@Sagit-chu
Copy link
Contributor

I hope it's easy to configure HTTPS

@Ornias1993
Copy link
Member Author

Just like any other TrueCharts chart

@stavros-k
Copy link
Member

I was just thinking, because there is some apps on nextcloud that needs additional packages to be installed in order for them to work. And also because we move to our in house non root containers.

IF you decide to create our own container for nextcloud, have an option to load and run a script on startup so if anyone want to add extra packages in the installation can drop a script in specific folder and let it install.

Of course something like this will be out of the support.

Other option is to include ALL dependencies for the current app list, but that would result in a very much larger image than needed.

@Ornias1993
Copy link
Member Author

@stavros-k Please, this is months away of being done and is currently blocked.
I appreciate the effort, but I haven't even started working on this yet and I highly doubt that I will read this once I do start working on it. While appreciate the feedback, I find it hard to work if everyone starts saying how they want this done before I even started the design phase.

@stavros-k
Copy link
Member

@stavros-k Please, this is months away of being done and is currently blocked.
I appreciate the effort, but I haven't even started working on this yet and I highly doubt that I will read this once I do start working on it. While appreciate the feedback, I find it hard to work if everyone starts saying how they want this done before I even started the design phase.

Sure, i understand, though of throwing the idea before starting, so there won't be need to redo things just to implement something!
We 'll talk again about this when the time comes!

@Ornias1993 Ornias1993 removed their assignment Apr 6, 2021
@Ornias1993
Copy link
Member Author

Ornias1993 commented Apr 6, 2021

I'm not 100% sure if I'm going to take this on as a maintainer, as I try to focus on things I actually use and i'm not sure if I'll keep using this in the future. Hence I removed my assignment.

@Ornias1993
Copy link
Member Author

As initial design for the postgresql database solution has been submitted as a draft PR, i'm going to consider this "Unblocked".

It should now be at least possible to make a start at adding this app if anyone wants to take this on!

@Ornias1993
Copy link
Member Author

Initial work on nextcloud has been merged into the develop train for testing

@truecharts-admin
Copy link
Collaborator

This issue is locked to prevent necro-posting on closed issues. Please create a new issue or contact staff on discord of the problem persists

@truecharts truecharts locked and limited conversation to collaborators Feb 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants