Handle start_pipeline() when response is None / Invalid

[chirodip98]

closes: #50387
Refference : https://cdap.atlassian.net/wiki/spaces/DOCS/pages/477560983/Lifecycle+Microservices#Start-a-Program

[chirodip98]

I would need some assistance to run System Tests for DataFusion , As I dont have any running instance.

[chirodip98]

@MaksYermak Kindly review : )

[jscheffl]

Or @VladaZakharova ?

[shahar1]

I prefer first to revert the original PR (#60701) to fix it quickly, reopen the original issue, and then you could solve it in this one.
After reverting, this will create conflicts the current PR.
I'm aware that it's a bit extra work for you, but I think that it's better to start from a more stable state.

I'll cancel the "Request changes" once the original PR is reverted.

[chirodip98]

Sure @shahar1 once the revert is done I will start working on the same !

[shahar1]

Sure @shahar1 once the revert is done I will start working on the same !

Sounds great.
To avoid confusion - I'll reopen the original issue and assign you to that.

[shahar1]

Sure @shahar1 once the revert is done I will start working on the same !

Sounds great. To avoid confusion - I'll reopen the original issue and assign you to that.

@chirodip98 You're good to go and start implementing.

Maintainers - please do not merge this PR unless we're certain that it passes the system tests successfully.

[chirodip98]

Sure. I will start working, However I need some guidance on running the system tests. As I don't have any Data fusion instance , how to get around this?
@shahar1

[chirodip98]

report.html

I need help with the System tests!!!

[shahar1]

report.html

I need help with the System tests!!!

Hey, please allow some time to Google's provider team to reply - it might take some time (as it is still weekend), and there's no guarantee that it will happen fast or anytime soon. In general, I'd kindly ask not to rush maintainers or other stakeholders to review your PRs - people maintain this project voluntarily, and their time and attention should be respected.
You're always welcome to start working on other PRs, there are many good first issues to work on.

[VladaZakharova]

report.html
I need help with the System tests!!!

Hey, please allow some time to Google's provider team to reply - it might take some time (as it is still weekend), and there's no guarantee that it will happen fast or anytime soon. In general, I'd kindly ask not to rush maintainers or other stakeholders to review your PRs - people maintain this project voluntarily, and their time and attention should be respected. You're always welcome to start working on other PRs, there are many good first issues to work on.

thank you, @shahar1
I think your comment is a really helpful point for all the starting developers in the Community :)

@chirodip98
thank you for your PR, but I want to remind you that there are a lot of opened issues in the Community and Composer for us. When creating a PR it is mostly the responsibility of the owner of the PR to verify that the changes are working correctly. Although I am happy to help you testing them, there is a list of priorities
I will try to run your changes this week to verify is it is working as expected