Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

feat: Add support for session renaming #505

Merged
merged 4 commits into from
Dec 21, 2021

Conversation

kyujin-cho
Copy link
Member

This PR adds new rename_session API, as described on lablup/backend.ai#296.

@codecov
Copy link

codecov bot commented Dec 16, 2021

Codecov Report

Merging #505 (b8485d5) into main (d8d525a) will increase coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #505      +/-   ##
==========================================
+ Coverage   48.84%   49.03%   +0.19%     
==========================================
  Files          54       54              
  Lines        8943     9340     +397     
==========================================
+ Hits         4368     4580     +212     
- Misses       4575     4760     +185     
Impacted Files Coverage Δ
src/ai/backend/manager/models/kernel.py 44.77% <0.00%> (+0.30%) ⬆️
src/ai/backend/manager/models/scaling_group.py 67.40% <0.00%> (+1.74%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8d525a...b8485d5. Read the comment docs.

@kyujin-cho kyujin-cho requested a review from achimnol December 16, 2021 04:54
@kyujin-cho kyujin-cho self-assigned this Dec 16, 2021
@kyujin-cho kyujin-cho added this to the 21.09 milestone Dec 16, 2021
@kyujin-cho kyujin-cho added the API label Dec 16, 2021
@achimnol achimnol changed the title feat: add support for session name renaming feat: Add support for session renaming Dec 21, 2021
@achimnol achimnol merged commit a508ebc into main Dec 21, 2021
@achimnol achimnol deleted the feature/running-session-name-update branch December 21, 2021 04:45
achimnol pushed a commit that referenced this pull request Dec 21, 2021
* Implement rename_session API

Backported-From: main (22.03)
Backported-To: 21.09
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants