You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/10/07 04:56:34 UTC

[GitHub] [spark] Yikun opened a new pull request, #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Yikun opened a new pull request, #38145:
URL: https://github.com/apache/spark/pull/38145

   ### What changes were proposed in this pull request?
   Add packages write permisson for parent workflows
   
   
   ### Why are the changes needed?
   To avoid the issue like:
   > The workflow is not valid. .github/workflows/build_main.yml (Line: 28, Col: 3): Error calling workflow 'apache/spark/.github/workflows/build_and_test.yml@6eedb0eaa72008858b77d8d36fde0938b0962922'. The nested job 'Base image build' is requesting 'packages: write', but is only allowed 'packages: none'.
   
   https://github.com/apache/spark/actions/runs/3202259476/workflow
   
   
   
   ### Does this PR introduce _any_ user-facing change?
   No, dev only
   
   
   ### How was this patch tested?
   see master merge results.


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org

[GitHub] [spark] Yikun commented on pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Posted by GitBox <gi...@apache.org>.
Yikun commented on PR #38145:
URL: https://github.com/apache/spark/pull/38145#issuecomment-1271111971

   wait until base image job passed


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org

[GitHub] [spark] Yikun commented on pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Posted by GitBox <gi...@apache.org>.
Yikun commented on PR #38145:
URL: https://github.com/apache/spark/pull/38145#issuecomment-1271113946

   Ready to go.
   
   https://github.com/Yikun/spark/actions/runs/3202348639/jobs/5231291445: base image passed
   https://github.com/Yikun/spark/actions/runs/3202348639/jobs/5231299323: some jobs are also OK to initialize container.


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org

[GitHub] [spark] Yikun commented on pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Posted by GitBox <gi...@apache.org>.
Yikun commented on PR #38145:
URL: https://github.com/apache/spark/pull/38145#issuecomment-1271116496

   @HyukjinKwon 
   
   From the latest merged commit CI result, work as execpected!
   https://github.com/apache/spark/actions/runs/3202391656/jobs/5231332667


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org

[GitHub] [spark] Yikun commented on pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Posted by GitBox <gi...@apache.org>.
Yikun commented on PR #38145:
URL: https://github.com/apache/spark/pull/38145#issuecomment-1271110996

   cc @HyukjinKwon 


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org

[GitHub] [spark] HyukjinKwon closed pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Posted by GitBox <gi...@apache.org>.
HyukjinKwon closed pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows
URL: https://github.com/apache/spark/pull/38145


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org

[GitHub] [spark] HyukjinKwon commented on pull request #38145: [SPARK-40696][INFRA][FOLLOWUP] Add packages write permisson for parent workflows

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on PR #38145:
URL: https://github.com/apache/spark/pull/38145#issuecomment-1271114710

   Merged to master.


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org