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

Add client event tests #20548

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add client event tests #20548

wants to merge 1 commit into from

Conversation

gosipyan
Copy link
Contributor

@gosipyan gosipyan commented Nov 4, 2024 •
edited
Loading

@gosipyan gosipyan force-pushed the sensor_test branch 11 times, most recently from b84eea1 to 4917c8e Compare November 7, 2024 10:59
@djoreilly
Copy link

The config generator has a --merge_file option, so can do this instead of the yq calls:

# cat init-config.json 
{
  "defaults": {
    "event_max_wait": 1,
    "event_max_wait_jitter": 1,
    "event_change_notify_all_clients": true
  },
  "Client": {
    "max_poll": 1,
    "max_poll_std": 1,
    "min_poll": 1,
    "default_max_wait": 1
  }
}

# velociraptor-client config generate --merge_file=init-config.json > server.conf
# velociraptor-client config client --config server.conf > /etc/velociraptor/client.config

@gosipyan gosipyan force-pushed the sensor_test branch 12 times, most recently from 0c6f8f0 to c672ae7 Compare November 11, 2024 13:44
@gosipyan gosipyan marked this pull request as ready for review November 11, 2024 14:15
@gosipyan gosipyan force-pushed the sensor_test branch 5 times, most recently from c18dcaa to b47bbfc Compare November 12, 2024 15:09
@gosipyan gosipyan force-pushed the sensor_test branch 13 times, most recently from b82f815 to f52fb42 Compare November 15, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gosipyan @djoreilly