You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by GitBox <gi...@apache.org> on 2022/08/23 14:02:39 UTC

[GitHub] [incubator-devlake] e2corporation opened a new pull request, #2808: fix: add missing pipeline var with logfile download

e2corporation opened a new pull request, #2808:
URL: https://github.com/apache/incubator-devlake/pull/2808

   ### ⚠️ Config-UI / Blueprints / Activity Status / **Logfile Download**
   
   - [ ] `Fix` Resolve missing `pipeline` object var when async `download` handler is called
   - [ ] `Fix` Relay `404` `message` from API when pipeline not found 
   - [ ] Test Logfile Download
   
   ### Description
   
   This PR resolves an issue with the Pipeline Logfile Download process where Pipeline ID defaulted to `0` due to missing pipeline object as argument.
   
   ### Does this close any open issues?
   #2765 
   
   


-- 
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@devlake.apache.org

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


[GitHub] [incubator-devlake] klesh merged pull request #2808: fix: add missing pipeline var with logfile download

Posted by GitBox <gi...@apache.org>.
klesh merged PR #2808:
URL: https://github.com/apache/incubator-devlake/pull/2808


-- 
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@devlake.apache.org

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


[GitHub] [incubator-devlake] e2corporation commented on pull request #2808: fix: add missing pipeline var with logfile download

Posted by GitBox <gi...@apache.org>.
e2corporation commented on PR #2808:
URL: https://github.com/apache/incubator-devlake/pull/2808#issuecomment-1225651895

   @klesh Please perform a local download test for pipeline log.


-- 
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@devlake.apache.org

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


[GitHub] [incubator-devlake] e2corporation commented on pull request #2808: fix: add missing pipeline var with logfile download

Posted by GitBox <gi...@apache.org>.
e2corporation commented on PR #2808:
URL: https://github.com/apache/incubator-devlake/pull/2808#issuecomment-1226680803

   @klesh This PR is ready for review & merge to main.


-- 
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@devlake.apache.org

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