Skip to content

Add trace-collector service#1

Draft
sankalpatimilsina12 wants to merge 1 commit intomainfrom
feature/trace-collector-service
Draft

Add trace-collector service#1
sankalpatimilsina12 wants to merge 1 commit intomainfrom
feature/trace-collector-service

Conversation

@sankalpatimilsina12
Copy link
Copy Markdown
Owner

Adds trace-collector service.

I am not sure about the .ssh and /dump mounts.

Comment thread docker-compose.yml Outdated
driver: local

trace-collector:
image: sankalpatimilsina/scheduler:latest
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What's this and why is it on your personal account? And let's not use latest please.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

This is supposed to be pointing to Docker hub.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Comment thread docker-compose.yml Outdated
network_mode: service:nfd
volumes:
- /home/.ssh:/root/.ssh:ro
- /home/ndnops/ndntdump-exp-2023:/dump
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We should probably rename the directory at this point, "exp-2023" is no longer very relevant.

@sankalpatimilsina12 sankalpatimilsina12 force-pushed the feature/trace-collector-service branch from 1199061 to 6d82848 Compare March 28, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants