8000 Maximum Memory shows -1 instead of Unlimited · Issue #5134 · Cacti/cacti · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Maximum Memory shows -1 instead of Unlimited #5134

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.

Sign up for GitHub

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

Jump to bottom
Closed
TheWitness opened this issue Jan 5, 2023 · 1 comment
Closed

Maximum Memory shows -1 instead of Unlimited #5134

TheWitness opened this issue Jan 5, 2023 · 1 comment
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Milestone
v1.2.24

Comments

@TheWitness
Copy link
Member
TheWitness commented Jan 5, 2023 •
edited
Loading

Describe the bug

This is primarily a visual issue where when have the memory set to "Unlimited" on the settings page, it displays as -1 on the Boost Status page as seen below.

image

Expected behavior

Show "Unlimited" there instead of -1
@TheWitness TheWitness added bug Undesired behaviour unverified Some days we don't have a clue confirmed Bug is confirm by dev team and removed unverified Some days we don't have a clue labels Jan 5, 2023
@TheWitness TheWitness added this to the v1.2.24 milestone Jan 5, 2023
TheWitness added a commit that referenced this issue Jan 5, 2023
@TheWitness
Fixing #5134 - Memory display
494ebd1 
Display of "Memory Limit" on Boost Status page shows -1 and not "Unlimited"
TheWitness added a commit that referenced this issue Jan 5, 2023
@TheWitness
Fixing #5134, #5135, #5138 from 1.2.x
4eab34c 
Porting forward these three bugs and updating the CHANGELOG for #5127
@TheWitness
Copy link
Member Author

Committed to develop now

TheWitness added a commit that referenced this issue Jan 6, 2023
@TheWitness
QA: Regression in #5134 Boost Memory
26b72d8 
Setting memory to unlimited now caused boost to crash due to an illegal memory limit of -1M.  This is actually not a regression, but was in place from the beginning.
TheWitness added a commit that referenced this issue Jan 6, 2023
@TheWitness
QA: Issue #5134 Invalid memory limit
b39417a 
Was not so much a regression, but as a result, invalid memory limit was being set.  This was possible since before 1.2.23.
@netniV netniV changed the title 1.2.23 - Display of "Memory Limit" on Boost Status page shows -1 and not "Unlimited" Maximum Memory shows -1 instead of Unlimited Feb 15, 2023
@netniV netniV added the resolved A fixed issue label Feb 15, 2023
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Milestone
v1.2.24
Development

No branches or pull requests
2 participants
@TheWitness @netniV
0