-
Notifications
You must be signed in to change notification settings - Fork 57
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
Improve messages in Helm deployment issues with custom chartDir option #1044
Improve messages in Helm deployment issues with custom chartDir option #1044
Conversation
This will bail out if the repository is ambigious about the deployment descriptor. It will fail if: * there is no deployment descriptor * there is more than one deployment descriptor For a successfull release to happen, developers are required to "clean" their repository of unrelated deployment desciptors that could interfere with the current version.
…m-chartdir-option # Conflicts: # CHANGELOG.md
@jafarre-bi, @jorge-romero, @hrcornejo -- can you review this? |
…m-chartdir-option # Conflicts: # CHANGELOG.md
@jafarre-bi, @jorge-romero, @hrcornejo -- can you review this? |
…m-chartdir-option # Conflicts: # CHANGELOG.md
resources/org/ods/component/RolloutOpenShiftDeploymentStage.deprecate-tailor.GString.txt
Show resolved
Hide resolved
resources/org/ods/component/RolloutOpenShiftDeploymentStage.deprecate-tailor.GString.txt
Show resolved
Hide resolved
* More than one deployment descriptor found. | ||
* No deployment descriptor found. | ||
|
||
Deployment descriptors found: ${files} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This message is intended for the end user. It may not be clear, if it shows an empty list in case there are no descriptors.
Ideally, we should be able to tell the user the exact reason of the failure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The end user is a developer. I would expect some prior knowledge here. Any suggestions what it should look like?
…m-chartdir-option # Conflicts: # CHANGELOG.md
…m-chartdir-option
#1044) Co-authored-by: serverhorror <serverhorror@users.noreply.github.com>
This will bail out if the repository is ambigious about the deployment descriptor.
It will fail if:
For a successfull release to happen, developers are required to "clean" their repository of unrelated deployment desciptors that could interfere with the current version.
related with #1038