I noticed that at least for heartbeat CLI tool, the logic for firedrill / no-firedrill needs to change.
In the past we had this as an argument to the messages, and messages were passing it around. However, currently the messages are independent of the firedrill mode, it only becomes relevant when we send the message to snews using send_messages() method. Thus, it needs to be moved there on the CLI tools as well.
I noticed that at least for heartbeat CLI tool, the logic for
firedrill / no-firedrill
needs to change.In the past we had this as an argument to the messages, and messages were passing it around. However, currently the messages are independent of the firedrill mode, it only becomes relevant when we send the message to snews using
send_messages()
method. Thus, it needs to be moved there on the CLI tools as well.