search

mesosphere / spark-build

Used to build the mesosphere/spark docker image and the DC/OS Spark package
52 stars 34 forks source link

[DCOS-58439][DS] Don't fail run.sh if previous infra components already installed #541

Closed alexeygorobets closed 5 years ago

alexeygorobets commented 5 years ago

What changes were proposed in this pull request?

Resolves DCOS-58439

How were these changes tested?

I have tested it manually: 1) Throw an error between Zookeeper and Kafka installations 2) Check if output infrastructure.json only contains Zookeeper 2) Rerun script without throwing an error 3) Check if output infrastructure.json contains all services (Zookeeper, Kafka, Cassandra)

akirillov commented 5 years ago

light ping @alexeygorobets and @mpereira: how shall we proceed with this PR?

akirillov commented 5 years ago

ping @alexeygorobets and @mpereira: how shall we proceed with this PR?

mpereira commented 5 years ago

Let's close this one for now.