Skip to content

Optimize docker image size#3

Merged
LordRalex merged 3 commits into
mainfrom
urielsalis-patch-1
Nov 17, 2022
Merged

Optimize docker image size#3
LordRalex merged 3 commits into
mainfrom
urielsalis-patch-1

Conversation

@urielsalis
Copy link
Copy Markdown
Member

Completly untested, this should reduce the image size by quite a bit

If someone doesnt get to testing it, I can try in a few hours

@urielsalis
Copy link
Copy Markdown
Member Author

urielsalis commented May 5, 2022

Final image size is 149mb vs 1.02gb

@LordRalex LordRalex closed this Jul 14, 2022
@LordRalex LordRalex reopened this Jul 14, 2022
@LordRalex LordRalex merged commit 35631c9 into main Nov 17, 2022
@LordRalex
Copy link
Copy Markdown
Member

Starting Modmail v3.6.1 (358a116) on Node.js 14.21.1 (arm64)
Loaded 14 values from environment variables
Configuration ok!
Using an SQLite database:
/usr/src/modmailbot/db/data.sqlite

n
$ npm install sqlite3 --save
Cannot find module '@mapbox/node-pre-gyp'
Require stack:

  • /usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3-binding.js
  • /usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3.js
  • /usr/src/modmailbot/node_modules/knex/lib/dialects/sqlite3/index.js
  • /usr/src/modmailbot/node_modules/knex/lib/dialects/index.js
  • /usr/src/modmailbot/node_modules/knex/lib/knex-builder/internal/config-resolver.js
  • /usr/src/modmailbot/node_modules/knex/lib/knex-builder/Knex.js
  • /usr/src/modmailbot/node_modules/knex/lib/index.js
  • /usr/src/modmailbot/node_modules/knex/knex.js
  • /usr/src/modmailbot/src/knex.js
  • /usr/src/modmailbot/src/main.js
  • /usr/src/modmailbot/src/index.js
    Error: Cannot find module '@mapbox/node-pre-gyp'
    Require stack:
  • /usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3-binding.js
  • /usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3.js
  • /usr/src/modmailbot/node_modules/knex/lib/dialects/sqlite3/index.js
  • /usr/src/modmailbot/node_modules/knex/lib/dialects/index.js
  • /usr/src/modmailbot/node_modules/knex/lib/knex-builder/internal/config-resolver.js
  • /usr/src/modmailbot/node_modules/knex/lib/knex-builder/Knex.js
  • /usr/src/modmailbot/node_modules/knex/lib/index.js
  • /usr/src/modmailbot/node_modules/knex/knex.js
  • /usr/src/modmailbot/src/knex.js
  • /usr/src/modmailbot/src/main.js
  • /usr/src/modmailbot/src/index.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:902:15)
    at Function.Module._load (internal/modules/cjs/loader.js:746:27)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at require (internal/modules/cjs/helpers.js:101:18)
    at Object. (/usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3-binding.js:1:16)
    at Module._compile (internal/modules/cjs/loader.js:1085:14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
    at Module.load (internal/modules/cjs/loader.js:950:32)
    at Function.Module._load (internal/modules/cjs/loader.js:790:12)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    Error: Error: Knex: run
    $ npm install sqlite3 --save
    Cannot find module '@mapbox/node-pre-gyp'
    Require stack:
  • /usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3-binding.js
  • /usr/src/modmailbot/node_modules/sqlite3/lib/sqlite3.js
  • /usr/src/modmailbot/node_modules/knex/lib/dialects/sqlite3/index.js
  • /usr/src/modmailbot/node_modules/knex/lib/dialects/index.js
    ...stack trace truncated to 8 lines

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.

2 participants