Skip to content

Conversation

@lidel
Copy link
Member

@lidel lidel commented Aug 27, 2025

lidel and others added 12 commits August 21, 2025 19:50
* fix: harness tests random panic

Connecting nodes in parallel can cause TLS handshake failures. For each node, connect to the other nodes serially. It is not necessary to connect in parallel as it does not save any significant time.

Closes #10932

(cherry picked from commit ae068a8)
* 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 <[email protected]>
(cherry picked from commit 15f723a)
* feat(ci): docker linting

adds hadolint to validate dockerfile best practices
configures project-specific rules in .hadolint.yaml

* fix(ci): enable hadolint console output

adds verbose and tty format to see linting results in CI logs

* test: trigger hadolint warning

remove --no-install-recommends to test CI output

* fix(ci): fail hadolint on warnings

stricter linting to catch all best practice violations

* fix: add --no-install-recommends to apt-get

reduces image size by avoiding unnecessary packages

* refactor: use WORKDIR instead of cd in dockerfile

replaces cd commands with WORKDIR for cleaner dockerfile
removes unnecessary hadolint ignore rules DL3003 and DL3009

* chore: simplify hadolint config

removes unnecessary override rules for cleaner config

(cherry picked from commit 82fef0c)
- updated changelog and contributors with mkreleaselog output
- improved mkreleaselog to work with kubo from any directory
- add emojis to all headers
- add overview section
- group IPNS features together
- fix IPNS flags to match actual implementation
- explain how 'auto' resolution depends on Routing.Type
- add command to inspect effective publishers
- simplify publishing behavior explanation
@lidel lidel requested a review from a team as a code owner August 27, 2025 21:01
keep -dev version from master and add v0.38 to changelog
@lidel lidel merged commit 81e354b into master Aug 27, 2025
15 checks passed
@lidel lidel deleted the merge-release-v0.37.0 branch August 27, 2025 21:54
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.

4 participants