Ignore negative value of grace-period

[xiangpengzhao]

If the value of flag --grace-period passed to oadm manage-node is negative, we'd better ignore it.

BTW, should we add an example of --grace-period in oadm_by_example_content.adoc#oadm-manage-node ?

[fabianofranz]

@xiangpengzhao thanks for the PR! You can add an example right in the examples section of the command and then run hack/update-generated-docs.sh which will update the examples document automatically.

[fabianofranz]

[test]

[openshift-bot]

Evaluated for origin test up to 3d5289a

[openshift-bot]

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7803/)

[xiangpengzhao]

Just modify an existed example and add description. PTAL @fabianofranz . Thanks!

[fabianofranz]

@xiangpengzhao just go ahead and add a separate example for --grace-period=60 instead of changing the existing one. No problem if they are similar. Having simple examples where each demonstrate one single feature of the command is encouraged and improve the understandability for our users. ;)

[xiangpengzhao]

Thanks @fabianofranz ! Agreed. I add two separate examples for flags --grace-period and --force. PTAL.

[fabianofranz]

LGTM [merge]

[openshift-bot]

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7924/) (Image: devenv-rhel7_4830)

[openshift-bot]

Evaluated for origin merge up to 0cb8cb9

[openshift-bot]

Origin Action Required: Please contact #openshift-dev to have this pull request manually reviewed and tested