Skip to content
This repository has been archived by the owner on Nov 17, 2020. It is now read-only.

Respect management.load_definitions for backwards compatibility #818

Closed
michaelklishin opened this issue Jun 9, 2020 · 0 comments · Fixed by #819
Closed

Respect management.load_definitions for backwards compatibility #818

michaelklishin opened this issue Jun 9, 2020 · 0 comments · Fixed by #819
Assignees
Labels
Milestone

Comments

@michaelklishin
Copy link
Member

Ever since definition load on boot has moved into the core, management.load_definitions is unintentionally ignored. Instead of trying to work around this in the core we should do it in this plugin, e.g. with an additional boot step.

@michaelklishin michaelklishin added this to the 3.8.5 milestone Jun 9, 2020
@michaelklishin michaelklishin self-assigned this Jun 9, 2020
michaelklishin added a commit that referenced this issue Jun 9, 2020
Previously we tried to rely on rabbitmq_management env
being loaded in the core. That is suboptimal and has
unintentional side effects.

References #2368, #2373.
Closes #818.
michaelklishin added a commit that referenced this issue Jun 9, 2020
Previously we tried to rely on rabbitmq_management env
being loaded in the core. That is suboptimal and has
unintentional side effects.

References rabbitmq/rabbitmq-management#2368, rabbitmq/rabbitmq-management#2373.
Closes #818.
michaelklishin added a commit that referenced this issue Jun 9, 2020
Previously we tried to rely on rabbitmq_management env
being loaded in the core. That is suboptimal and has
unintentional side effects.

References rabbitmq/rabbitmq-management#2368, rabbitmq/rabbitmq-management#2373.
Closes #818.
michaelklishin added a commit to rabbitmq/rabbitmq-server that referenced this issue Jun 9, 2020
michaelklishin added a commit to rabbitmq/rabbitmq-server that referenced this issue Jun 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant