)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"abce7eb79c011d1ec17a2d4ac29ea7853cef0b25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"09eeb07a_47f3182e","updated":"2022-06-10 06:05:38.000000000","message":"Need some more testing - we can\u0027t rely on CI for that specific thing.","commit_id":"f02b1c4eb1de6bc40f8dbf1b42f3fab70ece7a66"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"784c3c88aab10ec659579a1196f8e35e2a327b3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ff807c1e_5e62f111","updated":"2022-06-09 14:40:06.000000000","message":"Note: I\u0027ve tested this on a local undercloud, registered against a Satellite-6.7 instance.\nWithout the patch, the deploy fails around step_4, when it tries to (re)install net-snmp; with the patch, everything is fine in the install","commit_id":"f02b1c4eb1de6bc40f8dbf1b42f3fab70ece7a66"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"9b16acd25d09cc4f6abbc4bb2854b13a7ffcf713","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a828e640_4b4516b2","updated":"2022-06-10 09:08:18.000000000","message":"aw dang. This would work only on an Overcloud, not the Undercloud X(. Need to hit another part of the deploy...\n\nKeeping as WIP, I need to first correct the UC before being able to deploy an overcloud.....","commit_id":"f02b1c4eb1de6bc40f8dbf1b42f3fab70ece7a66"}],"tasks/main.yml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"41ff2a190002f87abe6a2e7073ffb46b8ccb75a7","unresolved":true,"context_lines":[{"line_number":60,"context_line":"    dest: /etc/profile.d/smdev-container-off.sh"},{"line_number":61,"context_line":"    content: |"},{"line_number":62,"context_line":"      # File managed by ansible-role-redhat-subscription"},{"line_number":63,"context_line":"      SMDEV_CONTAINER_OFF\u003d{{ rhsm_disable_container_check | bool }}"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"- block:"},{"line_number":66,"context_line":"    - include_tasks: \"{{ rhsm_method }}.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"eac846dc_b92b77b0","line":63,"updated":"2022-06-09 16:09:11.000000000","message":"this complements https://review.opendev.org/c/openstack/ansible-role-redhat-subscription/+/831810 , right? but this time for the host instead of containers, so we probably want an opposite behavior?","commit_id":"f02b1c4eb1de6bc40f8dbf1b42f3fab70ece7a66"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"0b4ffc316e8704d10f446ac81478b99bbac5dcda","unresolved":true,"context_lines":[{"line_number":60,"context_line":"    dest: /etc/profile.d/smdev-container-off.sh"},{"line_number":61,"context_line":"    content: |"},{"line_number":62,"context_line":"      # File managed by ansible-role-redhat-subscription"},{"line_number":63,"context_line":"      SMDEV_CONTAINER_OFF\u003d{{ rhsm_disable_container_check | bool }}"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"- block:"},{"line_number":66,"context_line":"    - include_tasks: \"{{ rhsm_method }}.yml\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9cd2769a_d767dc9e","line":63,"in_reply_to":"eac846dc_b92b77b0","updated":"2022-06-10 06:05:18.000000000","message":"hmmm, the rhsm_disable_container_check is set to \"false\" by default - meaning it\u0027s enabled (meh). IIRC, the other set of patches (in t-h-t and this very same project) were to address a host issue as well... We don\u0027t install packages within containers during the deploy, do we?!\n\nI\u0027ll do some test, lemme pass this patch as WIP in the meantime.","commit_id":"f02b1c4eb1de6bc40f8dbf1b42f3fab70ece7a66"}]}