-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Upgrade installation scripts and documentation #1332
Conversation
00a532b
to
d381ac4
Compare
3e0c305
to
13c5ac9
Compare
FYI: rebased branch and append
Thank you, @tiekoetter 👍 |
Running: There are warning about the virtualenv:
The command ends with the line above and the exit code 42. Is it normal ? https://github.com/dalf/searxng-install : the scripts runs in a GitHub Action. "Of course" there is a network issue: the gateway is reachable but not further. I wish to fix this issue so we can run these tests automatically. |
Would be cool to have an automated test 👍
Without a network no packages can be installed and the SearXNG brand (repo) can't be cloned.
Yes, the function https://github.com/return42/searxng/blob/searxng-install/utils/searxng.sh#L470-L474 You will see the same exit code, if you do not install https://github.com/return42/searxng/blob/searxng-install/utils/searxng.sh#L133 |
I've tried the next step: The full log:
|
Sorry, I do not understand the intention of your question. Isn't it clear, that https://github.com/return42/searxng/blob/searxng-install/utils/searxng.sh#L256-L287 If one command fails it does not make sense to continue with the next; When there is no clone, you can't install a pyenv, when there is no uwsgi service installed, you can't install a HTTP service for this uWSGI service .. and when there is a network issue it makes no sense to try an installation, since nothin can be loaded and installed from package manager repositories. |
The network issue is from this GitHub workflow (disabled for now) The log above comes from my try on my laptop: there is no network issue.
In addition to the test above I have updated https://github.com/dalf/searxng-install :
It is not surprising reading the doc, however I'm not sure how to solve that. |
Ah, OK .. thats good .. if you have problems building all containers, I recommend to run your first test in a single container, e.g. I often use archlinux for a first test; bevore remove all containers and all images (I have seen some issues building all containers due to the limited size of the lxc storage where all the containers and images are stored)
Does this work on your laptop .. If yes, install ngingx:
I not, please read the log thoroughly, are there any ERROR messages .. please post it here to see where it might fail. The message you posted above:
shows me that the clone was not built .. and the nessage
shows me, that the wget command from the archlinix package manager was not installed for some reason, I assume you have no more space left on your lxc storage. And this is the reason why the installation in the containers fails at different points (do you see a "no space left" message anywhere in the log?). About CI & LXC ..
:-) .. after the clone .. https://github.com/dalf/searxng-install/blob/main/.github/workflows/test2.yml#L17 and before the |
Ok I figured out the issue: I created a new branch from master and then merge the changes from this branch. the log about "Clone SearXNG sources" with the previous configuration (new branch from master and then merge searxng-install)
It seems to be a chicken / egg problem: after then git clone, the searxng user does not exist. why doesn't the script do the chown? See the log: https://github.com/dalf/searxng-install/runs/7062375666?check_suite_focus=true#step:2:180 |
The script should not change permissions of objects that are not a part of the installation itself .. the git clone in this folder is a "tool" to install, but it is not a part of the installation itself. I hope it was understandable what I wanted to express. |
13c5ac9
to
25dc5ad
Compare
FYI: in preparation to merge this PR, the branch has been rebased on master and the brand-commit has been removed (see HINT) A preview of the documentation about the upgrade is available from here: https://return42.github.io/searxng/admin/update-searxng.html#migrate-and-stay-tuned |
25dc5ad
to
58b7e67
Compare
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Since ./utils/searxng.sh is implemented, the old installation procedures from filtron, morty and searx can be removed. For users who want to upgrade, the procedures for removing old installations have still been retained. Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
58b7e67
to
e5323b8
Compare
Related: searxng#1332 Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
[mod] fix minor leftovers from PR #1332
A preview of the documentation about the upgrade is available from here:
https://return42.github.io/searxng/admin/update-searxng.html#migrate-and-stay-tuned
HINT: last patch on top needs to be removed before this PR is merged!
This brand is needed to test the installation procedures as long this PR is not merged into master.
What does this PR do?
Why is this change important?
The infrastructure of SearXNG has changed dramatically over the past year. The installation scripts and documentation have not been adapted so far.
How to test this PR locally?
First read the admin documentation, to see what have been changed ..
Test new
./utils/searxng.sh
script ..in LXCs, first build containers and install SearXNG by
install all
:Choose to
install nginx
:or
install apache
HTTP server:Run
instance check
:Now you can test the SearXNG instances from the conatiners, to get the URLs of the containers use:
Settings see https://github.com/return42/searxng/blob/searxng-install/utils/templates/etc/searxng/settings.yml#L3-L22
Features:
static_use_hash
) should workcloses: #1112