)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35967,"name":"Lindley Werner Soares Vieira","display_name":"Lindley Werner","email":"lindley.vieira@encora.com","username":"lindleywerner"},"change_message_id":"28bde60560c7666d3de7bde7bac961a9592b6c06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"d1c97a20_3c7e7192","updated":"2023-05-02 19:52:06.000000000","message":"Just a small refactoring","commit_id":"4cb19afa0b5b7dd725a159e09187b7eb64926b03"},{"author":{"_account_id":35920,"name":"dcaires","display_name":"Daniel Caires","email":"daniel.caires@encora.com","username":"Daniel.Caires"},"change_message_id":"1b2ddf047c2b4c627451d235b3fd1ec2e1cf62e9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"d715e034_212a1f57","updated":"2023-05-11 11:32:26.000000000","message":"I believe an old patch was commited and broke the code","commit_id":"55a838b7c413b8e8810e350de23e89d497f0db6f"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"a590386d06a4652cfcf3cd578ff94627823e7116","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"fc9cfc06_99d9ce3b","in_reply_to":"d715e034_212a1f57","updated":"2023-05-11 12:06:28.000000000","message":"Oh, indeed. What have I done? I\u0027ll fix this.","commit_id":"55a838b7c413b8e8810e350de23e89d497f0db6f"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"c1ab94a0dd64e2a29e45fa6ac8c20ebd1ccdf1c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"4cbe27b4_9baf3624","in_reply_to":"fc9cfc06_99d9ce3b","updated":"2023-05-11 12:28:09.000000000","message":"Ok, I reverted patchset 17. Good catch. Thanks!","commit_id":"55a838b7c413b8e8810e350de23e89d497f0db6f"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"4035c70e5e10916d1187e68b98d9c92f883c48da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"1d879fde_098af755","updated":"2023-05-23 14:27:03.000000000","message":"It would be good to remove, on the future, hard-coded passwords from the code if possible, for instance the script could ask for user input before starting","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"ddcc305ef96b725b9e2661057bc3e8d43ae714eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"fcb66626_e18709ac","in_reply_to":"1d879fde_098af755","updated":"2023-05-23 18:19:07.000000000","message":"We\u0027re working on that on another task/review, but we want to get this one in because it brings this script back to life and makes it work at least for the minimal setup without breaking too much of the current design.\n\nSubsequent changes might change the design of the code. FYI, here\u0027s where we\u0027re baking the password thing: https://review.opendev.org/c/starlingx/tools/+/883879","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"}],"deployment/virtualbox/pybox/README.md":[{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"5c5ee58ab91f070afc261bbbfb0fddc0d5436981","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Pybox"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"This automated installer provides you with an easy way to install"},{"line_number":4,"context_line":"StarlingX in many different configuration options. The following "}],"source_content_type":"text/x-gfm","patch_set":1,"id":"28f8d7cb_8ac0e1f4","line":1,"updated":"2023-04-19 16:49:05.000000000","message":"I renamed the file and this makes the review more difficult. I\u0027m sorry for that.\n\nI basically replaced the first paragraph with the whole first section of the document. And renamed the \"Installation\" section at the end while adding a step-by-step approach on how to get StarlingX installed with the code here.\n\nIs it ok to change it from a TXT file to a Markdown file? I was having trouble with identifying the patterns of what represents a title, subtitle etc and thought that Markdown would help with that.","commit_id":"f71bb467d7d999e820da19d42d015514bc2b9916"},{"author":{"_account_id":28739,"name":"Michel Thebeau","email":"michel.thebeau@windriver.com","username":"michel.thebeau"},"change_message_id":"dc892c915ff786240b0fc5b2fc42fbf1c392f9c4","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Pybox"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"This automated installer provides you with an easy way to install"},{"line_number":4,"context_line":"StarlingX in many different configuration options. The following "}],"source_content_type":"text/x-gfm","patch_set":1,"id":"662da6c4_0f06bd85","line":1,"in_reply_to":"28f8d7cb_8ac0e1f4","updated":"2023-04-19 17:29:18.000000000","message":"Renaming is fine.  Reviewers can cherry-pick and compare like this:\n\n  git diff \\ \n \n HEAD~:deployment/virtualbox/pybox/README.txt..HEAD:deployment/virtualbox/pybox/README.md","commit_id":"f71bb467d7d999e820da19d42d015514bc2b9916"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"9c99abe2400e279d266c2bfa30dc7d6edce238d1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Pybox"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"This automated installer provides you with an easy way to install"},{"line_number":4,"context_line":"StarlingX in many different configuration options. The following "}],"source_content_type":"text/x-gfm","patch_set":1,"id":"d4ec02df_5dece62e","line":1,"in_reply_to":"662da6c4_0f06bd85","updated":"2023-04-20 22:04:46.000000000","message":"Ack","commit_id":"f71bb467d7d999e820da19d42d015514bc2b9916"}],"deployment/virtualbox/pybox/configs/aio-sx/lab_setup1.sh":[{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"0509dbcd6eee53d66988f52576e8789efe668a20","unresolved":true,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"##This file makes the necesssary configuration for the unlock of the Controlller-0"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"GROUPNO\u003d0"},{"line_number":5,"context_line":"DATE_FORMAT\u003d\"%Y-%m-%d %T\""}],"source_content_type":"text/x-sh","patch_set":18,"id":"4d970c61_b0f13832","line":2,"range":{"start_line":2,"start_character":69,"end_line":2,"end_character":82},"updated":"2023-05-23 14:12:23.000000000","message":"nit: controller-0","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"0509dbcd6eee53d66988f52576e8789efe668a20","unresolved":true,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"##This file makes the necesssary configuration for the unlock of the Controlller-0"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"GROUPNO\u003d0"},{"line_number":5,"context_line":"DATE_FORMAT\u003d\"%Y-%m-%d %T\""}],"source_content_type":"text/x-sh","patch_set":18,"id":"2a7c7724_f36d065a","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":32},"updated":"2023-05-23 14:12:23.000000000","message":"nit: necessary","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"8a06051dfbd851e88b2943a479c2433b1b65dba9","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"##This file makes the necesssary configuration for the unlock of the Controlller-0"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"GROUPNO\u003d0"},{"line_number":5,"context_line":"DATE_FORMAT\u003d\"%Y-%m-%d %T\""}],"source_content_type":"text/x-sh","patch_set":18,"id":"60d2ead3_2a40b8bc","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":32},"in_reply_to":"2a7c7724_f36d065a","updated":"2023-05-23 16:25:27.000000000","message":"Ack","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"8a06051dfbd851e88b2943a479c2433b1b65dba9","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"##This file makes the necesssary configuration for the unlock of the Controlller-0"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"GROUPNO\u003d0"},{"line_number":5,"context_line":"DATE_FORMAT\u003d\"%Y-%m-%d %T\""}],"source_content_type":"text/x-sh","patch_set":18,"id":"9b9edfa8_192951fc","line":2,"range":{"start_line":2,"start_character":69,"end_line":2,"end_character":82},"in_reply_to":"4d970c61_b0f13832","updated":"2023-05-23 16:25:27.000000000","message":"Ack","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"ce91b82fabce5f22edbdd3d40b36c01e6325d110","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"##This file makes the necesssary configuration for the unlock of the Controlller-0"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"GROUPNO\u003d0"},{"line_number":5,"context_line":"DATE_FORMAT\u003d\"%Y-%m-%d %T\""}],"source_content_type":"text/x-sh","patch_set":18,"id":"ca85756e_d1b25c52","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":32},"in_reply_to":"60d2ead3_2a40b8bc","updated":"2023-05-23 19:20:59.000000000","message":"Done","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"ce91b82fabce5f22edbdd3d40b36c01e6325d110","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"##This file makes the necesssary configuration for the unlock of the Controlller-0"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"GROUPNO\u003d0"},{"line_number":5,"context_line":"DATE_FORMAT\u003d\"%Y-%m-%d %T\""}],"source_content_type":"text/x-sh","patch_set":18,"id":"94a63b01_640e3960","line":2,"range":{"start_line":2,"start_character":69,"end_line":2,"end_character":82},"in_reply_to":"9b9edfa8_192951fc","updated":"2023-05-23 19:20:59.000000000","message":"Done","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"0509dbcd6eee53d66988f52576e8789efe668a20","unresolved":true,"context_lines":[{"line_number":39,"context_line":"    if [ ${RET} -ne 0 ]; then"},{"line_number":40,"context_line":"        info \"COMMAND FAILED (rc\u003d${RET}): ${CMD}\""},{"line_number":41,"context_line":"        info \"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\""},{"line_number":42,"context_line":"        info \"Check \\\"${LOG_FILE}\\\" for more details, and re-run the failed\""},{"line_number":43,"context_line":"        info \"command manually before contacting the domain owner for assistance.\""},{"line_number":44,"context_line":"        exit 1"},{"line_number":45,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":18,"id":"9e106e19_59f1037d","line":42,"range":{"start_line":42,"start_character":54,"end_line":42,"end_character":64},"updated":"2023-05-23 14:12:23.000000000","message":"suggestion: \"fix the issues and re-run\"","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"ce91b82fabce5f22edbdd3d40b36c01e6325d110","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    if [ ${RET} -ne 0 ]; then"},{"line_number":40,"context_line":"        info \"COMMAND FAILED (rc\u003d${RET}): ${CMD}\""},{"line_number":41,"context_line":"        info \"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\""},{"line_number":42,"context_line":"        info \"Check \\\"${LOG_FILE}\\\" for more details, and re-run the failed\""},{"line_number":43,"context_line":"        info \"command manually before contacting the domain owner for assistance.\""},{"line_number":44,"context_line":"        exit 1"},{"line_number":45,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":18,"id":"edc7d156_f1819360","line":42,"range":{"start_line":42,"start_character":54,"end_line":42,"end_character":64},"in_reply_to":"1b148aa2_40c3d2bf","updated":"2023-05-23 19:20:59.000000000","message":"Done","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"8a06051dfbd851e88b2943a479c2433b1b65dba9","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    if [ ${RET} -ne 0 ]; then"},{"line_number":40,"context_line":"        info \"COMMAND FAILED (rc\u003d${RET}): ${CMD}\""},{"line_number":41,"context_line":"        info \"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\""},{"line_number":42,"context_line":"        info \"Check \\\"${LOG_FILE}\\\" for more details, and re-run the failed\""},{"line_number":43,"context_line":"        info \"command manually before contacting the domain owner for assistance.\""},{"line_number":44,"context_line":"        exit 1"},{"line_number":45,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":18,"id":"1b148aa2_40c3d2bf","line":42,"range":{"start_line":42,"start_character":54,"end_line":42,"end_character":64},"in_reply_to":"9e106e19_59f1037d","updated":"2023-05-23 16:25:27.000000000","message":"Ack","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"d5e2d776aeda3c814086c285b2399c2952d64548","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":19,"id":"e1c98845_ae3e96ac","side":"PARENT","updated":"2023-06-12 12:20:27.000000000","message":"Is this script really needed? It\u0027s name and content seems like it should be part of lab_setup.sh","commit_id":"b875514efc66013f55764e112d2c395cb0a0c650"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"6ad918fea4c0b2184413fa90ce609820b2e34491","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"ea6f1009_bf375a3e","side":"PARENT","in_reply_to":"68cf3c53_a8a3a86c","updated":"2023-06-12 14:52:35.000000000","message":"OK, thanks for clarifying","commit_id":"b875514efc66013f55764e112d2c395cb0a0c650"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"0c7277ff66418bfe6a27112019c072e258c15f15","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":19,"id":"68cf3c53_a8a3a86c","side":"PARENT","in_reply_to":"e1c98845_ae3e96ac","updated":"2023-06-12 13:35:25.000000000","message":"It could be part of `lab_setup.sh`, but our long-term plan is to \"deprecate\" the huge file and break it into smaller and more purpose-driven (like `network_setup.sh` and/or `applications_setup.sh` etc.) files. \n\nI\u0027m not happy with the name either, but this commit is just touching the bare minimum to make this script work, and the script has a set of 6 or 7 custom steps, and IIRC we used the first custom step to push and run this script, hence the temporary name.","commit_id":"b875514efc66013f55764e112d2c395cb0a0c650"}],"deployment/virtualbox/pybox/install_vbox.py":[{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"9f8c55488534d213cda5e6128b3cd5129727e506","unresolved":false,"context_lines":[{"line_number":790,"context_line":"            sftp_send("},{"line_number":791,"context_line":"                fp.name, remote_host\u003dip, remote_port\u003dport, destination\u003ddestination,"},{"line_number":792,"context_line":"                username\u003dvboxoptions.username, password\u003dvboxoptions.password)"},{"line_number":793,"context_line":"    else:"},{"line_number":794,"context_line":"        destination \u003d \"/home/\" + \\"},{"line_number":795,"context_line":"            vboxoptions.username + \"/stx_config.ini_centos\""},{"line_number":796,"context_line":"        sftp_send("}],"source_content_type":"text/x-python","patch_set":2,"id":"177e178a_b0a14948","line":793,"updated":"2023-04-20 17:51:45.000000000","message":"This `else` was unreachable.","commit_id":"f3572908d8e4e3b76ac7d17892e5a7b43b12b3de"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"9f8c55488534d213cda5e6128b3cd5129727e506","unresolved":false,"context_lines":[{"line_number":867,"context_line":"        send_dir(source\u003dlocal_path, remote_host\u003dip, remote_port\u003dport,"},{"line_number":868,"context_line":"                 destination\u003d\u0027/home/\u0027 + vboxoptions.username + \u0027/\u0027,"},{"line_number":869,"context_line":"                 username\u003dvboxoptions.username, password\u003dvboxoptions.password,"},{"line_number":870,"context_line":"                 follow_links\u003dfollow_links)"},{"line_number":871,"context_line":""},{"line_number":872,"context_line":""},{"line_number":873,"context_line":"@connect_to_serialz"}],"source_content_type":"text/x-python","patch_set":3,"id":"f7f83b87_0df94290","line":870,"updated":"2023-04-20 17:51:45.000000000","message":"The fix here is sending the `follow_links` parameter, which was not being sent before.\n\nThe early return was something that I just decided to do.","commit_id":"d42c4b9d57a048b67d055176d87a58773f9c6a0a"},{"author":{"_account_id":35967,"name":"Lindley Werner Soares Vieira","display_name":"Lindley Werner","email":"lindley.vieira@encora.com","username":"lindleywerner"},"change_message_id":"28bde60560c7666d3de7bde7bac961a9592b6c06","unresolved":true,"context_lines":[{"line_number":905,"context_line":"    elif stage \u003d\u003d 5:"},{"line_number":906,"context_line":"        setup_file \u003d \"lab_setup5.sh\""},{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    command \u003d f\u0027source /etc/platform/openrc; export PATH\u003d\"$PATH:/usr/local/bin\"; export PATH\u003d\"$PATH:/usr/bin\"; export PATH\u003d\"$PATH:/usr/local/sbin\"; export PATH\u003d\"$PATH:/usr/sbin\"; sh {setup_file}\u0027"},{"line_number":909,"context_line":""},{"line_number":910,"context_line":""},{"line_number":911,"context_line":"    _, _, exitcode \u003d run_ssh_cmd(ssh_client, command, timeout\u003dHostTimeout.LAB_INSTALL)"}],"source_content_type":"text/x-python","patch_set":13,"id":"b278a6a3_08739cf7","line":908,"range":{"start_line":908,"start_character":182,"end_line":908,"end_character":194},"updated":"2023-05-02 19:52:06.000000000","message":"You can substitute all these ifs by a \n...; sh lab_setup{stage}.sh ...","commit_id":"4cb19afa0b5b7dd725a159e09187b7eb64926b03"},{"author":{"_account_id":35967,"name":"Lindley Werner Soares Vieira","display_name":"Lindley Werner","email":"lindley.vieira@encora.com","username":"lindleywerner"},"change_message_id":"22729fe5a223d1f975e081d7a26a0ff247485870","unresolved":false,"context_lines":[{"line_number":905,"context_line":"    elif stage \u003d\u003d 5:"},{"line_number":906,"context_line":"        setup_file \u003d \"lab_setup5.sh\""},{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    command \u003d f\u0027source /etc/platform/openrc; export PATH\u003d\"$PATH:/usr/local/bin\"; export PATH\u003d\"$PATH:/usr/bin\"; export PATH\u003d\"$PATH:/usr/local/sbin\"; export PATH\u003d\"$PATH:/usr/sbin\"; sh {setup_file}\u0027"},{"line_number":909,"context_line":""},{"line_number":910,"context_line":""},{"line_number":911,"context_line":"    _, _, exitcode \u003d run_ssh_cmd(ssh_client, command, timeout\u003dHostTimeout.LAB_INSTALL)"}],"source_content_type":"text/x-python","patch_set":13,"id":"b005c43d_279785f0","line":908,"range":{"start_line":908,"start_character":182,"end_line":908,"end_character":194},"in_reply_to":"4ac483bf_5210c4b5","updated":"2023-05-03 11:57:49.000000000","message":"Done","commit_id":"4cb19afa0b5b7dd725a159e09187b7eb64926b03"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"d4b482a29322e0923f6166b93f9565c1592a123a","unresolved":false,"context_lines":[{"line_number":905,"context_line":"    elif stage \u003d\u003d 5:"},{"line_number":906,"context_line":"        setup_file \u003d \"lab_setup5.sh\""},{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    command \u003d f\u0027source /etc/platform/openrc; export PATH\u003d\"$PATH:/usr/local/bin\"; export PATH\u003d\"$PATH:/usr/bin\"; export PATH\u003d\"$PATH:/usr/local/sbin\"; export PATH\u003d\"$PATH:/usr/sbin\"; sh {setup_file}\u0027"},{"line_number":909,"context_line":""},{"line_number":910,"context_line":""},{"line_number":911,"context_line":"    _, _, exitcode \u003d run_ssh_cmd(ssh_client, command, timeout\u003dHostTimeout.LAB_INSTALL)"}],"source_content_type":"text/x-python","patch_set":13,"id":"95d059f7_dee90039","line":908,"range":{"start_line":908,"start_character":182,"end_line":908,"end_character":194},"in_reply_to":"4ac483bf_5210c4b5","updated":"2023-05-03 13:36:12.000000000","message":"Done. Task added to https://storyboard.openstack.org/#!/story/2005051","commit_id":"4cb19afa0b5b7dd725a159e09187b7eb64926b03"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"9dd4e182b718ac4fdde467433033ff2c01a21854","unresolved":true,"context_lines":[{"line_number":905,"context_line":"    elif stage \u003d\u003d 5:"},{"line_number":906,"context_line":"        setup_file \u003d \"lab_setup5.sh\""},{"line_number":907,"context_line":""},{"line_number":908,"context_line":"    command \u003d f\u0027source /etc/platform/openrc; export PATH\u003d\"$PATH:/usr/local/bin\"; export PATH\u003d\"$PATH:/usr/bin\"; export PATH\u003d\"$PATH:/usr/local/sbin\"; export PATH\u003d\"$PATH:/usr/sbin\"; sh {setup_file}\u0027"},{"line_number":909,"context_line":""},{"line_number":910,"context_line":""},{"line_number":911,"context_line":"    _, _, exitcode \u003d run_ssh_cmd(ssh_client, command, timeout\u003dHostTimeout.LAB_INSTALL)"}],"source_content_type":"text/x-python","patch_set":13,"id":"4ac483bf_5210c4b5","line":908,"range":{"start_line":908,"start_character":182,"end_line":908,"end_character":194},"in_reply_to":"b278a6a3_08739cf7","updated":"2023-05-02 20:30:43.000000000","message":"Good point. Already on it. This makes me think that we also need unit tests in these and I don\u0027t think I added this to our backlog in the related Story. Gonna do it right away.","commit_id":"4cb19afa0b5b7dd725a159e09187b7eb64926b03"}],"deployment/virtualbox/pybox/requirements.txt":[{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"0509dbcd6eee53d66988f52576e8789efe668a20","unresolved":true,"context_lines":[{"line_number":1,"context_line":"configparser"},{"line_number":2,"context_line":"paramiko"},{"line_number":3,"context_line":"pytest"},{"line_number":4,"context_line":"git+https://github.com/digidotcom/python-streamexpect#egg\u003dstreamexpect"},{"line_number":5,"context_line":"pexpect"},{"line_number":6,"context_line":""}],"source_content_type":"text/plain","patch_set":18,"id":"73bc1057_788fda2d","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":70},"updated":"2023-05-23 14:12:23.000000000","message":"I\u0027m wondering why this lib had to be changed. Also, it\u0027s a common practice to specify the version of the requirements, e.g. paramiko\u003e\u003d1.8.0","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"},{"author":{"_account_id":35896,"name":"Bruno Drugowick Muniz","display_name":"Bruno","email":"bruno.muniz@encora.com","username":"brunomuniz"},"change_message_id":"8a06051dfbd851e88b2943a479c2433b1b65dba9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"configparser"},{"line_number":2,"context_line":"paramiko"},{"line_number":3,"context_line":"pytest"},{"line_number":4,"context_line":"git+https://github.com/digidotcom/python-streamexpect#egg\u003dstreamexpect"},{"line_number":5,"context_line":"pexpect"},{"line_number":6,"context_line":""}],"source_content_type":"text/plain","patch_set":18,"id":"c7f74c0b_806b2a6f","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":70},"in_reply_to":"73bc1057_788fda2d","updated":"2023-05-23 16:25:27.000000000","message":"This is before my era 😊... I\u0027ll make a task to understand why this happened (I think it\u0027s a compatibility issue) and add the current working versions for all dependencies.\n\nEDIT: created task 48077 in https://storyboard.openstack.org/#!/story/2005051","commit_id":"14c0fd7bbc7fee75240e01374593c7e319d63495"}]}