You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "tvalentyn (via GitHub)" <gi...@apache.org> on 2023/05/19 21:19:58 UTC

[GitHub] [beam] tvalentyn opened a new pull request, #26797: Restore the behavior that failing to create a venv fails the pipeline.

tvalentyn opened a new pull request, #26797:
URL: https://github.com/apache/beam/pull/26797

   See: #26792 for context.


-- 
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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] tvalentyn commented on pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "tvalentyn (via GitHub)" <gi...@apache.org>.
tvalentyn commented on PR #26797:
URL: https://github.com/apache/beam/pull/26797#issuecomment-1555296935

   R: @lostluck 
   cc: @robertwb @phoerious 


-- 
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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] riteshghorse merged pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "riteshghorse (via GitHub)" <gi...@apache.org>.
riteshghorse merged PR #26797:
URL: https://github.com/apache/beam/pull/26797


-- 
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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] tvalentyn commented on pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "tvalentyn (via GitHub)" <gi...@apache.org>.
tvalentyn commented on PR #26797:
URL: https://github.com/apache/beam/pull/26797#issuecomment-1555962082

   R: @liferoad 


-- 
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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] github-actions[bot] commented on pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #26797:
URL: https://github.com/apache/beam/pull/26797#issuecomment-1555301557

   Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control


-- 
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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] riteshghorse commented on pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "riteshghorse (via GitHub)" <gi...@apache.org>.
riteshghorse commented on PR #26797:
URL: https://github.com/apache/beam/pull/26797#issuecomment-1556319982

   Failure not related to this PR, merging
   
   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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] codecov[bot] commented on pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #26797:
URL: https://github.com/apache/beam/pull/26797#issuecomment-1555292784

   ## [Codecov](https://app.codecov.io/gh/apache/beam/pull/26797?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   > Merging [#26797](https://app.codecov.io/gh/apache/beam/pull/26797?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (7bbd60a) into [master](https://app.codecov.io/gh/apache/beam/commit/87d9b843baa0f35ddb6121dfe463a1a19cbf916e?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) (87d9b84) will **increase** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #26797   +/-   ##
   =======================================
     Coverage   71.97%   71.97%           
   =======================================
     Files         747      747           
     Lines      101306   101306           
   =======================================
   + Hits        72914    72916    +2     
   + Misses      26933    26931    -2     
     Partials     1459     1459           
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | python | `80.95% <ø> (+<0.01%)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   [see 2 files with indirect coverage changes](https://app.codecov.io/gh/apache/beam/pull/26797/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   


-- 
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: github-unsubscribe@beam.apache.org

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


[GitHub] [beam] tvalentyn commented on pull request #26797: Restore the behavior that failing to create a venv fails the pipeline.

Posted by "tvalentyn (via GitHub)" <gi...@apache.org>.
tvalentyn commented on PR #26797:
URL: https://github.com/apache/beam/pull/26797#issuecomment-1555299527

   tested both default and opt-out codepaths on Dataflow.


-- 
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: github-unsubscribe@beam.apache.org

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