Skip to content

Commit 4c9e7a1

Browse files
authored
fix: do not fail on cluster name conflicts with 500 (#1171)
1 parent 3a359bf commit 4c9e7a1

1 file changed

Lines changed: 13 additions & 0 deletions

File tree

  • components/renku_data_services/crc

components/renku_data_services/crc/db.py

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1015,6 +1015,19 @@ async def insert(self, api_user: base_models.APIUser, cluster: ClusterSettings)
10151015

10161016
cluster_orm = ClusterORM.load(cluster)
10171017
async with self.session_maker() as session, session.begin():
1018+
res = await session.scalar(select(ClusterORM).where(ClusterORM.name == cluster_orm.name))
1019+
if res is not None:
1020+
raise errors.ConflictError(
1021+
message="Cannot create a cluster because the name is already used, please try a different name.",
1022+
quiet=True,
1023+
)
1024+
res = await session.scalar(select(ClusterORM).where(ClusterORM.config_name == cluster_orm.config_name))
1025+
if res is not None:
1026+
raise errors.ConflictError(
1027+
message="Cannot create a cluster because the config_name is already used, "
1028+
"please try a different one.",
1029+
quiet=True,
1030+
)
10181031
session.add(cluster_orm)
10191032
await session.flush()
10201033
await session.refresh(cluster_orm)

0 commit comments

Comments
 (0)