You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/04/29 07:00:38 UTC

[GitHub] [pulsar] horizonzy opened a new pull request, #15385: Commit for test ci, needn't view.

horizonzy opened a new pull request, #15385:
URL: https://github.com/apache/pulsar/pull/15385

   Commit for test ci, needn't view.


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] github-actions[bot] commented on pull request #15385: Commit for test ci, needn't view.

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1112937446

   @horizonzy:Thanks for your contribution. For this PR, do we need to update docs?
   (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md#documentation), which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy closed pull request #15385: [WIP] Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
horizonzy closed pull request #15385: [WIP] Commit for test ci, ignore this pr
URL: https://github.com/apache/pulsar/pull/15385


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] lhotari commented on pull request #15385: [WIP] Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
lhotari commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113068351

   > fine, after ci done at this time, I won't do it again.
   
   @horizonzy  +1. There are some instructions in https://lists.apache.org/thread/1ofdbd9j1hhj4c0rw3o7vv8y6rbsxrp0 about using "Personal CI".
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] lhotari commented on pull request #15385: [WIP] Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
lhotari commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113050807

   @horizonzy If you want to test the CI or modify it, please consider opening PRs to your own fork of apache/pulsar since we have very limited resources in apache/pulsar.
   In your own fork, you can run GitHub Actions workflows independently and your builds won't cause the build queue to grow in apache/pulsar.


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy commented on pull request #15385: [WIP] Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
horizonzy commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113059817

   > @horizonzy If you want to test the CI or modify it, please consider opening PRs to your own fork of apache/pulsar since we have very limited resources in apache/pulsar. In your own fork, you can run GitHub Actions workflows independently and your builds won't cause the build queue to grow in apache/pulsar.
   
   fine, after ci done at this time, I won't do it again.


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy commented on pull request #15385: Commit for test ci, needn't view.

Posted by GitBox <gi...@apache.org>.
horizonzy commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113016114

   > could you explain more why you split the test? it doesn't show up correctly in the CI results? can you provide an example?
   
   Not mint it, I'm trying to find the flaky test root case, split it just for find.


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy commented on pull request #15385: Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
horizonzy commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113046165

   > @horizonzy ok, I suggest you to set the pull as draft, add "WIP" in the title or create it in your fork to not cause confustion
   
   I try it at my fork and upstream both... It will be faster for test.


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] nicoloboschi commented on pull request #15385: Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
nicoloboschi commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113037192

   @horizonzy ok, I suggest you to set the pull as draft, add "WIP" in the title or create it in your fork to not cause confustion 


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] lhotari commented on pull request #15385: [WIP] Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
lhotari commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113071158

   @horizonzy The additional benefit of the "Personal CI" is that you get ssh access to the build VMs when the build is running. That is handled by this logic in the pulsar-ci.yaml GitHub Actions workflow file:
   https://github.com/apache/pulsar/blob/1682884d5d4860bfcbe25d54ac27e446f94f8716/.github/workflows/pulsar-ci.yaml#L77-L82
   and inline composite action implementation: https://github.com/apache/pulsar/blob/master/.github/actions/ssh-access/action.yml


-- 
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: commits-unsubscribe@pulsar.apache.org

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


[GitHub] [pulsar] horizonzy commented on pull request #15385: [WIP] Commit for test ci, ignore this pr

Posted by GitBox <gi...@apache.org>.
horizonzy commented on PR #15385:
URL: https://github.com/apache/pulsar/pull/15385#issuecomment-1113072073

   > @horizonzy The additional benefit of the "Personal CI" is that you get ssh access to the build VMs when the build is running. That is handled by this logic in the pulsar-ci.yaml GitHub Actions workflow file:
   
   Thanks your guide


-- 
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: commits-unsubscribe@pulsar.apache.org

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