feat(l2): always use monitor (#3759)

**Motivation**
Once all monitor issues are resolved, it should be used by default
<!-- Why does this pull request exist? What are its goals? -->

**Description**
Change the `--monitor` flag for the `--no-monitor` one, which turns off
the monitor
<!-- A clear and concise general description of the changes this PR
introduces -->

<!-- Link to issues: Resolves #111, Resolves #222 -->

It should only be merged after all other issues are resolved
#3513,
#3526,
#3534,
#3516,
#3520,
#3521,
#3522,
#3523,
#3524,
#3525,
#3527,
#3528,
#3529,
#3530,
#3531,
#3532,
#3695,
#3757,
#3736

Author: gianbelinche

cmd/ethrex/l2/initializers.rs

@@ -119,7 +119,7 @@ pub fn init_metrics(opts: &L1Options, tracker: TaskTracker) {
 }
 
 pub fn init_tracing(opts: &L2Options) {
-    if opts.sequencer_opts.monitor {
+    if !opts.sequencer_opts.no_monitor {
         let level_filter = EnvFilter::builder()
             .parse_lossy("debug,tower_http::trace=debug,reqwest_tracing=off,hyper=off,libsql=off,ethrex::initializers=off,ethrex::l2::initializers=off,ethrex::l2::command=off");
         let subscriber = tracing_subscriber::registry()

cmd/ethrex/l2/options.rs

@@ -95,7 +95,7 @@ pub struct SequencerOptions {
         env = "ETHREX_MONITOR",
         help_heading = "Sequencer options"
     )]
-    pub monitor: bool,
+    pub no_monitor: bool,
 }
 
 pub fn parse_signer(
@@ -207,7 +207,7 @@ impl TryFrom<SequencerOptions> for SequencerConfig {
                 aligned_sp1_elf_path: opts.aligned_opts.aligned_sp1_elf_path.unwrap_or_default(),
             },
             monitor: MonitorConfig {
-                enabled: opts.monitor,
+                enabled: !opts.no_monitor,
                 tick_rate: opts.monitor_opts.tick_rate,
                 batch_widget_height: opts.monitor_opts.batch_widget_height,
             },