)]}'
{"ansible/group_vars/all/ipa":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"bb59695fe0d854fd8fc1132e3b57da40c9faa172","unresolved":true,"context_lines":[{"line_number":78,"context_line":"# Building IPA images with ironic-python-agent-builder requires a newer DIB"},{"line_number":79,"context_line":"# than what is available in upper constraints for Ussuri. Default to using"},{"line_number":80,"context_line":"# upper constraints from Victoria instead."},{"line_number":81,"context_line":"os_images_upper_constraints_file: \"https://releases.openstack.org/constraints/upper/victoria\""},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"###############################################################################"},{"line_number":84,"context_line":"# Ironic Python Agent (IPA) images configuration."}],"source_content_type":"application/octet-stream","patch_set":5,"id":"9715915f_c6bd0555","line":81,"updated":"2021-02-12 09:26:39.000000000","message":"Another option is to pin IPA builder to a tag, which is an approach being tried in bifrost\u0027s train branch: https://review.opendev.org/c/openstack/bifrost/+/774237.\n\nGiven that the task variable has precedence, does this have any effect?","commit_id":"768f0229686dd1e76b2ef8ac1d62f95034e4d3c3"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"a60d3d2e80363435160e4def35836ab2c9c0dfa3","unresolved":true,"context_lines":[{"line_number":78,"context_line":"# Building IPA images with ironic-python-agent-builder requires a newer DIB"},{"line_number":79,"context_line":"# than what is available in upper constraints for Ussuri. Default to using"},{"line_number":80,"context_line":"# upper constraints from Victoria instead."},{"line_number":81,"context_line":"os_images_upper_constraints_file: \"https://releases.openstack.org/constraints/upper/victoria\""},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"###############################################################################"},{"line_number":84,"context_line":"# Ironic Python Agent (IPA) images configuration."}],"source_content_type":"application/octet-stream","patch_set":5,"id":"bbb5df41_14b777c4","line":81,"in_reply_to":"9715915f_c6bd0555","updated":"2021-02-12 12:09:57.000000000","message":"Unfortunately pinning doesn\u0027t work anymore out of box, because of the cryptography rust update. And since IPA-builder and DIB are branchless we may never get a fix 😞\n\nGood catch on the precedence, I must have tested with `os_images_upper_constraints_file` as an extra var…","commit_id":"768f0229686dd1e76b2ef8ac1d62f95034e4d3c3"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"5ee8a93c64cafb06a73fee9377c56a50963bd694","unresolved":true,"context_lines":[{"line_number":78,"context_line":"# Building IPA images with ironic-python-agent-builder requires a newer DIB"},{"line_number":79,"context_line":"# than what is available in upper constraints for Ussuri. Default to using"},{"line_number":80,"context_line":"# upper constraints from Victoria instead."},{"line_number":81,"context_line":"os_images_upper_constraints_file: \"https://releases.openstack.org/constraints/upper/victoria\""},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"###############################################################################"},{"line_number":84,"context_line":"# Ironic Python Agent (IPA) images configuration."}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7d52cab7_177fd69c","line":81,"in_reply_to":"bbb5df41_14b777c4","updated":"2021-02-12 13:55:41.000000000","message":"@Mark: I\u0027ve extracted the fix part from the CI test, please see https://review.opendev.org/c/openstack/kayobe/+/775386. Since it introduces a new variable we probably want to merge this into master and backport.","commit_id":"768f0229686dd1e76b2ef8ac1d62f95034e4d3c3"}],"playbooks/kayobe-seed-base/overrides.yml.j2":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"bb59695fe0d854fd8fc1132e3b57da40c9faa172","unresolved":true,"context_lines":[{"line_number":16,"context_line":"# NOTE(priteau): This is already set in ansible/group_vars/all/ipa but needs to"},{"line_number":17,"context_line":"# be redefined as an extra variable here because pip_upper_constraints_file"},{"line_number":18,"context_line":"# overrides it."},{"line_number":19,"context_line":"os_images_upper_constraints_file: \"https://releases.openstack.org/constraints/upper/victoria\""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# NOTE(mgoddard): We\u0027re using a cirros image, which doesn\u0027t require the"},{"line_number":22,"context_line":"# resolv.conf work around used for CentOS."}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"3f4aca7d_8035cb25","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":32},"updated":"2021-02-12 09:26:39.000000000","message":"I suppose that would trip up anyone building the image with a UC override.","commit_id":"768f0229686dd1e76b2ef8ac1d62f95034e4d3c3"}]}