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

feature: Standalone watcher #1560

Open
wants to merge 69 commits into
base: main
Choose a base branch
from
Open

Conversation

fregataa
Copy link
Contributor

@fregataa fregataa commented Sep 10, 2023

Changes

Follow-ups

The PRs below should be merged after this get merged

How to test

./backend.ai watcher start-server -f <CONFIG_TOML_PATH>

You can find a sample config toml file in each watcher repository.
storage watcher: https://github.com/lablup/backend.ai-storage-watcher
agent watcher: https://github.com/lablup/backend.ai-agent-watcher

If you want to run multiple watcher plugins in one watcher process, install the all watcher plugins and set all configs in one file and pass the path of the config file.

# Example config toml file to run agent-watcher and storage-watcher simultaneously

[watcher]
node-id = "i-something-special"
event-loop = "uvloop"
ipc-base-path = "/tmp/backend.ai/ipc"

[module.agent-watcher]
service-name = "backendai-agent"
do-mount-check = true
ipc-base-path = "/tmp/backend.ai/ipc"
ignore-mount-event = true

[module.agent-watcher.event]
connect-server = true
consumer-group = "agent-watcher"

[module.storage-watcher]
bai-uid = 502
bai-gid = 502
service-name = "backendai-storage"
do-mount-check = false

[module.agent-watcher.event]
connect-server = false
consumer-group = "storage-watcher"

#...

TODO

  • Port agent-watcher APIs
  • Bundle indivisual subapps e.g) agent watcher subapp, service watcher subapp

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Documentation

@fregataa fregataa added this to the 23.09 milestone Sep 10, 2023
@fregataa fregataa self-assigned this Sep 10, 2023
@github-actions github-actions bot added comp:storage-proxy Related to Storage proxy component size:XL 500~ LoC labels Sep 10, 2023
@fregataa fregataa added the urgency:3 Must be finished within a certain time frame. label Sep 13, 2023
@fregataa
Copy link
Contributor Author

Since control panel would be the one that requests to watcher servers, I made an issue about supporting HTTPS in control panel.
https://github.com/lablup/backend.ai-control-panel/issues/674

@fregataa fregataa marked this pull request as ready for review November 14, 2023 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:manager Related to Manager component comp:storage-proxy Related to Storage proxy component comp:webserver Related to Web Server component size:XL 500~ LoC urgency:3 Must be finished within a certain time frame.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants