-
Notifications
You must be signed in to change notification settings - Fork 243
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
We should validate config params for devfile #3791
Comments
@girishramnani Are we considering this for devfile v2 because it seems quite important to me. Whatever feature we say is there with odo, IMO we should cover all the cases which validates it. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
Stale issues rot after 30d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle rotten |
Rotten issues close after 30d of inactivity. Reopen the issue by commenting /close |
@openshift-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
We should validate memory limit, not specifically here may be during
odo push
, if memory limit specified in wrong format like756g
,odo config set
odo push
k8s api
silently ignores that, and component gets pushed successfully without applying limits.Originally posted by @adisky in #3702
The text was updated successfully, but these errors were encountered: