Is this missing a ``session.commit()``?