)]}'
{"id":"openstack%2Fdevstack-plugin-ceph~810059","triplet_id":"openstack%2Fdevstack-plugin-ceph~stable%2Fwallaby~I6cd5424f023d2a895f8ed259d21944490bc1de5f","project":"openstack/devstack-plugin-ceph","branch":"stable/wallaby","topic":"ceph-pacific-stable/wallaby","hashtags":[],"change_id":"I6cd5424f023d2a895f8ed259d21944490bc1de5f","subject":"Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs","status":"MERGED","created":"2021-09-21 02:22:48.000000000","updated":"2021-10-11 23:25:19.000000000","submitted":"2021-10-11 23:20:52.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"810059-ceph-pacific-stable/wallaby","cherry_pick_of_change":807706,"cherry_pick_of_patch_set":4,"meta_rev_id":"14098c0f2c2b4a07ac1ff1301a41b13ce25f9dc5","_number":810059,"virtual_id_number":810059,"owner":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-10-11 23:20:51.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-09-30 13:52:30.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":2,"date":"2021-10-06 15:11:51.000000000","_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},{"value":1,"date":"2021-09-30 03:08:01.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"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":{"approved":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":1,"date":"2021-10-06 15:11:51.000000000","_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},{"value":0,"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"}],"reviewers":{"CC":[{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"}],"REVIEWER":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-09-21 02:22:48.000000000","updated_by":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"reviewer":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"state":"REVIEWER"},{"updated":"2021-09-21 04:14:24.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":"2021-09-21 14:08:27.000000000","updated_by":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"reviewer":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"state":"CC"},{"updated":"2021-09-28 19:47:27.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"CC"},{"updated":"2021-09-28 19:47:27.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2021-09-30 13:52:30.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2021-10-06 15:11:51.000000000","updated_by":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"reviewer":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"state":"REVIEWER"}],"messages":[{"id":"503d558b8b3c47053f73b34995720299fc2fd7ec","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2021-09-21 02:22:48.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"32a65de56445afe970534971173211c491c77b50","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-09-21 04:14:24.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/858b762164c74fbc85c2504ff1cc9c39 : FAILURE in 18m 27s\n- devstack-plugin-ceph-cephfs-native https://zuul.opendev.org/t/openstack/build/1b48dd90e3674d26abbba2a4bca2644a : SUCCESS in 1h 30m 01s (non-voting)\n- devstack-plugin-ceph-cephfs-nfs https://zuul.opendev.org/t/openstack/build/6c966ea41e6b49c3bdff60b795193acf : SUCCESS in 1h 46m 38s (non-voting)\n- devstack-plugin-ceph-tempest-fedora-latest https://zuul.opendev.org/t/openstack/build/75d669405b164be6adf8f9f8bbc3a14d : FAILURE in 17m 08s (non-voting)\n- devstack-plugin-ceph-multinode-tempest-py3 https://zuul.opendev.org/t/openstack/build/b722078de63745749bf8db2cec69e9f0 : SUCCESS in 1h 20m 09s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/58e357a47d7848949925c3fd845f3c98 : SUCCESS in 10m 37s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/d62b2d5591e84b6a805733b4ac98c8d6 : SUCCESS in 3m 19s","accounts_in_message":[],"_revision_number":1},{"id":"4e721b0a1263eae7a5372028f7b54e1708ecc45b","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2021-09-21 22:12:42.000000000","message":"Patch Set 1:\n\nLooks like we need a backport of https://review.opendev.org/c/openstack/devstack/+/810202","accounts_in_message":[],"_revision_number":1},{"id":"519b4763f844ef9e42180581ba1f556def2d9d81","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2021-09-21 22:13:57.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"2ae1b8fa0be3dcbe89457a4f98af2a09543a58f7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-09-22 00:36:14.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/d4189a44bfdc4b688902f79d4f1c02a6 : SUCCESS in 1h 25m 14s\n- devstack-plugin-ceph-cephfs-native https://zuul.opendev.org/t/openstack/build/233b0ee2db254addb5c3f49c30633cbc : SUCCESS in 1h 18m 21s (non-voting)\n- devstack-plugin-ceph-cephfs-nfs https://zuul.opendev.org/t/openstack/build/9a1c61be7a254402894a6c0c8fd276eb : SUCCESS in 1h 31m 08s (non-voting)\n- devstack-plugin-ceph-tempest-fedora-latest https://zuul.opendev.org/t/openstack/build/a3b5cb8b591f4a158e65172ac647692d : FAILURE in 11m 15s (non-voting)\n- devstack-plugin-ceph-multinode-tempest-py3 https://zuul.opendev.org/t/openstack/build/c1d09e4bb3d1438a889b9703462d8693 : SUCCESS in 1h 37m 35s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/18acf06fd81d4513af9bd9703ed7b959 : SUCCESS in 6m 59s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/2ef1676ee8b3465b811e75613387aaab : SUCCESS in 4m 35s","accounts_in_message":[],"_revision_number":2},{"id":"00d7365d96f1a16869170ec69a4d0d928bb104eb","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-09-29 01:11:27.000000000","message":"Patch Set 2:\n\nBased on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\nI don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].","accounts_in_message":[],"_revision_number":2},{"id":"413cf08d6e9c10d769273c63b0cc46b4c5f69b47","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2021-09-29 17:11:57.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \n\u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\nAck - i could use help establishing if this will break stable branch jobs :)\n\nThe rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;","accounts_in_message":[],"_revision_number":2},{"id":"6f2841e83ed15e855b821b44612ef1d7093407fe","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-09-29 17:20:14.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \u003e \n\u003e \u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\u003e \n\u003e Ack - i could use help establishing if this will break stable branch jobs :)\n\u003e \n\u003e The rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;\n\n+1, let\u0027s add a nova DNM patch to test it. or I think we should add nova-ceph-mulitstore job in this plugin gate too?","accounts_in_message":[],"_revision_number":2},{"id":"f64ad50860549256d7ca20c03533a9c72e65a772","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-09-29 20:35:47.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \n\u003e \u003e \u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \u003e \u003e \n\u003e \u003e \u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\u003e \u003e \n\u003e \u003e Ack - i could use help establishing if this will break stable branch jobs :)\n\u003e \u003e \n\u003e \u003e The rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;\n\u003e \n\u003e +1, let\u0027s add a nova DNM patch to test it. or I think we should add nova-ceph-mulitstore job in this plugin gate too?\n\nRe: adding a job to this plugin\u0027s gate, doesn\u0027t running the existing devstack-plugin-ceph-tempest-py3 accomplish the same thing (installing N version of ceph)? IIUC only the stable jobs got broken by the recent incident, so maybe it should be that this plugin\u0027s gate could add stable versions of the existing devstack-plugin-ceph-tempest-py3 job, similar to how tempest does it [1]?\n\n[1] https://github.com/openstack/tempest/blob/669900f622ce46c62260910d2cf9fa1d32216191/zuul.d/stable-jobs.yaml","accounts_in_message":[],"_revision_number":2},{"id":"2a055c321fafab7f3cac6218176ccaf0952eedd2","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-09-29 20:39:50.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\u003e \u003e \u003e \n\u003e \u003e \u003e Ack - i could use help establishing if this will break stable branch jobs :)\n\u003e \u003e \u003e \n\u003e \u003e \u003e The rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;\n\u003e \u003e \n\u003e \u003e +1, let\u0027s add a nova DNM patch to test it. or I think we should add nova-ceph-mulitstore job in this plugin gate too?\n\u003e \n\u003e Re: adding a job to this plugin\u0027s gate, doesn\u0027t running the existing devstack-plugin-ceph-tempest-py3 accomplish the same thing (installing N version of ceph)? IIUC only the stable jobs got broken by the recent incident, so maybe it should be that this plugin\u0027s gate could add stable versions of the existing devstack-plugin-ceph-tempest-py3 job, similar to how tempest does it [1]?\n\u003e \n\u003e [1] https://github.com/openstack/tempest/blob/669900f622ce46c62260910d2cf9fa1d32216191/zuul.d/stable-jobs.yaml\n\nI see, as we fixed the master zuul configuration now we should not face such break. Tempest testing the stable job is because of tempest being branchless. Before devstack-plugin-ceph was branchless we used to have those stable job on master gate.\n\nI think things should be ok now but let\u0027s see in future if anything extra jobs we need to run on master.","accounts_in_message":[],"_revision_number":2},{"id":"8c91ed84b4bacc3abf5e51007004ec83e0d5f78c","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-09-29 20:46:37.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Ack - i could use help establishing if this will break stable branch jobs :)\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e The rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;\n\u003e \u003e \u003e \n\u003e \u003e \u003e +1, let\u0027s add a nova DNM patch to test it. or I think we should add nova-ceph-mulitstore job in this plugin gate too?\n\u003e \u003e \n\u003e \u003e Re: adding a job to this plugin\u0027s gate, doesn\u0027t running the existing devstack-plugin-ceph-tempest-py3 accomplish the same thing (installing N version of ceph)? IIUC only the stable jobs got broken by the recent incident, so maybe it should be that this plugin\u0027s gate could add stable versions of the existing devstack-plugin-ceph-tempest-py3 job, similar to how tempest does it [1]?\n\u003e \u003e \n\u003e \u003e [1] https://github.com/openstack/tempest/blob/669900f622ce46c62260910d2cf9fa1d32216191/zuul.d/stable-jobs.yaml\n\u003e \n\u003e I see, as we fixed the master zuul configuration now we should not face such break. Tempest testing the stable job is because of tempest being branchless. Before devstack-plugin-ceph was branchless we used to have those stable job on master gate.\n\u003e \n\u003e I think things should be ok now but let\u0027s see in future if anything extra jobs we need to run on master.\n\nAh, right. This was a strange case where we have branching *but* older branches were able to pick up configs from master ... Maybe for now we just wait and see, as you rightly point out, it is not typical and should not be necessary to run stable jobs on master for a branched repo.","accounts_in_message":[],"_revision_number":2},{"id":"8ab6c12ef2eba1ad772ad3a30d4166904f03672a","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2021-09-30 02:52:27.000000000","message":"Patch Set 2:\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e Ack - i could use help establishing if this will break stable branch jobs :)\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e The rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e +1, let\u0027s add a nova DNM patch to test it. or I think we should add nova-ceph-mulitstore job in this plugin gate too?\n\u003e \u003e \u003e \n\u003e \u003e \u003e Re: adding a job to this plugin\u0027s gate, doesn\u0027t running the existing devstack-plugin-ceph-tempest-py3 accomplish the same thing (installing N version of ceph)? IIUC only the stable jobs got broken by the recent incident, so maybe it should be that this plugin\u0027s gate could add stable versions of the existing devstack-plugin-ceph-tempest-py3 job, similar to how tempest does it [1]?\n\u003e \u003e \u003e \n\u003e \u003e \u003e [1] https://github.com/openstack/tempest/blob/669900f622ce46c62260910d2cf9fa1d32216191/zuul.d/stable-jobs.yaml\n\u003e \u003e \n\u003e \u003e I see, as we fixed the master zuul configuration now we should not face such break. Tempest testing the stable job is because of tempest being branchless. Before devstack-plugin-ceph was branchless we used to have those stable job on master gate.\n\u003e \u003e \n\u003e \u003e I think things should be ok now but let\u0027s see in future if anything extra jobs we need to run on master.\n\u003e \n\u003e Ah, right. This was a strange case where we have branching *but* older branches were able to pick up configs from master ... Maybe for now we just wait and see, as you rightly point out, it is not typical and should not be necessary to run stable jobs on master for a branched repo.\n\n\nThe \"nova-ceph-multistore\" job passed on the stable/wallaby branch with this change.\n\nhttps://zuul.opendev.org/t/openstack/build/c8b86c23753a40aaa112290497f0efff/log/controller/logs/ceph/ceph/ceph-osd.0_log.txt#2","accounts_in_message":[],"_revision_number":2},{"id":"a51eb1cf93d9691032c6c86ad7183aa306eec051","author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"date":"2021-09-30 03:08:01.000000000","message":"Patch Set 2: Code-Review+1\n\n\u003e Patch Set 2:\n\u003e \n\u003e \u003e Patch Set 2:\n\u003e \u003e \n\u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e \u003e Patch Set 2:\n\u003e \u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e \u003e Based on the discussion on the master change, this looks good for using pacific on stable/wallaby.\n\u003e \u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e \u003e I don\u0027t know enough about zuul to understand whether this has potential to break any nova stable branch jobs [after https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/811478 merges].\n\u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e Ack - i could use help establishing if this will break stable branch jobs :)\n\u003e \u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e \u003e The rationale for this change is that Pacific is the latest supported stable branch of ceph; while nautilus is now eol (https://docs.ceph.com/en/latest/releases/). Since wallaby is relatively recent, we should be to either test with Octopus or Pacific releases of Ceph;\n\u003e \u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e \u003e +1, let\u0027s add a nova DNM patch to test it. or I think we should add nova-ceph-mulitstore job in this plugin gate too?\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e Re: adding a job to this plugin\u0027s gate, doesn\u0027t running the existing devstack-plugin-ceph-tempest-py3 accomplish the same thing (installing N version of ceph)? IIUC only the stable jobs got broken by the recent incident, so maybe it should be that this plugin\u0027s gate could add stable versions of the existing devstack-plugin-ceph-tempest-py3 job, similar to how tempest does it [1]?\n\u003e \u003e \u003e \u003e \n\u003e \u003e \u003e \u003e [1] https://github.com/openstack/tempest/blob/669900f622ce46c62260910d2cf9fa1d32216191/zuul.d/stable-jobs.yaml\n\u003e \u003e \u003e \n\u003e \u003e \u003e I see, as we fixed the master zuul configuration now we should not face such break. Tempest testing the stable job is because of tempest being branchless. Before devstack-plugin-ceph was branchless we used to have those stable job on master gate.\n\u003e \u003e \u003e \n\u003e \u003e \u003e I think things should be ok now but let\u0027s see in future if anything extra jobs we need to run on master.\n\u003e \u003e \n\u003e \u003e Ah, right. This was a strange case where we have branching *but* older branches were able to pick up configs from master ... Maybe for now we just wait and see, as you rightly point out, it is not typical and should not be necessary to run stable jobs on master for a branched repo.\n\u003e \n\u003e \n\u003e The \"nova-ceph-multistore\" job passed on the stable/wallaby branch with this change.\n\u003e \n\u003e https://zuul.opendev.org/t/openstack/build/c8b86c23753a40aaa112290497f0efff/log/controller/logs/ceph/ceph/ceph-osd.0_log.txt#2\n\nExcellent, thank you Goutham! LGTM","accounts_in_message":[],"_revision_number":2},{"id":"0d5a601faed766bd113ee3c5b4c4a7c002b6e6c9","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2021-09-30 13:52:30.000000000","message":"Patch Set 2: Code-Review+2\n\nyeah and it is using pacific version as expected - https://zuul.opendev.org/t/openstack/build/c8b86c23753a40aaa112290497f0efff/log/job-output.txt#4910\n\nThanks Goutham","accounts_in_message":[],"_revision_number":2},{"id":"34d6d5bde1d74cc5817a31e62daa0810fd043251","author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"date":"2021-10-06 15:11:51.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\neverything seems ok, lgtm","accounts_in_message":[],"_revision_number":2},{"id":"8ad059f5f9efca61a9203ec15abf03a03d9999bc","author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"date":"2021-10-11 20:06:57.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"e6e1ab2487ce70d04e4c3fbf0070226f3087bed3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-11 21:58:11.000000000","message":"Patch Set 2:\n\nBuild succeeded (check pipeline).\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/ecfc776d626c4d048b55304f29514ffa : SUCCESS in 1h 31m 37s\n- devstack-plugin-ceph-cephfs-native https://zuul.opendev.org/t/openstack/build/547fed0b6f414a5992591eaf44fe6c39 : FAILURE in 1h 45m 37s (non-voting)\n- devstack-plugin-ceph-cephfs-nfs https://zuul.opendev.org/t/openstack/build/ff0433436e8341ecb99fc9d89f25f80b : SUCCESS in 1h 46m 10s (non-voting)\n- devstack-plugin-ceph-tempest-fedora-latest https://zuul.opendev.org/t/openstack/build/494fa27d76da48299d0ccb9cb0ca05fb : FAILURE in 12m 36s (non-voting)\n- devstack-plugin-ceph-multinode-tempest-py3 https://zuul.opendev.org/t/openstack/build/c775882cf97c459eaa171ee480b3d6ad : FAILURE in 1h 32m 00s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b9063beec7b9413e9bd4e6fffe202031 : SUCCESS in 6m 00s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/e2de6bd07f9146458ea0657c303950b9 : SUCCESS in 3m 14s","accounts_in_message":[],"_revision_number":2},{"id":"aec02f8991cfdd45fdbc6b83bcf31db1445084b0","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-11 21:58:56.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"79040ee053caa7ceb26da34095b8df3285b14b0e","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-11 23:20:51.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/40c368aedca745578419e13866aa70df : SUCCESS in 1h 17m 18s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/036a9efa34c74841a8b7cbc331dff61a : SUCCESS in 5m 35s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/9637b68aee21494d9d0c1779c7fc7453 : SUCCESS in 3m 21s","accounts_in_message":[],"_revision_number":2},{"id":"6b4fd58865cdf21f3b4475fc65e1006204070863","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-11 23:20:52.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":2},{"id":"14098c0f2c2b4a07ac1ff1301a41b13ce25f9dc5","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-10-11 23:25:19.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b56338316bac40e1b3e475fbfaef13b4 : SUCCESS in 56s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"447ec0965162f57a5e90a73b399358a11838c405","revisions":{"ef50e1c2f7751363c33106df7f59011da2af3c8d":{"kind":"REWORK","_number":1,"created":"2021-09-21 02:22:48.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/59/810059/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/59/810059/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/1"}}},"commit":{"parents":[{"commit":"78f6900a8784c63e8f4141046d5c7a91ebdf6755","subject":"Remove devstack-gate test hooks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/78f6900a8784c63e8f4141046d5c7a91ebdf6755"}]}],"author":{"name":"Giulio Fidente","email":"gfidente@redhat.com","date":"2021-09-07 11:23:20.000000000","tz":120},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2021-09-21 02:22:48.000000000","tz":0},"subject":"Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs","message":"Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs\n\nMove tempest py3 jobs to latest released stable Ceph, which is\nPacific.\n\nChange-Id: I6cd5424f023d2a895f8ed259d21944490bc1de5f\n(cherry picked from commit ea0c1c768c99437504a1a6f2971dd0c5c1563ca5)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ef50e1c2f7751363c33106df7f59011da2af3c8d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ef50e1c2f7751363c33106df7f59011da2af3c8d"}]},"branch":"refs/heads/stable/wallaby"},"447ec0965162f57a5e90a73b399358a11838c405":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2021-09-21 22:13:57.000000000","uploader":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"ref":"refs/changes/59/810059/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/59/810059/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/59/810059/2"}}},"commit":{"parents":[{"commit":"78f6900a8784c63e8f4141046d5c7a91ebdf6755","subject":"Remove devstack-gate test hooks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/78f6900a8784c63e8f4141046d5c7a91ebdf6755"}]}],"author":{"name":"Giulio Fidente","email":"gfidente@redhat.com","date":"2021-09-07 11:23:20.000000000","tz":120},"committer":{"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","date":"2021-09-21 22:13:53.000000000","tz":0},"subject":"Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs","message":"Use Ceph Pacific for devstack-plugin-ceph-tempest-py3 jobs\n\nMove tempest py3 jobs to latest released stable Ceph, which is\nPacific.\n\nDepends-On: I5fa2105ceb3b97a4e38926d76c1e4028f1108d4a\nChange-Id: I6cd5424f023d2a895f8ed259d21944490bc1de5f\n(cherry picked from commit ea0c1c768c99437504a1a6f2971dd0c5c1563ca5)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/447ec0965162f57a5e90a73b399358a11838c405"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/447ec0965162f57a5e90a73b399358a11838c405"}]},"branch":"refs/heads/stable/wallaby","description":"Edit commit message"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}