lablup / lablup/backend.ai

Align action names with the operation each one declares

Open
#13,880 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
670
Forks
183
Avg merge
17h 7m
Merged PRs (30d)
358

Description

카탈로그로 훑어 찾은 것. 액션 이름의 동사와 선언한 operation이 서로 다른 자리들이다. 감사 행에는 이름과 operation이 함께 남으므로, 목록에서 이름만 보고 무슨 동작인지 짐작할 수 없다.

이름은 get인데 SEARCH: get_artifact_revisions, get_all_images, get_image_by_id, get_image_by_identifier, get_images_by_canonicals. 동사가 read인 것: read_registry_quota (GET).

이름은 delete인데 PURGE: delete_retention_policy. PurgeEntityOpsAction을 상속해 행을 지운다.

vfolder의 스토리지 세션과 파일 연산: create_vfolder_download_session, create_vfolder_archive_download_session, create_vfolder_download_session_v2가 GET이고, create_vfolder_upload_session_v2와 delete_vfolder_files_v2가 UPDATE, list_vfolder_files_v2와 search_storage_host_permissions가 GET이다. 이름과 연산이 서로 다른 방향을 가리킨다.

dotfile, error_log, keypair의 create_/delete_가 UPDATE인 것은 규칙대로다 (필드 쓰기는 전부 UPDATE). 이 목록에 넣지 않는다.

할 일: 각 자리에서 이름과 operation 중 어느 쪽이 틀렸는지 정하고 맞춘다. action_name을 바꾸면 그 지점에서 감사 이력이 갈리므로, 이름을 바꿀지 operation 선언을 바꿀지는 건별로 판단한다. lookup 이름에 키를 넣지 않는 규칙을 어긴 lookup_keypair_owner_by_access_key도 같이 본다.

JIRA Issue: BA-7435

Contributor guide

Open the contributing guide

Research direction

Start by cataloging the action declarations and comparing each action name with its declared operation, including the listed vfolder, artifact, registry, and lookup_keypair_owner_by_access_key cases. Decide case by case whether the name or operation is wrong, preserve the audit-history implications of action_name changes, and finish with every identified mismatch aligned while leaving the stated UPDATE exceptions unchanged.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.