Guard length of url for bz reporting #5067
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
rvykydal
force-pushed
the
guard-length-of-url-for-bz-reporting
branch
2 times, most recently
from
8a821d5
to
e62415f
Compare
|
/kickstart-test --waive webui-only |
KKoukiou
reviewed
Aug 23, 2023
|
/webui-test Sorry, I need to see the status name to enforce it... |
rvykydal
force-pushed
the
guard-length-of-url-for-bz-reporting
branch
from
e62415f
to
65925a9
Compare
rvykydal
force-pushed
the
guard-length-of-url-for-bz-reporting
branch
from
65925a9
to
9ce0c38
Compare
|
So I added some comments in the code and removed the message informing about the Detail being cut from the Detail. The message asking to attach the log is still added after the cut detail.
|
rvykydal
force-pushed
the
guard-length-of-url-for-bz-reporting
branch
from
9ce0c38
to
16236fb
Compare
|
/kickstart-test --waive webui-only |
|
/webui-test |
rvykydal
force-pushed
the
guard-length-of-url-for-bz-reporting
branch
from
16236fb
to
4ac0e0c
Compare
|
/kickstart-test --waive webui-only |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Low priority, the first commit can be perhaps merged separately.
https://issues.redhat.com/browse/INSTALLER-3612
Example of truncated comment:
(The message should appear really exceptionally - we don't expect automatically generated Detail/comment to exceed the 7000 characters. It is there so it is clear what really happened (the comment was truncated) in such a rare case.)