The logic is not incorrect. The most correct way to do this is probably:

```python
        if "*" in items:
            for p in glob(self.NS + "*.json"):
                self.fs_util.delete(p)
        elif archive_mode:
            for item in items:
                file_name = self.NS + "-" + item + ".json"
                self.fs_util.delete(file_name)
        if archive_mode:
            self.suite_engine_proc.job_logs_archive(suite_name, items)
```

But that's does not give us more value (and it is an extra line).