)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"932fa6b45abbee5c40dc6b68cc3ea6e3c2be13ce","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Message Registry resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is one of the first patches to support Message Registry."},{"line_number":10,"context_line":"This delivers only Message Registry parsing to sushy fields."},{"line_number":11,"context_line":"Any other processing will be added in followup patches."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_f431b60b","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":51},"updated":"2018-07-27 11:24:45.000000000","message":"nit: may be Redfish Message Registry to make it obvious that we are not talking of some internal sushy feature","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"4568eab50b80b6e3c40e7124532fbd6df70df02d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add Message Registry resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is one of the first patches to support Message Registry."},{"line_number":10,"context_line":"This delivers only Message Registry parsing to sushy fields."},{"line_number":11,"context_line":"Any other processing will be added in followup patches."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_2765b9b0","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":51},"in_reply_to":"5f7c97a3_f431b60b","updated":"2018-07-27 15:50:12.000000000","message":"Done","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"932fa6b45abbee5c40dc6b68cc3ea6e3c2be13ce","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add Message Registry resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is one of the first patches to support Message Registry."},{"line_number":10,"context_line":"This delivers only Message Registry parsing to sushy fields."},{"line_number":11,"context_line":"Any other processing will be added in followup patches."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Message Registry will be used by sushy internally,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_f44c569e","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":4},"updated":"2018-07-27 11:24:45.000000000","message":"nit: repeating `This`","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"4568eab50b80b6e3c40e7124532fbd6df70df02d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add Message Registry resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is one of the first patches to support Message Registry."},{"line_number":10,"context_line":"This delivers only Message Registry parsing to sushy fields."},{"line_number":11,"context_line":"Any other processing will be added in followup patches."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Message Registry will be used by sushy internally,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_e70021b4","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":4},"in_reply_to":"5f7c97a3_f44c569e","updated":"2018-07-27 15:50:12.000000000","message":"Changed to \u0027it\u0027","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"932fa6b45abbee5c40dc6b68cc3ea6e3c2be13ce","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Any other processing will be added in followup patches."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Message Registry will be used by sushy internally,"},{"line_number":14,"context_line":"it will not be exposed to sushy users."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I9a1735230a8328fd8365e375889c6ab066c3df16"},{"line_number":17,"context_line":"Story: 2001791"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_b4101e5a","line":14,"range":{"start_line":14,"start_character":3,"end_line":14,"end_character":7},"updated":"2018-07-27 11:24:45.000000000","message":"nit: may be put it less definitely wrt what will happen in the future (at least I am personally have no clue what future holds)","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"4568eab50b80b6e3c40e7124532fbd6df70df02d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Any other processing will be added in followup patches."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Message Registry will be used by sushy internally,"},{"line_number":14,"context_line":"it will not be exposed to sushy users."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I9a1735230a8328fd8365e375889c6ab066c3df16"},{"line_number":17,"context_line":"Story: 2001791"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_a706a9cb","line":14,"range":{"start_line":14,"start_character":3,"end_line":14,"end_character":7},"in_reply_to":"5f7c97a3_b4101e5a","updated":"2018-07-27 15:50:12.000000000","message":"Done","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"}],"sushy/resources/constants.py":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"932fa6b45abbee5c40dc6b68cc3ea6e3c2be13ce","unresolved":false,"context_lines":[{"line_number":36,"context_line":"PARAMTYPE_STRING \u003d \u0027string\u0027"},{"line_number":37,"context_line":"PARAMTYPE_NUMBER \u003d \u0027number\u0027"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"# Severity related consants"},{"line_number":40,"context_line":"SEVERITY_OK \u003d \u0027ok\u0027"},{"line_number":41,"context_line":"SEVERITY_WARNING \u003d \u0027warning\u0027"},{"line_number":42,"context_line":"SEVERITY_CRITICAL \u003d \u0027critical\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_5472eae5","line":39,"range":{"start_line":39,"start_character":19,"end_line":39,"end_character":27},"updated":"2018-07-27 11:24:45.000000000","message":"nit: typo","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"4568eab50b80b6e3c40e7124532fbd6df70df02d","unresolved":false,"context_lines":[{"line_number":36,"context_line":"PARAMTYPE_STRING \u003d \u0027string\u0027"},{"line_number":37,"context_line":"PARAMTYPE_NUMBER \u003d \u0027number\u0027"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"# Severity related consants"},{"line_number":40,"context_line":"SEVERITY_OK \u003d \u0027ok\u0027"},{"line_number":41,"context_line":"SEVERITY_WARNING \u003d \u0027warning\u0027"},{"line_number":42,"context_line":"SEVERITY_CRITICAL \u003d \u0027critical\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_11323f10","line":39,"range":{"start_line":39,"start_character":19,"end_line":39,"end_character":27},"in_reply_to":"5f7c97a3_5472eae5","updated":"2018-07-27 15:50:12.000000000","message":"Done","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":16635,"name":"deray","email":"debayan.ray@gmail.com","username":"deray"},"change_message_id":"16c83635e3c4e9898b586116f6abec9e28215140","unresolved":false,"context_lines":[{"line_number":37,"context_line":"PARAMTYPE_NUMBER \u003d \u0027number\u0027"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"# Severity related constants"},{"line_number":40,"context_line":"SEVERITY_OK \u003d \u0027ok\u0027"},{"line_number":41,"context_line":"SEVERITY_WARNING \u003d \u0027warning\u0027"},{"line_number":42,"context_line":"SEVERITY_CRITICAL \u003d \u0027critical\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_f148182f","line":42,"range":{"start_line":40,"start_character":0,"end_line":42,"end_character":30},"updated":"2018-08-31 10:20:17.000000000","message":"Just thinking if the \u0027Status\u0027 related constants (L18-L20) can be reused for\nMessage Severity constants as well. More so when the long description for Severity says:\n\n    The value of this property shall be the severity of the condition\n    resulting in the message, as defined in the Status section of the\n    Redfish specification.\n\nReference: https://redfish.dmtf.org/schemas/v1/MessageRegistry.v1_1_1.json\n\nPossibly we can rename the existing constants as:\n\n    STATUS_HEALTH_OK \u003d \u0027ok\u0027\n    STATUS_HEALTH_WARNING \u003d \u0027warning\u0027\n    STATUS_HEALTH_CRITICAL \u003d \u0027critical\u0027\n\nAnd even the State constants as:\n\n    STATUS_STATE_ENABLED \u003d \u0027enabled\u0027\n    STATUS_STATE_DISABLED \u003d \u0027disabled\u0027\n    ...","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"},{"author":{"_account_id":16635,"name":"deray","email":"debayan.ray@gmail.com","username":"deray"},"change_message_id":"bc87a2a62a7a9df51409e612999d0675accd66ae","unresolved":false,"context_lines":[{"line_number":37,"context_line":"PARAMTYPE_NUMBER \u003d \u0027number\u0027"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"# Severity related constants"},{"line_number":40,"context_line":"SEVERITY_OK \u003d \u0027ok\u0027"},{"line_number":41,"context_line":"SEVERITY_WARNING \u003d \u0027warning\u0027"},{"line_number":42,"context_line":"SEVERITY_CRITICAL \u003d \u0027critical\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_07b772da","line":42,"range":{"start_line":40,"start_character":0,"end_line":42,"end_character":30},"in_reply_to":"3f79a3b5_002a072b","updated":"2018-09-03 10:52:46.000000000","message":"\u003e I was considering this too, but chose current implementation\n \u003e because the schema for these fields are different and maybe in\n \u003e future they will differ.\n \u003e \n \u003e Health mapping corresponds to the enum named \u0027Health\u0027 in [1],\n \u003e but Severity field is just a string with comment saying that it\n \u003e should map to the same values as Status (meaning Health enum).\n \u003e \n \u003e Need to clarify why they both are not enums, or are not the same\n \u003e enums (though name \u0027Health\u0027 can be confusing for \u0027Severity\u0027) just\n \u003e to be sure that there is nothing else.\n \u003e \n \u003e That\u0027s why I haven\u0027t included this in the followup patch now.\n\nTo be honest the Status-Health mapping to Severity was confusing to me also. It most probably makes sense to me to keep the values separate as you suggested.\n\n \u003e Also I think Health mapping is released and that would be change to public sushy API?\n \u003e \n \u003e Any thoughts?\n\nIt had occurred to me but of course there are ways we can avoid any break of backward compatibility in that sense. For instance, in this case putting a deprecated notice for the earlier vars and introducing new ones.\n\nOverall in the present context and for the time being I feel I am completely in agreement with your approach!","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"5906ecf9fc749b49e4ed4ca54f237a52c139e3e3","unresolved":false,"context_lines":[{"line_number":37,"context_line":"PARAMTYPE_NUMBER \u003d \u0027number\u0027"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"# Severity related constants"},{"line_number":40,"context_line":"SEVERITY_OK \u003d \u0027ok\u0027"},{"line_number":41,"context_line":"SEVERITY_WARNING \u003d \u0027warning\u0027"},{"line_number":42,"context_line":"SEVERITY_CRITICAL \u003d \u0027critical\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_002a072b","line":42,"range":{"start_line":40,"start_character":0,"end_line":42,"end_character":30},"in_reply_to":"3f79a3b5_f148182f","updated":"2018-09-01 14:17:08.000000000","message":"I was considering this too, but chose current implementation because the schema for these fields are different and maybe in future they will differ.\n\nHealth mapping corresponds to the enum named \u0027Health\u0027 in [1],\nbut Severity field is just a string with comment saying that it should map to the same values as Status (meaning Health enum).\n\nNeed to clarify why they both are not enums, or are not the same enums (though name \u0027Health\u0027 can be confusing for \u0027Severity\u0027) just to be sure that there is nothing else.\n\nThat\u0027s why I haven\u0027t included this in the followup patch now.\n\nAlso I think Health mapping is released and that would be change to public sushy API?\n\nAny thoughts?\n\n\n[1] https://redfish.dmtf.org/schemas/v1/Resource.json","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"}],"sushy/resources/mappings.py":[{"author":{"_account_id":16635,"name":"deray","email":"debayan.ray@gmail.com","username":"deray"},"change_message_id":"16c83635e3c4e9898b586116f6abec9e28215140","unresolved":false,"context_lines":[{"line_number":35,"context_line":"HEALTH_VALUE_MAP_REV \u003d ("},{"line_number":36,"context_line":"    utils.revert_dictionary(HEALTH_VALUE_MAP))"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"PARAMTYPE_MAP \u003d {"},{"line_number":39,"context_line":"    \u0027string\u0027: res_cons.PARAMTYPE_STRING,"},{"line_number":40,"context_line":"    \u0027number\u0027: res_cons.PARAMTYPE_NUMBER"},{"line_number":41,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_719f2865","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":13},"updated":"2018-08-31 10:20:17.000000000","message":"nit: s/PARAMTYPE_MAP/PARAMTYPE_VALUE_MAP\nto maintain consistency","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"5906ecf9fc749b49e4ed4ca54f237a52c139e3e3","unresolved":false,"context_lines":[{"line_number":35,"context_line":"HEALTH_VALUE_MAP_REV \u003d ("},{"line_number":36,"context_line":"    utils.revert_dictionary(HEALTH_VALUE_MAP))"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"PARAMTYPE_MAP \u003d {"},{"line_number":39,"context_line":"    \u0027string\u0027: res_cons.PARAMTYPE_STRING,"},{"line_number":40,"context_line":"    \u0027number\u0027: res_cons.PARAMTYPE_NUMBER"},{"line_number":41,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_2004e3c5","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":13},"in_reply_to":"3f79a3b5_719f2865","updated":"2018-09-01 14:17:08.000000000","message":"Done","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"}],"sushy/resources/messageregistry.py":[{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"932fa6b45abbee5c40dc6b68cc3ea6e3c2be13ce","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \"\"\"Template text of the message"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    Template can include placeholders for message arguments in form"},{"line_number":30,"context_line":"    %\u003cinteger\u003e where \u003cinteger\u003e denotes an order passed from MessageArgs."},{"line_number":31,"context_line":"    \"\"\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    number_of_args \u003d base.Field(\u0027NumberOfArgs\u0027, required\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_1f2d6b8f","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":72},"updated":"2018-07-27 11:24:45.000000000","message":"nit: may be replacing `order` by `position` would be easier on the reader just because `positional args` is a well-established parlance I think","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"4568eab50b80b6e3c40e7124532fbd6df70df02d","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \"\"\"Template text of the message"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    Template can include placeholders for message arguments in form"},{"line_number":30,"context_line":"    %\u003cinteger\u003e where \u003cinteger\u003e denotes an order passed from MessageArgs."},{"line_number":31,"context_line":"    \"\"\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    number_of_args \u003d base.Field(\u0027NumberOfArgs\u0027, required\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_044e7b6d","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":72},"in_reply_to":"5f7c97a3_1f2d6b8f","updated":"2018-07-27 15:50:12.000000000","message":"Done","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":26340,"name":"Ilya Etingof","email":"etingof@gmail.com","username":"etingof"},"change_message_id":"932fa6b45abbee5c40dc6b68cc3ea6e3c2be13ce","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    param_types \u003d base.Field(\u0027ParamTypes\u0027,"},{"line_number":39,"context_line":"                             adapter\u003dlambda x:"},{"line_number":40,"context_line":"                                 [res_maps.PARAMTYPE_MAP.get(v) for v in x])"},{"line_number":41,"context_line":"    \"\"\"Mapped MessageArg types, in order, for the message\"\"\""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    resolution \u003d base.Field(\u0027Resolution\u0027, required\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_7f625fdb","line":40,"range":{"start_line":40,"start_character":57,"end_line":40,"end_character":63},"updated":"2018-07-27 11:24:45.000000000","message":"would possible (?) `None` be ok?","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"4568eab50b80b6e3c40e7124532fbd6df70df02d","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    param_types \u003d base.Field(\u0027ParamTypes\u0027,"},{"line_number":39,"context_line":"                             adapter\u003dlambda x:"},{"line_number":40,"context_line":"                                 [res_maps.PARAMTYPE_MAP.get(v) for v in x])"},{"line_number":41,"context_line":"    \"\"\"Mapped MessageArg types, in order, for the message\"\"\""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    resolution \u003d base.Field(\u0027Resolution\u0027, required\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_64f5ef83","line":40,"range":{"start_line":40,"start_character":57,"end_line":40,"end_character":63},"in_reply_to":"5f7c97a3_7f625fdb","updated":"2018-07-27 15:50:12.000000000","message":"No, changed to [] and added a test. \nThis could happen when Redfish standard has added new param type, but sushy is not up-to-date or user is running older version.","commit_id":"547233f3ce4f3c5d739a641808022e64afbb4c71"}],"sushy/resources/registry/messageregistry.py":[{"author":{"_account_id":16635,"name":"deray","email":"debayan.ray@gmail.com","username":"deray"},"change_message_id":"16c83635e3c4e9898b586116f6abec9e28215140","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":2,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":3,"context_line":"#    a copy of the License at"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_f13c9831","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":1},"updated":"2018-08-31 10:20:17.000000000","message":"nit: s/messageregistry.py/message_registry.py\n\nJust an afterthought!","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"},{"author":{"_account_id":16635,"name":"deray","email":"debayan.ray@gmail.com","username":"deray"},"change_message_id":"bc87a2a62a7a9df51409e612999d0675accd66ae","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":2,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":3,"context_line":"#    a copy of the License at"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_87f8627c","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":1},"in_reply_to":"3f79a3b5_c0f2cfe3","updated":"2018-09-03 10:52:46.000000000","message":"I feel the backward compatibility thing will possibly be a hindrance here in the cleanup of sessionservice.py.","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"},{"author":{"_account_id":27909,"name":"Aija Jauntēva","email":"code@clusums.eu","username":"ajya"},"change_message_id":"5906ecf9fc749b49e4ed4ca54f237a52c139e3e3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":2,"context_line":"#    not use this file except in compliance with the License. You may obtain"},{"line_number":3,"context_line":"#    a copy of the License at"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_c0f2cfe3","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":1},"in_reply_to":"3f79a3b5_f13c9831","updated":"2018-09-01 14:17:08.000000000","message":"done,\nI was mislead by sessionservice and did not look further where there is simple_storage and ethernet_interface. Perhaps sessionservice needs some cleanup too?","commit_id":"6bddd338edac3d83fb6438e12bf4767c3f9fb48c"}]}