Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FLINK-17193] [k8s-python-example] build-example.sh improvements #99

Closed
wants to merge 2 commits into from

Conversation

aroch
Copy link
Contributor

@aroch aroch commented Apr 16, 2020

  • The helm template was printed out instead of being redirected to the resource YAML file.

  • In case the SDK distribution was not previously built - instead of failing the build, build the distribution and continue.

  • Abort script on failure. Previously, on failure the script would continue to run all steps and finish in an erroneous state. Also this saves 'if' statements to check exit codes after commands execute.

@sjwiesman sjwiesman assigned sjwiesman and unassigned sjwiesman Apr 16, 2020
@sjwiesman
Copy link
Contributor

Hi @aroch, thank you for fixing this. I will go ahead and merge it.

In the future, could you please open a Jira ticket for this type of change. It helps us keep track on of what is being worked on in the project.

@aroch aroch changed the title [k8s-python-example] build-example.sh improvements [FLINK-17193] [k8s-python-example] build-example.sh improvements Apr 16, 2020
@sjwiesman
Copy link
Contributor

Thank you!

@aroch
Copy link
Contributor Author

aroch commented Apr 16, 2020

Thanks @sjwiesman! Just opened an issue and updated accordingly. Was not completely sure about filling the JIRA. Would appreciate if you can take a look.

@sjwiesman sjwiesman closed this in 2ceb374 Apr 16, 2020
sjwiesman pushed a commit that referenced this pull request Apr 16, 2020
… distribution if was not previously built

This closes #99
@sjwiesman
Copy link
Contributor

Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants