You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/04/21 02:51:34 UTC

[GitHub] [incubator-kyuubi] cfmcgrady opened a new issue, #2430: [FlakyTest] ServiceControlCliArgumentsSuite

cfmcgrady opened a new issue, #2430:
URL: https://github.com/apache/incubator-kyuubi/issues/2430

   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   
   
   ### Search before asking
   
   - [X] I have searched in the [issues](https://github.com/apache/incubator-kyuubi/issues) and found no similar issues.
   
   
   ### Describe the subtask
   
   https://lists.apache.org/thread/g0ydjyo9893f0dvvmgg3sgkkfcr2xzm6
   
   ```
   Run starting. Expected test count is: 24
   ServiceControlCliArgumentsSuite:
   - test basic kyuubi service arguments parser *** FAILED ***
     "[zhangduo-ubuntu]" did not equal "[localhost]"
   (ServiceControlCliArgumentsSuite.scala:99)
     Analysis:
     "[zhangduo-ubuntu]" -> "[localhost]"
   - prints usage on empty input
   - prints error with unrecognized options
   - test invalid arguments
   - test extra unused arguments
   - test list action arguments
   - test get/delete action arguments
   - test with unknown host
   - test with invalid port specification
   - test create action arguments
   - test use default property value if missing
   - test use short options *** FAILED ***
     "[zhangduo-ubuntu]" did not equal "[localhost]"
   (ServiceControlCliArgumentsSuite.scala:333)
     Analysis:
     "[zhangduo-ubuntu]" -> "[localhost]"
   - test --help
   ```
   
   ### Parent issue
   
   #1733 
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


[GitHub] [incubator-kyuubi] Apache9 commented on issue #2430: [FlakyTest] ServiceControlCliArgumentsSuite

Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #2430:
URL: https://github.com/apache/incubator-kyuubi/issues/2430#issuecomment-1104670910

   It seems related to how we get the machine name. My machine name is zhangduo-ubuntu, so in general, 'zhangduo-ubuntu' points to the same machine with 'localhost'.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


[GitHub] [incubator-kyuubi] cxzl25 commented on issue #2430: [FlakyTest] ServiceControlCliArgumentsSuite

Posted by GitBox <gi...@apache.org>.
cxzl25 commented on issue #2430:
URL: https://github.com/apache/incubator-kyuubi/issues/2430#issuecomment-1108646789

   `--host` will now keep the original value in this PR(https://github.com/apache/incubator-kyuubi/issues/2461), so this problem should not occur.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org


[GitHub] [incubator-kyuubi] yaooqinn commented on issue #2430: [FlakyTest] ServiceControlCliArgumentsSuite

Posted by GitBox <gi...@apache.org>.
yaooqinn commented on issue #2430:
URL: https://github.com/apache/incubator-kyuubi/issues/2430#issuecomment-1104758605

   Look like the expected result in the test is hardcoded with `localhost`, while the actual one is getting from canonicalized hostname.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org