Skip to content

[puppet] PE and POSS require puppetserver_gem to get the dogapi gem in the JRuby env.#188

Merged
truthbk merged 3 commits intomasterfrom
jaime/fix_reporting_pe
Jun 22, 2016
Merged

[puppet] PE and POSS require puppetserver_gem to get the dogapi gem in the JRuby env.#188
truthbk merged 3 commits intomasterfrom
jaime/fix_reporting_pe

Conversation

@truthbk
Copy link
Copy Markdown
Member

@truthbk truthbk commented Jun 3, 2016

Why

As it turns out PE and POSS have the puppetserver sit on top of their own JRuby environment, with its own set of gems, etc. Fortunately the puppetserver_gem provider comes to the rescue to allow us to provide the required dogapi gem (if you want reporting enabled) into the environment.

@truthbk truthbk force-pushed the jaime/fix_reporting_pe branch 2 times, most recently from 798f95f to 0cc4a84 Compare June 21, 2016 10:30
@truthbk
Copy link
Copy Markdown
Member Author

truthbk commented Jun 21, 2016

Tested manually - successfully.

@hkaj
Copy link
Copy Markdown
Member

hkaj commented Jun 21, 2016

Nice, you can remove the comment about clientversion and :shipit:

@truthbk truthbk force-pushed the jaime/fix_reporting_pe branch from 0cc4a84 to 8153350 Compare June 21, 2016 11:41
…m in the JRuby env.

[reporting] updating README.
@truthbk truthbk force-pushed the jaime/fix_reporting_pe branch from 8153350 to bbb2ad1 Compare June 22, 2016 03:34
truthbk added 2 commits June 22, 2016 07:10
…m in the JRuby env.

[reporting] updating README.

[reporting] puppet DSL doesn't allow reassignment, use new var.
@truthbk truthbk merged commit f0cca16 into master Jun 22, 2016
@truthbk truthbk deleted the jaime/fix_reporting_pe branch June 22, 2016 05:29
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Jan 31, 2018
[puppet] PE and POSS require puppetserver_gem to get the dogapi gem in the JRuby env.
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