-
Notifications
You must be signed in to change notification settings - Fork 366
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
Unable to deploy to forge.puppet.com #487
Comments
Before the release we do a few tests, all with latest puppet and json_pure gem, which works without issues. |
@BanzaiMan is there anything I can help to fix this? I saw in the commit history that there were a few changes for the json/json_pure gem that are unreleased, not sure if they fix the issue. I'm not aware of a solution to test a patched dpl version or the master branch :( In our case travis automatically pulls the gem by some black magic. |
You can fork and fix the puppet deployment provider. The deployment fails now because the specification of |
This gem seems to be rather complex. I'm trying to built a solution that works at ruby2.3.1 again, but I'm not sure if this will work again. |
Pushed a potential fix to the deploy:
provider: puppetforge
edge:
branch: ha-bug-puppet-forge
⋮ # rest |
thanks for the fast investigation @BanzaiMan! The patch throws an argument error:
|
@bastelfreak Ooops. Sorry about that. Could you try again? |
Hi. The error changed: https://travis-ci.org/voxpupuli/puppet-community_kickstarts/jobs/149119338#L567
I don't quite understand why travis uses ruby1.9 here. The tests itself run fine on ruby231 |
Ooops. Sorry about that. There was a syntax error in my fix. Could you try again? |
Thanks, this works now: Are you able to publish a release soon? |
+1 |
HI @BanzaiMan, do you think you can do a release in the near future? |
During a release, travis installs the dpl gem. The latest released version is broken, the travis devs provided us a fix which is available in a certain branch: travis-ci/dpl#487 This fix is tested in several modules, fox example zabbix, gluster, minecraft
Hi @BanzaiMan, is it possible to get a release soon? |
@BanzaiMan Bump for a release. Thanks! |
Sorry about the delay. Could you try the updated commit before releasing this? Thanks. |
@BanzaiMan we just tested it, worked. https://travis-ci.org/voxpupuli/puppet-msoffice/jobs/172236476 |
Thanks for the confirmation. I'm closing this now. |
@BanzaiMan Would you be willing to re-open this ticket until the fix in #524 is published to rubygems so that travis uses it automatically without requiring a separate branch in |
1.8.24 should fix this problem now. |
@BanzaiMan Thanks so much, Hiro! |
Hi,
I'm currently unable to release anything to the forge, I think there is a dependency issue with json_pure? Here is the full built. And the logs:
The text was updated successfully, but these errors were encountered: