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

Fixes #31105 - Truncate description of CV env in candlepin #9089

Merged
merged 1 commit into from Jan 7, 2021

Conversation

sjha4
Copy link
Member

@sjha4 sjha4 commented Jan 4, 2021

To test: create a CV with a very long description and publish.
You'll see action : Actions::Candlepin::Environment::Create fail with error: description: size must be between 0 and 255

With this change we truncate the Candlepin env description to 255 chars.

@theforeman-bot
Copy link

Issues: #31105

Copy link
Member

@jturel jturel left a comment

Choose a reason for hiding this comment

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

ACK! Works for me with very long descriptions and when there is no description. Thanks @sjha4 !

@sjha4 sjha4 merged commit 84835e6 into Katello:master Jan 7, 2021
ranjan pushed a commit to ranjan/katello that referenced this pull request Feb 9, 2021
ianballou pushed a commit to ianballou/katello that referenced this pull request Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants