matrixorigin / matrixorigin/matrixone

[Bug]: test-iceberg-e2e-local fails because the nested adapter module is not tidy

Open
#27,721 2 comments 0 reactions 1 assignee Claimed by @Ariznawlll View on GitHub
kind/bug needs-triage
Dominant language
Go
Stars
1.9k
Forks
311
Avg merge
1d 3h
Merged PRs (30d)
768

Description

### MatrixOne commit

`d57f99abc0b9756e07aeb09ba642471db8c40ad7`

### What happened

The repository-provided local Iceberg E2E gate cannot reach the Docker or SQL test stages on the latest `main` because the nested Iceberg adapter module is not tidy.

### Reproduction

```bash
env -u GOROOT make test-iceberg-e2e-local
```

The gate stops at:

```text
[iceberg-ci] go test -tags iceberggo -count=1 ./...
go: updates to go.mod needed; to update it:
go mod tidy
make: *** [test-iceberg-e2e-local] Error 1
```

The failing command is run in `pkg/iceberg/adapter/iceberggo`, which has its own `go.mod` and `go.sum`.

### Root-cause evidence

Running the non-mutating diagnostic below in the nested module:

```bash
env -u GOROOT go mod tidy -diff
```

reports required changes, including:

- add indirect `github.com/BurntSushi/toml v1.2.1`
- update `github.com/klauspost/compress` from `v1.18.4` to `v1.18.7`
- update `github.com/pierrec/lz4/v4` from `v4.1.25` to `v4.1.26`
- update several `golang.org/x/*` indirect dependencies
- remove stale indirect dependencies and checksums

The root dependency upgrades were introduced by #27531, while the independent adapter module was not updated with them.

After temporarily running `go mod tidy` in the nested module, the same adapter test invoked by the gate passes under the E2E script's CGO environment:

```text
ok github.com/matrixorigin/matrixone/pkg/iceberg/adapter/iceberggo 0.453s
```

### Expected behavior

`make test-iceberg-e2e-local` should pass its adapter dependency preflight on a clean checkout and continue to the Nessie/MinIO/MatrixOne E2E stages without modifying tracked module files.

### Actual behavior

The official E2E target fails deterministically at the nested adapter module dependency check before any Iceberg SQL E2E case runs.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.