fleet icon indicating copy to clipboard operation
fleet copied to clipboard

passing osquery configuration to launcher

Open groob opened this issue 6 years ago • 1 comments

Moving from code review in #1582

The current implementation of the launcher is somewhat opinionated about osquery config, so when the configuration endpoint was added in https://github.com/kolide/fleet/pull/1544/files#diff-57894187d53a170dbc47ebddb3139466R54 the following change was made:

	// Launcher manages plugins so remove them from configuration if they exist.
	for _, optionName := range []string{"distributed_plugin", "logger_plugin"} {
		if _, ok := config.Options[optionName]; ok {
			delete(config.Options, optionName)
		}
	}

Is it necessary to remove those options? Especially if the user wants to configure multiple logger_plugins.

// cc @marpaia @zwass

groob avatar Oct 24 '17 19:10 groob

The reason why this is here is because the default configuration options in fleet specify that the tls logger and distributed plugins should be used. This is problematic for the launcher because obviously the launcher should continue to use the kolide_grpc plugins, not tls. This was added as a quick fix, but it is obviously not a sustainable solution to this problem.

marpaia avatar Oct 25 '17 03:10 marpaia