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.

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

Release v0.18.0 #581

Merged
merged 25 commits into from
Feb 20, 2024
Merged

Release v0.18.0 #581

merged 25 commits into from
Feb 20, 2024

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Feb 20, 2024

No description provided.

hacdias and others added 24 commits January 10, 2024 16:52
golang/go@edfe078 was released in go1.20 and our `go.mod` file indicates:
```
go 1.20
```
So this is not needed anymore.

However none of this matter since c28c847 embeded and modified the ServeContent function in our codebase.
So I copied my upstream fix there.
They don't share state, on my computer time to test went down from 20s to 10s with ~200% CPU usage.
Time went down from 25s to 0.5s

I also removed `TestFuzzBuzhashChunking` since I rewrote it a `testing.F` fuzz test and yet it wasn't able to find any new interesting input.
This also include cleanup for session code.
…ksBackend`

This allows the `BlocksBackend` to inject a session in the context, it remove the problems where we keep reseting the bitswap session each time we call the `fetcher`.
<!--
Please update the CHANGELOG.md if you're modifying Go files. If your change does not require a changelog entry, please do one of the following:
- add `[skip changelog]` to the PR title
- label the PR with `skip/changelog`
-->
…ksBackend` (#550)

This allows the `BlocksBackend` to inject a session in the context, it remove the problems where we keep reseting the bitswap session each time we call the `fetcher`.
… inside newSession

This was not wrong however it was confusing, grabSessionFromContext would show up twice in logs.
This is accessible through the blockservice object.
… valid leading elements

We would scan the valid leading elements twice.
* fix(gateway): bound negative indexes to size of file
* fix: adjust negative to when from is negative too
* chore: gateway-conformance@v0.5

https://github.com/ipfs/gateway-conformance/releases/tag/v0.5.0

---------

Co-authored-by: Marcin Rataj <lidel@lidel.org>
@hacdias hacdias self-assigned this Feb 20, 2024
@hacdias hacdias changed the base branch from main to release February 20, 2024 08:36
@hacdias hacdias added release Merging this PR will create a tagged release and removed skip/changelog labels Feb 20, 2024
@hacdias hacdias changed the title Release vX.Y.Z Release v0.18.0 Feb 20, 2024
Copy link

Suggested version: v0.18.0

Comparing to: v0.17.0 (diff)

Changes in go.mod file(s):

(empty)

gorelease says:

# diagnostics
go.mod: the following requirements are needed
	go.opentelemetry.io/otel/exporters/jaeger@v1.14.0
	go.uber.org/atomic@v1.11.0
Run 'go mod tidy' to add missing requirements.
required module github.com/microcosm-cc/bluemonday@v1.0.1 retracted by module author: Retract older versions as only latest is to be depended upon

# summary
Suggested version: v0.17.1

gocompat says:

Your branch is up to date with 'origin/main'.

Automatically created GitHub Release

A draft GitHub Release has been created.
It is going to be published when this PR is merged.
You can modify its' body to include any release notes you wish to include with the release.

Copy link

codecov bot commented Feb 20, 2024

Codecov Report

Attention: 80 lines in your changes are missing coverage. Please review.

Comparison is base (8530184) 65.68% compared to head (4f5b2c0) 65.63%.

Impacted file tree graph

@@             Coverage Diff             @@
##           release     #581      +/-   ##
===========================================
- Coverage    65.68%   65.63%   -0.06%     
===========================================
  Files          207      207              
  Lines        25590    25612      +22     
===========================================
+ Hits         16809    16810       +1     
- Misses        7314     7331      +17     
- Partials      1467     1471       +4     
Files Coverage Δ
examples/gateway/common/handler.go 95.40% <100.00%> (-0.16%) ⬇️
gateway/gateway.go 75.94% <ø> (-9.10%) ⬇️
gateway/handler.go 78.27% <100.00%> (+1.73%) ⬆️
gateway/handler_unixfs_file.go 83.63% <ø> (+2.68%) ⬆️
gateway/headers.go 100.00% <100.00%> (ø)
namesys/namesys.go 56.31% <100.00%> (+1.18%) ⬆️
namesys/namesys_cache.go 47.91% <100.00%> (+8.89%) ⬆️
gateway/hostname.go 74.18% <33.33%> (+0.79%) ⬆️
gateway/metrics.go 78.42% <40.00%> (-0.82%) ⬇️
routing/http/client/client.go 73.00% <69.23%> (-1.60%) ⬇️
... and 3 more

... and 12 files with indirect coverage changes

@hacdias hacdias marked this pull request as ready for review February 20, 2024 08:43
@hacdias hacdias requested review from lidel and a team as code owners February 20, 2024 08:43
@hacdias hacdias merged commit c336fc9 into release Feb 20, 2024
19 checks passed
@hacdias hacdias deleted the release-v0.18.0 branch February 20, 2024 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Merging this PR will create a tagged release
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants