)]}'
{"config/release/centosci/master-current-tripleo.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e783013cceba7f74a22ed9510c0cadbdf44978f0","unresolved":true,"context_lines":[{"line_number":59,"context_line":"  - type: generic"},{"line_number":60,"context_line":"    reponame: quickstart-centos-ceph-octopus"},{"line_number":61,"context_line":"    filename: quickstart-centos-ceph-octopus.repo"},{"line_number":62,"context_line":"    baseurl: http://mirror.centos.org/centos/8/storage/x86_64/ceph-octopus/"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"  - type: generic"},{"line_number":65,"context_line":"    reponame: quickstart-centos-opstools"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"12bb1f41_60428ac7","line":62,"updated":"2021-03-26 08:26:50.000000000","message":"let\u0027s ask the ceph team to ack this bit too... will reach out to them after i vote here","commit_id":"4278f19776300a39b7df4c93c8c85396b1da0cb9"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"5a7a225ecdca5e700800405272418d077fdff074","unresolved":false,"context_lines":[{"line_number":59,"context_line":"  - type: generic"},{"line_number":60,"context_line":"    reponame: quickstart-centos-ceph-octopus"},{"line_number":61,"context_line":"    filename: quickstart-centos-ceph-octopus.repo"},{"line_number":62,"context_line":"    baseurl: http://mirror.centos.org/centos/8/storage/x86_64/ceph-octopus/"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"  - type: generic"},{"line_number":65,"context_line":"    reponame: quickstart-centos-opstools"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"35799880_bea7ddfb","line":62,"in_reply_to":"12bb1f41_60428ac7","updated":"2021-03-26 08:35:45.000000000","message":"+1, ceph-ansible is now available from the octopus repo, and it works as usual (scenario004) w/ octopus containers.\nThere\u0027s no reason to have nautilus here, and I probably forgot to remove it from this file when I made [1] and [2] :/\n\n[1] https://review.opendev.org/c/openstack/tripleo-quickstart/+/780008\n[2] https://review.opendev.org/c/openstack/tripleo-quickstart/+/779657","commit_id":"4278f19776300a39b7df4c93c8c85396b1da0cb9"}],"config/release/tripleo-ci/CentOS-8/wallaby.yml":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"aafed714109db5b7e0d9abd43406b049079a7e7f","unresolved":false,"context_lines":[{"line_number":6,"context_line":"distro_deps_repo_name: deps"},{"line_number":7,"context_line":"dlrn_hash_tag: \"{{ job.provider_dlrn_hash_tag_branch[release]|default(\u0027current-tripleo\u0027) }}\""},{"line_number":8,"context_line":"dlrn_hash_tag_newest: current"},{"line_number":9,"context_line":"validate_on: rdo"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# job.build_container_images is defined in zuul as a"},{"line_number":12,"context_line":"# direct var.  adding build_container_images as tqe"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"af46d451_c10760b1","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":16},"updated":"2021-03-29 12:57:59.000000000","message":"Earlier, I was following master.yml to create wallaby.yml\n\nmaster.yml don\u0027t have \"validate_on\" defined.\n\nIf validate_on is not defined this tasks execute- https://opendev.org/openstack/tripleo-quickstart/src/branch/master/tests/validate-release-config.yml#L9\n\n^^ above tasks fails for wallaby as current-tripleo-rdo images dir doesn\u0027t exists yet. We haven\u0027t got a current-tripleo-rdo promotion\n\n~~~\nhttps://images.rdoproject.org/centos8/wallaby/rdo_trunk/current-tripleo-rdo/ironic-python-agent.tar\nhttps://images.rdoproject.org/centos8/wallaby/rdo_trunk/current-tripleo-rdo/overcloud-full.tar\n~~~\n\n\nAdding \"validate_on\" var - All the stable branches release file - victoria/ussuri/train have this var defined. \n\nhttps://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/victoria.yml#L9\nhttps://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/ussuri.yml#L37","commit_id":"173c7d10f0b30030588a781feef1b466edbb0246"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"8cf2ca719a1c69b6964d32611e709c8e1e3344fa","unresolved":false,"context_lines":[{"line_number":6,"context_line":"distro_deps_repo_name: deps"},{"line_number":7,"context_line":"dlrn_hash_tag: \"{{ job.provider_dlrn_hash_tag_branch[release]|default(\u0027current-tripleo\u0027) }}\""},{"line_number":8,"context_line":"dlrn_hash_tag_newest: current"},{"line_number":9,"context_line":"validate_on: rdo"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# job.build_container_images is defined in zuul as a"},{"line_number":12,"context_line":"# direct var.  adding build_container_images as tqe"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"905e632a_c96e6a78","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":16},"in_reply_to":"34283392_cb44db2e","updated":"2021-03-30 08:27:17.000000000","message":"Same as my other answer.","commit_id":"173c7d10f0b30030588a781feef1b466edbb0246"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1598ee09e4a789a8540684070d33c6976729f5e4","unresolved":false,"context_lines":[{"line_number":6,"context_line":"distro_deps_repo_name: deps"},{"line_number":7,"context_line":"dlrn_hash_tag: \"{{ job.provider_dlrn_hash_tag_branch[release]|default(\u0027current-tripleo\u0027) }}\""},{"line_number":8,"context_line":"dlrn_hash_tag_newest: current"},{"line_number":9,"context_line":"validate_on: rdo"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"# job.build_container_images is defined in zuul as a"},{"line_number":12,"context_line":"# direct var.  adding build_container_images as tqe"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"34283392_cb44db2e","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":16},"in_reply_to":"af46d451_c10760b1","updated":"2021-03-30 07:03:47.000000000","message":"hi, i see that current-tripleo-rdo isn\u0027t there but current-tripleo is\nhttps://images.rdoproject.org/centos8/wallaby/rdo_trunk/current-tripleo/\n\nperhaps that is why it is now passing but wasn\u0027t yesterday i.e. did you push those images to current-tripleo?","commit_id":"173c7d10f0b30030588a781feef1b466edbb0246"}],"config/release/trunk/master.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e783013cceba7f74a22ed9510c0cadbdf44978f0","unresolved":true,"context_lines":[{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"},{"line_number":70,"context_line":"    # https://bugs.launchpad.net/tripleo/+bug/1909191"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"11415191_30955e02","side":"PARENT","line":67,"updated":"2021-03-26 08:26:50.000000000","message":"did you ask the ceph folks about this?","commit_id":"bfb6e2b48321ca7c4f7c287524bb836d1e898488"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"7dd1e367306eaf1bffe26a7467fef50a5691a4a5","unresolved":true,"context_lines":[{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"},{"line_number":70,"context_line":"    # https://bugs.launchpad.net/tripleo/+bug/1909191"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3b005c1b_6de93f62","side":"PARENT","line":67,"in_reply_to":"11415191_30955e02","updated":"2021-03-26 08:36:15.000000000","message":"Hey Marios,\n\nI haven\u0027t reached out to them because I have found patches from Ceph team, where they have already removed ceph-nautlius from config/release/tripleo-ci/CentOS-8/promotion-testing-hash-master.yml  and config/release/tripleo-ci/CentOS-8/master.yml . \n\nI think its just that they missed to update centosci/master-current-tripleo.yml \u0026 trunk/master.yml in [1] and [2]\n\n[1] https://review.opendev.org/c/openstack/tripleo-quickstart/+/780008\n[2] https://review.opendev.org/c/openstack/tripleo-quickstart/+/779657","commit_id":"bfb6e2b48321ca7c4f7c287524bb836d1e898488"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"5a7a225ecdca5e700800405272418d077fdff074","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"},{"line_number":70,"context_line":"    # https://bugs.launchpad.net/tripleo/+bug/1909191"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"108b2d55_5a5e5eca","side":"PARENT","line":67,"in_reply_to":"11415191_30955e02","updated":"2021-03-26 08:35:45.000000000","message":"Same as my previous comment","commit_id":"bfb6e2b48321ca7c4f7c287524bb836d1e898488"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"8309f56c83fd83e3a564a49132926f4fdba2079c","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"},{"line_number":70,"context_line":"    # https://bugs.launchpad.net/tripleo/+bug/1909191"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9d698cad_019d901b","side":"PARENT","line":67,"in_reply_to":"1360a6cf_e9df9ae3","updated":"2021-03-26 09:09:40.000000000","message":"Thank you for confirming, Francesco","commit_id":"bfb6e2b48321ca7c4f7c287524bb836d1e898488"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"6b8c2648d0248c5c5330e556f3d0f2ce884a4126","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"},{"line_number":70,"context_line":"    # https://bugs.launchpad.net/tripleo/+bug/1909191"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1360a6cf_e9df9ae3","side":"PARENT","line":67,"in_reply_to":"3b005c1b_6de93f62","updated":"2021-03-26 08:46:32.000000000","message":"Exactly, thanks for updating them!","commit_id":"bfb6e2b48321ca7c4f7c287524bb836d1e898488"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6731d052d581b0e0ca0935b884e1738e899d3736","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"},{"line_number":70,"context_line":"    # https://bugs.launchpad.net/tripleo/+bug/1909191"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"01787752_bd777163","side":"PARENT","line":67,"in_reply_to":"9d698cad_019d901b","updated":"2021-03-29 07:00:54.000000000","message":"ack thank you Sandeep \u0026 Francesco","commit_id":"bfb6e2b48321ca7c4f7c287524bb836d1e898488"}],"config/release/trunk/wallaby.yml":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"430c4b82cbfec17f49241908ae270bea43ce4e86","unresolved":true,"context_lines":[{"line_number":1,"context_line":"release: wallaby"},{"line_number":2,"context_line":"distro_ver: centos8"},{"line_number":3,"context_line":"dlrn_hash_tag: current-tripleo-rdo"},{"line_number":4,"context_line":"docker_image_tag: \"{{ dlrn_hash|default(dlrn_hash_tag) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c252c013_c9e9c112","line":1,"updated":"2021-03-25 12:13:03.000000000","message":"If this file is still used, it should refer to stream repos, images etc. Otherwise, let;s ditch it.","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"e0aa9d9f2efcc2d07c0d2e2ab8978094a92fd04d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"release: wallaby"},{"line_number":2,"context_line":"distro_ver: centos8"},{"line_number":3,"context_line":"dlrn_hash_tag: current-tripleo-rdo"},{"line_number":4,"context_line":"docker_image_tag: \"{{ dlrn_hash|default(dlrn_hash_tag) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9352c9e8_b0e95ffa","line":1,"in_reply_to":"5d8058ed_64342070","updated":"2021-03-26 07:45:30.000000000","message":"Done","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a95d70db415654313924fd26e49544124e60b383","unresolved":true,"context_lines":[{"line_number":1,"context_line":"release: wallaby"},{"line_number":2,"context_line":"distro_ver: centos8"},{"line_number":3,"context_line":"dlrn_hash_tag: current-tripleo-rdo"},{"line_number":4,"context_line":"docker_image_tag: \"{{ dlrn_hash|default(dlrn_hash_tag) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5d8058ed_64342070","line":1,"in_reply_to":"a6364823_ba3cb765","updated":"2021-03-26 05:21:26.000000000","message":"https://review.opendev.org/c/openstack/tripleo-quickstart/+/783048 is taking care of other releases, wallaby can be udpated in this patch itself.","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"88a0a24bef581a36ca4033232cfc159904914441","unresolved":true,"context_lines":[{"line_number":1,"context_line":"release: wallaby"},{"line_number":2,"context_line":"distro_ver: centos8"},{"line_number":3,"context_line":"dlrn_hash_tag: current-tripleo-rdo"},{"line_number":4,"context_line":"docker_image_tag: \"{{ dlrn_hash|default(dlrn_hash_tag) }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a6364823_ba3cb765","line":1,"in_reply_to":"c252c013_c9e9c112","updated":"2021-03-25 13:01:59.000000000","message":"Looking at [1] my understanding was this files are needed by quickstart script.\n\nfor ex. quickstart.sh -R [victoria|ussuri].\n\nif ^^ is true: other release file also need update.. May be in a separate patch before..\n\n[1] https://github.com/openstack/tripleo-quickstart/commit/d53ac0d9fc2dce5eebceb5cc63e509f418a9f794","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6d99c83493b06de5071b3a12d6c72112acf1abba","unresolved":true,"context_lines":[{"line_number":63,"context_line":"    filename: quickstart-centos-extras.repo"},{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5047d88f_3b9432ac","line":66,"updated":"2021-03-23 10:50:43.000000000","message":"worth pinging the ceph team jfulton gfidente fpantano to sync on this point\n\nWe probably want to merge as is and update this once it is ready","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"e0aa9d9f2efcc2d07c0d2e2ab8978094a92fd04d","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    filename: quickstart-centos-extras.repo"},{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"48d348da_4fe22c72","line":66,"in_reply_to":"344cabc0_911ceb2a","updated":"2021-03-26 07:45:30.000000000","message":"Thank you Yatin,\n\nI have removed ceph nautilus from centosci/master-current-tripleo.yml \u0026 trunk/master.yml as well in same patch, Also updated wallaby trunk/wallaby.yml, it was missed in in [1] \u0026 [2].\n\n[1] https://review.opendev.org/c/openstack/tripleo-quickstart/+/780008\n[2] https://review.opendev.org/c/openstack/tripleo-quickstart/+/779657","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"a95d70db415654313924fd26e49544124e60b383","unresolved":true,"context_lines":[{"line_number":63,"context_line":"    filename: quickstart-centos-extras.repo"},{"line_number":64,"context_line":"    baseurl: http://mirror.centos.org/centos/8/extras/x86_64/os/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    # TODO: This is temporary and ideally we should use ceph octopus repository"},{"line_number":67,"context_line":"    # when the release is W+. Octopus repos doesn\u0027t have ceph-ansible. Revisit"},{"line_number":68,"context_line":"    # when switch from ceph-ansible to cephadm is complete (Ceph nautilus"},{"line_number":69,"context_line":"    # repos to be removed on W+. For more detail see:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"344cabc0_911ceb2a","line":66,"in_reply_to":"5047d88f_3b9432ac","updated":"2021-03-26 05:21:26.000000000","message":"can be updated to wallaby already, i see master trunk file also not updated, so both can be updated together here itself.","commit_id":"614d2eae9f6d392528b917dc320c80b311c15d1b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1598ee09e4a789a8540684070d33c6976729f5e4","unresolved":true,"context_lines":[{"line_number":5,"context_line":"dlrn_baseurl: \"https://trunk.rdoproject.org/{{ distro_ver }}-{{ release }}\""},{"line_number":6,"context_line":"overcloud_image_url: \"https://images.rdoproject.org/{{ distro_ver }}/{{ release }}/rdo_trunk/{{ dlrn_hash|default(dlrn_hash_tag) }}/overcloud-full.tar\""},{"line_number":7,"context_line":"ipa_image_url: \"https://images.rdoproject.org/{{ distro_ver }}/{{ release }}/rdo_trunk/{{ dlrn_hash|default(dlrn_hash_tag) }}/ironic-python-agent.tar\""},{"line_number":8,"context_line":"validate_on: rdo"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"docker_registry_host: \u003e-"},{"line_number":11,"context_line":"  {% if build_container_images|default(false)|bool -%}"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ae08f44a_a174e92e","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":16},"updated":"2021-03-30 07:03:47.000000000","message":"as far as i can see this in the only diff from v5 https://review.opendev.org/c/openstack/tripleo-quickstart/+/782211/5..6 i.e. you added validate_on to the trunk/wallaby.yml \n\nbut now the molecule test is passing, what changed? perhaps you pushed the images to current-tripleo there? https://images.rdoproject.org/centos8/wallaby/rdo_trunk/current-tripleo/","commit_id":"965e2cae38497549dafd3bd34a357fe00fcc43de"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"8cf2ca719a1c69b6964d32611e709c8e1e3344fa","unresolved":false,"context_lines":[{"line_number":5,"context_line":"dlrn_baseurl: \"https://trunk.rdoproject.org/{{ distro_ver }}-{{ release }}\""},{"line_number":6,"context_line":"overcloud_image_url: \"https://images.rdoproject.org/{{ distro_ver }}/{{ release }}/rdo_trunk/{{ dlrn_hash|default(dlrn_hash_tag) }}/overcloud-full.tar\""},{"line_number":7,"context_line":"ipa_image_url: \"https://images.rdoproject.org/{{ distro_ver }}/{{ release }}/rdo_trunk/{{ dlrn_hash|default(dlrn_hash_tag) }}/ironic-python-agent.tar\""},{"line_number":8,"context_line":"validate_on: rdo"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"docker_registry_host: \u003e-"},{"line_number":11,"context_line":"  {% if build_container_images|default(false)|bool -%}"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"79427cdc_cbc76a82","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":16},"in_reply_to":"ae08f44a_a174e92e","updated":"2021-03-30 08:27:17.000000000","message":"Hello Marios,\n\nWhen \"validate_on\" is not defined this play[1] executes.\n\nEarlier, tox-molecule was failing on play[1] as current-tripleo-rdo images directories[2] doesn\u0027t exists yet because we haven\u0027t got a current-tripleo-rdo promotion\n\nAdding \"validate_on\" var skip running of [1]. stable branches release files - victoria/ussuri/train have this var defined. \n\nhttps://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/victoria.yml#L9\nhttps://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/ussuri.yml#L37\n\n\n[1] https://opendev.org/openstack/tripleo-quickstart/src/branch/master/tests/validate-release-config.yml#L9\n[2]\n~~~\nhttps://images.rdoproject.org/centos8/wallaby/rdo_trunk/current-tripleo-rdo/ironic-python-agent.tar\nhttps://images.rdoproject.org/centos8/wallaby/rdo_trunk/current-tripleo-rdo/overcloud-full.tar\n~~~","commit_id":"965e2cae38497549dafd3bd34a357fe00fcc43de"}]}