Petsc conifg opts changed

yes, I agree this needs cleanup, the `self.cfg.update` should be done outside of the `if`/`else`