aai-institute / aai-institute/lakefs-spec

[upstream] Tag names cannot contain spaces

オープン
#282 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
bug triage
主要言語
Python
スター
65
フォーク
7
PR マージ指標
30日以内にマージされた PR はありません

説明

### What happened?

Creating tags as such:

```py
with fs.transaction(repository.id) as tx:
sha = tx.rev_parse("main")
tag = tx.tag(sha, "v2 pipapo hello")
```

does not work because the lakeFS API rejects tag names containing spaces. It says

```
lakefs.exceptions.BadRequestException: code: 400, reason: Bad Request, body: {'message': 'argument tagID: tag id: invalid value: validation error'}
```
but with no mention as to what the validation mechanism (probably regex?) actually is. This would benefit from a bit of validation before sending the API request, to give the user a clearer idea of what actually went wrong.

IMO, it doesn't actually matter where that validation is happening, so it might as well be in lakeFS's Python wrapper project.

### What did you expect to happen?

The tag being created just fine.

### lakeFS-spec version

HEAD

### lakeFS version

latest

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。