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

create guid_owncloud_infinite_scale.rst #1582

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pbielefeldt
Copy link

Copy link
Contributor

@EV21 EV21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi 👋,

as a lab contributor I would like to give you some feedback.

  • Please rename the file so it starts with guide_ (the e is missing) so the guide will be present in the preview and final deployment (link section)
    The current version with the wrong link can be found at that location 👀

  • Please just use the domain isabell.uber.space and no sub domains. The notice via .. include:: includes/web-domain-list.rst is usually enough the keep the guide short and users can simply follow the link to the manual.

  • Please use the supervisor daemon (note: I used ~/bin/ocis so the command works like this)

    Service Daemon
    --------------
    
    Create `~/etc/services.d/ocis.ini`
    
    .. code-block:: ini
    
      [program:ocis]
      environment=
        OCIS_URL="https://isabell.uber.space",
        PROXY_TLS="false",
        PROXY_HTTP_ADDR="0.0.0.0:9200",
        PROXY_LOG_LEVEL="warn"
      command=ocis server
      startsecs=30
      autostart=yes
      autorestart=yes
    
    .. include:: includes/supervisord.rst
  • Move the part with the web backend further down after the service part (after starting the service) so there shouldn't be a "NOT OK" status
    This should be enough:

    .. note:: oCIS is running on port 9200.
    
    .. include:: includes/web-backend.rst
  • I think ~/bin/ocis is a better place to put the binary file (the config path defaults to ~/.ocis/config so we don't need to worry about the working directory)

    curl https://download.owncloud.com/ocis/ocis/stable/4.0.1/ocis-4.0.1-linux-amd64 --output ~/bin/ocis
    chmod +x ~/bin/ocis
    ocis init
  • Maybe mention that all app related files and (encrypted) user files will be stored at ~/.ocis

    /home/isabell/.ocis
    |-- config
    |-- idm
    |-- idp
    |-- nats
    |-- search
    |-- storage
    `-- store

There are a few more things to do, but I leave that to the official reviewers.

@noave noave marked this pull request as draft September 21, 2023 07:32
@noave
Copy link
Contributor

noave commented Sep 21, 2023

@pbielefeldt thanks for starting the guide, do you want to take for those changes or should we take over here?

@pbielefeldt
Copy link
Author

@pbielefeldt thanks for starting the guide, do you want to take for those changes or should we take over here?

@noave Thanks for asking.
Currently, I don't have the capacity myself to push this forward.
I also noticed that with this guide, the installation will be pretty unstable; this might need more investigation anyway (and more knowledge about ownCloud InfintScale than I currently have).

Hence, if there's someone who wants to and can continue on working on this guide, that would be great!

(Also: I can't compile the repo and I have no knowledge about the markup language used here, so that's an additional issue :-) ...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants