Merge pull request #13 from hnez/systemd-notify-early

ingres: poll: do not poll before notifying systemd

src/ingres/poll.rs

@@ -160,7 +160,7 @@ impl Poller {
     /// How far back to go in the run history is decided by `MAX_NEW_RUN_AGE`,
     /// the most recent run id already known for the repository and the list
     /// of runs the `create::jobs::Manager` is interested in.
-    pub async fn poll_once(&self) -> octocrab::Result<()> {
+    async fn poll_once(&self) -> octocrab::Result<()> {
         let cfg = self.config.get();
 
         // These are runs for which we have jobs in "interesting" states,

src/main.rs

@@ -47,10 +47,6 @@ async fn forrest() -> anyhow::Result<()> {
     // missed webhooks.
     let poller = ingres::Poller::new(config.clone(), auth.clone(), job_manager);
 
-    // Make sure we can reach GitHub and our authentication works before
-    // signaling readiness to systemd.
-    poller.poll_once().await?;
-
     log::info!("Startup complete. Handling requests");
 
     // Notify systemd that we are ready to handle requests.