Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/rabbitmq] Rename RABBITMQ_VHOST #2853

Merged
merged 4 commits into from
Dec 7, 2017
Merged

Conversation

arthur-c
Copy link
Contributor

Rename RABBITMQ_VHOST to RABBITMQ_DEFAULT_VHOST to work with latest rabbitmq:3 images.
Source: https://hub.docker.com/_/rabbitmq/

Tested with helm version:

Client: &version.Version{SemVer:"v2.7.0", GitCommit:"08c1144f5eb3e3b636d9775617287cc26e53dba4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.7.0", GitCommit:"08c1144f5eb3e3b636d9775617287cc26e53dba4", GitTreeState:"clean"}

and it's working well

=INFO REPORT==== 24-Nov-2017::14:24:19 ===
Adding vhost 'myvhost'

=INFO REPORT==== 24-Nov-2017::14:24:19 ===
Creating user 'guest'

=INFO REPORT==== 24-Nov-2017::14:24:19 ===
Setting user tags for user 'guest' to [administrator]

=INFO REPORT==== 24-Nov-2017::14:24:19 ===
Setting permissions for 'guest' in 'myvhost' to '.*', '.*', '.*'

With RABBITMQ_VHOST it sticks to /vhost when rabbitmqVhostis set to myvhost

Rename RABBITMQ_VHOST to RABBITMQ_DEFAULT_VHOST to work with latest rabbitmq:3 images. 
Source: https://hub.docker.com/_/rabbitmq/
@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 24, 2017
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Nov 24, 2017
@arthur-c arthur-c changed the title Rename RABBITMQ_VHOST [stable/rabbitmq] Rename RABBITMQ_VHOST Nov 25, 2017
@unguiculus
Copy link
Member

/ok-to-test

@k8s-ci-robot k8s-ci-robot removed the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Dec 7, 2017
@unguiculus unguiculus self-assigned this Dec 7, 2017
@unguiculus
Copy link
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 7, 2017
@unguiculus unguiculus merged commit d8859b2 into helm:master Dec 7, 2017
@arthur-c arthur-c deleted the patch-1 branch December 9, 2017 17:03
@colingabr
Copy link
Contributor

colingabr commented Jan 18, 2018

This breaks vhosts for bitnami rabbitmq docker images, which are the default images for this chart. Refer to the bitnami repo https://github.com/bitnami/bitnami-docker-rabbitmq

  • RABBITMQ_VHOST: RabbitMQ application vhost. Default: /

There are no references to RABBITMQ_DEFAULT_VHOST

root@q-rabbitmq-781131022-nbjm1:/# env | grep "VHOST"
RABBITMQ_VHOST=/
RABBITMQ_DEFAULT_VHOST=/foo
root@q-rabbitmq-781131022-nbjm1:/# rabbitmqctl list_vhosts
warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell)
Listing vhosts ...
/

@colingabr
Copy link
Contributor

Here's the PR to restore this: #3363

intelfx pushed a commit to intelfx/charts that referenced this pull request Oct 19, 2018
* Rename RABBITMQ_VHOST

Rename RABBITMQ_VHOST to RABBITMQ_DEFAULT_VHOST to work with latest rabbitmq:3 images. 
Source: https://hub.docker.com/_/rabbitmq/

* Bump rabbitmq chart version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants