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

feat(docker): add monolithic memory checks #175

Merged
merged 3 commits into from
Jan 30, 2024
Merged

Conversation

Gaitholabi
Copy link
Contributor

No description provided.

@bytestream
Copy link
Member

I've made some changes:

  • warning rather than error, the system locking up only happens after running docker compose up -d
  • reduced 4gb because a t3.small seems to actually have 1.9G of physical RAM..
  • check if /proc/meminfo exists rather than looking at the OS

I now shows:

$ bash setup.sh 
 ____                               _   ____       _
/ ___| _   _ _ __  _ __   ___  _ __| |_|  _ \ __ _| |
\___ \| | | | '_ \| '_ \ / _ \| '__| __| |_) / _` | |
 ___) | |_| | |_) | |_) | (_) | |  | |_|  __/ (_| | |
|____/ \__,_| .__/| .__/ \___/|_|   \__|_|   \__,_|_|
            |_|   |_|

##### ##### ##### ##### ##### ##### ##### ##### ##### ##### #####
                         !! WARNING !!                           
##### ##### ##### ##### ##### ##### ##### ##### ##### ##### #####

The system has less than 4GB of total memory (RAM + Swap),
detected 3999999KB. The system may crash if launched with
insufficient memory.

Greater than 4000000KB is recommended.

Press CTRL+C to exit and add more RAM/Swap. Continuing in 15s...

##### ##### ##### ##### ##### ##### ##### ##### ##### ##### #####

^C

WDYT?

@bytestream bytestream changed the base branch from master to 5.x January 30, 2024 17:39
@Gaitholabi
Copy link
Contributor Author

I've made some changes:

  • warning rather than error, the system locking up only happens after running docker compose up -d
  • reduced 4gb because a t3.small seems to actually have 1.9G of physical RAM..
  • check if /proc/meminfo exists rather than looking at the OS

I now shows:

$ bash setup.sh 
 ____                               _   ____       _
/ ___| _   _ _ __  _ __   ___  _ __| |_|  _ \ __ _| |
\___ \| | | | '_ \| '_ \ / _ \| '__| __| |_) / _` | |
 ___) | |_| | |_) | |_) | (_) | |  | |_|  __/ (_| | |
|____/ \__,_| .__/| .__/ \___/|_|   \__|_|   \__,_|_|
            |_|   |_|

##### ##### ##### ##### ##### ##### ##### ##### ##### ##### #####
                         !! WARNING !!                           
##### ##### ##### ##### ##### ##### ##### ##### ##### ##### #####

The system has less than 4GB of total memory (RAM + Swap),
detected 3999999KB. The system may crash if launched with
insufficient memory.

Greater than 4000000KB is recommended.

Press CTRL+C to exit and add more RAM/Swap. Continuing in 15s...

##### ##### ##### ##### ##### ##### ##### ##### ##### ##### #####

^C

WDYT?

Looks reasonable 👍

@bytestream bytestream merged commit bf719f7 into 5.x Jan 30, 2024
2 checks passed
@bytestream bytestream deleted the add-memory-checks branch January 30, 2024 22:29
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