)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"94cf4de59ca2ffebd7db994c613a06c82ba1698f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5f0e753d_a670fe38","updated":"2023-06-27 17:06:45.000000000","message":"Bug #1735051 has been closed but this patch is still up.\n\nI set up a devstack and:\n\n$ openstack domain list\n+---------+---------+---------+--------------------+\n| ID      | Name    | Enabled | Description        |\n+---------+---------+---------+--------------------+\n| default | Default | True    | The default domain |\n+---------+---------+---------+--------------------+\n\nI think this patch is technically right (the id should be \"default\", lower d, not \"Default\", capital d). But the config works with:\n\n    project_domain_name \u003d Default\n    user_domain_name \u003d Default\n\nand:\n\n    project_domain_name \u003d default\n    user_domain_name \u003d default\n\nand:\n\n    project_domain_id \u003d default\n    user_domain_id \u003d default\n    \nBut not with:\n\n    project_domain_id \u003d Default\n    user_domain_id \u003d Default\n    \n\n$ openstack image list\nHttpException: 401: Client Error for url: http://192.168.121.98/image/v2/images, This server could not verify that you are authorized to access the document you requested. Either you supplied the wrong credentials (e.g., bad password), or your browser does not understand how to supply the credentials required.\u003cbr /\u003e\u003cbr /\u003e\n\n\ntl;dr: names are not case-sensitive, but ids are.","commit_id":"a4aa56f0eb474a45a3d83e53a53295b1d56a80b0"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"dac4dc57a023d197afa3f59fb22cabff261dc9c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"53b2a7a2_9bdf4fdf","updated":"2024-06-25 18:33:12.000000000","message":"Note to reviewers: the initial patch (6.5 years ago) fixed an issue that arose when using the wrong case (Default vs default, not the upper/lower d) as the project_domain_name. The way it fixed it was by switching to project_domain_id.\n\nThe current example works (as far as I know) but I think we might still want to use ids rather than names, as ids are unique, but I don\u0027t think names are.\n\nI don\u0027t have strong feelings about this, I\u0027m mostly trying to merge or abandon this patch and make sure it does not stay in the queue for a few more years :)","commit_id":"77ed805fd338e6126b74fe94f3a126819bfef15d"}]}