)]}'
{"ironic_python_agent/utils.py":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"9be46a59b5d232195b7574f12f0d50bf7699c4ba","unresolved":true,"context_lines":[{"line_number":839,"context_line":"            execute(\u0027chronyc\u0027, \u0027shutdown\u0027, check_exit_code\u003d[0, 1])"},{"line_number":840,"context_line":"            # force a time sync now"},{"line_number":841,"context_line":"            query \u003d \"server \" + CONF.ntp_server + \" iburst\""},{"line_number":842,"context_line":"            execute(\"chronyd -q \\\u0027%s\\\u0027\" % query, shell\u003dTrue)"},{"line_number":843,"context_line":"            LOG.debug(\u0027Set software clock using chrony\u0027)"},{"line_number":844,"context_line":"        except (processutils.ProcessExecutionError,"},{"line_number":845,"context_line":"                errors.CommandExecutionError) as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fe5e2d1d_5c790795","line":842,"updated":"2021-07-16 09:53:43.000000000","message":"after this, should we start the daemon to keep the clock in sync?","commit_id":"5531d5cee744f90ff24231a9ef467282b254adc2"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"a1c47b3055e1b6fe126c3aa85f4b7054a2c7d8ed","unresolved":true,"context_lines":[{"line_number":839,"context_line":"            execute(\u0027chronyc\u0027, \u0027shutdown\u0027, check_exit_code\u003d[0, 1])"},{"line_number":840,"context_line":"            # force a time sync now"},{"line_number":841,"context_line":"            query \u003d \"server \" + CONF.ntp_server + \" iburst\""},{"line_number":842,"context_line":"            execute(\"chronyd -q \\\u0027%s\\\u0027\" % query, shell\u003dTrue)"},{"line_number":843,"context_line":"            LOG.debug(\u0027Set software clock using chrony\u0027)"},{"line_number":844,"context_line":"        except (processutils.ProcessExecutionError,"},{"line_number":845,"context_line":"                errors.CommandExecutionError) as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ccd1945b_dd15d94d","line":842,"in_reply_to":"2f617dec_065aa80e","updated":"2021-07-16 12:50:30.000000000","message":"right, I\u0027ll be optimistic for once :)","commit_id":"5531d5cee744f90ff24231a9ef467282b254adc2"},{"author":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"change_message_id":"0761176854f204bb1e49cfd57b922882b70ff097","unresolved":true,"context_lines":[{"line_number":839,"context_line":"            execute(\u0027chronyc\u0027, \u0027shutdown\u0027, check_exit_code\u003d[0, 1])"},{"line_number":840,"context_line":"            # force a time sync now"},{"line_number":841,"context_line":"            query \u003d \"server \" + CONF.ntp_server + \" iburst\""},{"line_number":842,"context_line":"            execute(\"chronyd -q \\\u0027%s\\\u0027\" % query, shell\u003dTrue)"},{"line_number":843,"context_line":"            LOG.debug(\u0027Set software clock using chrony\u0027)"},{"line_number":844,"context_line":"        except (processutils.ProcessExecutionError,"},{"line_number":845,"context_line":"                errors.CommandExecutionError) as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"2f617dec_065aa80e","line":842,"in_reply_to":"fe5e2d1d_5c790795","updated":"2021-07-16 10:05:31.000000000","message":"I was thinking this as well initially, but we do not do this for the other method either.\n(Also, I would not expect the clock to drift away quickly enough to cause harm during the life-time of the IPA when we have sync-ed it just now.)","commit_id":"5531d5cee744f90ff24231a9ef467282b254adc2"}]}