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

Paperless ngx #652

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Paperless ngx #652

wants to merge 15 commits into from

Conversation

viktor-c
Copy link

I hereby request to add paperless_ngx to the list of apps, since paperless_ng has not been developed for a while and it's original repo has been archived.
I based the files on hello_world example and used the paperless_ngx official example docker-compose file

This PR which is just a couple of hours younger as mine does not integrate tests.

Since I am a hobby developer I could not get tests working, appreciate some help in that direction: python venv and vagrant are working; test.sh output is blank, github-test-matrix.sh outputs ::set-output name=matrix::[""]

@viktor-c
Copy link
Author

viktor-c commented Feb 19, 2023

Pre-commit without issues.
Fixed some linting errors
Setting paperless_ngx_enabled to true and then to false should trigger stopping all containers, but these tasks always get skipped in ansible ? Any idea why?

@viktor-c
Copy link
Author

As of now, testing the role also works.

@viktor-c
Copy link
Author

Setting paperless_ngx_enabled to true and then to false should trigger stopping all containers, but these tasks always get skipped in ansible ? Any idea why?

fixed that with above commit

@viktor-c
Copy link
Author

viktor-c commented Apr 1, 2023

Ran CI and Integration (only passing for paperless-ngx).

Don't know why the other roles don't pass molecule tests, could be something to do with: #655 (comment) which fixes the tests.

@viktor-c
Copy link
Author

viktor-c commented Apr 16, 2023

Integration

CI

Thanks to last Pullrequest (as of 16.04.2023) CI passes all tests.

Copy link

@tanc tanc left a comment

Choose a reason for hiding this comment

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

Works well except for a mismatch on the container names within the network for gotenberg and tika making then inaccessible within the network from paperless.

roles/paperless_ngx/defaults/main.yml Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
roles/paperless_ngx/defaults/main.yml Outdated Show resolved Hide resolved
roles/paperless_ngx/defaults/main.yml Outdated Show resolved Hide resolved
gcuendet added a commit to gcuendet/ansible-nas that referenced this pull request Mar 10, 2024
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.

None yet

2 participants