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

External RabbitMQ issues partition issue which isn't true #374

Open
matthiasdeblock opened this issue Oct 25, 2017 · 4 comments
Open

External RabbitMQ issues partition issue which isn't true #374

matthiasdeblock opened this issue Oct 25, 2017 · 4 comments
Assignees
Labels
Milestone

Comments

@matthiasdeblock
Copy link
Contributor

matthiasdeblock commented Oct 25, 2017

Problem description

RabbitMQ is triggering a partition issue while it should trigger something else:

[INFO] Storagerouter Id: qsjZ10r5Wv2RBwEM
[INFO] Environment Os: Ubuntu 16.04 xenial
[INFO] Hostname: stor-01.ds1
[INFO] Cluster Id: IOip4i7eYCoH7W6f
[INFO] Storagerouter Type: MASTER
[INFO] Starting OpenvStorage Healthcheck version 3.4.0-1
[INFO] ======================
[INFO] Pre-check: verification of RabbitMQ cluster.
[FAILED] RabbitMQ has partition issues: STOP, RabbitMQ is not running.
[INFO] Recap of Health Check module ovs test verify-rabbitmq-test!
[INFO] ======================
[INFO] SUCCESS=0 FAILED=1 SKIPPED=0 WARNING=0 EXCEPTION=0

According to the code it should print another message when it is an externally managed RabbitMQ:

        if not RabbitMQ.INTERNAL:
            return 'UNKNOWN', "Unable to check the management plugin, this is not an internal RabbitMQ from ovs."

Workaround

Manually enable the rabbitmq_management worked for the externally managed rabbitMQ.

@matthiasdeblock matthiasdeblock changed the title External RabbitMQ issues partition issue which wasn't correct due to non enabled rabbitmq_management External RabbitMQ issues partition issue which isn't true Oct 25, 2017
@matthiasdeblock
Copy link
Contributor Author

@JeffreyDevloo , why is the check RabbitMQ.INTERNAL implemented? Can't we just make sure that it always enables the rabbitmq_management? Or was this for a certain reason?

@JeffreyDevloo
Copy link
Contributor

Legacy most likely

@wimpers
Copy link

wimpers commented Oct 25, 2017

Since the whole conecpt of the HC is that it only detects problems and doesn't make any changes, I propose following:

  • HC checks if internal or external RabbitMQ and reports accordingly
  • If enable_management is off , return UNKNOWN and the correct error "Unable to check the management plugin, this is not an internal RabbitMQ from ovs."

@wimpers
Copy link

wimpers commented Oct 25, 2017

After discussion with @matthiasdeblock , it seems that the HC will enable the setting if it was disabled in case of an internally managed RabbitMQ. This violates the basic principle that the HC doesn't change anything in your environment. So what needs to happen

  • In case internally managed the framework should during setup enable the plugin. (if not yet the case)
  • In case externally managed, the ansible script enabled the plugin https://github.com/openvstorage/dev_ops/issues/92
  • The path in the healthcheck to enbale the plugin should be removed
  • HC checks if internal or external RabbitMQ and reports accordingly
  • If enable_management is off , return UNKNOWN and the correct error "Unable to check the management plugin. Please enable the plugin."

@matthiasdeblock matthiasdeblock self-assigned this Nov 21, 2017
@wimpers wimpers added this to the J milestone Dec 14, 2017
@wimpers wimpers modified the milestones: J, K Jan 16, 2018
@wimpers wimpers modified the milestones: K, M Mar 6, 2018
@wimpers wimpers modified the milestones: M, Roadmap May 4, 2018
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