matrixorigin / matrixorigin/matrixone
[Bug]: test-iceberg-e2e-local fails because the nested adapter module is not tidy
- 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
Assessment
This issue has not been assessed yet.