)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"00f4867779fb62618dfdad64fdb2d21698e07141","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5ab7a00b_5d469268","updated":"2021-10-14 16:36:10.000000000","message":"I\u0027m extremely confused by this change. It does not remove any pecan imports, just moves them around. It doesn\u0027t seem to work towards the goal as written in the storyboard story.","commit_id":"c99b3471d179b6e4021e4cb3da24fd2f2b1451ea"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"74c0677169d3e4e4d03fe5789db3bacd18652983","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"42e789c7_6ff00567","updated":"2021-10-18 04:29:03.000000000","message":"Thankyou for taking on this outreachy intern task, but it is not quite implementing what was requested. Please see the review comments I left in base.py","commit_id":"c99b3471d179b6e4021e4cb3da24fd2f2b1451ea"}],"ironic/tests/unit/api/base.py":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"74c0677169d3e4e4d03fe5789db3bacd18652983","unresolved":true,"context_lines":[{"line_number":24,"context_line":"from urllib import parse as urlparse"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"from oslo_config import cfg"},{"line_number":27,"context_line":"import pecan"},{"line_number":28,"context_line":"import pecan.testing"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from ironic.api.controllers.v1 import node as api_node"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f0fa817_e148c1ab","side":"PARENT","line":27,"updated":"2021-10-18 04:29:03.000000000","message":"Our import policy is that we import packages, not package contents. So this import is fine.","commit_id":"a4a6f26333be31b84a9b1a874dde506e61d407d3"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"74c0677169d3e4e4d03fe5789db3bacd18652983","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"from oslo_config import cfg"},{"line_number":27,"context_line":"import pecan"},{"line_number":28,"context_line":"import pecan.testing"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from ironic.api.controllers.v1 import node as api_node"},{"line_number":31,"context_line":"from ironic.tests.unit.db import base as db_base"}],"source_content_type":"text/x-python","patch_set":1,"id":"0713e43c_169ba072","side":"PARENT","line":28,"updated":"2021-10-18 04:29:03.000000000","message":"The intent of Story: 2009257 is to remove this import entirely, and implement a load_test_app method inside BaseApiTest and use that instead of pecan.testing.load_test_app","commit_id":"a4a6f26333be31b84a9b1a874dde506e61d407d3"}]}