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

Support Jolokia agent in proxy mode #4421

Closed
ppf2 opened this issue May 30, 2017 · 10 comments
Closed

Support Jolokia agent in proxy mode #4421

ppf2 opened this issue May 30, 2017 · 10 comments

Comments

@ppf2
Copy link
Member

ppf2 commented May 30, 2017

Currently, we do not support the Jolokia agent when it is run in proxy mode. There are users out there with IT restrictions and not able to get approvals for installing a direct agent so supporting the proxy mode will be helpful for these situations.

@tsg
Copy link
Contributor

tsg commented May 31, 2017

I wonder how come the proxy mode is not supported? Any idea on what exactly we're missing? Or is one of the limitations listed there affecting us?

@ruflin
Copy link
Contributor

ruflin commented May 31, 2017

The reason I think it is not going to work in proxy mode is that proxy mode requires the url, username and password passed. And that is not possible in our config at the moment:

  {
    "type":"READ"
    "mbean":"java.lang:type=Threading",
    "attribute":"ThreadCount",
    "target": { 
                "url":"service:jmx:rmi:///jndi/rmi://jboss-as:8686/jmxrmi",
                "password":"admin",
                "user":"s!cr!t"
              },
  }

whale-shark added a commit to whale-shark/beats that referenced this issue Jan 21, 2018
@ruflin ruflin added the module label Feb 26, 2018
@ruflin
Copy link
Contributor

ruflin commented Feb 26, 2018

@whale-shark Looking at this issue I saw that you did a commit referencing this issue. Does the commit you made work? If yes, any chance to contribute it back?

whale-shark added a commit to whale-shark/beats that referenced this issue Feb 26, 2018
@whale-shark
Copy link
Contributor

@ruflin Thank you for pointing out. I read a procedure and made a request. Please tell me if there is any mistake. #6475

@ruflin
Copy link
Contributor

ruflin commented Feb 27, 2018

@whale-shark Awesome. Left some comments on the PR.

Are you using this code on your end in production / testing? Did you compile your own beat?

@whale-shark
Copy link
Contributor

@ruflin Thank you for checking.

I built it locally on my CentOS.
I was able to get the HeapMemoryUsage from remote Java VM by using the result of building.

The bundled test did not work well from the beginning, so I could not do it.
For more details, I wrote in PR.

@balamacab
Copy link

Any binary to test it?

@ruflin
Copy link
Contributor

ruflin commented Mar 15, 2018

@balamacab For a binary you would have to check out #6475 and run make or go build on your end.

@dedemorton
Copy link
Contributor

@ruflin Can this issue be closed? The docs were updated with this commit. Since the options are now documented, users will expect this to be supported. Not clear to me from this exchange whether we can say that's the case.

@ruflin
Copy link
Contributor

ruflin commented Aug 16, 2018

Yes, proxy mode should now be supported. @jsoriano Please correct me if that is not the case.

@ruflin ruflin closed this as completed Aug 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants