linuxserver / docker-snipe-it

Alpine/Nginx container for the Asset Management software Snipe-IT
GNU General Public License v3.0
123 stars 26 forks source link

Remove base packages #51

Closed nemchik closed 1 year ago

nemchik commented 1 year ago

Also Fix dependencies

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-77db8818-pr-51/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-77db8818-pr-51/shellcheck-result.xml

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-77db8818-pr-51/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-77db8818-pr-51/shellcheck-result.xml

thespad commented 1 year ago

Probably need to add an APP_URL to the CI envs or disable the web CI otherwise it'll just fail.

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-77db8818-pr-51/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-77db8818-pr-51/shellcheck-result.xml

nemchik commented 1 year ago

CI screenshots are still broken. I'm not sure how to fix them. I can disable, but if someone has a suggestion on making them work I don't mind to push that change.

thespad commented 1 year ago

It won't work without the DB in place, the setup page just doesn't load unless it can connect to the db. Tested it locally to try and figure it out.

nemchik commented 1 year ago

Turned it off in https://github.com/linuxserver/docker-snipe-it/pull/51/commits/53d724c3777c98e5cb37ced2904721a3e59e6615 I edited here in GH web, so the Jenkinsfile is not updated. CI for this PR will still have broken screenshot, but future builds should skip the screenshot.

LinuxServer-CI commented 1 year ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-ca133780-pr-51/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v6.0.14-pkg-ca133780-pr-51/shellcheck-result.xml