reverse launch order

Author: upsj

src/config.rs

@@ -88,9 +88,9 @@ impl<'de> serde::Deserialize<'de> for BoolOrString {
 pub struct GitLabRunnerInstance {
     /// Tags whose associated jobs will be run by this runner
     pub tags: Vec<String>,
-    /// Order in which the instances' launch processes should be executed
-    /// All jobs without an order will be launched last
-    pub launch_order: Option<u32>,
+    /// Priority in which the instances' launch processes should be executed, higher priority means earlier launch.
+    /// All jobs without a priority will be launched last
+    pub launch_priority: Option<u32>,
     /// Variables to be expanded in the template instantiation
     /// Naming to avoid confusing with environment variables
     pub config_variables: HashMap<String, String>,
@@ -241,7 +241,7 @@ pub fn get_default_config() -> GitLabRunnersConfig {
             "test-runner".to_owned(),
             GitLabRunnerInstance {
                 tags: vec!["tag-1".to_owned(), "tag-2".to_owned()],
-                launch_order: None,
+                launch_priority: None,
                 config_variables: [("VARIABLE", "value")]
                     .map(|(k, v)| (k.to_owned(), v.to_owned()))
                     .into_iter()

src/run.rs

@@ -178,8 +178,10 @@ async fn run_impl(paths: &cli::Paths, state: &MetaRunnerState) -> anyhow::Result
         }
         grouped_matched_jobs.get_mut(name).unwrap().1.push(job);
     }
+    // sort descending by priority
     let mut grouped_matched_jobs: Vec<_> = grouped_matched_jobs.into_iter().collect();
-    grouped_matched_jobs.sort_by_key(|(_, (instance, _))| instance.launch_order);
+    grouped_matched_jobs.sort_by_key(|(_, (instance, _))| instance.launch_priority);
+    grouped_matched_jobs.reverse();
     // Dispatch jobs
     let mut queue = Vec::new();
     // this unwrap can't fail because we ran check_config::check