Skip to content

.gitmodules: remove shallow=true config from nvmeof/gateway#63729

Merged
tchaikov merged 1 commit intoceph:mainfrom
athanatos:sjust/wip-71568
Jun 5, 2025
Merged

.gitmodules: remove shallow=true config from nvmeof/gateway#63729
tchaikov merged 1 commit intoceph:mainfrom
athanatos:sjust/wip-71568

Conversation

@athanatos
Copy link
Contributor

#61264 reintroduced https://tracker.ceph.com/issues/67640 fixed by
383091e.

Setting shallow=true for the nvmeof/gateway submodule is problematic because the ceph.git submodule sha1 is only very rarely the head sha1 of the default
branch.

Fixes: https://tracker.ceph.com/issues/71568

Show available Jenkins commands

ceph#61264 reintroduced
https://tracker.ceph.com/issues/67640 fixed by
383091e.

Setting shallow=true for the nvmeof/gateway submodule
is problematic because the ceph.git submodule sha1
is only very rarely the head sha1 of the default
branch.

Fixes: https://tracker.ceph.com/issues/71568
Signed-off-by: Samuel Just <sjust@redhat.com>
@athanatos athanatos requested review from a team, chardan and irq0 June 4, 2025 21:21
@tchaikov tchaikov merged commit 2c842c0 into ceph:main Jun 5, 2025
14 checks passed
@tchaikov
Copy link
Contributor

tchaikov commented Jun 5, 2025

hi @chardan and @irq0, sorry for merging this change without your reviews which were requested by Sam, because this is apparently a regression, and per https://tracker.ceph.com/issues/71568, this should addressed ASAP. so i took the liberty and merged it.

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.

2 participants