)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"a707fefa832698f98fa7dd900f612e063a295ecd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0ee458cf_64dafab4","updated":"2022-08-10 11:54:56.000000000","message":"Not sure this is the right way or I miss any master related repo file, let me know what do you think.\nNote: this patch is related to https://review.opendev.org/c/openstack/tripleo-ci/+/852715 \nbut we should coordinate the way we bump master to quincy to avoid any CI breakage","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"53393816f5889119c605363e5e38f04b5a88c679","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1be89b01_584bc0d4","updated":"2022-08-10 15:17:28.000000000","message":"putting -W until we know which is the -stable-quincy upstream container that can be used in master","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"468ee843ea1998fe1d67e866c88de09be3fe56a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2cb320f7_8277763f","updated":"2022-08-12 07:43:30.000000000","message":"recheck","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f332e5613de5dbf1765f6279d4b24f85919023e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"385ff19e_48fbb5a5","updated":"2022-08-17 14:59:08.000000000","message":"As per discussions, master.yml and testing-hash-master.yml will be enough for our upstream jobs. Afte getting this change merged, we need proceed with ceph-daemon tag promotion. Once delerean-deps gets updated to provide quincy packages, we can revert this change.\nThanks Francesco.","commit_id":"a7c3728e4d021c706ebcb9716c0bec49b07b5a31"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"421db6b2814fb88b0c167e09a0ca516ad92dbfe2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e1526991_6647a3b7","updated":"2022-08-17 15:00:04.000000000","message":"ack from me as discussed ","commit_id":"a7c3728e4d021c706ebcb9716c0bec49b07b5a31"}],"config/release/tripleo-ci/CentOS-9/master.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0e481e5cbd1ecd16e10d6e27f4f9675ac316e288","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"23160894_7221136b","line":185,"updated":"2022-08-11 07:55:01.000000000","message":"but how is this not a replacement?\ni was expecting to see the ceph-pacific repos here being replaced by quincy?\n\nbut we don\u0027t have pacific here?","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"d4b7836f36e9bbfec1b00bf18e7a18e1dc4852f7","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fd56e48b_bc1c66b6","line":185,"in_reply_to":"23160894_7221136b","updated":"2022-08-11 08:06:36.000000000","message":"Right, and I\u0027m not sure why -pacific isn\u0027t there!\nI\u0027m wondering if we use in tripleo-ci an image which already contains that package (see [0]), but I\u0027m not sure. I think having the repo here doesn\u0027t hurt in any case, but let me know if you have any clues about the missing -pacific repo there!\n\n[0] https://review.opendev.org/c/openstack/tripleo-puppet-elements/","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3f14f8bfb7a4f0fc2555195d0c67f803cad71412","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"70448277_4ef8a25c","line":185,"in_reply_to":"23645000_97c4399e","updated":"2022-08-17 07:15:20.000000000","message":"@Doug what do you mean by \u0027updated to provide quincy\u0027 cos as i wrote earlier it is already there both p and q \nhttp://mirror.stream.centos.org/SIGs/9-stream/storage/x86_64/\n\ncan we just update the ContainerCephDaemonImage to point to quincy and be done with it? i.e. just the https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915 part and maybe other places there like \nhttps://opendev.org/openstack/tripleo-heat-templates/src/commit/0ea892ec87d16a84442fd1eb86de3ec1d821b4f3/deployment/cephadm/ceph-base.yaml#L201","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1cc8e2003d0afdc25880b2767c54bf29f6e20a26","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"84239d5a_bfc33998","line":185,"in_reply_to":"3c0df564_e600191d","updated":"2022-08-17 13:13:06.000000000","message":"ack on the plan, as long as we are also going to update delorean-deps for master to point to quincy instead of pacific.\n\ni.e. ok to do this for now but lets not change that we are relying on delorean-deps for ceph unless there is a reason to do that in the longer term agree?","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9fffe685d6945fb2e1689afb404c52e6984d7952","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"23645000_97c4399e","line":185,"in_reply_to":"67f6b764_25f717d5","updated":"2022-08-16 18:03:16.000000000","message":"Agree that delorean-deps should be updated to provide quincy.\nJust as reminder, once we change to quincy repos, scenario001 and scenario004 will start to fail [1] until we promote a new ceph-daemon tag (which we can do quickly in a test project):\n\n\"ERROR ERROR: Container release pacific !\u003d cephadm release quincy; please use matching version of cephadm (pass --allow-mismatched-release to continue anyway)\"\n\n-\u003e If it is easier to merge in quickstart + promote container image, we can proceed with this patch and revert after delorean-deps change.\n\n[1] https://logserver.rdoproject.org/27/36727/54/check/tripleo-ci-centos-9-scenario001-standalone/405412f/logs/undercloud/var/log/ceph/cephadm.log.txt.gz\n\ntestproject: https://review.rdoproject.org/r/c/testproject/+/36727","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"6823c1de2c496754eab8913c69c283065fd67dce","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"803f48b5_6b9623b7","line":185,"in_reply_to":"70448277_4ef8a25c","updated":"2022-08-17 07:39:38.000000000","message":"Hi, I agree w/ Douglas about merging this change to provide quincy repos (to get the cephadm \"quincy\" rpm) until [1] is updated and provides quincy repos for master.\n@Marios this is not about the image, but the rpm that should be installed in the host and it should come from the right repo.\nIf you look at [2], it depends-on [3], which is something similar to this change to make sure we install cephadm from \"Quincy\" on the host (we basically override from quickstart what delorean provides).\nWhen we know we have the right rpm, then we can work on the image change, which is not ContainerCephDaemonImage anymore, but Douglas job that promotes the image within tripleomaster namespace (with -current-ceph tag).\nThis means that we need to coordinate the merge process here, as we need to promote at the same time:\n\n1. the rpm repository through this change\n2. the container image used by tripleo-ci via [4] and [5]\n\nDoug can you confirm this is enough to trigger a new promotion and get Quincy?\n\nNote:\nwe\u0027re moving to a new major, and it requires a new setup in CI, which is something we can\u0027t automate in this particular scenario.\nAlso, the quickstart change can be reverted as soon as [1] has quincy in master.\n\n[1] https://trunk.rdoproject.org/centos9-master/delorean-deps.repo\n[2] https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915\n[3] https://review.opendev.org/c/openstack/tripleo-quickstart/+/790670\n[4] https://review.opendev.org/c/openstack/tripleo-ci/+/852715/2/zuul.d/standalone-jobs.yaml\n[5] https://review.rdoproject.org/r/c/rdo-jobs/+/44518","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"0924872c027da2ab2c7217cd87975662957373d3","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"67f6b764_25f717d5","line":185,"in_reply_to":"788e8990_49b61c77","updated":"2022-08-16 11:03:56.000000000","message":"ack and thanks Marios, \nit wasn\u0027t clear to me where we used to override the ceph repo.\nIn theory, quickstart should be able to override the repos coming from delorean (and not the reverse), and I tried this approach testing the new quincy container through [0] and [1] (you can see the 17.x package has been installed and the quincy -testing repo is here [2]).\nHowever, I\u0027m not sure this is the right approach you\u0027d like to take, because I\u0027m not aware about the reason that led to remove the ceph related repos from quickstart and move them explicitly to delorean.\nChanging delorean works for me, but having control of the tripleo-ci in quickstart should be better.\nAdding Alfredo who can help w/ more history on this area.\n\n[0] https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915\n[1] https://review.opendev.org/c/openstack/tripleo-quickstart/+/790670\n[2] https://b948bf55ea2cf3af9f35-c96c299047b55dcdeaefef8e344ceab6.ssl.cf2.rackcdn.com/778915/113/check/tripleo-ci-centos-9-scenario001-standalone/8ca8d2a/logs/undercloud/etc/yum.repos.d/index.html","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"c061dc3b8d04b9efae06c7af7445d09f3ec3ae74","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a5c398e6_96648ff2","line":185,"in_reply_to":"803f48b5_6b9623b7","updated":"2022-08-17 12:29:36.000000000","message":"@marios, by \"update to quincy\" I am refering to \"deloream-deps.repo\", \nwhich points to pacific only:\n\"baseurl\u003dhttp://mirror.stream.centos.org/SIGs/9-stream/storage/$basearch/ceph-pacific/\"\n@marios, it is easier and faster to run ceph promotion job and get it promoted.\n@fpantano and @marios, the following testproject[1] already validates new tag so we can just run the promotion job, which will take a few minutes to promote the tag.\nBy merging [2], next run of ceph-pipeline (friday) will try to promote new quincy tag, but we don\u0027t need to wait so long, we can do that in testproject.\nAnd [3] doesn\u0027t affect promotions or check/gates, so it can be merged at any time.\n\n[1] https://review.rdoproject.org/r/c/testproject/+/36727/55#message-af7c4978d7a7bc046af7ce5d7b80285d60f32c75\n[2] https://review.rdoproject.org/r/c/rdo-jobs/+/44518\n[3] https://review.opendev.org/c/openstack/tripleo-ci/+/852715/2/zuul.d/standalone-jobs.yaml","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"4cbe2a76f0a0c2730f6ac265b35ce651686f19d7","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7efb96f4_0b5e38d3","line":185,"in_reply_to":"84239d5a_bfc33998","updated":"2022-08-17 13:19:46.000000000","message":"I agree, after quickstart is able to deploy ceph w/ quincy in master, we\u0027re going to work w/ amoralej to change delorean-deps and provide the quincy repo from there.","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"9719704104194be88a14ca2f51f34a9314c451be","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3c0df564_e600191d","line":185,"in_reply_to":"a5c398e6_96648ff2","updated":"2022-08-17 12:56:24.000000000","message":"Doug, this is a good plan. Let\u0027s move forward as long as you\u0027re ready to trigger the testproject and the other related jobs!","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3436d952f63aacf41d938d2aab4bbd3bd37bb4a5","unresolved":true,"context_lines":[{"line_number":182,"context_line":"    filename: quickstart-centos-highavailability.repo"},{"line_number":183,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/9-stream/HighAvailability/x86_64/os/"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"  - type: generic"},{"line_number":186,"context_line":"    reponame: quickstart-centos-ceph-quincy"},{"line_number":187,"context_line":"    filename: quickstart-centos-ceph-quincy.repo"},{"line_number":188,"context_line":"    baseurl: ${NODEPOOL_CENTOS_MIRROR}/SIGs/9-stream/storage/x86_64/ceph-quincy/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"788e8990_49b61c77","line":185,"in_reply_to":"fd56e48b_bc1c66b6","updated":"2022-08-16 07:11:36.000000000","message":"checking a currently running scenario1 job, i only see cephadm in the list of installed packages at [1]\n\ncephadm.noarch                                   2:16.2.9-1.el9s                       @centos9-storage                   \n\nand that centos9-storage repo is there [2]\n\n[centos9-storage]\nname\u003dcentos9-storage\nbaseurl\u003dhttp://mirror-int.iad.rax.opendev.org/centos-stream/SIGs/9-stream/storage/$basearch/ceph-pacific/\nenabled\u003d1\ngpgcheck\u003d0\nmodule_hotfixes\u003d1\n\n\nso that delorean-deps may overwrite what you add here unless you play with priorities but i don\u0027t think we want to do that\n\n\nit answers our question - we are getting ceph from the delorean-deps directly [3]\n\nand that storage sig already has both quincy and pacific at [4]\n\nso we should not need to do this change. instead we should just switch to installing quincy instaed of pacific can we not specify that in another way?\n\n[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_894/845860/20/check/tripleo-ci-centos-9-scenario001-standalone/8945d65/logs/undercloud/var/log/extra/package-list-installed.txt\n[2] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_894/845860/20/check/tripleo-ci-centos-9-scenario001-standalone/8945d65/logs/undercloud/etc/yum.repos.d/delorean-master-deps.repo\n[3] https://trunk.rdoproject.org/centos9-master/delorean-deps.repo\n[4] http://mirror.stream.centos.org/SIGs/9-stream/storage/x86_64/","commit_id":"9f86c425ea39250bb9145d2c3c6cbab97316e9c2"}]}