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

Nextcloud: upgrade to 14.0.0 #5578

Closed
alepolidori opened this issue Sep 6, 2018 · 19 comments
Closed

Nextcloud: upgrade to 14.0.0 #5578

alepolidori opened this issue Sep 6, 2018 · 19 comments
Labels
verified All test cases were verified successfully

Comments

@alepolidori
Copy link

alepolidori commented Sep 6, 2018

Changes

Upgrade to version 14.0.0

Description

Todo based on warning message after upgrade from 13.0.6

screenshot from 2018-09-06 15-45-34

Components

nethserver-nextcloud
nextcloud

See also
Changelog

@alepolidori
Copy link
Author

alepolidori commented Sep 10, 2018

  • add missing db indexes when upgrade
  • add http header
  • fix server mail configuration

alepolidori added a commit to alepolidori/nethserver-nextcloud that referenced this issue Sep 12, 2018
alepolidori added a commit to alepolidori/nethserver-nextcloud that referenced this issue Sep 12, 2018
alepolidori added a commit to alepolidori/nethserver-nextcloud that referenced this issue Sep 12, 2018
alepolidori added a commit to alepolidori/nethserver-nextcloud that referenced this issue Sep 12, 2018
@alepolidori
Copy link
Author

After installed nethserver-nextcloud and nextcloud this is the "Overview" page:

image

gsanchietti added a commit to NethServer/nethserver-nextcloud that referenced this issue Sep 12, 2018
@gsanchietti
Copy link
Member

gsanchietti commented Sep 12, 2018

In nethserver-testing:

  • nextcloud-14.0.0-1.el7.noarch.rpm

Upstream release: https://github.com/nextcloud/server-packages/releases/tag/centos-14.0.0

@nethbot
Copy link
Member

nethbot commented Sep 12, 2018

in 7.5.1804/testing:

@gsanchietti
Copy link
Member

Test case 1

  • Install on a clean machine with OpenLDAP
  • Check everything works as expected

Test case 2

  • Install on a clean machine with AD
  • Check everything works as expected

Test case 3

  • Update and existing machine
  • Check everything works as expected

@gsanchietti gsanchietti added the testing Packages are available from testing repositories label Sep 12, 2018
@DavidePrincipi
Copy link
Member

DavidePrincipi commented Sep 13, 2018

Test case 3 - VERIFIED

Seems Ok, but httpd restart is really slow (due to my reverse proxy configuration). Can we just reload it?

https://github.com/NethServer/nethserver-nextcloud/blob/71729a9dcd2cec4c34af98f6dc5afcaf64dba540/createlinks#L42

@alepolidori
Copy link
Author

Test case 3

Seems Ok, but httpd restart is really slow (due to my reverse proxy configuration). Can we just reload it?

https://github.com/NethServer/nethserver-nextcloud/blob/71729a9dcd2cec4c34af98f6dc5afcaf64dba540/createlinks#L42

I think it can be done, but this will not reload the unit configuration file of systemd, if it will be changed in the future. Probably, NethServer has some automatic mechanism to manage this changes, is that it ?

@DavidePrincipi
Copy link
Member

It seems httpd unit configuration is not altered by any package here. If that happens the package which alters it is responsible for calling systemctl daemon-reload accordingly.

markVnl pushed a commit to markVnl/nethserver-nextcloud that referenced this issue Sep 13, 2018
@dz00te
Copy link
Member

dz00te commented Sep 13, 2018

Test Case 3 - ok on a working install with openldap

i only see some error before nethserver-nextcloud-update, after that all ok

esmith::event[32736]: The current PHP memory limit is below the recommended value of 512MB.
su: (to apache) root on none
....
esmith::event[32736]: Action: /etc/e-smith/events/nethserver-nextcloud-update/S30nethserver-nextcloud-occ-conf SUCCESS [22.828166]
systemd: Reloading.

@mrmarkuz
Copy link
Member

Test Case 2 - fresh install with AD - ok
...except of the PHP memory limit warning

@gsanchietti
Copy link
Member

I think we can release for the restart action for now, but we should explore if the reload is enough.

@alepolidori please increase memory limit during nethserver-nextcloud-occ-config action. If necessary, increase the memory limit for the whole app.

@nethbot
Copy link
Member

nethbot commented Sep 17, 2018

in 7.5.1804/testing:

1 similar comment
@nethbot
Copy link
Member

nethbot commented Sep 17, 2018

in 7.5.1804/testing:

@dz00te
Copy link
Member

dz00te commented Sep 17, 2018

installed new package on ldap+ previous 14 test
installed on a clean nextcloud13
in both cases no more warning in log about memory limit

@markVnl
Copy link
Member

markVnl commented Sep 17, 2018

an update and a new install on (x86_64) system with local AD worked as expected and did not throw warnings about memory limit.

alepolidori added a commit to alepolidori/nethserver-nextcloud that referenced this issue Sep 24, 2018
alepolidori added a commit to alepolidori/nethserver-nextcloud that referenced this issue Sep 24, 2018
gsanchietti added a commit to NethServer/nethserver-nextcloud that referenced this issue Sep 24, 2018
Change 'smtp', 'logfile' and admin email configurations

NethServer/dev#5578
@nethbot
Copy link
Member

nethbot commented Sep 24, 2018

in 7.5.1804/testing:

@NethServer NethServer deleted a comment from nethbot Sep 24, 2018
@gsanchietti gsanchietti removed the testing Packages are available from testing repositories label Sep 24, 2018
@gsanchietti
Copy link
Member

On a machine without mail server installed, mail configuration is invalid and raises an error when trying to share a file with another user.

gsanchietti added a commit to NethServer/nethserver-nextcloud that referenced this issue Sep 24, 2018
Revert "Change 'smtp', 'logfile' and admin email configurations"

NethServer/dev#5578
@nethbot
Copy link
Member

nethbot commented Sep 24, 2018

in 7.5.1804/testing:

@NethServer NethServer deleted a comment from nethbot Sep 24, 2018
@gsanchietti gsanchietti added the testing Packages are available from testing repositories label Sep 24, 2018
@alepolidori alepolidori added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Sep 25, 2018
@nethbot
Copy link
Member

nethbot commented Sep 25, 2018

in 7.5.1804/updates:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified All test cases were verified successfully
Projects
None yet
Development

No branches or pull requests

7 participants