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

Remove base64 decoding/encoding of rabbitmq-server.conf for bosh cli v2 compat #63

Closed
MatthiasWinzeler opened this issue Aug 1, 2017 · 2 comments
Labels

Comments

@MatthiasWinzeler
Copy link

Hi guys

The property rabbitmq-server.config allows specifying additional rabbitmq server config.

Currently, it expects a base64 encoded string as described in the docs.

However, this relies on the bosh cli to interpret ERB expressions in the deployment manifest. This was the case for the old ruby cli, but the new golang cli (v2) does (intentionally) not support this:
cloudfoundry/bosh-cli#282

This forces bosh cli v2 users of this release to somehow (manually) preprocess the bosh deployment manifest before deploying (or just put base64 in the deployment manifest - either of it is not that nice) - it would be nicer if we could put the config as plain string, i.e. rabbitmq-server.raw_config or similar... Would this be possible?

I can't see why passing rabbitmq-server.config as a plain string could lead to issues, i.e.:

rabbitmq-server:
  config: | 
    [{rabbit, [{vm_memory_high_watermark,0.5}]}].
@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@SemanticallyNull
Copy link
Contributor

Hi @MatthiasWinzeler,

Thanks for your issue. We now recommend generating the Base64 encoded value outside of the BOSH interpolation process, rather than using ERB to generate it inline in the manifest.

Due to other dependencies we cannot change the way the release handles this input, and we also cannot add another property that could configure this at the moment.

Thanks,
@BenChapman & @dlresende

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

No branches or pull requests

3 participants