Commit Graph

4 Commits

Author SHA1 Message Date
Hector Sanjuan
46324f085d fix: disable telemetry in test profile (#10931)
* Tests: disable telemetry in tests by default

Disable the plugin in cli tests and sharness by default. Enable only in
telemetry tests.

There are cases when tests get stuck or get killed and leave daemons hanging around. We don't want to be getting telemetry from those.

* sharness: attempt to fix

* sharness: add missing --bool flag

* fix(ci): add omitempty to Plugin.Config field

The sharness problem is that when the telemetry plugin is configured
initially with 'ipfs config --bool', it creates a structure without
the 'Config: null' field, but when the config is copied and replaced,
it expects the structure to be preserved.

Adding omitempty ensures the Config field is omitted from JSON when
nil, making the config structure consistent between initial creation
and replacement operations.

---------

Co-authored-by: Marcin Rataj <lidel@lidel.org>
(cherry picked from commit 15f723a15e)
2025-08-27 16:26:50 +02:00
Steven Allen
f0b5fe9733 go-ipfs-config: plugins: don't omit empty config values
Turns out the go-ipfs config logic really doesn't play well with this. Let's
just keep them and avoid encoding empty values another way.
2019-08-29 13:56:13 -07:00
Steven Allen
25e3e1974b go-ipfs-config: nit: omit empty plugin values
Let's try to slim down default configs a bit.
2019-08-29 10:25:35 -07:00
Steven Allen
a04a858922 go-ipfs-config: add plugins config section
* Allow users to store plugin specific config options.
* Allow users to disable specific plugins.

Eventually, we can use this to allow loading plugins from non-standard locations.
2019-08-29 10:15:31 -07:00