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

Server Preparation for Ubuntu 22.04 #369

Merged
merged 1 commit into from
Jun 2, 2022
Merged

Server Preparation for Ubuntu 22.04 #369

merged 1 commit into from
Jun 2, 2022

Conversation

pako81
Copy link

@pako81 pako81 commented May 19, 2022

WHAT Needs to be Documented?

  • Add Server Preparation for Ubuntu 22.04

WHERE Does This Need To Be Documented (Link)?

WHY Should This Change Be Made?

Ubuntu 22.04 is now officially supported

(Optional) What Type Of Content Change Is This?

  • New Content Addition
  • Old Content Deprecation
  • Existing Content Simplification
  • Bug Fix to Existing Content

(Optional) Which Manual Does This Relate To?

  • Admin Manual
  • Developer Manual
  • User Manual
  • Android
  • iOS
  • Branded Clients
  • Desktop Client
  • Other

Sorry, something went wrong.

@mmattel
Copy link
Contributor

mmattel commented May 20, 2022

Some things are needed to embed (link) this page properly. I will take this branch and do this. The rest is looking good to me, approving when the embedding is done 👍

mmattel
mmattel previously approved these changes May 20, 2022
@mmattel
Copy link
Contributor

mmattel commented May 20, 2022

Backport to 10.10 and 10.9

@mmattel mmattel dismissed their stale review May 20, 2022 11:54

an issue raised

@mmattel
Copy link
Contributor

mmattel commented May 20, 2022

There is an open issue to be solved first:
owncloud/core#40071 ([QA] Encryption cannot be enabled on Ubuntu 22.04)

We need to add content because of MariaDB 10.6 in the DB section, see core #39286

We need to add content to the maintenance upgrade section in the documentation as this effects a lot of topics especially when uograding from U20.04 to U22.04 !!

@mmattel
Copy link
Contributor

mmattel commented May 21, 2022

Because this is getting a bigger task than expected, I have cherry-picked all relevant PR into this one so we only have to maintain this PR only for the U22.04/PHP7.4 stuff.

@mmattel mmattel mentioned this pull request May 21, 2022
12 tasks
Copy link
Contributor

@phil-davis phil-davis left a comment

Choose a reason for hiding this comment

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

Looks good - only a few minor things.

Copy link
Contributor

@jnweiger jnweiger left a comment

Choose a reason for hiding this comment

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

@pako81 did php7.4-phpseclib work for you?

php7.4-intl php7.4-json php7.4-mbstring \
php7.4-mysql php7.4-ssh2 php7.4-xml \
php7.4-zip php7.4-apcu php7.4-redis \
php7.4-ldap php7.4-opcache php7.4-phpseclib\
Copy link
Contributor

Choose a reason for hiding this comment

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

On Ubuntu 22.04 with Ondrej repos I cannot find php7.4-phpseclib

Should that be php-phpseclib instead? That exists, but is probably meant for php8, not sure if that also works with 7.4 ...

Copy link
Author

Choose a reason for hiding this comment

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

@jnweiger You are right, php7.4-phpseclib does not exist on Ondrej repo, I may have done an error while documenting. Not sure if we want to document at this point php-phpseclib as required package

Copy link
Contributor

Choose a reason for hiding this comment

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

php-phpseclib (or however it get named when using ondrej) is required when using https://doc.owncloud.com/server/latest/admin_manual/enterprise/external_storage/sharepoint-integration_configuration.html#introduction.

If this library is (currently) not available with ondrej, we have to inform him and ask for adding it so it can be downloaded from his source.

Copy link
Contributor

@mmattel mmattel Jun 1, 2022

Choose a reason for hiding this comment

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

Note, seems that php-phpseclib is part of the ondrej package, see: oerdnj/deb.sury.org#445

But when looking into https://launchpad.net/~ondrej/+archive/ubuntu/php for both 20.04 (bionic) and 22.04 (jammy), the extension is not present...?

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, I think I got the solution 😄 and I will update the PR accordingly

From: https://phpseclib.com/docs/why

Why phpseclib?
phpseclib provides pure-PHP implementations of SSH2, SFTP, RSA, DSA, Elliptic Curves, AES, ChaCha20, X.509, CSR, CRL, SPKAC, etc.

Portability
The only requirement that phpseclib 3.0 has is that you must be using PHP 5.6+.

Extensions like bcmath, gmp, libsodium and openssl, if they're available, for speed, but they're not required.

And when installing it with sudo apt install php-phpseclib you will find the follonwing files:

dpkg -L php-phpseclib

/.
/usr
/usr/share
/usr/share/doc
/usr/share/doc/php-phpseclib
/usr/share/doc/php-phpseclib/changelog.Debian.gz
/usr/share/doc/php-phpseclib/copyright
/usr/share/doc/php-phpseclib/examples
/usr/share/doc/php-phpseclib/examples/openssl.cnf
/usr/share/php
/usr/share/php/phpseclib
/usr/share/php/phpseclib/Crypt
/usr/share/php/phpseclib/Crypt/AES.php
/usr/share/php/phpseclib/Crypt/Base.php
/usr/share/php/phpseclib/Crypt/Blowfish.php
/usr/share/php/phpseclib/Crypt/DES.php
/usr/share/php/phpseclib/Crypt/Hash.php
/usr/share/php/phpseclib/Crypt/RC2.php
/usr/share/php/phpseclib/Crypt/RC4.php
/usr/share/php/phpseclib/Crypt/RSA.php
/usr/share/php/phpseclib/Crypt/Random.php
...

Copy link
Contributor

Choose a reason for hiding this comment

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

update applied, all good

@mmattel
Copy link
Contributor

mmattel commented Jun 2, 2022

Squashed the commits ( 9-->1)

Copy link
Contributor

@EParzefall EParzefall left a comment

Choose a reason for hiding this comment

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

only language fixes necessary

@mmattel
Copy link
Contributor

mmattel commented Jun 2, 2022

Removed the opcache extension (not necesssayr as it is part of the php delivery) as discussed with @pako81

@mmattel mmattel dismissed jnweiger’s stale review June 2, 2022 09:49

The issue with php-phpseclib is fixed now and documented properly

@mmattel
Copy link
Contributor

mmattel commented Jun 2, 2022

Backport to 10.10, 10.9 and 10.8

@mmattel mmattel merged commit b56d5b3 into owncloud:master Jun 2, 2022
mmattel added a commit that referenced this pull request Jun 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants