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.

Sign up for GitHub

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

Jump to bottom

Definitions export from management UI does not include vhost metadata #10515

Open
lukebakken opened this issue Feb 7, 2024 · 2 comments · May be fixed by #10539
Open

Definitions export from management UI does not include vhost metadata #10515

lukebakken opened this issue Feb 7, 2024 · 2 comments · May be fixed by #10539
Assignees
@michaelklishin @lukebakken
Labels
bug

Comments

@lukebakken
Copy link
Collaborator

lukebakken commented Feb 7, 2024 •
edited

https://vmware.slack.com/archives/C0RDGG81Z/p1707224857559109

Reproduction steps:

Create vhost

rabbitmqctl add_vhost test --default-queue-type quorum

Export using CLI

rabbitmqctl export_definitions test-vhost-with-queue-type-from-cli.json

Note that the above file includes vhost metadata:

    "vhosts": [
        {
            "limits": [],
            "metadata": {
                "description": "Default virtual host",
                "tags": []
            },
            "name": "/"
        },
        {
            "limits": [],
            "metadata": {
                "description": "",
                "tags": [],
                "default_queue_type": "quorum"
            },
            "name": "test"
        }
    ],

Export using management UI

Note that the exported file does NOT contain metadata:

    "vhosts": [
        {
            "name": "/"
        },
        {
            "name": "test"
        }
    ],
@lukebakken lukebakken added the bug label Feb 7, 2024
@lukebakken lukebakken mentioned this issue Feb 7, 2024
Open
@michaelklishin
Copy link
Member

@lukebakken thank you for digging in and providing the details.

@lukebakken
Copy link
Collaborator Author

cc @pvladMQ

michaelklishin added a commit that referenced this issue Feb 12, 2024
@michaelklishin
Definition export: include virtual host metadata
2ca0648 
when exporting over the HTTP API. CLI tools
use rabbit_definitions:all_definitions/0 which
already include the metadata.

Closes #10515
@michaelklishin michaelklishin linked a pull request Feb 12, 2024 that will close this issue
Draft
michaelklishin added a commit that referenced this issue Feb 12, 2024
@michaelklishin
Add a test (assertion) for #10515
1f7c6a0
michaelklishin added a commit that referenced this issue Feb 13, 2024
@michaelklishin
Definition export: include virtual host metadata
64227ba 
when exporting over the HTTP API. CLI tools
use rabbit_definitions:all_definitions/0 which
already include the metadata.

Closes #10515
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michaelklishin michaelklishin

@lukebakken lukebakken

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

WIP Definition export: include virtual host metadata rabbitmq/rabbitmq-server
2 participants
@michaelklishin @lukebakken