coocoobau opened a new issue, #8624: URL: https://github.com/apache/incubator-devlake/issues/8624
### Search before asking - [x] I had searched in the [issues](https://github.com/apache/incubator-devlake/issues?q=is%3Aissue) and found no similar issues. ### What happened I was forced to select that I searched in the issues and found nothing similar but I did find. Issues #8148 and #8442 are related as well as PR #8583 which introduced the new behaviour while fixing the zero time. The new issue is for nullable `published_at` field. This would be null when the release is in Draft mode and the pipeline will fail. `backend/plugins/github/tasks/release_convertor.go:76 (PublishedAt: *githubRelease.PublishedAt)` is the culprit. Screenshot of db table `_tool_github_releases` <img width="770" height="119" alt="image" src="https://github.com/user-attachments/assets/7b33b5e0-3eb3-4ea7-8ac8-a0cc4265a606"; /> Stack Trace: `attached stack trace -- stack trace: | github.com/apache/incubator-devlake/server/services.runPipeline | /app/server/services/pipeline_runner.go:79 | [...repeated from below...] Wraps: (2) Error running pipeline 29. Wraps: (3) attached stack trace -- stack trace: | github.com/apache/incubator-devlake/server/services.RunTasksStandalone | /app/server/services/task.go:217 | github.com/apache/incubator-devlake/server/services.(*pipelineRunner).runPipelineStandalone.func1 | /app/server/services/pipeline_runner.go:42 | github.com/apache/incubator-devlake/core/runner.runPipelineTasks | /app/core/runner/run_pipeline.go:90 | github.com/apache/incubator-devlake/core/runner.RunPipeline | /app/core/runner/run_pipeline.go:54 | github.com/apache/incubator-devlake/server/services.(*pipelineRunner).runPipelineStandalone | /app/server/services/pipeline_runner.go:38 | github.com/apache/incubator-devlake/server/services.runPipeline | /app/server/services/pipeline_runner.go:76 | github.com/apache/in cubator-devlake/server/services.RunPipelineInQueue.func1 | /app/server/services/pipeline.go:371 | runtime.goexit | /usr/local/go/src/runtime/asm_amd64.s:1598 Wraps: (4) attached stack trace | -- stack trace: | | github.com/apache/incubator-devlake/server/services.RunTasksStandalone.func1 | | /app/server/services/task.go:189 | Wraps: (2) Error running task 265. | Wraps: (3) attached stack trace | -- stack trace: | | github.com/apache/incubator-devlake/core/runner.RunTask.func1 | | /app/core/runner/run_task.go:75 | | runtime.gopanic | | /usr/local/go/src/runtime/panic.go:884 | | runtime.panicmem | | /usr/local/go/src/runtime/panic.go:260 | | runtime.sigpanic | | /usr/local/go/src/runtime/signal_unix.go:841 | | github.com/apache/incubator-devlake/plugins/github/tasks.ConvertRelease.func1 | | /app/plugins/github/tasks/release_convertor.go:76 | | github.com/apache/incubator-devlake/helpers/pluginhelper/api.(*DataConverter).Execute | | /app/helpers/pluginhelper/api/data_convertor.go:105 | | github.com/apache/incubator-devlake/plugins/github/tasks.ConvertRelease | | /app/plugins/github/tasks/release_convertor.go:103 | | github.com/apache/incubator-devlake/core/runner.runSubtask | | /app/core/runner/run_task.go:425 | | github.com/apache/incubator-devlake/core/runner.RunPluginSubTasks | | /app/core/runner/run_task.go:330 | | github.com/apache/incubator-devlake/core/runner.RunPluginTask | | /app/core/runner/run_task.go:165 | | github.com/apache/incubator-devlake/core/runner.RunTask | | /app/core/runner/run_task.go:139 | | github.com/apache/incubator-devlake/server/services.runTaskStandalone | | /app/server/services/task_runner.go:114 | | github.com/apache/incubator-devlake/server/services.RunTasksStandalone.func1 | | /app/server/services/task.go:187 | | runtime.goexit | | /usr/local/go/src/runtime/asm_amd64.s:1598 | Wraps: (4) run task failed with panic (github.com/apache/incubator-devlake/plugins/github/tasks.ConvertRelease.func1:76) | Wraps: (5) runtime error: invalid memory address or nil pointer dereference | Error types: (1) *withstack.withStack (2) *errutil.withPrefix (3) *withstack.withStack (4) *errutil.withPrefix (5) runtime.errorString Error types: (1) *withstack.withStack (2) *errutil.withPrefix (3) *withstack.withStack (4) *errutil.leafError` ### What do you expect to happen As far as I am concerned, if `published_at` is `null` the entire row/release should be skipped as it is a draft release. ### How to reproduce 1. Create a GitHub release in Draft mode. 2. Execute the transform data pipeline. ### Anything else _No response_ ### Version v1.0.3-beta7 ### Are you willing to submit PR? - [ ] Yes I am willing to submit a PR! ### Code of Conduct - [x] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
