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

(GH-905) Ensure release failure includes error message #929

Merged
merged 1 commit into from
Feb 3, 2021
Merged

(GH-905) Ensure release failure includes error message #929

merged 1 commit into from
Feb 3, 2021

Conversation

michaeltlombardi
Copy link
Contributor

@michaeltlombardi michaeltlombardi commented Dec 10, 2020

Prior to this commit, a failure to publish to the forge returns the string representing a Net::HTTPBadRequest object instead of text including the actual error message. This commit ensures that the body of the error is instead written to the error stream.

Resolves #905

@coveralls
Copy link

coveralls commented Dec 10, 2020

Coverage Status

Coverage remained the same at 91.237% when pulling ed6880f on michaeltlombardi:gh-905/master/add-release-failure-context into 1800e59 on puppetlabs:main.

Base automatically changed from master to main January 11, 2021 22:22
@michaeltlombardi
Copy link
Contributor Author

Failures in acceptance testing seem unrelated, I'm not sure what's going on there.

@DavidS
Copy link
Contributor

DavidS commented Feb 3, 2021

Some of those failures are addressed in 4abaa34 - I'm still working on the sudden time out on the erroring cell

Prior to this commit, a failure to publish to the forge returns
the string representing a Net::HTTPBadRequest object instead of
text including the actual error message. This commit ensures that
the body of the error is instead written to the error stream.
Copy link
Contributor

@DavidS DavidS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rebased on top of the fixes in #939 ; code looks fine

@DavidS DavidS merged commit 6b35c66 into puppetlabs:main Feb 3, 2021
@michaeltlombardi michaeltlombardi deleted the gh-905/master/add-release-failure-context branch February 3, 2021 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide more context when forge upload fails
3 participants