Skip to content

Commit 75d6ba1

Browse files
committed
Fix service account requiring access to every resource to delete a database
1 parent f2125a8 commit 75d6ba1

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ fn setup_logger() -> Result<(), log::SetLoggerError> {
3737
})
3838
.level(if cfg!(debug_assertions) {
3939
// TODO: Add a flag to enable debug logging
40-
log::LevelFilter::Info
40+
log::LevelFilter::Debug
4141
} else {
4242
log::LevelFilter::Debug
4343
})

src/routes/delete_database.rs

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,19 @@ async fn delete_database(
3838
.as_str(),
3939
), //.labels(format!("app.kubernetes.io/managed-by=Moonscale").as_str()
4040
)
41-
.await?;
41+
.await;
4242

43-
for item in list.items {
43+
if list.is_err() {
44+
warn!(
45+
"Failed to list resources while deleting instance {} ({}/{})",
46+
instance,
47+
ar.kind,
48+
list.err().unwrap()
49+
);
50+
continue;
51+
}
52+
53+
for item in list.unwrap().items {
4454
let name = item.name_any();
4555
let ns = item.metadata.namespace.map(|s| s + "/").unwrap_or_default();
4656

0 commit comments

Comments
 (0)