Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Implement check-and-pull mechanism for session creation lifecycle #2721

Conversation

fregataa
Copy link
Member

@fregataa fregataa commented Aug 16, 2024

ref #2276 (Step 2)

As is

Pending -> Scheduled -> Preparing (-> Pulling) -> Running -> ...

To do

Pending -> Scheduled -> Preparing (-> Pulling) -> Prepared -> Preparing(After #3114 Creating) -> Running -> ...

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Mention to the original issue
  • API server-client counterparts (e.g., manager API -> client SDK)
  • Documentation
    • Contents in the docs directory
    • docstrings in public interfaces and type annotations

📚 Documentation preview 📚: https://sorna--2721.org.readthedocs.build/en/2721/


📚 Documentation preview 📚: https://sorna-ko--2721.org.readthedocs.build/ko/2721/

Copy link
Member Author

fregataa commented Aug 16, 2024

@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 913c5b1 to 4d6c7c0 Compare August 16, 2024 06:54
@fregataa fregataa added this to the 24.09 milestone Aug 16, 2024
@fregataa fregataa changed the title feat: Apply 'check-and-pull' mechanism to session creation lifecycle feat: Apply check-and-pull mechanism to session creation lifecycle Aug 16, 2024
@fregataa fregataa force-pushed the topic/08-10-feat_add_check-and-pull_rpc_function branch from e777812 to 186819e Compare August 19, 2024 10:56
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 4d6c7c0 to b2640fc Compare August 19, 2024 10:56
@fregataa fregataa force-pushed the topic/08-10-feat_add_check-and-pull_rpc_function branch from 186819e to 72350c8 Compare August 25, 2024 06:37
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from b2640fc to fbe61c2 Compare August 25, 2024 06:37
@fregataa fregataa force-pushed the topic/08-10-feat_add_check-and-pull_rpc_function branch from 72350c8 to 0927a15 Compare August 31, 2024 16:16
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from fbe61c2 to 2ad51d8 Compare August 31, 2024 16:17
@fregataa fregataa force-pushed the topic/08-10-feat_add_check-and-pull_rpc_function branch from 0927a15 to 2dbab11 Compare September 1, 2024 08:51
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 2ad51d8 to 5fac8cc Compare September 1, 2024 08:52
@fregataa fregataa force-pushed the topic/08-10-feat_add_check-and-pull_rpc_function branch from 03030bd to e054703 Compare September 1, 2024 13:51
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch 2 times, most recently from f3976df to 66a1cbb Compare September 2, 2024 02:31
@fregataa fregataa force-pushed the topic/08-10-feat_add_check-and-pull_rpc_function branch from e054703 to c26d5ee Compare September 2, 2024 08:26
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 66a1cbb to 0621dbb Compare September 2, 2024 08:26
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 61fa41f to 1cf8073 Compare November 19, 2024 12:40
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 1cf8073 to 22258b1 Compare November 22, 2024 04:51
@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 22258b1 to b5741af Compare November 25, 2024 08:55
Comment on lines 299 to +301
load_aliases=False,
*,
loading_options: Iterable[RelationLoadingOption] = tuple(),

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

write a issue to refactor this bool flag

Comment on lines +3281 to +3283
(KernelRow.image == image)
& (KernelRow.agent == agent_id)
& (KernelRow.status == KernelStatus.SCHEDULED)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sa.and_?

@fregataa fregataa force-pushed the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch from 0b24b49 to 3ce06ff Compare November 26, 2024 08:19
Copy link

@HyeockJinKim HyeockJinKim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fregataa fregataa added this pull request to the merge queue Nov 26, 2024
Merged via the queue into main with commit 56cf12b Nov 26, 2024
24 checks passed
@fregataa fregataa deleted the topic/08-16-feat_apply_check-and-pull_mechanism_to_session_creation_lifecycle branch November 26, 2024 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:docs Documentations comp:common Related to Common component comp:manager Related to Manager component size:L 100~500 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants