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

fix: Always set the scaling group when creating sessions #472

Merged
merged 4 commits into from
Aug 27, 2021

Conversation

achimnol
Copy link
Member

  • Setting it as None may schedule the session in non-allowed scaling groups.

* Setting it as None may schedule the session in non-allowed scaling groups.
@achimnol achimnol added this to the 20.09 milestone Aug 27, 2021
@achimnol achimnol added the bug label Aug 27, 2021
@codecov
Copy link

codecov bot commented Aug 27, 2021

Codecov Report

Merging #472 (5d550d4) into main (8c2b448) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #472      +/-   ##
==========================================
- Coverage   48.69%   48.66%   -0.03%     
==========================================
  Files          54       54              
  Lines        8874     8879       +5     
==========================================
  Hits         4321     4321              
- Misses       4553     4558       +5     
Impacted Files Coverage Δ
src/ai/backend/manager/models/scaling_group.py 65.66% <0.00%> (ø)
src/ai/backend/manager/registry.py 16.95% <0.00%> (-0.09%) ⬇️
src/ai/backend/manager/scheduler/dispatcher.py 18.23% <ø> (ø)

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 8c2b448...5d550d4. Read the comment docs.

@achimnol achimnol merged commit 7a42620 into main Aug 27, 2021
@achimnol achimnol deleted the fix/set-session-scaling-group-always branch August 27, 2021 15:57
achimnol added a commit that referenced this pull request Aug 27, 2021
* Setting it as None may schedule the session in non-allowed scaling groups.
* Add server-side warning log for scaling group fallback

Backported-From: main (21.09)
Backported-To: 21.03
achimnol added a commit that referenced this pull request Aug 27, 2021
* Setting it as None may schedule the session in non-allowed scaling groups.
* Add server-side warning log for scaling group fallback

Backported-From: main (21.09)
Backported-To: 20.09
lizable pushed a commit that referenced this pull request Sep 15, 2021
* Setting it as None may schedule the session in non-allowed scaling groups.
* Add server-side warning log for scaling group fallback
lizable pushed a commit that referenced this pull request Sep 15, 2021
* Setting it as None may schedule the session in non-allowed scaling groups.
* Add server-side warning log for scaling group fallback
# 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.

1 participant