Skip to content

Commit a5471ae

Browse files
authored
Merge pull request #83 from vohoanglong0107/feat-make-refresh-more-robust
feat: make refresh more robust
2 parents 151951e + ca7886e commit a5471ae

File tree

1 file changed

+16
-3
lines changed

1 file changed

+16
-3
lines changed

src/bors/handlers/mod.rs

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -125,10 +125,23 @@ pub async fn handle_bors_global_event<Client: RepositoryClient>(
125125
let span = tracing::info_span!("Repository reload").entered();
126126

127127
match ctx.repository_loader.load_repositories().await {
128-
Ok(repos) => {
128+
Ok(reloaded_repos) => {
129129
let mut repositories = ctx.repositories.write().unwrap();
130-
// TODO: keep old repos in case of repo loading failure
131-
*repositories = repos;
130+
for repo in repositories.values() {
131+
if !reloaded_repos.contains_key(&repo.repository) {
132+
tracing::info!("Repository {} was not reloaded", repo.repository);
133+
}
134+
}
135+
for repo in reloaded_repos.values() {
136+
if repositories
137+
.insert(repo.repository.clone(), repo.clone())
138+
.is_some()
139+
{
140+
tracing::info!("Repository {} was reloaded", repo.repository);
141+
} else {
142+
tracing::info!("Repository {} was added", repo.repository);
143+
}
144+
}
132145
}
133146
Err(error) => {
134147
span.log_error(error);

0 commit comments

Comments
 (0)