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.

Sign up for GitHub

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

Jump to bottom

Skip loading local config in designated mgr/ag cli commands #1681

Closed
achimnol opened this issue Nov 1, 2023 · 0 comments · Fixed by #1686
Closed

Skip loading local config in designated mgr/ag cli commands #1681

achimnol opened this issue Nov 1, 2023 · 0 comments · Fixed by #1686
Labels
comp:cli Related to CLI component effort:normal Need to understand a few modules / some extent of contextual or historical information. urgency:2 With time limit, it should be finished within it; otherwise, resolve it when no other chores.
Milestone
23.09

Comments

@achimnol
Copy link
Member

achimnol commented Nov 1, 2023

By default, backend.ai mgr, backend.ai ag, and similar per-service CLI commands load the local configuration (manager.toml, agent.toml, ...) even when the specific command does not actually require the full local configuration.

Examples of such command is backend.ai mgr gql show, backend.ai mgr generate-api-keypair, etc.

Let's allow them to be runnable without having the configuration files, by extending CLIContext.
@achimnol achimnol added type:feature Add new features comp:cli Related to CLI component effort:normal Need to understand a few modules / some extent of contextual or historical information. urgency:2 With time limit, it should be finished within it; otherwise, resolve it when no other chores. labels Nov 1, 2023
@achimnol achimnol added this to the 23.09 milestone Nov 1, 2023
@achimnol achimnol mentioned this issue Nov 2, 2023
Merged
1 task
@achimnol achimnol removed the type:feature Add new features label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
comp:cli Related to CLI component effort:normal Need to understand a few modules / some extent of contextual or historical information. urgency:2 With time limit, it should be finished within it; otherwise, resolve it when no other chores.
Projects
None yet
Milestone
23.09
Development

Successfully merging a pull request may close this issue.

fix: Lazy-load the local config in the manager CLI lablup/backend.ai
1 participant
@achimnol