)]}'
{"id":"openstack%2Fdevstack-plugin-ceph~956417","triplet_id":"openstack%2Fdevstack-plugin-ceph~master~I6340ec1dde49f7b6da351f98ee369bc438b1d117","project":"openstack/devstack-plugin-ceph","branch":"master","attention_set":{"38153":{"account":{"_account_id":38153,"name":"Tobias Haslop","display_name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","username":"tobias-haslop"},"last_update":"2025-08-04 19:41:33.000000000","reason":"\u003cGERRIT_ACCOUNT_16643\u003e replied on the change","reason_account":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"}},"10459":{"account":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"last_update":"2025-08-04 19:41:33.000000000","reason":"\u003cGERRIT_ACCOUNT_16643\u003e replied on the change","reason_account":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"}}},"removed_from_attention_set":{},"hashtags":[],"change_id":"I6340ec1dde49f7b6da351f98ee369bc438b1d117","subject":"Fix mixup of RGW_PORT with obsolete CEPH_RGW_PORT","status":"NEW","created":"2025-08-02 02:08:35.000000000","updated":"2025-08-04 19:41:33.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":2,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"dd4195ddc230444084a3b517ff54c478cf06e703","_number":956417,"virtual_id_number":956417,"owner":{"_account_id":38153,"name":"Tobias Haslop","display_name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","username":"tobias-haslop"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2025-08-02 18:38:16.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"value":2,"date":"2025-08-04 19:41:33.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-08-02 04:37:04.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2025-08-04 19:41:33.000000000","updated_by":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"reviewer":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"state":"REVIEWER"},{"updated":"2025-08-04 19:41:33.000000000","updated_by":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"reviewer":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"state":"REVIEWER"}],"messages":[{"id":"691885641ba9117865ac74068f4fc01f0abe9219","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":38153,"name":"Tobias Haslop","display_name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","username":"tobias-haslop"},"date":"2025-08-02 02:08:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"837259f19de128d97f0073a69dc0f8b3ea8feab0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-08-02 04:37:04.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/5ce21dac9ffd4f1eacefea1ace7a9de6\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/e16d3353bb4a44489db8a7bc75222971 : SUCCESS in 1h 24m 30s\n- devstack-plugin-ceph-multinode-tempest-py3 https://zuul.opendev.org/t/openstack/build/c5d2e97b57d444b7ae65a2ffb80296a0 : SUCCESS in 2h 19m 59s\n- devstack-plugin-ceph-multinode-cephfs-nfs-cephadm https://zuul.opendev.org/t/openstack/build/8adc605411194368969954d89af33496 : SUCCESS in 1h 15m 15s (non-voting)\n- devstack-plugin-ceph-cephfs-native https://zuul.opendev.org/t/openstack/build/afe9d33fb53e46a3a2b622ee02bb9e1b : FAILURE in 57m 33s (non-voting)\n- devstack-plugin-ceph-cephfs-nfs-standalone https://zuul.opendev.org/t/openstack/build/9fec73ecb2d7424bb96b1519fb7640aa : FAILURE in 42m 57s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/17e0812568554124b2aae869173bcf2f : SUCCESS in 3m 38s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/f297cf80facf401ba7aeb3094b020b30 : SUCCESS in 2m 42s","accounts_in_message":[],"_revision_number":1},{"id":"36a992b2e0a85f3226689caf4dc269d02e868195","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":38153,"name":"Tobias Haslop","display_name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","username":"tobias-haslop"},"date":"2025-08-02 16:07:54.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.\n\nOutdated Votes:\n* Verified+1\n","accounts_in_message":[],"_revision_number":2},{"id":"6612d841ab366337be9e9263948877374e444fae","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-08-02 18:38:16.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/7e84f542455749e2aa9cec4786bb6f3d\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/fc6d26dd5e1e465e9d8130fe1cd5e2fd : SUCCESS in 1h 32m 55s\n- devstack-plugin-ceph-multinode-tempest-py3 https://zuul.opendev.org/t/openstack/build/6724eb5fc4484277a4f7b55161453277 : SUCCESS in 2h 25m 35s\n- devstack-plugin-ceph-multinode-cephfs-nfs-cephadm https://zuul.opendev.org/t/openstack/build/b0c79a385de34da292a938d7a783fa6a : SUCCESS in 1h 36m 32s (non-voting)\n- devstack-plugin-ceph-cephfs-native https://zuul.opendev.org/t/openstack/build/fb7d170ef25b4e58bbc579a4f2320f07 : SUCCESS in 36m 23s (non-voting)\n- devstack-plugin-ceph-cephfs-nfs-standalone https://zuul.opendev.org/t/openstack/build/d82ba3387f744622b586e9018acda48d : SUCCESS in 34m 49s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/dba4a59ded4942efb94d1258b1cdeea4 : SUCCESS in 5m 59s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/342782f9518748ff87e79704d27e0d6d : SUCCESS in 3m 09s","accounts_in_message":[],"_revision_number":2},{"id":"dd4195ddc230444084a3b517ff54c478cf06e703","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2025-08-04 19:41:33.000000000","message":"Patch Set 2: Code-Review+2\n\n(2 comments)","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"159327785df58b03314ad9b65a8ec930d1c074e3","revisions":{"ddee93b802902f773674515cf972a1b8910f5bbd":{"kind":"REWORK","_number":1,"created":"2025-08-02 02:08:35.000000000","uploader":{"_account_id":38153,"name":"Tobias Haslop","display_name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","username":"tobias-haslop"},"ref":"refs/changes/17/956417/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/17/956417/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/1"}}},"commit":{"parents":[{"commit":"0cc01e63dbba41d943ff5fb50ae08828efd89509","subject":"Fix orch apply rgw call","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/0cc01e63dbba41d943ff5fb50ae08828efd89509"}]}],"author":{"name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","date":"2025-08-01 22:25:14.000000000","tz":120},"committer":{"name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","date":"2025-08-02 01:37:05.000000000","tz":120},"subject":"Fix mixup of RGW_PORT with obsolete CEPH_RGW_PORT","message":"Fix mixup of RGW_PORT with obsolete CEPH_RGW_PORT\n\nWhen the cephadm script was introduced as an alternative to the ceph\nscript in bba01bdd2f7c50635bb7656e8c043064b422e085, the ceph script\nalready defined and used a CEPH_RGW_PORT variable. The new cephadm\nscript defined and used another variable RGW_PORT. This change fixes a\nusage of CEPH_RGW_PORT in the cephadm script by using RGW_PORT instead.\nAfaict, this usage has always been faulty, because the ceph script and\nthe cephadm script were alternatives, i.e. one or the other would be\nsourced but not both. It would have been overlooked if one defined\nCEPH_RGW_PORT (and RGW_PORT) with the same value in localrc instead of\nletting the devstack-plugin-ceph define the variables.\n\nWhen the ceph script was removed in\n784716d949c64b3ee653f95004ae978c60d295c3, the faulty usage in the\ncephadm script became the only reference of CEPH_RGW_PORT.\n\nBoth CEPH_RGW_PORT and RGW_PORT had/have 8080 as their default value.\n\nChange-Id: I6340ec1dde49f7b6da351f98ee369bc438b1d117\nSigned-off-by: Tobias Haslop \u003ctobiashaslop+ubuntuone@hotmail.de\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ddee93b802902f773674515cf972a1b8910f5bbd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ddee93b802902f773674515cf972a1b8910f5bbd"}]},"branch":"refs/heads/master"},"159327785df58b03314ad9b65a8ec930d1c074e3":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2025-08-02 16:07:54.000000000","uploader":{"_account_id":38153,"name":"Tobias Haslop","display_name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","username":"tobias-haslop"},"ref":"refs/changes/17/956417/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/17/956417/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/17/956417/2"}}},"commit":{"parents":[{"commit":"98b16b0e9b846c5baed79b2ce77382e267780839","subject":"Fix orch apply rgw call","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/98b16b0e9b846c5baed79b2ce77382e267780839"}]}],"author":{"name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","date":"2025-08-01 22:25:14.000000000","tz":120},"committer":{"name":"Tobias Haslop","email":"tobiashaslop+ubuntuone@hotmail.de","date":"2025-08-02 15:05:43.000000000","tz":120},"subject":"Fix mixup of RGW_PORT with obsolete CEPH_RGW_PORT","message":"Fix mixup of RGW_PORT with obsolete CEPH_RGW_PORT\n\nWhen the cephadm script was introduced as an alternative to the ceph\nscript in bba01bdd2f7c50635bb7656e8c043064b422e085, the ceph script\nalready defined and used a CEPH_RGW_PORT variable. The new cephadm\nscript defined and used another variable RGW_PORT. This change fixes a\nusage of CEPH_RGW_PORT in the cephadm script by using RGW_PORT instead.\nAfaict, this usage has always been faulty, because the ceph script and\nthe cephadm script were alternatives, i.e. one or the other would be\nsourced but not both. It would have been overlooked if one defined\nCEPH_RGW_PORT (and RGW_PORT) with the same value in localrc instead of\nletting the devstack-plugin-ceph define the variables.\n\nWhen the ceph script was removed in\n784716d949c64b3ee653f95004ae978c60d295c3, the faulty usage in the\ncephadm script became the only reference of CEPH_RGW_PORT.\n\nBoth CEPH_RGW_PORT and RGW_PORT had/have 8080 as their default value.\n\nChange-Id: I6340ec1dde49f7b6da351f98ee369bc438b1d117\nSigned-off-by: Tobias Haslop \u003ctobiashaslop+ubuntuone@hotmail.de\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/159327785df58b03314ad9b65a8ec930d1c074e3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/159327785df58b03314ad9b65a8ec930d1c074e3"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"}},{"label":"Workflow","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}