search

datastax / pulsar-ansible

Apache License 2.0
4 stars 9 forks source link

Pass path to pulsar binary to avoid bin not found on control node #17

Closed MMirelli closed 1 year ago

MMirelli commented 2 years ago

Fixes #16 by injecting an environment variable based on the global value of tgt_pulsar_inst_dir.

dave2wave commented 2 years ago

Some considerations

  1. The pulsar installation on the omb-runner is shared by all EVMT deployments.
  2. The nbs4j framework is adjusted to use the pulsar-admin on client[0].

I think we need to discuss control node requirements for Pulsar-Ansible and EVMT.

MMirelli commented 2 years ago

I think we need to discuss control node requirements for Pulsar-Ansible and EVMT.

I see your concern and I agree. IMO, it would be best to align the behaviour across the testing tools. So either we run pulsar binaries from the ansible control node or from a client, in this way we make sure we are using the same pulsar version and we reduce the complexity of the solution.

The pulsar installation on the omb-runner is shared by all EVMT deployments.

A nice feature would be to add a property in a global config file to select the pulsar version to use both on the server (VMs running pulsar) and client side (node where we are issuing pulsar-admin commands).

MMirelli commented 2 years ago

I would say this should be merged after https://github.com/datastax/pulsar-ansible/issues/24 is done.