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

3.7.0 introduced a weird bug that breaks some checks on some configurations #507

Closed
remh opened this issue May 15, 2013 · 5 comments
Closed
Assignees

Comments

@remh
Copy link
Contributor

remh commented May 15, 2013

It happens when we try to launch a java process (e.g. nodetool, jmxconnector)

Could not reserve enough space for object heap
Could not create the Java virtual machine.

It doesn't happen on all configurations though.

My guess is that it's related to this pull request:
#429

Need to be fixed asap.
ETA of the release: end of this week.

@ghost ghost assigned remh May 15, 2013
@remh
Copy link
Contributor Author

remh commented May 15, 2013

Commenting this line:
https://github.com/DataDog/dd-agent/blob/master/util.py#L271

Fixes the issue confirming that it's coming from that pull request.

@remh
Copy link
Contributor Author

remh commented May 15, 2013

So the solutions are:

  1. reverting this feature
  2. feature flag it and let the user customize the limits in datadog.conf, should it be enable or disable by default ?

My thoughts would be the solution number 2, disabled by default.

@alq666
Copy link
Member

alq666 commented May 15, 2013

I vote for number 2, disabled by default so that most folks don't even risk a chance to get hit by this.

@remh
Copy link
Contributor Author

remh commented May 15, 2013

cool that's what i did :)

@remh
Copy link
Contributor Author

remh commented May 15, 2013

Fixed by f2304b9

@remh remh closed this as completed May 15, 2013
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

No branches or pull requests

2 participants