Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

attempt to determine the correct gem provider #530

Merged
merged 1 commit into from
Mar 8, 2016

Conversation

jhoblitt
Copy link
Member

@jhoblitt jhoblitt commented Mar 8, 2016

jenkins::cli::config currently installs the retries gem needed by
the xtypes as a convenience -- it is not clear if functionality should be
in this class at all or if it should merely be a vehicle for passing
data into the catalog. Puppetserver users will still have to manually
install the retries gem as there is no reliable way to determine if an
agent run is configuring a puppet master.

resolves #434
resolves #498

@jenkinsadmin
Copy link

Thank you for this pull request! Please check this document for how the Jenkins project handles pull requests.

`jenkins::cli::config` currently installs the `retries` gem needed by
the xtypes as a convenience -- it is not clear if functionality should be
in this class at all or if it should merely be a vehicle for passing
data into the catalog.  Puppetserver users will still have to manually
install the `retries` gem as there is no reliable way to determine if an
agent run is configuring a puppet master.

resolves voxpupuli#434
resolves voxpupuli#498
@jhoblitt jhoblitt force-pushed the maint/gem-provider branch from 0f7a348 to b66e2f6 Compare March 8, 2016 15:21
jhoblitt added a commit that referenced this pull request Mar 8, 2016
attempt to determine the correct gem provider
@jhoblitt jhoblitt merged commit bcca89d into voxpupuli:master Mar 8, 2016
@jhoblitt jhoblitt deleted the maint/gem-provider branch March 8, 2016 16:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
2 participants